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
35coreos-live/coreos-livepxe-rootfs: retry fetching rootfs forever #964
Conversation
Still need to test this. |
overlay.d/05core/usr/lib/dracut/modules.d/35coreos-live/coreos-livepxe-rootfs.sh
Outdated
Show resolved
Hide resolved
OK, tweaked and tested now! |
overlay.d/05core/usr/lib/dracut/modules.d/35coreos-live/coreos-livepxe-rootfs.sh
Outdated
Show resolved
Hide resolved
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.
LGTM
I am interested in @bgilbert's thoughts on retrying forever, though.
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.
I think it makes sense to retry forever. We'd just fail outright otherwise.
overlay.d/05core/usr/lib/dracut/modules.d/35coreos-live/coreos-livepxe-rootfs.sh
Outdated
Show resolved
Hide resolved
overlay.d/05core/usr/lib/dracut/modules.d/35coreos-live/coreos-livepxe-rootfs.sh
Outdated
Show resolved
Hide resolved
We don't know how long it will take for networking to come up, so let's match Ignition's semantic of retrying to fetch remote resources forever. `curl` doesn't natively have support for this, so wrap it in a `while` loop. This also works around the fact that el8's `curl` doesn't support `--retry-all-errors`. This was previously added in coreos#929.
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.
LGTM
For coreos/fedora-coreos-config#964 for https://gitlab.cee.redhat.com/coreos/redhat-coreos/-/merge_requests/1252#note_2289576. ``` Dusty Mabe (5): manifests: f34: Delete coreos-reset-stub-resolv-selinux-context.{path,service}. overrides: drop kernel 5.10.19 freeze overrides: drop graduated podman override overlay.d/35coreos-network: handle new nm-run.service in initramfs overlay.d/35coreos-network: don't bring up NM if not needed Jonathan Lebon (3): tests/kola/rootless-systemd: add back sleep overrides: fast-track rpm-ostree-2021.4 35coreos-live/coreos-livepxe-rootfs: retry fetching rootfs forever Luca BRUNO (1): manifest-lock: drop graduated overrides Miroslav Lichvar (1): overlay.d/20platform-chrony: Fix misconfiguration with leap-smearing servers Timothée Ravier (2): tests/kola/rpm-ostree-countme: Update for pre-release repo count tests/kola/toolbox: Retry 'toolbox create' to avoid infra flakes ```
We don't know how long it will take for networking to come up, so let's
match Ignition's semantic of retrying to fetch remote resources forever.
curl
doesn't natively have support for this, so wrap it in awhile
loop. This also works around the fact that el8's
curl
doesn't support--retry-all-errors
.This was previously added in #929.