Skip to content

pkg/option: disable K8sEventHandover if Cilium is running without KVStore#12146

Merged
aanm merged 1 commit intocilium:masterfrom
aanm:pr/fix-policy-delete
Jun 18, 2020
Merged

pkg/option: disable K8sEventHandover if Cilium is running without KVStore#12146
aanm merged 1 commit intocilium:masterfrom
aanm:pr/fix-policy-delete

Conversation

@aanm
Copy link
Copy Markdown
Member

@aanm aanm commented Jun 17, 2020

Running Cilium with K8sEventHandover and without a KVStore configured
causes it on not being able to delete any CNP after being created.

Fixes: e4e83e8 ("daemon: Allow kvstore to be unconfigured")
Signed-off-by: André Martins andre@cilium.io

stop Cilium from hanging on CNP or CCNP events from Kubernetes if running with 'k8s-event-handover=true' and 'kvstore=""'

…tore

Running Cilium with K8sEventHandover and without a KVStore configured
causes it on not being able to delete any CNP after being created.

Fixes: e4e83e8 ("daemon: Allow kvstore to be unconfigured")
Signed-off-by: André Martins <andre@cilium.io>
@aanm aanm added kind/bug This is a bug in the Cilium logic. release-note/bug This PR fixes an issue in a previous release of Cilium. needs-backport/1.6 labels Jun 17, 2020
@aanm aanm requested a review from a team June 17, 2020 16:44
@aanm aanm requested a review from a team as a code owner June 17, 2020 16:44
@aanm
Copy link
Copy Markdown
Member Author

aanm commented Jun 17, 2020

test-me-please

@coveralls
Copy link
Copy Markdown

Coverage Status

Coverage decreased (-0.008%) to 37.064% when pulling 617582a on aanm:pr/fix-policy-delete into 989eced on cilium:master.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

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

None yet

Development

Successfully merging this pull request may close these issues.

7 participants