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

Create Region lookup composition #3575

Closed
Tracked by #3469
mproffitt opened this issue Jul 15, 2024 · 3 comments
Closed
Tracked by #3469

Create Region lookup composition #3575

mproffitt opened this issue Jul 15, 2024 · 3 comments
Assignees
Labels

Comments

@mproffitt
Copy link

mproffitt commented Jul 15, 2024

In order to ensure the region is correctly passed to the composition for building the RDS cache cluster, a new composition needs to be created that wraps rdscacheclusters.xdatabases.crossplane.giantswarm.io

This composition will be given all the parameters for the rdscacheclusters along with additional clusterName and clusterNamespace

The purpose of this composition will then

  • Block Infrastructure creation
  • Look up clusters.cluster.x-k8s.io
  • Retrieve spec.infrastructureRef
  • Determine if this is an AWS cluster or AWS Managed cluster
    • If not, abort on a composition assertion as this composition currently only supports AWS
  • Perform a secondary lookup to retrieve the AWSCluster or AWSManagedCluster resource
  • Patch region back to status.region
  • Feed status.region to rdscacheclusters.xdatabases.crossplane.giantswarm.io.spec.region
  • Unblock infrastructure creation
@github-project-automation github-project-automation bot moved this to Inbox 📥 in Roadmap Jul 15, 2024
@mproffitt mproffitt self-assigned this Jul 15, 2024
@architectbot architectbot added the team/honeybadger Team Honey Badger label Jul 15, 2024
@mproffitt mproffitt added kind/story team/honeybadger Team Honey Badger and removed team/honeybadger Team Honey Badger labels Jul 15, 2024
@mproffitt
Copy link
Author

Testing requires crossplane to be deployed to a management cluster as this requires access to CAPI/CAPA resources

@mproffitt
Copy link
Author

Done

@github-project-automation github-project-automation bot moved this from Inbox 📥 to Done ✅ in Roadmap Aug 19, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
Archived in project
Development

No branches or pull requests

2 participants