Skip to content

[v14] Add vpc-id label to RDS Auto-Discover converter#35890

Merged
marcoandredinis merged 1 commit intobranch/v14from
bot/backport-35775-branch/v14
Dec 20, 2023
Merged

[v14] Add vpc-id label to RDS Auto-Discover converter#35890
marcoandredinis merged 1 commit intobranch/v14from
bot/backport-35775-branch/v14

Conversation

@marcoandredinis
Copy link
Copy Markdown
Contributor

Backport #35775 to branch/v14

changelog: add vpc-id as a label to auto-discovered RDS databases

@marcoandredinis marcoandredinis marked this pull request as ready for review December 19, 2023 12:20
@github-actions github-actions Bot added backport database-access Database access related issues and PRs discovery size/sm labels Dec 19, 2023
@marcoandredinis marcoandredinis force-pushed the bot/backport-35775-branch/v14 branch from 970df8a to cb7a6f4 Compare December 19, 2023 21:40
This PR adds a new label to RDS Instances and RDS Clusters to identify
its VPC ID.

For Clusters, an additional fetch was necessary because RDS
DescribeDBCluster does not return its VPC. So, a DescribeDBInstance must
be performed to obtain that information from one of its members.

This will allow us to deploy a DatabaseService in a specific VPC and
only proxy the Databases that also belong to that VPC.
@marcoandredinis marcoandredinis force-pushed the bot/backport-35775-branch/v14 branch from cb7a6f4 to c82c59d Compare December 20, 2023 07:27
@marcoandredinis marcoandredinis added this pull request to the merge queue Dec 20, 2023
Merged via the queue into branch/v14 with commit 78c49cd Dec 20, 2023
@marcoandredinis marcoandredinis deleted the bot/backport-35775-branch/v14 branch December 20, 2023 08:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

backport database-access Database access related issues and PRs discovery size/sm

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants