Skip to content

aws oidc skip aurora clusters without instances#47529

Merged
GavinFrazar merged 1 commit intomasterfrom
gavinfrazar/aurora-cluster-instance-not-found
Oct 15, 2024
Merged

aws oidc skip aurora clusters without instances#47529
GavinFrazar merged 1 commit intomasterfrom
gavinfrazar/aurora-cluster-instance-not-found

Conversation

@GavinFrazar
Copy link
Copy Markdown
Contributor

@GavinFrazar GavinFrazar commented Oct 11, 2024

Changelog: Fixed an issue that prevented RDS Aurora discovery configuration in the AWS OIDC enrollment wizard when any cluster existed without member instances.

Closes #47197

Instead of returning an error, just skip the cluster(s) that have no instances.

If any cluster in a region/vpc had no member instances, then the enrollment wizard would fail to list all of the aurora clusters in that region/vpc. Now it lists the ones that have member instances instead of giving up.

I verified that this fix works in a manual test as well by deploying to my cloud staging tenant.

@GavinFrazar GavinFrazar added discover Issues related to Teleport Discover backport/branch/v14 labels Oct 11, 2024
Comment thread lib/integrations/awsoidc/listdatabases.go Outdated
Instead of returning an error, just skip the cluster that have no
instances.
@GavinFrazar GavinFrazar force-pushed the gavinfrazar/aurora-cluster-instance-not-found branch from 64e12df to 22b232f Compare October 15, 2024 19:59
@public-teleport-github-review-bot public-teleport-github-review-bot Bot removed the request for review from atburke October 15, 2024 22:03
@GavinFrazar GavinFrazar added this pull request to the merge queue Oct 15, 2024
Merged via the queue into master with commit 48fa242 Oct 15, 2024
@GavinFrazar GavinFrazar deleted the gavinfrazar/aurora-cluster-instance-not-found branch October 15, 2024 23:20
@public-teleport-github-review-bot
Copy link
Copy Markdown

@GavinFrazar See the table below for backport results.

Branch Result
branch/v14 Failed
branch/v15 Failed
branch/v16 Create PR

mvbrock pushed a commit that referenced this pull request Oct 16, 2024
Instead of returning an error, just skip the cluster that have no
instances.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

discover Issues related to Teleport Discover size/sm

Projects

None yet

Development

Successfully merging this pull request may close these issues.

RDS Discover flow error database cluster <db-cluster> has no instance

3 participants