-
Notifications
You must be signed in to change notification settings - Fork 1.5k
OCPBUGS-62790: Use separate tmpfs for ostree checkout on live ISO #10133
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
|
@zaneb: This pull request references Jira Issue OCPBUGS-62790, which is valid. 3 validation(s) were run on this bug
Requesting review from QA contact: The bug has been updated to refer to the pull request using the external bug tracker. In response to this:
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 openshift-eng/jira-lifecycle-plugin repository. |
a658402 to
92c5f9b
Compare
|
This works and is independent of coreos internals, unlike #10055. Plus no resizing of a mounted filesystem. I think this is the way to go. /verified by @zaneb |
|
@zaneb: This PR has been marked as verified by In response to this:
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 openshift-eng/jira-lifecycle-plugin repository. |
|
/cherry-pick release-4.20 |
|
@zaneb: once the present PR merges, I will cherry-pick it on top of In response to this:
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. |
Installations using ABI/assisted with 16GiB of RAM on the bootstrap node were failing with "no space left on device" during bootstrapping. The live ISO environment uses a tmpfs mounted at /var that is sized at 50% of available RAM. On systems with 16GiB of RAM, this provides only 8GiB of tmpfs space. At the beginning of the bootstrap process, node-image-pull.sh creates an ostree checkout underneath /var/ostree-container. When this is added to the regular disk space usage of the later parts of the bootstrap, the peak tmpfs usage hits around 9.4GiB. This fix creates a separate 4GiB tmpfs for /var/ostree-container, so that it is not subject to the limits on the size of /var.
|
@zaneb: 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-sigs/prow repository. I understand the commands that are listed here. |
|
coreos/fedora-coreos-config#499 describes the original reason for using an xfs filesystem on top of tmpfs, which was related to an SELinux issue at startup. Since that isn't a concern here, we can greatly simplify by dropping the xfs part. /verified by @zaneb |
|
@zaneb: This PR has been marked as verified by In response to this:
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 openshift-eng/jira-lifecycle-plugin repository. |
|
@mhanss: This PR has been marked as verified by In response to this:
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 openshift-eng/jira-lifecycle-plugin repository. |
|
/approve |
|
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: pawanpinjarkar 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 |
|
/lgtm |
00584fe
into
openshift:main
|
@zaneb: Jira Issue Verification Checks: Jira Issue OCPBUGS-62790 Jira Issue OCPBUGS-62790 has been moved to the MODIFIED state and will move to the VERIFIED state when the change is available in an accepted nightly payload. 🕓 In response to this:
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 openshift-eng/jira-lifecycle-plugin repository. |
|
@zaneb: new pull request created: #10140 In response to this:
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. |
Installations using ABI/assisted with 16GiB of RAM on the bootstrap node
were failing with "no space left on device" during bootstrapping. The
live ISO environment uses a tmpfs mounted at /var that is sized at 50%
of available RAM. On systems with 16GiB of RAM, this provides only 8GiB
of tmpfs space.
At the beginning of the bootstrap process, node-image-pull.sh creates an
ostree checkout underneath /var/ostree-container. When this is added to
the regular disk space usage of the later parts of the bootstrap, the
peak tmpfs usage hits around 9.4GiB.
This fix creates a separate 5GiB tmpfs to house /var/ostree-container,
so that it is not subject to the limits on the size of /var.