-
Notifications
You must be signed in to change notification settings - Fork 175
osbuild-manifests: enable kubevirt image for s390x #4093
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
base: main
Are you sure you want to change the base?
Conversation
I suspect if you were to try to boot these images they wouldn't boot (hint: Can you work with someone (maybe @jschintag) to verify the generated disk images work? |
@@ -98,7 +98,10 @@ pipelines: | |||
mpp-if: arch == 'aarch64' | |||
then: arm64 | |||
else: | |||
mpp-format-string: '{arch}' | |||
mpp-if: arch == 's390x' | |||
then: s390x |
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.
Is this needed? I mean the default would return s390x anyway right?
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.
we probably need to pass some value here, so I would say yes?
I know for this particular one there's no "mapping" that's happening, but I don't think it hurts.
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.
oh I see now the else below..
yeah, the else: mpp-format-string: '{arch}'
should already cover this.
hi,
|
Maybe since @nikita-dubrovskii knows OSBuild well and overlaps with your TZ better you can do a screenshare to debug what is going on? In the mean time feel free to push up the lastest version of your code here. |
fc78841
to
cbc74c5
Compare
By adding just the i ran |
cbc74c5
to
24f2cac
Compare
Hi Dusty,
|
@madhu-pillai: The following test failed, say
Full PR test history. Your PR dashboard. 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-sigs/prow repository. I understand the commands that are listed here. |
@@ -52,6 +52,41 @@ pipelines: | |||
partition: | |||
mpp-format-int: '{image.layout[''boot''].partnum}' | |||
target: /boot | |||
- mpp-if: arch == 's390x' |
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.
In the other platforms where we have this we also have this comment above which I think is useful:
# If on s390x then run zipl, which must run after the kernel
# arguments get finalized in the coreos.platform stage above
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.
also, if you rebase on top of latest :main
I think it will fix the CI failures.
Hi,
Based on the jira ticket https://issues.redhat.com/browse/COS-3221 .
https://issues.redhat.com/browse/OCPBUGS-54751
Have done the testing using coreos-assember image with changes in osbuild-manifests. Able to generate kubevirt archive for FCOS as well as RHCOS.
RHCOS
FCOS