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

Validate static labels assigned to Kubernetes service #22701

Merged
merged 2 commits into from
Mar 8, 2023

Conversation

tigrato
Copy link
Contributor

@tigrato tigrato commented Mar 7, 2023

This PR introduces the static label's key validation to Kubernetes Service. Services with invalid labels failed to heartbeat their clusters, but the service kept working. This change forces the service to fail fast during startup if any key is invalid.

Fixes #22696

lib/kube/proxy/server.go Outdated Show resolved Hide resolved
This PR introduces the static label's key validation to Kubernetes Service.
Services with invalid labels fail to hearbeat their clusters but the
service is kept working. This change forces the service to fail fast
during startup if any key is invalid.

Fixes #22696
@tigrato tigrato force-pushed the tigrato/validate-invalid-keys branch from f2affd7 to eadbe9b Compare March 7, 2023 15:51
@public-teleport-github-review-bot public-teleport-github-review-bot bot removed the request for review from jakule March 7, 2023 16:44
@tigrato tigrato enabled auto-merge March 7, 2023 16:45
@tigrato tigrato added this pull request to the merge queue Mar 8, 2023
Merged via the queue into master with commit 1fd58c5 Mar 8, 2023
@public-teleport-github-review-bot

@tigrato See the table below for backport results.

Branch Result
branch/v10 Failed
branch/v11 Failed
branch/v12 Failed

tigrato added a commit that referenced this pull request Mar 8, 2023
This PR introduces the static label's key validation to Kubernetes Service.
Services with invalid labels fail to hearbeat their clusters but the
service is kept working. This change forces the service to fail fast
during startup if any key is invalid.

Fixes #22696
tigrato added a commit that referenced this pull request Mar 8, 2023
This PR introduces the static label's key validation to Kubernetes Service.
Services with invalid labels fail to hearbeat their clusters but the
service is kept working. This change forces the service to fail fast
during startup if any key is invalid.

Fixes #22696
tigrato added a commit that referenced this pull request Mar 8, 2023
This PR introduces the static label's key validation to Kubernetes Service.
Services with invalid labels fail to hearbeat their clusters but the
service is kept working. This change forces the service to fail fast
during startup if any key is invalid.

Fixes #22696
@public-teleport-github-review-bot

@tigrato See the table below for backport results.

Branch Result
branch/v10 Failed
branch/v11 Create PR
branch/v12 Create PR

tigrato added a commit that referenced this pull request Mar 9, 2023
This PR introduces the static label's key validation to Kubernetes Service.
Services with invalid labels fail to hearbeat their clusters but the
service is kept working. This change forces the service to fail fast
during startup if any key is invalid.

Fixes #22696
tigrato added a commit that referenced this pull request Mar 9, 2023
This PR introduces the static label's key validation to Kubernetes Service.
Services with invalid labels fail to hearbeat their clusters but the
service is kept working. This change forces the service to fail fast
during startup if any key is invalid.

Fixes #22696
tigrato added a commit that referenced this pull request Mar 9, 2023
This PR introduces the static label's key validation to Kubernetes Service.
Services with invalid labels fail to hearbeat their clusters but the
service is kept working. This change forces the service to fail fast
during startup if any key is invalid.

Fixes #22696
@tigrato tigrato deleted the tigrato/validate-invalid-keys branch June 1, 2023 18:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

kubernetes_service starts with invalid labels
3 participants