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

renovate: Add explicit gitAuthor #24739

Merged
merged 1 commit into from Apr 5, 2023

Conversation

gandro
Copy link
Member

@gandro gandro commented Apr 4, 2023

This commit adds an explicit gitAuthor to Renovate. The intent behind this is that it will match the author line in the Signed-off-by field:

https://docs.renovatebot.com/presets-default/#gitsignoff

This fixes an issue where previously checkpatch failed with the following error:

Error: ERROR:NO_AUTHOR_SIGN_OFF: Missing Signed-off-by: line by nominal patch author '"renovate[bot]" <29139614+renovate[bot]@users.noreply.github.com>'

Manually tested here:

⚠️ Note that this changes the Renovate git commit author, meaning Renovate updates before and after this PR is merged will have a different commit author (old: renovate[bot]" <29139614+renovate[bot]@users.noreply.github.com>, new: Renovate Bot <bot@renovateapp.com>

@gandro gandro added the release-note/ci This PR makes changes to the CI. label Apr 4, 2023
@gandro gandro requested a review from a team as a code owner April 4, 2023 16:01
@gandro gandro requested review from bimmlerd and aanm April 4, 2023 16:01
Copy link
Member

@bimmlerd bimmlerd left a comment

Choose a reason for hiding this comment

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

If the name change is an issue, I think we can use gitmailmap to remap?

This commit adds an explicit gitAuthor to Renovate. The intent behind
this is that it will match the author line in the `Signed-off-by` field:

 https://docs.renovatebot.com/presets-default/#gitsignoff

This fixes an issue where previously checkpatch failed with the
following error:
```
Error: ERROR:NO_AUTHOR_SIGN_OFF: Missing Signed-off-by: line by nominal patch author '"renovate[bot]" <29139614+renovate[bot]@users.noreply.github.com>'
```

Signed-off-by: Sebastian Wicki <sebastian@isovalent.com>
@gandro gandro force-pushed the pr/gandro/fix-renovate-signoff branch from 77a8d00 to 534d906 Compare April 5, 2023 08:36
@gandro gandro requested a review from a team as a code owner April 5, 2023 08:36
@gandro
Copy link
Member Author

gandro commented Apr 5, 2023

If the name change is an issue, I think we can use gitmailmap to remap?

Thanks! I added a mailmap entry (and changed the Renovate config to keep using the old renovate[bot] name, which I think also helps in keeping things consistent)

@squeed squeed merged commit e0f7fac into cilium:master Apr 5, 2023
40 of 42 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
release-note/ci This PR makes changes to the CI.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants