Mailinglist Archive: opensuse-bugs (5295 mails)

< Previous Next >
[Bug 758879] NetworkManager doesn't write NTP servers provided by DHCP to netconfig

https://bugzilla.novell.com/show_bug.cgi?id=758879

https://bugzilla.novell.com/show_bug.cgi?id=758879#c12


Ralf Habacker <ralf.habacker@xxxxxxxxxx> changed:

What |Removed |Added
----------------------------------------------------------------------------
CC| |ralf.habacker@xxxxxxxxxx

--- Comment #12 from Ralf Habacker <ralf.habacker@xxxxxxxxxx> 2013-11-10
14:14:04 UTC ---
(In reply to comment #6)
Sorry for the incomplete filenames in comment #5. Every filename should be
/var/run/netconfig/NetworkManager.netconfig

Looking at the NetworkManager-0.9.8.8 source (Factory). There's a
write_to_netconfig() call for DNSSERVERS but no explicit one for NTPSERVERS.

In nm_dhcp_dhclient_create_config() in
src/dhcp-manager/nm-dhcp-dhclient-utils.c there is:

add_also_request (alsoreq, "ntp-servers");

but I expect that to only populate the lease file. Then every cases of
write_to_netconfig() is in src/dns-manager/nm-dns-manager.c All of them in
dispatch_netconfig() which although it sounds generic enough to be more than
just DNS configuration is called by update_dns().

Following it up through parent calls I guess that essentially
device_state_changed() is incomplete in the case of NM_DEVICE_STATE_ACTIVATED.
It has this comment:

/* Add device's new IPv4 and IPv6 configs to DNS */

and appears to update the /var/run/netconfig/NetworkManager.netconfig as if
DNS
was the only state netconfig could use from there when in fact if there was an
ultimate addition of write_to_netconfig() of the NTPSERVERS variable this
would
be fixed.

The bug should be assigned to one of the current maintainers of
NetworkManager.


and appears to update the /var/run/netconfig/NetworkManager.netconfig as if
DNS
was the only state netconfig could use from there when in fact if there was an
ultimate addition of write_to_netconfig() of the NTPSERVERS variable this
would
be fixed.

The bug should be assigned to one of the current maintainers of
NetworkManager.

The patch from https://bugzilla.novell.com/show_bug.cgi?id=798886 fixes exactly
this issue.

--
Configure bugmail: https://bugzilla.novell.com/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are on the CC list for the bug.

< Previous Next >