Closed chr15p closed 4 months ago
@chr15p: This pull request references Jira Issue OCPBUGS-33741, which is invalid:
Comment /jira refresh
to re-evaluate validity if changes to the Jira bug are made, or edit the title of this pull request to link to a different bug.
The bug has been updated to refer to the pull request using the external bug tracker.
[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: chr15p
The full list of commands accepted by this bot can be found here.
The pull request process is described here
Probably worth getting @yevgeny-shnaidman's review as well.
/jira refresh
@chr15p: This pull request references Jira Issue OCPBUGS-33741, which is valid. The bug has been moved to the POST state.
@chr15p, a couple of comments: 1) it is better to make that change in the upstream first and then backport it back to the downstream. 2) there is an inherent race condition with the current implementation: master will taint the node, which in turn means that there is a chance that a worker pod will not run (or won't be running a sufficient amout of time) on the node, which in turn means that the NFD labels will not be present on the node. I think that the solution should be updating the worker daemonset tolerations based on the presence of the Taints field in the CR
@chr15p: all tests passed!
Full PR test history. Your PR dashboard.
/lgtm
@chr15p: Jira Issue OCPBUGS-33741: All pull requests linked via external trackers have merged:
Jira Issue OCPBUGS-33741 has been moved to the MODIFIED state.
[ART PR BUILD NOTIFIER]
This PR has been included in build cluster-nfd-operator-container-v4.17.0-202406101017.p0.gb8c857c.assembly.stream.el9 for distgit cluster-nfd-operator. All builds following this will include this PR.
This adds a new option to the NodeFeatureDiscoverySpec
Spec.enableTaints
when this is set to true it will add the--enable-taints
flag to the master, allowing NFD to set taints as described in the upstream docsThis option is disabled by default, and not set in the sample configs, as taints support is experimental.