Skip to content
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

Fixes for 1.4.1 #331

Merged
merged 1 commit into from Jul 7, 2023
Merged

Fixes for 1.4.1 #331

merged 1 commit into from Jul 7, 2023

Conversation

gkurz
Copy link
Member

@gkurz gkurz commented Jul 5, 2023

This applies selected fixes from the devel branch to main for the 1.4.1 release :

The DaemonSet launched by the controller doesn't put tolerations on the
monitor pods. This prevents the monitor pods to run on tainted nodes and
break metrics for sandboxed containers.

The monitor pods must run on every node where kata is deployed, no
matter any taint the user might have set. Enforce a toleration that
matches all possible taints.

Fixes https://issues.redhat.com/browse/KATA-2121

Signed-off-by: Greg Kurz <groug@kaod.org>
(cherry picked from commit 7f004df)
Signed-off-by: Greg Kurz <groug@kaod.org>
@openshift-ci openshift-ci bot requested review from jensfr and pmores July 5, 2023 13:19
@gkurz
Copy link
Member Author

gkurz commented Jul 5, 2023

/override ci/prow/check

@openshift-ci
Copy link

openshift-ci bot commented Jul 5, 2023

@gkurz: Overrode contexts on behalf of gkurz: ci/prow/check

In response to this:

/override ci/prow/check

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.

@openshift-ci
Copy link

openshift-ci bot commented Jul 5, 2023

@gkurz: The following test failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name Commit Details Required Rerun command
ci/prow/sandboxed-containers-operator-e2e d39747b link false /test sandboxed-containers-operator-e2e

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.

Copy link
Contributor

@jensfr jensfr left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm

@gkurz gkurz merged commit 1a660d8 into openshift:main Jul 7, 2023
3 of 4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants