Skip to content
This repository has been archived by the owner on Aug 2, 2022. It is now read-only.

allow customizing docker registry #358

Conversation

tareksha
Copy link
Contributor

@tareksha tareksha commented Aug 26, 2020

Description of changes:

Allow the consumer to specify an alternative docker registry in the helm chart. This allows installing OpenDistro in non-public environments that rely on private docker registries. Keep the default as the standard implicit docker registry: docker.io

Test Results:

Image in templates/elasticsearch/es-data-sts.yaml with default values:

image: docker.io/amazon/opendistro-for-elasticsearch:1.9.0

Image in templates/elasticsearch/es-data-sts.yaml with setting global.registry=my.registry.org in values:

image: my.registry.org/amazon/opendistro-for-elasticsearch:1.9.0

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

@gaiksaya
Copy link
Member

Hi @tareqhs ,

Thank you for your contribution. I can see that you have not included any test results in the PR body. We would like you to share the test results and to see if this change causes any issues or any breaking changes.

@gaiksaya gaiksaya added Helm Helm chart improvements enhancement New feature or request labels Aug 26, 2020
@tareksha
Copy link
Contributor Author

Hi @gaiksaya , added result from local tests

Copy link
Member

@gaiksaya gaiksaya left a comment

Choose a reason for hiding this comment

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

Thank you for contributing!

@peterzhuamazon peterzhuamazon merged commit 4583d22 into opendistro-for-elasticsearch:master Sep 3, 2020
@tareksha tareksha deleted the chart_docker_registry branch September 6, 2020 12:25
@tareksha
Copy link
Contributor Author

tareksha commented Sep 6, 2020

thank you @gaiksaya & @peterzhuamazon . when is this expected to be released?

@peterzhuamazon
Copy link
Contributor

@tareqhs it will be included in the next upcoming release. Thanks.

@NovikovEvgeny
Copy link

Hi,

is there any way to pull only specific image from the private registry, but keep pulling other images from the default registy?

Use case: we use own ODFE docker image with additional plugins installed (download it from the private registry), but want to use default kibana image and default busybox (and other helper images) from docker.io

In helm chart version 1.9.0 and lower we just used

elasticsearch.image: privateRegistryName/imageName:imageTag
kibana.image: amazon/opendistro-for-elasticsearch-kibana:1.9.0

but now helm chart forces us to have all images in the same registry...

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
community enhancement New feature or request Helm Helm chart improvements
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants