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 warning log for list IPV6 address: move IPV4 to IPv6. #16475

Merged
merged 1 commit into from Jun 16, 2021
Merged

fix warning log for list IPV6 address: move IPV4 to IPv6. #16475

merged 1 commit into from Jun 16, 2021

Conversation

lic17
Copy link
Contributor

@lic17 lic17 commented Jun 8, 2021

fix a small error for a log.
to list IPV6 address, but print the log is list IPv4.

@lic17 lic17 requested review from a team and borkmann June 8, 2021 17:01
@maintainer-s-little-helper
Copy link

Commit 424c77a53ae53e384aff26287eb4067b3afa6ded 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 Jun 8, 2021
@lic17
Copy link
Contributor Author

lic17 commented Jun 8, 2021

@jrfastab I create a new PR for the PR #16294(fix warning log for list IPV6 address: move IPV4 to IPv6 )

@lic17 lic17 changed the title fix warning log for list IPV6 address: move IPV4 to IPv6 fix warning log for list IPV6 address: move IPV4 to IPv6. Signed-off-by: Li Cheng 1076366950@qq.com Jun 8, 2021
@lic17 lic17 changed the title fix warning log for list IPV6 address: move IPV4 to IPv6. Signed-off-by: Li Cheng 1076366950@qq.com fix warning log for list IPV6 address: move IPV4 to IPv6. Jun 8, 2021
Signed-off-by: Li Cheng 1076366950@qq.com
@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 Jun 8, 2021
@borkmann borkmann added the release-note/misc This PR makes changes that have no direct user impact. label Jun 11, 2021
@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 Jun 11, 2021
@borkmann borkmann added the area/daemon Impacts operation of the Cilium daemon. label Jun 11, 2021
@borkmann
Copy link
Member

test-me-please

@aanm aanm merged commit 3a55d74 into cilium:master Jun 16, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/daemon Impacts operation of the Cilium daemon. 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.

None yet

3 participants