Skip to content

Conversation

darox
Copy link
Contributor

@darox darox commented Sep 19, 2024

This PR adds LoadBalancer as option in the coment section for the service type of Hubble relay.

@darox darox requested review from a team as code owners September 19, 2024 08: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 Sep 19, 2024
@darox darox force-pushed the add-loadbalancer-service-type-hubble-relay branch from 3d5c41b to 08aae88 Compare September 19, 2024 09:07
@darox darox force-pushed the add-loadbalancer-service-type-hubble-relay branch from 08aae88 to cea2bfe Compare September 19, 2024 09:41
@brlbil
Copy link
Contributor

brlbil commented Sep 25, 2024

/test

@darox
Copy link
Contributor Author

darox commented Oct 4, 2024

can we get it merged? @brlbil

@brlbil brlbil added the release-note/misc This PR makes changes that have no direct user impact. label Oct 4, 2024
@maintainer-s-little-helper maintainer-s-little-helper bot added ready-to-merge This PR has passed all tests and received consensus from code owners to merge. and removed dont-merge/needs-release-note-label The author needs to describe the release impact of these changes. labels Oct 4, 2024
@brlbil
Copy link
Contributor

brlbil commented Oct 4, 2024

@darox The PR was missing the release label. I have added one and it is now ready to merge.

@darox
Copy link
Contributor Author

darox commented Oct 4, 2024

@darox The PR was missing the release label. I have added one and it is now ready to merge.

thank you for the swift response.

@julianwiedmann julianwiedmann added this pull request to the merge queue Oct 4, 2024
Merged via the queue into cilium:main with commit 3d0e548 Oct 4, 2024
63 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
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
Development

Successfully merging this pull request may close these issues.

4 participants