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

Issues wish KubernetesResourceUtil.santizeName in the operator #20854

Closed
shawkins opened this issue Jun 7, 2023 · 1 comment
Closed

Issues wish KubernetesResourceUtil.santizeName in the operator #20854

shawkins opened this issue Jun 7, 2023 · 1 comment

Comments

@shawkins
Copy link
Contributor

shawkins commented Jun 7, 2023

Description

The usage of a santizeName is problematic: fabric8io/kubernetes-client#5220

It limits the combined name to 63 characters, when the actual limit is 253. This means a long name on the keycloak resource will cause the suffixes to be dropped completely - however that isn't currently a major issue as none of the resources created by the operator are of the same type.

It's a little more of an issue when there's support for multiple keycloaks in a single namespace - this truncation will make it more likely we'll create a collision.

@shawkins
Copy link
Contributor Author

Obviated by #20906 - the upstream issue will still need to be worked to properly docuement and likely add more methods.

@shawkins shawkins closed this as not planned Won't fix, can't repro, duplicate, stale Jun 28, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant