Skip to content

Allow configuring the PIA canary pods image in chart values#97

Closed
LeBoucEtMistere wants to merge 1 commit intorestatedev:mainfrom
qube-rt:94-make-pia-canary-image-configurable
Closed

Allow configuring the PIA canary pods image in chart values#97
LeBoucEtMistere wants to merge 1 commit intorestatedev:mainfrom
qube-rt:94-make-pia-canary-image-configurable

Conversation

@LeBoucEtMistere
Copy link
Copy Markdown

Adresses issue #94

This allows configuring which container image is used for PIA canary pods instead of relying on a hardcoded value. This is controlled by helm chart values.

@github-actions
Copy link
Copy Markdown


Thank you for your submission, we really appreciate it. Like many open-source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution. You can sign the CLA by just posting a Pull Request Comment same as the below format.


I have read the CLA Document and I hereby sign the CLA


Arthur Depasse seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.
You can retrigger this bot by commenting recheck in this Pull Request. Posted by the CLA Assistant Lite bot.

Copy link
Copy Markdown
Contributor

@tillrohrmann tillrohrmann left a comment

Choose a reason for hiding this comment

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

Thanks a lot for contributing this improvement @LeBoucEtMistere. The changes look good to me. I will merge this PR once you have accepted the CLA.

@LeBoucEtMistere
Copy link
Copy Markdown
Author

Thanks, waiting for my company to review and approve the CLA, as this is a work-related contribution, will let you know when it's done.

@LeBoucEtMistere
Copy link
Copy Markdown
Author

@tillrohrmann I see the fix was already merged as part of another PR, I think I can close this one then ? What's the ETA for a release of the new version with the change ?

@tillrohrmann
Copy link
Copy Markdown
Contributor

I think we are going to create a release of the restate-operator within the next few days. @lukebond please correct me if I am over promising things.

@lukebond
Copy link
Copy Markdown
Contributor

yes i'll do one either today or tomorrow 👍 i was waiting for another feature to land, which now has

@lukebond
Copy link
Copy Markdown
Contributor

@LeBoucEtMistere i didn't notice your PR for this i'm sorry! i went and reimplemented it having only seen the issue. thank you for your contribution!

nevertheless the feature is on main now and i'll do a release today or tomorrow.

@lukebond lukebond closed this Mar 30, 2026
@github-actions github-actions bot locked and limited conversation to collaborators Mar 30, 2026
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants