-
Notifications
You must be signed in to change notification settings - Fork 2.8k
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
Fix agent dashboard ConfigMap #28542
Conversation
Good catch :-) |
Can you please write a more descriptive release-node entry? Then we can merge this. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Two requests:
- Please write a more descriptive release-note. Include a short summary of what went wrong
- Add a "Fixes: (commit id") line in the commit message, pointing to the commit that introduced the bug. This makes it easier to "follow the chain". Thanks!
@squeed I changed it. Is it OK like this? |
@bakito Sorry, I wasn't specific enough -- the "fixes: xxxx" should be in the commit message (i.e. not the first line). Then GitHub automatically parses it and we can follow bugs up the chain. Fix that and we can merge this :-) |
6b64a4f
to
87e8714
Compare
/test |
fixes 9ec6e0d Signed-off-by: bakito <github@bakito.ch>
/test |
Please ensure your pull request adheres to the following guidelines:
description and a
Fixes: #XXX
line if the commit addresses a particularGitHub issue.
Fixes: <commit-id>
tag, thenplease add the commit author[s] as reviewer[s] to this issue.
This PR fixes the rendering of the cilium agent dashboard ConfigMap if enabled.