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(busybox): allow specifying a custom image #380

Merged
merged 1 commit into from
May 16, 2022

Conversation

hazzadous
Copy link
Contributor

In cases where you do not have access to dockerhub, it becomes necessary
to be able to specify an image from a different repo.

Refers #379

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

In cases where you do not have access to dockerhub, it becomes necessary
to be able to specify an image from a different repo.

Refers #379
@hazzadous
Copy link
Contributor Author

Closing as setting default repo as mentioned in the referenced ticket is probably a better approach

@hazzadous hazzadous closed this May 12, 2022
@dvirp-dt
Copy link

dvirp-dt commented May 16, 2022

Closing as setting default repo as mentioned in the referenced ticket is probably a better approach

Hey @hazzadous, after quick research I've understood there is no quick way to configure the registry in cluster-lever.
Can we please reopen this PR?

@hazzadous
Copy link
Contributor Author

@dvirp-dt ah that's a shame, I'll open and add some reviewers.

@hazzadous hazzadous reopened this May 16, 2022
@hazzadous hazzadous requested review from guidoiaquinti, fuziontech and benjackwhite and removed request for fuziontech May 16, 2022 14:55
@hazzadous hazzadous marked this pull request as ready for review May 16, 2022 14:55
@dvirp-dt
Copy link

@dvirp-dt ah that's a shame, I'll open and add some reviewers.

Thanks.

@hazzadous hazzadous merged commit 401922d into main May 16, 2022
@hazzadous hazzadous deleted the allow-busybox-specification branch May 16, 2022 16:05
@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.

fuziontech added a commit that referenced this pull request May 17, 2022
* main:
  Bump version to 20.0.1
  Regenerate chart ALL_VALUES.md
  feat(busybox): allow specifying a custom image (#380)
fuziontech added a commit that referenced this pull request May 17, 2022
* main:
  Bump version to 20.0.1
  Regenerate chart ALL_VALUES.md
  feat(busybox): allow specifying a custom image (#380)
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.

4 participants