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

Helm chart - Add configuration for image pull secrets #3428

Merged
merged 2 commits into from
Sep 18, 2024

Conversation

Arkaniad
Copy link
Contributor

This minor change should allow the use of private container images with this chart by specifying image pull secrets.

Pull Request Checklist

Signed-off-by: Rhea Danzey <[email protected]>

@Arkaniad Arkaniad requested a review from a team as a code owner September 18, 2024 16:50
Signed-off-by: Rhea Danzey <[email protected]>
Copy link
Contributor

@S7evinK S7evinK left a comment

Choose a reason for hiding this comment

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

Thanks!

@S7evinK S7evinK merged commit 152626c into main Sep 18, 2024
2 checks passed
@S7evinK S7evinK deleted the rdanzey/helm-pull-secrets branch September 18, 2024 18:22
S7evinK pushed a commit that referenced this pull request Sep 20, 2024
Forgotten in #3428 - Pull
secrets need to be used for signing-key job since it uses Dendrite
container as well

### Pull Request Checklist

<!-- Please read
https://matrix-org.github.io/dendrite/development/contributing before
submitting your pull request -->

* [x] I have added Go unit tests or [Complement integration
tests](https://github.com/matrix-org/complement) for this PR _or_ I have
justified why this PR doesn't need tests
* [x] Pull request includes a [sign off below using a legally
identifiable
name](https://matrix-org.github.io/dendrite/development/contributing#sign-off)
_or_ I have already signed off privately

Signed-off-by: `Rhea Danzey <[email protected]>`

---------

Signed-off-by: Rhea Danzey <[email protected]>
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