Skip to content

Remove orphaned field from operandconfig#2776

Open
qpdpQ wants to merge 3 commits intoIBM:masterfrom
qpdpQ:update_config
Open

Remove orphaned field from operandconfig#2776
qpdpQ wants to merge 3 commits intoIBM:masterfrom
qpdpQ:update_config

Conversation

@qpdpQ
Copy link
Copy Markdown
Contributor

@qpdpQ qpdpQ commented Mar 24, 2026

What this PR does / why we need it:
Remove orphaned field from operandconfig, after user remove it from CommonService CR.

Which issue(s) this PR fixes:
Fixes #
https://github.ibm.com/IBMPrivateCloud/roadmap/issues/68616
https://github.ibm.com/IBMPrivateCloud/roadmap/issues/69023

qpdpQ added 2 commits March 24, 2026 12:16
Signed-off-by: Allen Li <liyuchen223@gmail.com>
Signed-off-by: Allen Li <liyuchen223@gmail.com>
@qpdpQ
Copy link
Copy Markdown
Contributor Author

qpdpQ commented Mar 24, 2026

Test image: quay.io/yuchen_li1/common-service-operator-amd64:remove-orphaned-field
Test log

I0324 20:11:21.290630       1 operandconfig.go:587] Removing orphaned fields from OperandConfig

After removing the following field about zenFrontDoor from cs cr

  services:
    - name: ibm-im-operator
      spec:
        authentication:
          config:
            zenFrontDoor: false

cr operator will remove it from operandconfig
Screenshot 2026-03-24 at 5 00 38 PM

Signed-off-by: Allen Li <liyuchen223@gmail.com>
@Daniel-Fan
Copy link
Copy Markdown
Contributor

Thank you @qpdpQ

I just want to be cautious and do not wanna take the risk for un-stabilizing the 4.18 build since we are approaching Dcut.

Can we hold this PR and merge it for 4.19?

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.

2 participants