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

Helm-charts: Fix the values.yaml comment that was misleading #10515

Merged

Conversation

soumynathan
Copy link
Contributor

@soumynathan soumynathan commented Mar 9, 2020

This patch fixes the values.yaml comment for the 'ipvlan'
related key (primaryDevice instead of masterDevice) that mismatches
with the code and the documentation.

Singed-off-by: Swaminathan Vasudevan svasudevan@suse.com


This change is Reviewable

@soumynathan soumynathan requested a review from a team March 9, 2020 18:25
@maintainer-s-little-helper
Copy link

Commit ed65571c0cb2dc504bb28b40a9b60df64998af9b 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 the dont-merge/needs-sign-off The author needs to add signoff to their commits before merge. label Mar 9, 2020
@maintainer-s-little-helper
Copy link

Release note label not set, please set the appropriate release note.

1 similar comment
@maintainer-s-little-helper
Copy link

Release note label not set, please set the appropriate release note.

@joestringer
Copy link
Member

@soumynathan FYI you can use git commit -s to sign off your commits, and then it will add the last line for you. This avoids troublesome typos like what the maintainer's little helper is complaining about right now :)

@joestringer joestringer added the release-note/misc This PR makes changes that have no direct user impact. label Mar 9, 2020
This patch fixes the values.yaml comment for the 'ipvlan'
related key (primaryDevice instead of masterDevice) that mismatches
with the code and the documentation.

Signed-off-by: Swaminathan Vasudevan <svasudevan@suse.com>
@soumynathan soumynathan force-pushed the fix-default-values-comment-in-cilium branch from ed65571 to 99a2898 Compare March 9, 2020 18:30
@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 Mar 9, 2020
@joestringer joestringer requested review from borkmann and brb March 9, 2020 18:31
@coveralls
Copy link

coveralls commented Mar 9, 2020

Coverage Status

Coverage decreased (-0.01%) to 45.623% when pulling 99a2898 on soumynathan:fix-default-values-comment-in-cilium into 9a48426 on cilium:master.

@joestringer joestringer merged commit ed74481 into cilium:master Mar 9, 2020
1.8.0 automation moved this from In progress to Merged Mar 9, 2020
@maintainer-s-little-helper maintainer-s-little-helper bot added this to Needs backport from master in 1.7.2 Mar 9, 2020
@maintainer-s-little-helper maintainer-s-little-helper bot moved this from Needs backport from master to Backport pending to v1.7 in 1.7.2 Mar 16, 2020
@maintainer-s-little-helper maintainer-s-little-helper bot moved this from Backport pending to v1.7 to Backport done to v1.7 in 1.7.2 Mar 18, 2020
@maintainer-s-little-helper maintainer-s-little-helper bot moved this from Backport pending to v1.7 to Backport done to v1.7 in 1.7.2 Mar 18, 2020
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.7.2
Backport done to v1.7
1.8.0
  
Merged
Development

Successfully merging this pull request may close these issues.

None yet

5 participants