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

feat(load-balancer): Add new node-selector annotation #514

Merged
merged 8 commits into from
Sep 14, 2023

Conversation

xoxys
Copy link
Contributor

@xoxys xoxys commented Sep 14, 2023

Fixes: #506, #373

@xoxys xoxys requested a review from a team as a code owner September 14, 2023 10:18
@xoxys xoxys changed the title Add node selector Add new node-selector annotation Sep 14, 2023
Copy link
Member

@apricote apricote 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 very much @xoxys! This works very well in my tests, I only have some UX suggestions left :)

hcloud/load_balancers.go Outdated Show resolved Hide resolved
hcloud/load_balancers.go Outdated Show resolved Hide resolved
hcloud/load_balancers.go Outdated Show resolved Hide resolved
internal/annotation/load_balancer.go Outdated Show resolved Hide resolved
@apricote
Copy link
Member

Dont worry about the failing e2e tests. They are currently broken for external contributors. I ran them locally and everything passed :)

Co-authored-by: Julian Tölle <julian.toelle97@gmail.com>
@apricote apricote changed the title Add new node-selector annotation feat(load-balancer): Add new node-selector annotation Sep 14, 2023
xoxys and others added 3 commits September 14, 2023 15:52
Co-authored-by: Julian Tölle <julian.toelle97@gmail.com>
Co-authored-by: Julian Tölle <julian.toelle97@gmail.com>
Co-authored-by: Julian Tölle <julian.toelle97@gmail.com>
@xoxys
Copy link
Contributor Author

xoxys commented Sep 14, 2023

Thanks for the fast review! Also thanks for adding explanations to all suggestions, too often this is missing and contributor don't understand the suggestions or cant learn from it.

@xoxys xoxys requested a review from apricote September 14, 2023 13:54
@apricote apricote merged commit db2e6dc into hetznercloud:main Sep 14, 2023
apricote pushed a commit that referenced this pull request Sep 18, 2023
🤖 I have created a release *beep* *boop*
---


##
[1.18.0](v1.17.2...v1.18.0)
(2023-09-18)


### Features

* build with Go 1.21
([#516](#516))
([7bf7e71](7bf7e71))
* **chart:** configure additional tolerations
([#518](#518))
([0d25cb6](0d25cb6)),
closes
[#512](#512)
* **chart:** support running multiple replicas with leader election
([4b18ee5](4b18ee5))
* **load-balancer:** Add new node-selector annotation
([#514](#514))
([db2e6dc](db2e6dc))
* test against kubernetes v1.28 and drop v1.24
([#500](#500))
([3adf781](3adf781))

---
This PR was generated with [Release
Please](https://github.com/googleapis/release-please). See
[documentation](https://github.com/googleapis/release-please#release-please).
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

LoadBalancer targets too many nodes
2 participants