Skip to content

Implement dynamic default and fallback channels for operators in Oper…#2462

Merged
ibm-ci-bot merged 2 commits intoIBM:masterfrom
YCShen1010:cd_kc_version
Apr 3, 2025
Merged

Implement dynamic default and fallback channels for operators in Oper…#2462
ibm-ci-bot merged 2 commits intoIBM:masterfrom
YCShen1010:cd_kc_version

Conversation

@YCShen1010
Copy link
Contributor

What this PR does / why we need it:
Enhance the Common Service Operator to support dynamic default channel fallback channels management for operators in the OperandRegistry.
Cherry-pick: #2455

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

…andRegistry (IBM#2455)

* add process dynamic channel function

Signed-off-by: YuChen <yuchen.shen@mail.utoronto.ca>

* update cpp map with supported version

Signed-off-by: YuChen <yuchen.shen@mail.utoronto.ca>

* add ConfigMaps predicate functions in cs controller setup

Signed-off-by: YuChen <yuchen.shen@mail.utoronto.ca>

* polish the code

Signed-off-by: YuChen <yuchen.shen@mail.utoronto.ca>

* fix lint error

Signed-off-by: YuChen <yuchen.shen@mail.utoronto.ca>

* fix multi reconciliation problem

Signed-off-by: YuChen <yuchen.shen@mail.utoronto.ca>

---------

Signed-off-by: YuChen <yuchen.shen@mail.utoronto.ca>
@qpdpQ
Copy link
Contributor

qpdpQ commented Mar 26, 2025

@YCShen1010 we might need to remove forceUpdateODLMCRs variable in InstallOrUpdateOpreg in no-olm-controller as well

Signed-off-by: YuChen <yuchen.shen@mail.utoronto.ca>
Copy link
Contributor

@Daniel-Fan Daniel-Fan left a comment

Choose a reason for hiding this comment

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

/lgtm

@ibm-ci-bot
Copy link
Contributor

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: Daniel-Fan, YCShen1010

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Details Needs approval from an approver in each of these files:
  • OWNERS [Daniel-Fan,YCShen1010]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@ibm-ci-bot ibm-ci-bot added the lgtm label Apr 3, 2025
@ibm-ci-bot ibm-ci-bot merged commit e55cddb into IBM:master Apr 3, 2025
5 of 7 checks passed
@YCShen1010 YCShen1010 deleted the cd_kc_version branch April 4, 2025 13:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants