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
overlay: initramfs teardown: add support for coreos.force_persist_ip karg #1045
overlay: initramfs teardown: add support for coreos.force_persist_ip karg #1045
Conversation
also:
|
overlay.d/05core/usr/lib/dracut/modules.d/35coreos-ignition/coreos-teardown-initramfs.sh
Show resolved
Hide resolved
Everything is now newer than the minimimum requirement of NetworkManager 1.26.0.
…karg This karg will override the check to see if Networking Configuration is in the real root (i.e. written by Ignition most likely) and force propagation of initramfs networking anyway. It should only be used in cases where you know exactly what you are doing. For example, in the problem described in https://bugzilla.redhat.com/show_bug.cgi?id=1958930 the networking configs didn't conflict at all, thus this could be used safely. This option will help users who get stuck like they were in that bug to get unstuck easily and then change their strategy later. Fixes: coreos/fedora-coreos-tracker#853
8199fd3
to
a6c28d2
Compare
forcepropagate=0 | ||
if dracut_func getargbool 0 'coreos.force_persist_ip'; then | ||
forcepropagate=1 | ||
echo "info: coreos.force_persist_ip detected: will force network config propagation" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Overall, this looks fine to me, so
/lgtm
That said, I wonder if it makes sense to put something like
cat <<EOM
================================================================
Kernel arg "coreos.force_persist_up=1" should be used rarely
and in exceptional cirucmstances. This is unlikely the bit you
want...
================================================================
EOM
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yeah I thought about it. I doubt anyone other than us are looking that close in the logs. I figure we can keep the only place it's documented with a fairly large disclaimer about its use.
Bring in coreos.force_persist_ip support. coreos/fedora-coreos-config#1045 Should help with https://bugzilla.redhat.com/show_bug.cgi?id=1958930 ``` $ git shortlog 8600cd7..43eb876 Dusty Mabe (1): overlay: initramfs teardown: add support for coreos.force_persist_ip karg ```
Bring in coreos.force_persist_ip support. coreos/fedora-coreos-config#1045 Should help with https://bugzilla.redhat.com/show_bug.cgi?id=1958930 ``` $ git shortlog 8600cd7..43eb876 Dusty Mabe (1): overlay: initramfs teardown: add support for coreos.force_persist_ip karg ```
This karg will override the check to see if Networking Configuration is
in the real root (i.e. written by Ignition most likely) and force
propagation of initramfs networking anyway. It should only be used in
cases where you know exactly what you are doing. For example, in the
problem described in https://bugzilla.redhat.com/show_bug.cgi?id=1958930
the networking configs didn't conflict at all, thus this could be used
safely. This option will help users who get stuck like they were in that
bug to get unstuck easily and then change their strategy later.
Fixes: coreos/fedora-coreos-tracker#853