Project

General

Profile

Actions

Bug #509

closed

Review branch add_iface_korutopi

Added by Pierre-Louis Bonicoli about 7 years ago. Updated about 7 years ago.

Status:
Resolved
Priority:
Normal
Category:
System :: Network
Start date:
2017-02-06
Due date:
% Done:

100%

Estimated time:
Patch Available:
Confirmed:
Yes
Branch:
Entity:
DuckCorp
Security:
No
Help Needed:

Description

Please, could you review the branch called add_iface_korutopi (admin repository) ?

Notes:

The current content of /etc/resolv.conf is:

domain duckcorp.org
search duckcorp.org hq.duckcorp.org lan
nameserver 192.168.1.2

after these modifications, content will be:
search duckcorp.org hq.duckcorp.org
nameserver 192.168.1.2
options timeout:1 edns0


Related issues 2 (0 open2 closed)

Copied from DuckCorp Infrastructure - Bug #506: Review branch orfeo_remove_old_ipResolvedMarc Dequènes2017-02-06

Actions
Copied to DuckCorp Infrastructure - Review #510: Review branch specify_FQDN_and_remote_portResolvedMarc Dequènes2017-02-06Actions
Actions #1

Updated by Pierre-Louis Bonicoli about 7 years ago

  • Copied from Bug #506: Review branch orfeo_remove_old_ip added
Actions #2

Updated by Pierre-Louis Bonicoli about 7 years ago

  • Copied to Review #510: Review branch specify_FQDN_and_remote_port added
Actions #3

Updated by Marc Dequènes about 7 years ago

In the running config I see:

default via 192.168.1.2 dev eth0

which does not match the gateway defined in the patch. In fact '/etc/network/interfaces' was probably used as a template for the patch.

I guess it differs because the network settings are ultimately defined in LXC and the one in the system config does not change it. Anyway it is better if it matches to avoid mistakes, or if we define other network components one da and play with ifupdown; this would avoid breaking things.

The rest is fine.

Actions #4

Updated by Pierre-Louis Bonicoli about 7 years ago

  • Status changed from New to In Progress

Indeed, I didn't check the running config but /etc/network/interfaces, thanks !

Branch updated and renamed: korutopi_iface_DNS.

Actions #5

Updated by Marc Dequènes about 7 years ago

  • Category set to System :: Network
  • Status changed from In Progress to Resolved
  • % Done changed from 0 to 100

It's fine, thanks. Merged.

Actions

Also available in: Atom PDF