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

fix(clickhouse-operator): pin docker image version #371

Merged
merged 4 commits into from
May 10, 2022

Conversation

hazzadous
Copy link
Contributor

@hazzadous hazzadous commented May 10, 2022

Previously we were using the latest available image. Here we pin to the
latest image as of writing.

It looks like they version the chart and the operator image the same. It
may be worth pinning them to the same version. However, this will cause
the operator to downgrade in existing installs, so here I'm opting to
try to keep the image version the same as current installs.

Closes #370

Description

Type of change

  • Breaking change (fix or feature that would cause existing functionality to not work as expected)
  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)

How has this been tested?

Checklist

  • I have commented my code, particularly in hard-to-understand areas
  • I have made corresponding changes to the documentation
  • I have added tests that prove my fix is effective or that my feature works

Previously we were using the latest available image. Here we pin to the
latest image.

It looks like they version the chart and the operator image the same. It
may be worth pinning them to the same version. However, this will cause
the operator to downgrade in existing installs, so here I'm opting to
try to keep the image version the same as current installs.
@guidoiaquinti
Copy link
Contributor

I think you need to update the test snapshots but overall LGTM. Thank you!

Harry Waye added 2 commits May 10, 2022 09:08
@hazzadous hazzadous merged commit 8b8e92c into main May 10, 2022
@hazzadous hazzadous deleted the fix/pin-operator-image branch May 10, 2022 09:23
@posthog-bot
Copy link

We probably forgot to bump the chart with this merge. No worries head over to https://github.com/PostHog/charts-clickhouse/actions/workflows/bump-version-workflow-dispatch.yaml to trigger the chart update.

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.

clickhouse-operator image is hard coded
3 participants