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

Document how to provide container registry credentials in air-gapped environment #7256

Merged
merged 4 commits into from
Nov 14, 2023

Conversation

otteryoudoing
Copy link
Contributor

@otteryoudoing otteryoudoing commented Oct 19, 2023

Adding information on how to use credentials with a custom repo.

Added details to initial custom repo example.

Also provided a link later on to another example in Github.

@botelastic botelastic bot added the triage label Oct 19, 2023
@thbkrkr thbkrkr added the >docs Documentation label Oct 23, 2023
@botelastic botelastic bot removed the triage label Oct 23, 2023
docs/operating-eck/air-gapped.asciidoc Outdated Show resolved Hide resolved
@@ -53,6 +66,8 @@ The operator expects container images to be located at specific repositories in
* +my.registry/kibana/kibana:{version}+
* +my.registry/apm/apm-server:{version}+

You can provide credentials to a custom repo by adding the imagePullSecrets parameter to your configuration as shown here - https://github.com/elastic/cloud-on-k8s/issues/2580#issuecomment-589602837
Copy link
Contributor

@thbkrkr thbkrkr Oct 23, 2023

Choose a reason for hiding this comment

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

We prefer to avoid the indirection to GitHub and simply put the information in the documentation.

Suggested change
You can provide credentials to a custom repo by adding the imagePullSecrets parameter to your configuration as shown here - https://github.com/elastic/cloud-on-k8s/issues/2580#issuecomment-589602837
You can provide credentials to your private container registry by setting the `imagePullSecrets` field through the `spec.podTemplate` section of your Elastic resource specification, check <<{p}-customize-pods,how to customize the Elastic resources Pods>> and link:https://kubernetes.io/docs/tasks/configure-pod-container/pull-image-private-registry/[how to setup a Secret containing your registry credentials].

Copy link
Contributor

Choose a reason for hiding this comment

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

Due to lack of activity, I applied the suggestion. I hope it is okay for you.

@thbkrkr thbkrkr changed the base branch from 2.9 to main November 10, 2023 17:04
otteryoudoing and others added 2 commits November 10, 2023 18:05
Adding information on how to use credentials with a custom repo.

Added details to initial custom repo example

also provided a link later on to another example in Github
@thbkrkr thbkrkr changed the title Update air-gapped.asciidoc Document how to provide container registry credentials in air-gapped environment Nov 13, 2023
@thbkrkr thbkrkr requested a review from naemono November 13, 2023 17:14
Copy link
Contributor

@naemono naemono left a comment

Choose a reason for hiding this comment

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

This looks good.

Copy link
Contributor

@barkbay barkbay left a comment

Choose a reason for hiding this comment

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

LGTM

@thbkrkr thbkrkr merged commit a46fb29 into main Nov 14, 2023
5 checks passed
@thbkrkr thbkrkr deleted the otteryoudoing-patch-1 branch November 16, 2023 16:56
robbavey pushed a commit to robbavey/cloud-on-k8s that referenced this pull request Dec 20, 2023
…environment (elastic#7256)

Co-authored-by: Thibault Richard <thb.krkr@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
>docs Documentation v2.11.0
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants