Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

avahi-daemon.service.in: start after network-online #59

Closed
wants to merge 1 commit into from
Closed

avahi-daemon.service.in: start after network-online #59

wants to merge 1 commit into from

Conversation

mgorse
Copy link

@mgorse mgorse commented Jun 23, 2016

We are seeing this in a log while beta testing SLE 12 SP2
May 31 11:16:35 linux avahi-daemon[801]: Failed to open /etc/resolv.conf: No such file or directory

So it appears that avahi-daemon wants resolv.conf to be set up correctly; maybe it makes sense to ensure that NetworkManager or wicked have started first.

We may need resolv.conf to be set up, so ensure that NetworkManager
or wicked have already started before initializing.
@lathiat
Copy link
Contributor

lathiat commented Jun 24, 2016

Thanks for your contribution.

This functionality is used mainly for the seldom-used publish-dns-servers, publish-resolv-conf-dns-servers and publish-domain options which advertise your DNS services on the LAN for avahi-dnsconfd to consume. These options are also re-read when reload config is called, which can also happen on inotify - though we currently don't inotify on resolv.conf (and I'm not sure how that would work on the non existant file anyway)

avahi-dnsconfd and these options have long outlived their life, they have since been replaced by extensions to the radvd protocol and optionally DHCPv6. I am not really aware of people actually using them in production and I do not believe any other implementation has implemented using these. They will likely be removed in some future release where we have a good reason to break backwards compatibility at the same time.

From my understanding of systemd units, it appears that the "Wants" and "After" combination makes it a total loose coupling, so this wouldn't cause any problems but it also seems an unnecessary dependency. Instead of this change, I propose that we squelch this resolv.conf error unless the options that use it are actually configured.

Do you see any problem with that?

@mgorse
Copy link
Author

mgorse commented Jun 24, 2016 via email

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants