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

[repo] Inconsistent use of release name for resolving service locations #41

Closed
eawagner opened this issue Aug 27, 2024 · 2 comments · Fixed by #42
Closed

[repo] Inconsistent use of release name for resolving service locations #41

eawagner opened this issue Aug 27, 2024 · 2 comments · Fixed by #42
Assignees
Labels
bug Something isn't working

Comments

@eawagner
Copy link

Describe the bug a clear and concise description of what the bug is.

When using a release name it does not get templated consistently. For instance to generate the service name for the server it will use the opencti.fullname helper function, but the init container on the worker will simply use $.Release.Name. This means that we must use the release name override or simply use the release name of opencti when deploying to get consistent behavior.

What's your helm version?

3.15.4

What's your kubectl version?

1.30

Chart version

1.0.0

Explain bug

No response

What you expected to happen?

Ideally all instances of $.Release.Name would get replaced with the opencti.fullname helper function when referencing the service.

How to reproduce it?

simply run helm install debugging opencti/opencti and the worker will fail to start because it will try to look for debugging-server instead of the service which is named debugging-opencti-server.

Enter the changed values of values.yaml?

No response

Enter the command that you execute when chart failing/misfunctioning

helm install debugging opencti/opencti

More info

No response

@eawagner eawagner added the bug Something isn't working label Aug 27, 2024
@ialejandro
Copy link
Member

ialejandro commented Aug 27, 2024

Thx for the report! On our CI use https://github.com/devops-ia/helm-opencti/tree/main/charts/opencti/ci and force use fullnameOverride to force the same name on all services and dependencies, so I don't know about this bug. My next-steps will test every allowed option on values to check each template.

@eawagner
Copy link
Author

Thank you for the fast turnaround!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants