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.

Sign up for GitHub

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

Jump to bottom

Sort out dns for CAPA/CAPV hybrid MC (likely via dns-operator-route53) #3624

Closed
5 tasks done
Tracked by #3585
anvddriesch opened this issue Aug 7, 2024 · 3 comments
Closed
5 tasks done
Tracked by #3585

Sort out dns for CAPA/CAPV hybrid MC (likely via dns-operator-route53) #3624

anvddriesch opened this issue Aug 7, 2024 · 3 comments
Assignees
@anvddriesch
Labels
team/rocket Team Rocket

Comments

@anvddriesch
Copy link

anvddriesch commented Aug 7, 2024 •
edited
Loading

Tasks
Beta Give feedback
@anvddriesch anvddriesch mentioned this issue Aug 7, 2024
Closed
@anvddriesch anvddriesch self-assigned this Aug 7, 2024
@architectbot architectbot added the team/bigmac Team BigMac label Aug 7, 2024
@anvddriesch
Copy link
Author

For now we have deployed the operator manually to goat and created a set of credentials for it.
It works just fine creating the needed dns entries. However, the operator tries and fails (due to rbac) to reconcile the AWS clusters and we should prevent that from happening since it causes errors.
Furthermore, if we want this operator to be part only of capa-capv installations we need to add it to a suitable collection which can be applied to such installations so that flux keeps it up to date.

@anvddriesch anvddriesch mentioned this issue Aug 19, 2024
Merged
1 task
@anvddriesch
Copy link
Author

The open PR disables AWSCluster infrastructure in the operator and also pushes it to a new collection that we can apply to the MC https://github.com/giantswarm/vsphere-aws-addons-app-collection

@anvddriesch anvddriesch added team/rocket Team Rocket and removed team/bigmac Team BigMac labels Aug 19, 2024
This was referenced Aug 19, 2024
Merged
Merged
Merged
Merged
Merged
anvddriesch added a commit to giantswarm/management-cluster-bases that referenced this issue Aug 20, 2024
@anvddriesch
add vsphere aws addons collection (#157)
77db33d 
giantswarm/roadmap#3624

We need a separate addons collection for capv on capa due to the fact
that it uses a different dns operator.

## Some hints on changes to this repository

### Public information only

This is a public repository. The content and commit history is visible
to everyone.

Do not provide any **customer-specific details**. Use the customer's
repository for that.

### Descriptive PR title

Please write a descriptive pull request title. In this repo we don't
maintain a changelog file, so the PR title (becoming the commit message
after squash-merge) is the main information to understand a change in
retrospect.
@anvddriesch
Copy link
Author

So because dns-operator-route53 is not used on capz hybrid and therefore not present in the existing addons collection I have added a new collection specifically for aws/vsphere hybrid. (The one linked above)
We have decided that adding both collections is tedious and so I have also added all the other needed addon apps there and it's enough now to only deploy vsphere-aws-addons-app-collection on a capa MC. (you won't need the vsphere-addons-app-collection)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@anvddriesch anvddriesch

Labels
team/rocket Team Rocket
Projects
Roadmap
Archived in project
Milestone
No milestone
Development

No branches or pull requests
2 participants
@anvddriesch @architectbot