Skip to content

feat(opentelemetry-operator): add imageRegistry support [#2245] - #2322

Open
vkajtarGoTo wants to merge 6 commits into
open-telemetry:mainfrom
vkajtarGoTo:feat/opentelemetry-operator/2245-add-imageregistry
Open

feat(opentelemetry-operator): add imageRegistry support [#2245]#2322
vkajtarGoTo wants to merge 6 commits into
open-telemetry:mainfrom
vkajtarGoTo:feat/opentelemetry-operator/2245-add-imageregistry

Conversation

@vkajtarGoTo

@vkajtarGoTo vkajtarGoTo commented Jul 21, 2026

Copy link
Copy Markdown

Description

This PR implements the imageRegistry feature. Using this feature, a registry mirror, cache, or proxy can be used for all images managed by the operator.

Link to tracking issue

Fixes #2245

Authorship

  • I, a human, wrote this pull request description myself.

@vkajtarGoTo
vkajtarGoTo requested review from a team and Allex1 as code owners July 21, 2026 12:04
@linux-foundation-easycla

linux-foundation-easycla Bot commented Jul 21, 2026

Copy link
Copy Markdown

CLA Signed
The committers listed above are authorized under a signed CLA.

  • ✅ login: vkajtarGoTo / name: Viktor Kajtar (ff73228)

@vkajtarGoTo vkajtarGoTo changed the title feat(opentelemetry-operator): add imageregistry support [#2245] feat(opentelemetry-operator): add imageRegistry support [#2245] Jul 21, 2026
Comment thread charts/opentelemetry-operator/Chart.yaml Outdated
Comment thread charts/opentelemetry-operator/README.md Outdated
Comment on lines +59 to 60
registry: ""
repository: ghcr.io/open-telemetry/opentelemetry-operator/opentelemetry-operator

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

If we're adding the registry, wouldn't it make more sense to split this by default?

Suggested change
registry: ""
repository: ghcr.io/open-telemetry/opentelemetry-operator/opentelemetry-operator
registry: ghcr.io
repository: open-telemetry/opentelemetry-operator/opentelemetry-operator

Here and any other places where the repository is defined.

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

I suppose this would be a breaking change.

@github-actions

Copy link
Copy Markdown
Contributor

This PR was marked stale due to lack of activity. It will be closed in 14 days.

@github-actions github-actions Bot added the Stale label Aug 18, 2026
@marcschaeferger

Copy link
Copy Markdown
Contributor

Any Update?

@github-actions

github-actions Bot commented Sep 9, 2026

Copy link
Copy Markdown
Contributor

This PR was marked stale due to lack of activity. It will be closed in 14 days.

@github-actions github-actions Bot added the Stale label Sep 9, 2026
@marcschaeferger marcschaeferger added enhancement New feature or request chart:operator Issue related to opentelemetry-operator helm chart waiting-for-author needs-rebase labels Sep 10, 2026
@marcschaeferger

Copy link
Copy Markdown
Contributor

@vkajtarGoTo Pls rebase and apply the feedback/review above.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

chart:operator Issue related to opentelemetry-operator helm chart enhancement New feature or request needs-rebase Stale waiting-for-author

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[opentelemetry-operator] Seperate the registry from the repository

4 participants