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

fix: do not overwrite manually created operator configuration resource #302

Merged
merged 2 commits into from
Mar 10, 2025

Conversation

basti1302
Copy link
Member

...with values provided via Helm.

Previously, when instructed to create/update the operator configuration
resource via Helm, the operator manager would update any operator
configuration resource it finds with the provided value. This would
likely lead to unexpected results if people transition from a manually
configured operator configuration resource to a Helm-managed values.

Now the operator will refuse to overwrite the existing resource in this
scenario and log an error.

Also: Describe the behavior around creating the operator configuration
resource via Helm in more detail in the docs.

@basti1302 basti1302 force-pushed the document-auto-opconfres-handling-better branch from aba457c to 42bf1a1 Compare March 10, 2025 15:18
...with values provided via Helm.

Previously, when instructed to create/update the operator configuration
resource via Helm, the operator manager would update any operator
configuration resource it finds with the provided value. This would
likely lead to unexpected results if people transition from a manually
configured operator configuration resource to a Helm-managed values.

Now the operator will refuse to overwrite the existing resource in this
scenario and log an error.

Also: Describe the behavior around creating the operator configuration
resource via Helm in more detail in the docs.
@basti1302 basti1302 force-pushed the document-auto-opconfres-handling-better branch from 42bf1a1 to 9cfacff Compare March 10, 2025 16:43
@basti1302 basti1302 enabled auto-merge (rebase) March 10, 2025 16:45
@basti1302 basti1302 merged commit e982343 into main Mar 10, 2025
11 checks passed
@basti1302 basti1302 deleted the document-auto-opconfres-handling-better branch March 10, 2025 16:53
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.

1 participant