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

cni: Fix noisy warning "Unknown CNI chaining configuration" #9937

Merged
merged 1 commit into from
Jan 22, 2020

Conversation

tgraf
Copy link
Member

@tgraf tgraf commented Jan 22, 2020

The following warning has been printed for every CNI ADD and CNI DEL:

Unknown CNI chaining configuration name 'cilium'

Prevent a chaining plugin to use the name "cilium" and only perform a chaining
plugin lookup if the configuration is different than the default name "cilium".


This change is Reviewable

The following warning has been printed for every CNI ADD and CNI DEL:

    Unknown CNI chaining configuration name 'cilium'

Prevent a chaining plugin to use the name "cilium" and only perform a chaining
plugin lookup if the configuration is different than the default name "cilium".

Signed-off-by: Thomas Graf <thomas@cilium.io>
@tgraf tgraf added kind/bug This is a bug in the Cilium logic. pending-review area/cni Impacts the Container Networking Interface between Cilium and the orchestrator. release-note/bug This PR fixes an issue in a previous release of Cilium. labels Jan 22, 2020
@tgraf tgraf requested a review from a team January 22, 2020 16:45
@maintainer-s-little-helper maintainer-s-little-helper bot added this to In progress in 1.7.0 Jan 22, 2020
@maintainer-s-little-helper maintainer-s-little-helper bot added this to Needs backport from master in 1.6.6 Jan 22, 2020
@tgraf
Copy link
Member Author

tgraf commented Jan 22, 2020

test-me-please

@aanm aanm added this to the 1.7 milestone Jan 22, 2020
@aanm aanm merged commit 9718de7 into master Jan 22, 2020
1.7.0 automation moved this from In progress to Merged Jan 22, 2020
@aanm aanm deleted the pr/tgraf/avoid-cni-warning branch January 22, 2020 20:29
@maintainer-s-little-helper maintainer-s-little-helper bot moved this from Needs backport from master to Backport pending to v1.6 in 1.6.6 Jan 24, 2020
@maintainer-s-little-helper maintainer-s-little-helper bot moved this from Backport pending to v1.6 to Backport done to v1.6 in 1.6.6 Jan 24, 2020
@maintainer-s-little-helper maintainer-s-little-helper bot moved this from Backport pending to v1.6 to Backport done to v1.6 in 1.6.6 Jan 24, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/cni Impacts the Container Networking Interface between Cilium and the orchestrator. kind/bug This is a bug in the Cilium logic. release-note/bug This PR fixes an issue in a previous release of Cilium.
Projects
No open projects
1.6.6
Backport done to v1.6
1.7.0
  
Merged
Development

Successfully merging this pull request may close these issues.

None yet

3 participants