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
Add kernelType to MachineConfig #1330
Add kernelType to MachineConfig #1330
Conversation
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.
@sinnykumari is there any background on this change?
Also I would do an e2e on this when you are finished.
83f571d
to
b24edd0
Compare
b24edd0
to
e3cbdd5
Compare
We should provide the real time kernel install and update pathes until the openshift/machine-config-operator#1330 will be merged. Signed-off-by: Artyom Lukianov <alukiano@redhat.com>
We should provide the real time kernel install and update pathes until the openshift/machine-config-operator#1330 will be merged. Signed-off-by: Artyom Lukianov <alukiano@redhat.com>
e3cbdd5
to
1eacdea
Compare
1eacdea
to
18d108e
Compare
18d108e
to
5db5a59
Compare
There were lot of changes since the original implementation were made. To avoid confusion and get a fresh review, have done force push. This should be good for review now. |
just a super minor nit but the rest looks super sane will let others to review again before getting this in |
templates/common/_base/units/machine-config-daemon-host.service
Outdated
Show resolved
Hide resolved
Looking at this, I can definitely see the cost to the MCO of shipping kernel-rt the way we are; it would be simplest with two |
Also, while updating the RHCOS machine with realtime kernel, check if new kernel-rt packages are available. Update rt-kernel on host only when OS update contains updated kernel-rt.
e048de0
to
36ccb1c
Compare
Retrying as it says /test e2e-aws |
/skip |
1 similar comment
/skip |
b74cd5c
to
67f3f11
Compare
I moved following rpm-ostree command execution back in the respective if condition so that we never accidentally run it.
|
67f3f11
to
d9df159
Compare
/skip |
d9df159
to
8ee18d9
Compare
op e2e failing for the usual flakes
|
/retest e2e-gcp-op |
/test e2e-gcp-op |
/retest |
/lgtm |
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: runcom, sinnykumari 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 |
@sinnykumari: The following test failed, say
Full PR test history. Your PR dashboard. Please help us cut down on flakes by linking to an open issue when you hit one in your PR. 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. |
/skip |
By default we use traditional kernel on RHCOS system. With this,
one can create a MachineConfig specifying
kernelType: realtime
.Once this MachineConfig is applied to OpenShift cluster,
targetted RHCOS node should switch to using realtime kernel.
Currently, realtime kernel packages are shipped as rpm files in
RHCOS OSContainer. MCO fetches those rpms from OSContainer and
uses rpm-ostree to layer realtime kernel on host.