Skip to content

feat(chart): make retina-operator resources configurable - #2592

Merged
Evan Baker (rbtr) merged 1 commit into
microsoft:mainfrom
darkweaver87:feat/configurable-operator-resources
Aug 31, 2026
Merged

Evan Baker (rbtr) merged 1 commit into
microsoft:mainfrom
darkweaver87:feat/configurable-operator-resources

Conversation

@darkweaver87

@darkweaver87 Rémi BUISSON (darkweaver87) commented Jul 28, 2026 •

Copy link
Copy Markdown
Contributor

Motivation

The retina-operator container's resources are hardcoded in the deployment template, so they can't be tuned without patching the rendered manifest — while the agent, relay and ui all expose resources in values. This adds operator.resources, defaulted to the current values, so rendered output is unchanged.

@darkweaver87

Copy link
Copy Markdown
Contributor Author

@microsoft-github-policy-service agree company="TraefikLabs"

@github-actions

Copy link
Copy Markdown

This PR will be closed in 7 days due to inactivity.

@github-actions github-actions Bot added the meta/waiting-for-author Blocked and waiting on the author label Aug 28, 2026
@rbtr Evan Baker (rbtr) removed the meta/waiting-for-author Blocked and waiting on the author label Aug 28, 2026
@rbtr

Copy link
Copy Markdown
Collaborator

lgtm but same signing feedback as the other PR

Signed-off-by: darkweaver87 <remi-buisson@orange.fr>
@darkweaver87
Rémi BUISSON (darkweaver87) force-pushed the feat/configurable-operator-resources branch from 826613b to b49d41d Compare August 31, 2026 07:28
@darkweaver87

Copy link
Copy Markdown
Contributor Author

Thanks Evan Baker (@rbtr), done :-)

@nddq

Copy link
Copy Markdown
Member

/azp run

@azure-pipelines

Copy link
Copy Markdown
Azure Pipelines:
Successfully started running 1 pipeline(s).

@rbtr
Evan Baker (rbtr) added this pull request to the merge queue Aug 31, 2026
Merged via the queue into microsoft:main with commit 5db5829 Aug 31, 2026
46 of 47 checks passed
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.

3 participants