-
Notifications
You must be signed in to change notification settings - Fork 2.9k
pkg/machine/e2e: skip rosetta test #26293
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
Conversation
|
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: Luap99 The full list of commands accepted by this bot can be found here. The pull request process is described here DetailsNeeds approval from an approver in each of these files:
Approvers can indicate their approval by writing |
|
LGTM |
|
[NON-BLOCKING] Packit jobs failed. @containers/packit-build please check. Everyone else, feel free to ignore. |
|
ssh to running machine and check proxy settings In the proxy test, that one is new I see it on #26292 as well I though it was start a flake but given it fails in this exact tests I wonder if something "bad" got into the image now that we are using the latest coreos all of the sudden... |
We disabled it in the machine os image so it can no longer be used currently. Signed-off-by: Paul Holzinger <[email protected]>
|
Ok I found a workaround, something is very broken in the new image that setting a basic env var via environment.d causes boot failures. |
Do not write /etc/environment.d files, something is broken in the new image that causes the boot to fail when any basic var is set there. Signed-off-by: Paul Holzinger <[email protected]>
|
LGTM |
|
/lgtm |
e8e5dce
into
containers:main
We disabled it in the machine os image so it can no longer be used currently.
Does this PR introduce a user-facing change?