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

[opentelemetry-collector] add service.loadbalancerIP #551

Merged
merged 5 commits into from
Dec 14, 2022

Conversation

Stelminator
Copy link
Contributor

@Stelminator Stelminator commented Dec 9, 2022

I know it's deprecated because it doesn't support dualstack, but, that doesn't mean it doesn't still work, and afaik, there isn't yet an alternative.

kubernetes/kubernetes#107235

I'm targeting GKE, and have confirmed that this works as expected in that environment for IPV4 reserved external regional addresses.

@Stelminator Stelminator marked this pull request as ready for review December 9, 2022 03:31
@puckpuck puckpuck added the chart:collector Issue related to opentelemetry-collector helm chart label Dec 11, 2022
@Stelminator Stelminator force-pushed the loadbalancerIP branch 2 times, most recently from ce1f8b7 to f5db55b Compare December 12, 2022 21:53
@Stelminator Stelminator changed the title add service.loadbalancerIP [opentelemetry-collector] add service.loadbalancerIP Dec 12, 2022
@puckpuck
Copy link
Contributor

Can you add a commented entry for this in values.yaml with how to use it?

@Stelminator
Copy link
Contributor Author

@puckpuck

Can you add a commented entry for this in values.yaml with how to use it?

is this about what you had in mind? 035fce9

  # type: LoadBalancer
  # loadBalancerIP: 1.2.3.4

@TylerHelmuth TylerHelmuth merged commit a97f7f4 into open-telemetry:main Dec 14, 2022
JaredTan95 pushed a commit to JaredTan95/opentelemetry-helm-charts that referenced this pull request Jan 19, 2023
)

* add service.loadBalancerIP

* example

* bump chart version

* make generate-examples CHARTS=opentelemetry-collector

Co-authored-by: Tyler Helmuth <12352919+TylerHelmuth@users.noreply.github.com>
moh-osman3 pushed a commit to moh-osman3/opentelemetry-helm-charts that referenced this pull request Apr 12, 2023
)

* add service.loadBalancerIP

* example

* bump chart version

* make generate-examples CHARTS=opentelemetry-collector

Co-authored-by: Tyler Helmuth <12352919+TylerHelmuth@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
chart:collector Issue related to opentelemetry-collector helm chart
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants