Skip to content

Conversation

gandro
Copy link
Member

@gandro gandro commented Sep 29, 2020

v1.7 backports 2020-09-29

Once this PR is merged, you can update the PR labels via:

$ for pr in 13228; do contrib/backporting/set-labels.py $pr done 1.7; done

This backport slightly modifies the upstream commit to avoid issues with the unit tests in v1.7

@gandro gandro requested review from a team as code owners September 29, 2020 09:14
@maintainer-s-little-helper maintainer-s-little-helper bot added backport/1.8 kind/backports This PR provides functionality previously merged into master. labels Sep 29, 2020
@gandro gandro changed the base branch from v1.8 to v1.7 September 29, 2020 09:14
@gandro
Copy link
Member Author

gandro commented Sep 29, 2020

test-backport-1.7

Edit: Travis CI was somehow stuck (probably because I messed up the base branch originally), force-pushed to re-trigger Travis.

[ upstream commit f3424a3 ]

Reserved and CIDR identities are local to the agent and not stored in
the kvstore. This commit changes the identity cache to avoid performing
a kvstore lookup for CIDR entries (which is currently done when a CIDR
identity is released).

This is a follow-up to
#13205 (comment)

Signed-off-by: Sebastian Wicki <sebastian@isovalent.com>
@gandro gandro force-pushed the pr/v1.7-backport-2020-09-29-13228 branch from 0b588f6 to d27764b Compare September 29, 2020 13:59
@gandro
Copy link
Member Author

gandro commented Sep 29, 2020

test-backport-1.7

Edit: Failures seem unrelated
https://jenkins.cilium.io/job/Cilium-PR-Ginkgo-Tests-K8s/3555/

@gandro
Copy link
Member Author

gandro commented Sep 29, 2020

test-missed-k8s

@christarazi christarazi merged commit fb10a31 into v1.7 Sep 29, 2020
@christarazi christarazi deleted the pr/v1.7-backport-2020-09-29-13228 branch September 29, 2020 21:31
@joestringer
Copy link
Member

@gandro Agree, they are unrelated. I dug a bit further on the K8sPolicyTest GuestBook Examples checks policy example and found evidence of #12795.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
kind/backports This PR provides functionality previously merged into master.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants