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

Use global.imagePullSecrets at solr-operator helm chart #338 #339

Closed
wants to merge 1 commit into from

Conversation

j-coll
Copy link

@j-coll j-coll commented Oct 13, 2021

Attempting to push to release-0.4 as it's just a minimal change (I'd love to see a 0.4.1). Please, let me know if the PR should be against main instead.

@HoustonPutman
Copy link
Contributor

@j-coll , thanks for the contribution. This looks great!

I think we are aiming to have a swift release of v0.5.0 (within 2-3 weeks hopefully). Therefore I don't think we are going to do a v0.4.1 release.

Could you rebase this off of main, change the target branch for this PR to main and also add a changelog entry?

@HoustonPutman
Copy link
Contributor

In the meantime if you need a quick fix, I would look at using serviceAccounts to store your imagePullSecret info, then use that service account for the Solr Operator and the Zookeeper Operator.

https://kubernetes.io/docs/tasks/configure-pod-container/configure-service-account/#add-imagepullsecrets-to-a-service-account

@j-coll
Copy link
Author

j-coll commented Oct 13, 2021

Sure thing, I will rebase it. And thanks for service-account advice!

@j-coll j-coll changed the base branch from release-0.4 to main October 13, 2021 16:57
@j-coll j-coll changed the base branch from main to release-0.4 October 13, 2021 16:57
@j-coll j-coll closed this Oct 13, 2021
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