Skip to content

Handle server side default for TrafficDistribution#8994

Merged
rhr323 merged 1 commit intoelastic:mainfrom
rhr323:ignore-trafficDistribution-for-serverside-values
Jan 12, 2026
Merged

Handle server side default for TrafficDistribution#8994
rhr323 merged 1 commit intoelastic:mainfrom
rhr323:ignore-trafficDistribution-for-serverside-values

Conversation

@rhr323
Copy link
Contributor

@rhr323 rhr323 commented Jan 7, 2026

This PR fixes a bug related to the TrafficDistribution field that was added to the Kubernetes Service spec (kubernetes/api@0a84c1b).

If the API server defaults TrafficDistribution to PreferClose on Services, this can cause issues since ECK doesn't currently preserve this server-side defaulted value in its reconciliation logic. If ECK removes it and the API server then adds it back, ECK could get into an infinite reconciliation loop.

I confirmed that manually patching the Service – ECK does remove the trafficDistribution field.

Fixes: #8812

@prodsecmachine
Copy link
Collaborator

prodsecmachine commented Jan 7, 2026

Snyk checks have passed. No issues have been found so far.

Status Scanner Critical High Medium Low Total (0)
Open Source Security 0 0 0 0 0 issues
Licenses 0 0 0 0 0 issues

💻 Catch issues earlier using the plugins for VS Code, JetBrains IDEs, Visual Studio, and Eclipse.

@botelastic botelastic bot added the triage label Jan 7, 2026
@rhr323 rhr323 added the >bug Something isn't working label Jan 8, 2026
@botelastic botelastic bot removed the triage label Jan 8, 2026
@rhr323 rhr323 requested a review from a team January 8, 2026 08:24
@rhr323
Copy link
Contributor Author

rhr323 commented Jan 8, 2026

buildkite test this -f p=kind -m s=9.2.0

@rhr323 rhr323 merged commit 88e50d2 into elastic:main Jan 12, 2026
10 checks passed
@rhr323 rhr323 deleted the ignore-trafficDistribution-for-serverside-values branch January 12, 2026 09:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

>bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Intermittent reconciliation run loop issue

3 participants