Skip to content

Replace deployment-names annotation with helmcharts#1447

Merged
gusevda merged 1 commit intomainfrom
replace-deployment-names-with-helmcharts
Mar 27, 2026
Merged

Replace deployment-names annotation with helmcharts#1447
gusevda merged 1 commit intomainfrom
replace-deployment-names-with-helmcharts

Conversation

@gusevda
Copy link
Contributor

@gusevda gusevda commented Mar 27, 2026

What does this PR do?

Replaces the legacy giantswarm.io/deployment-names annotation with the existing giantswarm.io/helmcharts annotation for mapping catalog entities to their Kubernetes deployments.

Changes:

  • Remove GS_DEPLOYMENT_NAMES constant, isEntityDeploymentsAvailable(), and getDeploymentNamesFromEntity() from entity.ts

  • Update useCatalogEntitiesForDeployments hook to build the reverse lookup map from helmcharts chart names instead of deployment-names

  • Document helmcharts, helmchart-versions, and helmchart-app-versions annotations in docs/metadata.md

  • Update plugin README and AI chat skill docs

  • A changeset describing the change and affected packages was added. (more info)

…y mapping

Use the existing `giantswarm.io/helmcharts` annotation instead of the legacy
`giantswarm.io/deployment-names` to map catalog entities to their Kubernetes
deployments. The helmcharts annotation provides more accurate chart name data
including bundle sub-charts. Also document the helmcharts, helmchart-versions,
and helmchart-app-versions annotations in metadata docs.
@gusevda gusevda marked this pull request as ready for review March 27, 2026 09:47
@gusevda gusevda requested a review from a team as a code owner March 27, 2026 09:47
@gusevda gusevda requested a review from marians March 27, 2026 09:47
@gusevda gusevda merged commit a79d45d into main Mar 27, 2026
6 checks passed
@gusevda gusevda deleted the replace-deployment-names-with-helmcharts branch March 27, 2026 10:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants