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.

Sign up for GitHub

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

Jump to bottom

.github, bpf: Update reference to cilium-checkpatch image #14700

Merged
merged 1 commit into from
Jan 25, 2021
Merged

.github, bpf: Update reference to cilium-checkpatch image #14700

merged 1 commit into from
Jan 25, 2021

Conversation

pchaigno
Copy link
Member

Update the cilium-checkpatch image used in our GitHub action to avoid the Co-authored-by false positive.

Related: cilium/image-tools#97

@pchaigno
.github, bpf: Update reference to cilium-checkpatch image
9830ad7 
Signed-off-by: Paul Chaignon <paul@cilium.io>
@pchaigno pchaigno added area/CI Continuous Integration testing issue or flake release-note/ci This PR makes changes to the CI. sig/contributing Impacts contribution workflow, guidelines, and tools. labels Jan 22, 2021
@pchaigno pchaigno requested review from qmonnet and a team January 22, 2021 16:56
@pchaigno pchaigno requested review from a team as code owners January 22, 2021 16:56
@maintainer-s-little-helper maintainer-s-little-helper bot added this to In progress in 1.10.0 Jan 22, 2021
qmonnet
qmonnet approved these changes Jan 22, 2021
View reviewed changes
Copy link
Member

@qmonnet qmonnet left a comment

Choose a reason for hiding this comment

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

Thank you!

@qmonnet qmonnet removed their assignment Jan 22, 2021
@pchaigno pchaigno added the ready-to-merge This PR has passed all tests and received consensus from code owners to merge. label Jan 23, 2021
@aanm aanm merged commit bceadb0 into master Jan 25, 2021
@aanm aanm deleted the pr/pchaigno/update-cilium-checkpatch branch January 25, 2021 09:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@borkmann borkmann borkmann approved these changes

@christarazi christarazi christarazi approved these changes

@qmonnet qmonnet qmonnet approved these changes

Assignees
No one assigned
Labels
area/CI Continuous Integration testing issue or flake ready-to-merge This PR has passed all tests and received consensus from code owners to merge. release-note/ci This PR makes changes to the CI. sig/contributing Impacts contribution workflow, guidelines, and tools.
Projects
No open projects
1.10.0
Done
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
6 participants
@pchaigno @borkmann @christarazi @qmonnet @aanm @michi-covalent