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

nfd-worker: disable sources more easily #670

Merged
merged 2 commits into from
Dec 2, 2021

Conversation

marquiz
Copy link
Contributor

@marquiz marquiz commented Nov 25, 2021

Make it easier to disable single sources by prefixing the source name
with a dash ('-') in the core.sources config option (or -sources cmdline
flag).

@k8s-ci-robot k8s-ci-robot added cncf-cla: yes Indicates the PR's author has signed the CNCF CLA. size/M Denotes a PR that changes 30-99 lines, ignoring generated files. labels Nov 25, 2021
@k8s-ci-robot k8s-ci-robot added the approved Indicates a PR has been approved by an approver from all required OWNERS files. label Nov 25, 2021
@ArangoGutierrez
Copy link
Contributor

@marquiz rebase 😁

@k8s-ci-robot k8s-ci-robot added the needs-rebase Indicates a PR cannot be merged because it has merge conflicts with HEAD. label Dec 1, 2021
@marquiz marquiz force-pushed the devel/sources-disable branch 2 times, most recently from bdc36ff to 2c3624e Compare December 2, 2021 08:30
@k8s-ci-robot k8s-ci-robot removed the needs-rebase Indicates a PR cannot be merged because it has merge conflicts with HEAD. label Dec 2, 2021
@marquiz
Copy link
Contributor Author

marquiz commented Dec 2, 2021

@marquiz rebase

Done!

I also added a separate commit (somewhat piggy-backing this PR) that fixes some previous mistakes in the documentation of core.labelSources config option

Make it easier to disable single sources by prefixing the source name
with a dash ('-') in the core.sources config option (or -sources cmdline
flag).
Copy link
Contributor

@ArangoGutierrez ArangoGutierrez left a comment

Choose a reason for hiding this comment

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

Tested
/lgtm

@k8s-ci-robot k8s-ci-robot added the lgtm "Looks good to me", indicates that a PR is ready to be merged. label Dec 2, 2021
@k8s-ci-robot
Copy link
Contributor

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: ArangoGutierrez, marquiz

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 /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@ArangoGutierrez
Copy link
Contributor

[eduardo@fedora node-feature-discovery]$ kubectl describe node    
Name:               minikube          
Roles:              control-plane,master         
Labels:             beta.kubernetes.io/arch=amd64
                    beta.kubernetes.io/os=linux
                    feature.node.kubernetes.io/kernel-config.NO_HZ=true
                    feature.node.kubernetes.io/kernel-config.NO_HZ_IDLE=true
                    feature.node.kubernetes.io/kernel-version.full=4.19.202
                    feature.node.kubernetes.io/kernel-version.major=4
                    feature.node.kubernetes.io/kernel-version.minor=19
                    feature.node.kubernetes.io/kernel-version.revision=202
                    kubernetes.io/arch=amd64                                                                                                                                                                                                  
                    kubernetes.io/hostname=minikube                                                                                                                                                                                           
                    kubernetes.io/os=linux                                                                                                                                                                                                    
                    minikube.k8s.io/commit=76b94fb3c4e8ac5062daf70d60cf03ddcc0a741b                                                                                                                                                           
                    minikube.k8s.io/name=minikube                                                                                                                                                                                             
                    minikube.k8s.io/updated_at=2021_12_02T11_35_49_0700                                                                                                                                                                       
                    minikube.k8s.io/version=v1.24.0                                                                                                                                                                                           
                    node-role.kubernetes.io/control-plane=                                                                                                                                                                                    
                    node-role.kubernetes.io/master=                                                                                                                                                                                           
                    node.kubernetes.io/exclude-from-external-load-balancers=                                                                                                                                                                  

tested as described in the documentation

@k8s-ci-robot k8s-ci-robot merged commit 14ab512 into kubernetes-sigs:master Dec 2, 2021
@marquiz marquiz deleted the devel/sources-disable branch December 2, 2021 17:53
@marquiz marquiz mentioned this pull request Dec 22, 2021
22 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
approved Indicates a PR has been approved by an approver from all required OWNERS files. cncf-cla: yes Indicates the PR's author has signed the CNCF CLA. lgtm "Looks good to me", indicates that a PR is ready to be merged. size/M Denotes a PR that changes 30-99 lines, ignoring generated files.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants