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

Add JUMO to active Cilium users #18626

Merged
merged 1 commit into from
Mar 1, 2022
Merged

Add JUMO to active Cilium users #18626

merged 1 commit into from
Mar 1, 2022

Conversation

mantoine96
Copy link
Contributor

@mantoine96 mantoine96 commented Jan 26, 2022 •
edited
Loading

Add JUMO to the list of Cilium users.

Please let me know if I need to add anything else to my PR :) Thanks!

@mantoine96 mantoine96 requested a review from a team as a code owner January 26, 2022 09:54
@maintainer-s-little-helper maintainer-s-little-helper bot added the dont-merge/needs-release-note-label The author needs to describe the release impact of these changes. label Jan 26, 2022
@sayboras sayboras added the release-note/misc This PR makes changes that have no direct user impact. label Jan 28, 2022
@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 Jan 28, 2022
pchaigno
pchaigno reviewed Feb 27, 2022
View reviewed changes
Copy link
Member

@pchaigno pchaigno 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 adding JUMO to the list! It seems this pull request fell through the cracks. I'm sorry for the delay in reviewing.

I have only one minor comment below and we should be good to merge.

USERS.md Outdated Show resolved Hide resolved
@maintainer-s-little-helper

This comment was marked as resolved.

Sign in to view
@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 Feb 28, 2022
@mantoine96 mantoine96 requested a review from pchaigno February 28, 2022 14:25
@maintainer-s-little-helper

This comment was marked as resolved.

Sign in to view
@mantoine96
Add JUMO to active Cilium users
a9d1c70 
Signed-off-by: Matthieu Antoine <matthieu.antoine@jumo.world>
@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 Feb 28, 2022
@pchaigno pchaigno added the ready-to-merge This PR has passed all tests and received consensus from code owners to merge. label Feb 28, 2022
@borkmann borkmann merged commit 33bbc08 into cilium:master Mar 1, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@pchaigno pchaigno pchaigno approved these changes

Assignees
No one assigned
Labels
ready-to-merge This PR has passed all tests and received consensus from code owners to merge. release-note/misc This PR makes changes that have no direct user impact.
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@mantoine96 @pchaigno @borkmann @sayboras