Skip to content

[v18] Add status field to Kubernetes clusters to support discovery workflows#62598

Merged
tigrato merged 2 commits intobranch/v18from
tigrato/eksautocleanupv18
Jan 7, 2026
Merged

[v18] Add status field to Kubernetes clusters to support discovery workflows#62598
tigrato merged 2 commits intobranch/v18from
tigrato/eksautocleanupv18

Conversation

@tigrato
Copy link
Copy Markdown
Contributor

@tigrato tigrato commented Jan 5, 2026

Backport of #62161 to branch/v18

Changelog: Added cleanup of access entries for EKS auto-discovered clusters when they no longer match the filtering criteria and are removed.

#62161)

* Add status field to Kubernetes clusters to support discovery workflows

Adds a new Status field to KubernetesClusterV3 containing discovery-related
information. The status includes cloud provider-specific data, starting with
AWS which tracks the ARN for access setup, integration name, and the assumed
role used during discovery.

This enables the discovery service to persist state about clusters it discovers,
which is needed to properly cleanup created access entries when a cluster is
removed or no longer matches the label filtering. Without this information,
dangling AWS resources would be left behind after discovery changes.

Signed-off-by: Tiago Silva <tiago.silva@goteleport.com>

* handle code review comments

* handle code review comments

* add comment

---------

Signed-off-by: Tiago Silva <tiago.silva@goteleport.com>
@tigrato tigrato marked this pull request as ready for review January 5, 2026 11:19
@public-teleport-github-review-bot public-teleport-github-review-bot bot removed the request for review from smallinsky January 7, 2026 14:08
@tigrato tigrato enabled auto-merge January 7, 2026 15:41
@tigrato tigrato added this pull request to the merge queue Jan 7, 2026
Merged via the queue into branch/v18 with commit 7b67776 Jan 7, 2026
42 checks passed
@tigrato tigrato deleted the tigrato/eksautocleanupv18 branch January 7, 2026 16:14
@doggydogworld doggydogworld mentioned this pull request Jan 20, 2026
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.

4 participants