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

Do not crash on dangling symlink /etc/resolv.conf (#2019579) #3702

Conversation

rvykydal
Copy link
Contributor

No description provided.

@rvykydal rvykydal added the master Please, use the `f39` label instead. label Nov 12, 2021
@rvykydal rvykydal force-pushed the do-not-crash-on-dangling-etc-resolv-conf-symlink branch from 6fa3633 to 033a908 Compare November 12, 2021 14:47
@rvykydal rvykydal changed the title Do not crash on dangling symlink to /etc/resolv.conf (#2019579) Do not crash on dangling symlink /etc/resolv.conf (#2019579) Nov 12, 2021
@M4rtinK
Copy link
Contributor

M4rtinK commented Nov 15, 2021

/kickstart-test --testtype smoke

Copy link
Contributor

@M4rtinK M4rtinK left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good to me.

Copy link
Contributor

@poncovka poncovka left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good to me. Thanks!

@rvykydal rvykydal merged commit db4f61b into rhinstaller:master Nov 16, 2021
@VladimirSlavik
Copy link
Contributor

Hopefully it never becomes a directory...

jcmdln added a commit to jcmdln/infranetic that referenced this pull request Jan 17, 2022
Build the latest two stable releases of Fedora as a smoke test. This
currently doesn't work because of a bug in Fedora 35 where during a
kickstart the /etc/resolv.conf symlink for systemd-resolved isn't
created:

https://bugzilla.redhat.com/show_bug.cgi?id=2018913
https://bugzilla.redhat.com/show_bug.cgi?id=2019579
Do not crash on dangling symlink /etc/resolv.conf (#2019579) rhinstaller/anaconda#3702
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
master Please, use the `f39` label instead.
4 participants