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.

Sign up for GitHub

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

Jump to bottom

kafka: Use policy identity cache to lookup identity for L3 dependant rules #2813

Merged
merged 1 commit into from
Feb 15, 2018
Merged

kafka: Use policy identity cache to lookup identity for L3 dependant rules #2813

merged 1 commit into from
Feb 15, 2018

Conversation

manalibhutiyani
Copy link

@manalibhutiyani manalibhutiyani commented Feb 14, 2018 •
edited

kafka: Use policy identity cache to lookup identity for L3 dependant rules
Fixes Issue: #2824

kafka: Use policy identity cache to lookup identity for L3 dependant rules

@manalibhutiyani manalibhutiyani requested a review from a team as a code owner February 14, 2018 02:16
@manalibhutiyani
Copy link
Author

test-me-please

@manalibhutiyani manalibhutiyani requested a review from a team as a code owner February 14, 2018 22:35
@manalibhutiyani manalibhutiyani requested a review from a team February 14, 2018 22:35
@manalibhutiyani manalibhutiyani changed the title TEST branch: DO NOT REVIEW kafka learning policy kafka: Use policy identity cache to lookup identity for L3 dependant rules Feb 15, 2018
@manalibhutiyani manalibhutiyani added the release-note/minor This PR changes functionality that users may find relevant to operating Cilium. label Feb 15, 2018
@tgraf
kafka: Use policy identity cache to lookup identity for L3 dependant …
e8bea0f 
…rules

Fixes Issue: #2824

Signed-off-by: Thomas Graf <thomas@cilium.io>
Signed-off-by: Manali Bhutiyani <manali@covalent.io>
@tgraf
Copy link
Member

tgraf commented Feb 15, 2018

Excellent catch @manalibhutiyani 🥇

@tgraf
Copy link
Member

tgraf commented Feb 15, 2018

test-me-please

@tgraf tgraf 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. area/proxy Impacts proxy components, including DNS, Kafka, Envoy and/or XDS servers. stable/needs-backport and removed release-note/minor This PR changes functionality that users may find relevant to operating Cilium. labels Feb 15, 2018
@tgraf tgraf merged commit 4566a55 into master Feb 15, 2018
@tgraf tgraf deleted the learning-policy-kafka branch February 15, 2018 07:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@rlenglet rlenglet rlenglet approved these changes

@tgraf tgraf tgraf approved these changes

Assignees
No one assigned
Labels
area/proxy Impacts proxy components, including DNS, Kafka, Envoy and/or XDS servers. 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
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
3 participants
@manalibhutiyani @tgraf @rlenglet