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

docs: Fix wording around labels configuration #14064

Merged
merged 1 commit into from
Nov 17, 2020

Conversation

joestringer
Copy link
Member

@joestringer joestringer commented Nov 17, 2020

ParseLabelPrefixCfg() appends the user-specified list to the user-specified file for labels. By default, the file is empty, which
readLabelPrefixCfgFrom() interprets as "use the default list". This means that by default, when the user specifies a set of labels, it will append to the default list rather than replacing it.

/cc @ArthurChiao

ParseLabelPrefixCfg() appends the user-specified list to the
user-specified file for labels. By default, the file is empty, which
readLabelPrefixCfgFrom() interprets as "use the default list". This
means that by default, when the user specifies a set of labels, it will
append to the default list rather than replacing it.

Reported-by: Arthur Chiao <arthurchiao@hotmail.com>
Signed-off-by: Joe Stringer <joe@cilium.io>
@joestringer joestringer added release-note/misc This PR makes changes that have no direct user impact. needs-backport/1.8 labels Nov 17, 2020
@joestringer joestringer requested a review from a team as a code owner November 17, 2020 17:31
@maintainer-s-little-helper maintainer-s-little-helper bot added this to In progress in 1.10.0 Nov 17, 2020
@maintainer-s-little-helper maintainer-s-little-helper bot added this to Needs backport from master in 1.8.6 Nov 17, 2020
@aanm aanm merged commit 461a09f into cilium:master Nov 17, 2020
@joestringer joestringer deleted the submit/labels-cfg-docs branch November 17, 2020 19:34
@joestringer joestringer added this to Needs backport from master in 1.9.1 Nov 17, 2020
@maintainer-s-little-helper maintainer-s-little-helper bot moved this from Needs backport from master to Backport pending to v1.8 in 1.8.6 Nov 19, 2020
@maintainer-s-little-helper maintainer-s-little-helper bot moved this from Needs backport from master to Backport pending to v1.8 in 1.8.6 Nov 19, 2020
@maintainer-s-little-helper maintainer-s-little-helper bot moved this from Backport pending to v1.8 to Backport done to v1.8 in 1.8.6 Nov 30, 2020
@maintainer-s-little-helper maintainer-s-little-helper bot moved this from Backport pending to v1.8 to Backport done to v1.8 in 1.8.6 Nov 30, 2020
@aanm aanm mentioned this pull request Dec 4, 2020
@aanm aanm added this to Needs backport from master in 1.9.2 Dec 4, 2020
@aanm aanm removed this from Needs backport from master in 1.9.1 Dec 4, 2020
@aanm aanm mentioned this pull request Dec 4, 2020
@joestringer joestringer removed this from Needs backport from master in 1.9.2 Jan 18, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
release-note/misc This PR makes changes that have no direct user impact.
Projects
No open projects
1.8.6
Backport done to v1.8
Development

Successfully merging this pull request may close these issues.

None yet

4 participants