-
Notifications
You must be signed in to change notification settings - Fork 419
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
MCO-1505: allows user-configurable resource requests and limits via MachineOSConfig annotations #4946
base: main
Are you sure you want to change the base?
MCO-1505: allows user-configurable resource requests and limits via MachineOSConfig annotations #4946
Conversation
@cheesesashimi: This pull request references MCO-1505 which is a valid jira issue. Warning: The referenced jira issue has an invalid target version for the target branch this PR targets: expected the story to target the "4.19.0" version, but no target version was set. 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. |
Skipping CI for Draft Pull Request. |
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: cheesesashimi 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 |
@cheesesashimi: This pull request references MCO-1505 which is a valid jira issue. Warning: The referenced jira issue has an invalid target version for the target branch this PR targets: expected the story to target the "4.19.0" version, but no target version was set. 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. |
ceb45c6
to
ee2b811
Compare
ee2b811
to
bee5f07
Compare
@cheesesashimi: 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. |
- What I did
To allow resource requests and limits to be set for on-cluster layering (OCL), I added the capability of setting these values via an annotation on the MachineOSConfig object. Additionally, in the absence of these annotations being set, we now have a default resource request set that mirrors the resource requests of the rest of the MCO's components.
- How to verify it
machineconfiguration.openshift.io/cpu-request
machineconfiguration.openshift.io/cpu-limit
machineconfiguration.openshift.io/memory-request
machineconfiguration.openshift.io/memory-limit
machineconfiguration.openshift.io/storage-request
machineconfiguration.openshift.io/storage-limit
machineconfiguration.openshift.io/ephemeral-storage-request
machineconfiguration.openshift.io/ephemeral-storage-limit
In the absence of those values being set, CPU should be set to
20m
and memory should be set to50Mi
. The default resource request values are the same as the ones applied to the various MCO components.- Description for the changelog
Allows user-configurable resource requests and limits via MachineOSConfig annotations