clastix / kamaji

Kamaji is the Hosted Control Plane Manager for Kubernetes.
https://kamaji.clastix.io
Apache License 2.0
992 stars 90 forks source link

fix(migrate): preventing 63 characters pod name limit #391

Closed prometherion closed 9 months ago

prometherion commented 9 months ago

Closes #390.

netlify[bot] commented 9 months ago

Deploy Preview for kamaji-documentation canceled.

Name Link
Latest commit 10190d7c0ce361758bbdf71704ea546023f8a0a3
Latest deploy log https://app.netlify.com/sites/kamaji-documentation/deploys/657742136c6a21000877fc4e
prometherion commented 9 months ago

@sn4psh0t JFI, we're already relying on Job labels which refer to the namespaced name of the Tenant Control Plane in migration.

https://github.com/clastix/kamaji/blob/6eb3171817883aa69b6a67ea23e01995ea1cafe4/internal/resources/datastore/datastore_migrate.go#L102-L103

Easy fix (famous last words ™️)