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
libvirt: Bump bootstrap size (primarily for OKD) #4364
libvirt: Bump bootstrap size (primarily for OKD) #4364
Conversation
f3c6c4c
to
17829ad
Compare
(OK this time actually tested locally) |
Would you mind making the same change to baremetal's bootstrap volume? @hroyrh testing baremetal IPI on OKD has run into the same problem with 8GB disk being too small. installer/data/data/baremetal/bootstrap/main.tf Lines 7 to 12 in 8b21f01
|
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.
/approve
17829ad
to
2c9ddac
Compare
OK took a stab at that, though I don't have the test setup for it handy. |
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: staebler The full list of commands accepted by this bot can be found here. The pull request process is described here
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
2c9ddac
to
b06fbbb
Compare
Ah, looks like the baremetal side needed two separate volumes. Took a crack at that. |
People have been hitting space issues with OKD (FCOS) which is 8GB by default. Eventually I think we should support something like simulated OpenStack instance sizes for libvirt. The FCOS/RHCOS difference of 8GB vs 16GB is something I'd eventually like to fix by making RHCOS 8GB too for consistency.
b06fbbb
to
2d6bb07
Compare
Thank you for applying the baremetal fix. This LGTM. /lgtm |
/hold Feel free to remove when ready. Libvirt failures seem unrelated but I'm not an expert on that platform. |
I think that's a known SDN flake, see https://bugzilla.redhat.com/show_bug.cgi?id=1886922 |
/retest Please review the full test history for this PR and help us cut down flakes. |
7 similar comments
/retest Please review the full test history for this PR and help us cut down flakes. |
/retest Please review the full test history for this PR and help us cut down flakes. |
/retest Please review the full test history for this PR and help us cut down flakes. |
/retest Please review the full test history for this PR and help us cut down flakes. |
/retest Please review the full test history for this PR and help us cut down flakes. |
/retest Please review the full test history for this PR and help us cut down flakes. |
/retest Please review the full test history for this PR and help us cut down flakes. |
/retest Please review the full test history for this PR and help us cut down flakes. |
7 similar comments
/retest Please review the full test history for this PR and help us cut down flakes. |
/retest Please review the full test history for this PR and help us cut down flakes. |
/retest Please review the full test history for this PR and help us cut down flakes. |
/retest Please review the full test history for this PR and help us cut down flakes. |
/retest Please review the full test history for this PR and help us cut down flakes. |
/retest Please review the full test history for this PR and help us cut down flakes. |
/retest Please review the full test history for this PR and help us cut down flakes. |
@cgwalters: The following tests 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/test-infra repository. I understand the commands that are listed here. |
The FCOS/RHCOS difference of 8GB vs 16GB is something
I'd eventually like to fix by making RHCOS 8GB too for
consistency.
The installer can easily resize.