Skip to content

[tempo-distributed] fixing imagePullSecrets for enterprise components #3054

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

Merged
merged 13 commits into from
Apr 23, 2024

Conversation

jordan-simonovski
Copy link
Contributor

Ran into the following issue when trying to deploy GET via ArgoCD:

Error: template: tempo-distributed/templates/enterprise-gateway/gateway-dep.yaml:40:23: executing "tempo-distributed/templates/enterprise-gateway/gateway-dep.yaml" at <.Values.image.pullSecrets>: nil pointer evaluating interface {}.pullSecrets Use --debug flag to render out invalid YAML

After some debugging, I found some differences in the way that the pullSecrets are derived in the adminApi and the enterpriseGateway which I've now standardised to fix up some of the complexity when configuring this.

@zanhsieh zanhsieh changed the title fixing imagePullSecrets for enterprise components [tempo-distributed] fixing imagePullSecrets for enterprise components Apr 2, 2024
@jordan-simonovski jordan-simonovski force-pushed the fix-pullsecrets-templating branch from 20c4081 to 19becf2 Compare April 3, 2024 00:41
Signed-off-by: Jordan Simonovski <jsimonovski@atlassian.com>
The config overrides from the values.yml need to be added to the README.md to ensure up-to-date documentation for charts.

Signed-off-by: Jordan Simonovski <jsimonovski@atlassian.com>
Signed-off-by: Jordan Simonovski <jsimonovski@atlassian.com>
Signed-off-by: Jordan Simonovski <jsimonovski@atlassian.com>
@jordan-simonovski jordan-simonovski force-pushed the fix-pullsecrets-templating branch from b9bdae2 to d739d7e Compare April 17, 2024 09:36
jordan-simonovski and others added 2 commits April 19, 2024 12:28
Adding standardised imagepullsecrets support to tokengenJob

Signed-off-by: Jordan Simonovski <jsimonovski@atlassian.com>
@zalegrala
Copy link
Contributor

This looks good to me. We need a version bump again for the PR that went in before this one.

jordan-simonovski and others added 7 commits April 21, 2024 12:48
Signed-off-by: Jordan Simonovski <jsimonovski@atlassian.com>
…for future UID change

Signed-off-by: Zach Leslie <zach.leslie@grafana.com>
Signed-off-by: Jordan Simonovski <jsimonovski@atlassian.com>
Signed-off-by: Jordan Simonovski <jsimonovski@atlassian.com>
Signed-off-by: Jordan Simonovski <jsimonovski@atlassian.com>
Signed-off-by: Jordan Simonovski <jsimonovski@atlassian.com>
Copy link
Contributor

@zalegrala zalegrala left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good to me. Thanks for the contribution.

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.

3 participants