-
Notifications
You must be signed in to change notification settings - Fork 6.5k
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
Update node-feature-discovery to v0.16.4 #11250
Update node-feature-discovery to v0.16.4 #11250
Conversation
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: mzaian 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 |
/retest-failed |
47e8d1c
to
fd63ff8
Compare
fd63ff8
to
4fe7b92
Compare
/ok-to-test |
/lgtm |
4fe7b92
to
676ba15
Compare
676ba15
to
7701fda
Compare
I think this PR needs a rebase to fix CI. |
7701fda
to
f6ae200
Compare
f6ae200
to
3ea519e
Compare
/retitle Update node-feature-discovery to v0.16.4 |
@mzaian Node Feature Discovery has changes in the readiness and liveness probe detection method, which needs to be fixed. Kubespray version: kubespray/roles/kubernetes-apps/node_feature_discovery/templates/nfd-master.yaml.j2 Lines 35 to 49 in 1093c76
Node Feature Discovery version: https://github.com/kubernetes-sigs/node-feature-discovery/blob/a851aae9142e44b0df261f3db63cd0fb31c04fdf/deployment/base/master/master-deployment.yaml#L31-L41 |
3ea519e
to
d418069
Compare
CI passed. /lgtm |
What type of PR is this?
/kind feature
What this PR does / why we need it:
This PR update node-feature-discovery to v0.16.4
Update api crds & clusterrole
Does this PR introduce a user-facing change?: