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

releases App is in 'already-exists' status after creating a new MC with mc-bootstrap #3570

Closed
fiunchinho opened this issue Jul 10, 2024 · 1 comment
Assignees
Labels
team/turtles Team Turtles

Comments

@fiunchinho
Copy link
Member

fiunchinho commented Jul 10, 2024

When creating a new MC with mc-bootstrap, the releases-capa app is in already-exists status

reason: 'object already exists: (rendered manifests contain a resource that already
      exists. Unable to continue with install: Release "aws-25.0.0" in namespace ""
      exists and cannot be imported into the current release: invalid ownership metadata;
      label validation error: missing key "app.kubernetes.io/managed-by": must be
      set to "Helm"; annotation validation error: missing key "meta.helm.sh/release-name":
      must be set to "releases-capa"; annotation validation error: missing key "meta.helm.sh/release-namespace":
      must be set to "giantswarm")'
    status: already-exists

The Release CR is manually applied here https://github.com/giantswarm/mc-bootstrap/blob/main/scripts/install-releases-crds.sh#L22, and I guess the app is coming from the cluster chart , so they conflict.

I'm not sure if this step is necessary https://github.com/giantswarm/mc-bootstrap/blob/main/Makefile.custom.mk#L494 , because the CRD is managed by flux and the Release CR will be created by the releases-capa app.

@github-project-automation github-project-automation bot moved this to Inbox 📥 in Roadmap Jul 10, 2024
@weseven weseven added the team/turtles Team Turtles label Jul 11, 2024
@yulianedyalkova yulianedyalkova moved this from Inbox 📥 to Backlog 📦 in Roadmap Jul 11, 2024
@yulianedyalkova yulianedyalkova moved this from Backlog 📦 to Up Next ➡️ in Roadmap Jul 16, 2024
@weseven weseven self-assigned this Jul 29, 2024
@weseven weseven moved this from Up Next ➡️ to In Progress ⛏️ in Roadmap Jul 29, 2024
@weseven
Copy link

weseven commented Jul 31, 2024

Fixed with https://github.com/giantswarm/mc-bootstrap/pull/969: now the releases chart for the providers (releases-capa, releases-azure) is installed through helm during bootstrap, so flux should be able to adopt it once it comes up in a later stage and no longer complain about the missing annotations and labels.

@weseven weseven closed this as completed Jul 31, 2024
@github-project-automation github-project-automation bot moved this from In Progress ⛏️ to Done ✅ in Roadmap Jul 31, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
team/turtles Team Turtles
Projects
Archived in project
Development

No branches or pull requests

2 participants