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

feat: support imagePullSecrets on the service account as well as the deployment #28

Conversation

knechtionscoding
Copy link
Contributor

Closes #27

@ChrisJBurns
Copy link
Contributor

@knechtionscoding will need to rebase with master, make sure to follow the steps outlined in the new contributing docs as well just so we avoid any future issues as we've run into a few issues with bad kubernetes manifests being created from the charts - this was mainly by not running helm template before a PR was raised.

@knechtionscoding
Copy link
Contributor Author

@ChrisJBurns Sorry, just got back to this. I've updated the chart to template properly, and followed the new contributing documentation.

charts/pact-broker/values.yaml Outdated Show resolved Hide resolved
@ChrisJBurns ChrisJBurns merged commit 3a39ffc into pact-foundation:master Feb 3, 2023
@knechtionscoding knechtionscoding deleted the feat/service-account-image-pull-secrets branch February 7, 2023 14:57
roulettedares pushed a commit to roulettedares/pact-broker-chart that referenced this pull request Mar 13, 2023
…deployment (pact-foundation#28)

* feat: support imagePullSecrets on the service account as well as the deployment
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.

Allow imagePullSecrets to be specified on the SA not just the deployment
2 participants