Skip to content

Custom entitlement key#2770

Merged
Daniel-Fan merged 9 commits intoIBM:masterfrom
qpdpQ:custom-entitlement
Mar 23, 2026
Merged

Custom entitlement key#2770
Daniel-Fan merged 9 commits intoIBM:masterfrom
qpdpQ:custom-entitlement

Conversation

@qpdpQ
Copy link
Copy Markdown
Contributor

@qpdpQ qpdpQ commented Mar 18, 2026

What this PR does / why we need it:
Adding support for customer provided entitlement key

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

qpdpQ and others added 3 commits March 17, 2026 10:02
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 18, 2026

test steps:

  1. deploy cs operator with image quay.io/yuchen_li1/common-service-operator-amd64:custom-entitlement
  2. add entitlementKeyName: custom-entitlement to commonservice CR
  3. see if operandconfig contain updated entitlement key name
Screenshot 2026-03-18 at 1 53 33 PM

qpdpQ added 2 commits March 18, 2026 10:57
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 18, 2026

I was wondering if we need an extra field entitlementkeyName, or we can use imagePullSecret

Signed-off-by: Allen Li <liyuchen223@gmail.com>
Copy link
Copy Markdown
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.

@qpdpQ
Can you please review the comments?

Also please update the CRD in helm chart to include the latest field changes.

Comment thread helm/values.yaml Outdated
qpdpQ added 2 commits March 23, 2026 10:12
Signed-off-by: Allen Li <liyuchen223@gmail.com>
Signed-off-by: Allen Li <liyuchen223@gmail.com>
@Daniel-Fan
Copy link
Copy Markdown
Contributor

Thank you for the update @qpdpQ

This PR is getting flagged by the Detect Secrets check in SPS. Could you please follow the instructions to update the baseline file if this is a false positive: https://github.com/IBM/detect-secrets/tree/master

@qpdpQ qpdpQ force-pushed the custom-entitlement branch from 7770622 to 0552cb9 Compare March 23, 2026 18:32
Signed-off-by: Allen Li <liyuchen223@gmail.com>
Copy link
Copy Markdown
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.

Thank you @qpdpQ

The change looks good to me!

@Daniel-Fan Daniel-Fan merged commit f27aff7 into IBM:master Mar 23, 2026
9 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.

2 participants