Skip to content

Clean up POWER wrapper in libguestfish#2718

Merged
jlebon merged 1 commit intocoreos:mainfrom
ravanelli:follow-up
Feb 22, 2022
Merged

Clean up POWER wrapper in libguestfish#2718
jlebon merged 1 commit intocoreos:mainfrom
ravanelli:follow-up

Conversation

@ravanelli
Copy link
Copy Markdown
Member

Signed-off-by: Renata Ravanelli rravanel@redhat.com

- It is a follow up for the issue openshift/os#720  and for coreos#2714

- POWER never really used the newGuestfish function in
mantle/platform/qemu.go, with the new tests now using setupPreboot function
that was recently added, the issue happened because there is no call for the
POWER wrapper in the newGuestfish. Nonetheless, it is not a libguestfish issue
since POWER8 (cluster) needs the vsmt set to 8.

Signed-off-by: Renata Ravanelli <rravanel@redhat.com>
@ravanelli ravanelli requested a review from jlebon February 22, 2022 14:26
Copy link
Copy Markdown
Member

@jlebon jlebon left a comment

Choose a reason for hiding this comment

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

Thanks!

@jlebon jlebon enabled auto-merge (rebase) February 22, 2022 14:34
@ravanelli ravanelli requested a review from miabbott February 22, 2022 14:50
Copy link
Copy Markdown
Member

@miabbott miabbott left a comment

Choose a reason for hiding this comment

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

LGTM!

@ravanelli
Copy link
Copy Markdown
Member Author

/retest-required

@miabbott
Copy link
Copy Markdown
Member

I think the ci/prow/rhcos test is going to fail until we land openshift/os#718

And since this PR really doesn't touch the x86_64 case we can /override if we want

@jlebon
Copy link
Copy Markdown
Member

jlebon commented Feb 22, 2022

/override ci/prow/rhcos

@openshift-ci
Copy link
Copy Markdown

openshift-ci Bot commented Feb 22, 2022

@jlebon: Overrode contexts on behalf of jlebon: ci/prow/rhcos

Details

In response to this:

/override ci/prow/rhcos

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.

@jlebon jlebon merged commit 057f85a into coreos:main Feb 22, 2022
@ravanelli ravanelli deleted the follow-up branch December 5, 2022 13:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants