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
Bump node-problem-detector to v0.8.5 #96716
Conversation
@tosi3k: This issue is currently awaiting triage. If a SIG or subproject determines this is a relevant issue, they will accept it by applying the The 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. |
kubernetes.io/cluster-service: "true" | ||
spec: | ||
containers: | ||
- name: node-problem-detector | ||
image: k8s.gcr.io/node-problem-detector:v0.8.0 | ||
image: k8s.gcr.io/node-problem-detector:v0.8.5 |
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.
It seems this image (or k8s.gcr.io/node-problem-detector/node-problem-detector:v0.8.5
) is not yet available.
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.
Yup, that's why I put this PR on hold in the first place :).
/test pull-kubernetes-node-crio-e2e |
@tosi3k - please open k/k issue for this one - this is a bug and I want this to land in 1.20 (and we should then cherrypick back) |
Done - I updated the PR's description with the URL to the issue. |
@tosi3k - was the image already built? |
Not yet, unfortunately :/. I will unhold the PR (and rerun the tests) once it's released. |
v0.8.5 is released:
/hold cancel |
/retest |
a69132b
to
8f2dd3a
Compare
/test pull-kubernetes-e2e-kind-ipv6 |
@wojtek-t PTAL. |
/lgtm Adding to milestone as this is fixing a bug. |
@tosi3k - once this merges, please cherrypick to previous releases |
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: kemooo3333, tosi3k, wojtek-t 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 |
…96716-upstream-release-1.17 Automated cherry pick of #96716: Bump node-problem-detector to v0.8.5
…96716-upstream-release-1.19 Automated cherry pick of #96716: Bump node-problem-detector to v0.8.5
…96716-upstream-release-1.18 Automated cherry pick of #96716: Bump node-problem-detector to v0.8.5
What type of PR is this?
/kind feature
What this PR does / why we need it:
NPD v0.8.0 doesn't detect OOMs in newer Linux kernels. v0.8.5 contains a fix that enables the NPD to detect them in both older and newer kernels.
Which issue(s) this PR fixes:
Fixes #96746.
Special notes for your reviewer:
Holding since the Docker image for NPD v0.8.5 is not released yet.
/hold
/sig scalability
/assign @wojtek-t
SHA512 digest of the v0.8.5 tar package:
Does this PR introduce a user-facing change?: