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

fix(docs): bandwidth-manager install error #17338

Merged
merged 1 commit into from Sep 9, 2021
Merged

Conversation

withlin
Copy link
Contributor

@withlin withlin commented Sep 8, 2021

fix(docs): bandwidth-manager install error

Signed-off-by: JinLin Fu withlin@apache.org

@withlin withlin requested a review from a team September 8, 2021 09:28
@withlin withlin requested a review from a team as a code owner September 8, 2021 09:28
@maintainer-s-little-helper
Copy link

Commit 91ee7b6fefd332e3598b341dc46a38aaa420c457 does not contain "Signed-off-by".

Please follow instructions provided in https://docs.cilium.io/en/stable/contributing/development/contributing_guide/#developer-s-certificate-of-origin

@maintainer-s-little-helper maintainer-s-little-helper bot added dont-merge/needs-sign-off The author needs to add signoff to their commits before merge. dont-merge/needs-release-note-label The author needs to describe the release impact of these changes. labels Sep 8, 2021
Copy link
Member

@borkmann borkmann left a comment

Choose a reason for hiding this comment

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

Thanks for spotting!

@joestringer joestringer added needs-backport/1.10 release-note/misc This PR makes changes that have no direct user impact. labels Sep 8, 2021
@maintainer-s-little-helper maintainer-s-little-helper bot removed dont-merge/needs-release-note-label The author needs to describe the release impact of these changes. labels Sep 8, 2021
@maintainer-s-little-helper maintainer-s-little-helper bot added this to Needs backport from master in 1.10.5 Sep 8, 2021
@joestringer
Copy link
Member

Thanks for the submission! To satisfy the bot, can you add the Signoff line from your PR into the commit message itself using git commit --amend and repush?

Signed-off-by: JinLin Fu withlin@apache.org
@maintainer-s-little-helper maintainer-s-little-helper bot removed the dont-merge/needs-sign-off The author needs to add signoff to their commits before merge. label Sep 9, 2021
@withlin
Copy link
Contributor Author

withlin commented Sep 9, 2021

Done

Copy link
Member

@joestringer joestringer left a comment

Choose a reason for hiding this comment

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

Thanks!

FYI in future if you need it, you can use git commit -s for --signoff and it will inject the signoff into the commit message based on your git.user configuration, then you can just configure it once and don't need to type it out each time.

@joestringer joestringer merged commit 712af8e into cilium:master Sep 9, 2021
@maintainer-s-little-helper maintainer-s-little-helper bot moved this from Needs backport from master to Backport pending to v1.10 in 1.10.5 Sep 14, 2021
@maintainer-s-little-helper maintainer-s-little-helper bot moved this from Backport pending to v1.10 to Backport done to v1.10 in 1.10.5 Oct 13, 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.10.5
Backport done to v1.10
Development

Successfully merging this pull request may close these issues.

None yet

6 participants