Skip to content

[rhcos-4.13] mantle/platform: allow systemctl is-system-running nonzero exit codes#3497

Merged
dustymabe merged 1 commit intocoreos:rhcos-4.13from
openshift-cherrypick-robot:cherry-pick-3487-to-rhcos-4.13
Jun 2, 2023
Merged

[rhcos-4.13] mantle/platform: allow systemctl is-system-running nonzero exit codes#3497
dustymabe merged 1 commit intocoreos:rhcos-4.13from
openshift-cherrypick-robot:cherry-pick-3487-to-rhcos-4.13

Conversation

@openshift-cherrypick-robot
Copy link
Copy Markdown

This is an automated cherry-pick of #3487

/assign travier

The `CheckMachine()` call tolerates `systemctl is-system-running`
returning e.g. `starting` and `initializing` in addition to the usual
`running`. But by design, the command returns a nonzero exit code in
those cases.

Since we're explicitly checking the output string against the acceptable
states, just ignore the command exit code so that we don't fail if the
system is still starting.

Prep for adding a test in which we expect the system to still be
`starting`.
@openshift-ci
Copy link
Copy Markdown

openshift-ci Bot commented Jun 1, 2023

Hi @openshift-cherrypick-robot. Thanks for your PR.

I'm waiting for a coreos member to verify that this patch is reasonable to test. If it is, they should reply with /ok-to-test on its own line. Until that is done, I will not automatically test new commits in this PR, but the usual testing commands by org members will still work. Regular contributors should join the org to skip this step.

Once the patch is verified, the new status will be reflected by the ok-to-test label.

I understand the commands that are listed here.

Details

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

@travier
Copy link
Copy Markdown
Member

travier commented Jun 1, 2023

/ok-to-test

@dustymabe
Copy link
Copy Markdown
Member

LGTM

@travier
Copy link
Copy Markdown
Member

travier commented Jun 1, 2023

/retest

@openshift-ci
Copy link
Copy Markdown

openshift-ci Bot commented Jun 1, 2023

@openshift-cherrypick-robot: The following test failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name Commit Details Required Rerun command
ci/prow/rhcos d497b37 link true /test rhcos

Full PR test history. Your PR dashboard.

Details

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@dustymabe
Copy link
Copy Markdown
Member

the CI failure here is ext.config.shared.kdump.crash timing out because it needs coreos/fedora-coreos-config#2448 (comment)

merging over red so we can unblock that other PR too.

@dustymabe dustymabe merged commit 4d18b37 into coreos:rhcos-4.13 Jun 2, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants