Skip to content

Adds documentation for limiting identity-relevant labels used when evaluating Cilium Identities#12517

Merged
qmonnet merged 2 commits intocilium:masterfrom
seanmwinn:cilium-11540
Jul 15, 2020
Merged

Adds documentation for limiting identity-relevant labels used when evaluating Cilium Identities#12517
qmonnet merged 2 commits intocilium:masterfrom
seanmwinn:cilium-11540

Conversation

@seanmwinn
Copy link
Copy Markdown
Contributor

Adds documentation for limiting identity-relevant labels used when evaluating Cilium Identities
Fixes: #11540
Signed-off-by: Sean Winn sean@isovalent.com

Fixes: #11540
Signed-off-by: Sean Winn <sean@isovalent.com>
@seanmwinn seanmwinn requested a review from a team as a code owner July 13, 2020 18:27
@maintainer-s-little-helper maintainer-s-little-helper Bot added the dont-merge/needs-release-note-label The author needs to describe the release impact of these changes. label Jul 13, 2020
@seanmwinn seanmwinn added the release-note/misc This PR makes changes that have no direct user impact. label Jul 13, 2020
@maintainer-s-little-helper maintainer-s-little-helper Bot removed the dont-merge/needs-release-note-label The author needs to describe the release impact of these changes. label Jul 13, 2020
@coveralls
Copy link
Copy Markdown

coveralls commented Jul 13, 2020

Coverage Status

Coverage decreased (-0.003%) to 37.002% when pulling a1a8e23 on seanmwinn:cilium-11540 into b0610a4 on cilium:master.

@pchaigno pchaigno added the area/documentation Impacts the documentation, including textual changes, sphinx, or other doc generation code. label Jul 13, 2020
Copy link
Copy Markdown
Member

@pchaigno pchaigno left a comment

Choose a reason for hiding this comment

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

Looks great! Mostly nits below from me :-)

Comment thread Documentation/gettingstarted/index.rst Outdated
Comment thread Documentation/gettingstarted/identity-relevant-labels.rst Outdated
Comment thread Documentation/gettingstarted/identity-relevant-labels.rst Outdated
Comment thread Documentation/gettingstarted/identity-relevant-labels.rst Outdated
Comment thread Documentation/gettingstarted/identity-relevant-labels.rst Outdated
Comment thread Documentation/gettingstarted/identity-relevant-labels.rst Outdated
Comment thread Documentation/gettingstarted/identity-relevant-labels.rst
Comment thread Documentation/gettingstarted/identity-relevant-labels.rst Outdated
Comment thread Documentation/gettingstarted/identity-relevant-labels.rst Outdated
Signed-off-by: Sean Winn <sean@isovalent.com>
@qmonnet qmonnet merged commit a5fe80e into cilium:master Jul 15, 2020
@qmonnet
Copy link
Copy Markdown
Member

qmonnet commented Jul 15, 2020

Seeing that this is marked for backports, but that Documentation/concepts/scalability/index.rst does not exist in v1.6/v1.7, where would you like to include the file?

@christarazi
Copy link
Copy Markdown
Member

This PR has two commits which should have been squashed into one. We should look out for things like this next time before merging. :)

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

Labels

area/documentation Impacts the documentation, including textual changes, sphinx, or other doc generation code. release-note/misc This PR makes changes that have no direct user impact.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Docs: Provide description/example of using non-default "labels"

7 participants