[patch] remove ibm_entitlement_key from Tekton param flow and source from secrets#2405
[patch] remove ibm_entitlement_key from Tekton param flow and source from secrets#2405jigneshchauhan2001 wants to merge 4 commits into
Conversation
60db85c to
61696c9
Compare
whitfiea
left a comment
There was a problem hiding this comment.
The changes update all the references to the entitlement key but i don't think it is updating all the pipelines to use this workspace that contains it i.e. i don't think aiservice or backup/restore pipelines use pipeline-additional-configs workspace so those need updating.
Also the backup/restore needs the secret set like it does in the app.py for install.
The secretKeyRef is also set as optional: true, when it shouldn't really be optional.
5ecfc1b to
8119ce5
Compare
| dynClient=self.dynamicClient, | ||
| namespace=pipelinesNamespace, | ||
| restoreConfigs=self.configSecret, | ||
| ibm_entitlement_key=self.getParam("ibm_entitlement_key") if self.getParam("include_dro") == "true" else None, |
There was a problem hiding this comment.
why is there an if condition on include_dro?
There was a problem hiding this comment.
I have removed if condition on incldue_dro in ibm_entitlement_key initialisation.
| workspace: shared-configs | ||
| workspace: shared-additional-configs |
There was a problem hiding this comment.
We can't change the existing workspace, we should just add the workspace
There was a problem hiding this comment.
I have restored the existing workspace.
Summary
ibm_entitlement_keywas appearing as plaintext in pod environment variables and was being passed through the MAS install PipelineRun into pod specs.This change removes
ibm_entitlement_keyfrom the Tekton parameter flow under /cli/tekton/src and updates affected task pods to readIBM_ENTITLEMENT_KEYfrom Kubernetes secretpipeline-additional-configs.It also applies the SLS speific-fix in /cli/tekton/src/tasks/dependencies/sls.yml.j2 so
IBM_ENTITLEMENT_KEYis not injected into the SLS pod for newer SLS channel paths (sls_channel > 3.7.0).Changes
ibm_entitlement_key moved to Secrets
$(params.ibm_entitlement_key)withvalueFrom.secretKeyRefin affected task templates under:ibm_entitlement_keyTekton param wiring from affected pipeline/taskdef templates under:SLS fix:
- /cli/tekton/src/tasks/dependencies/sls.yml.j2
IBM_ENTITLEMENT_KEYwhensls_channel > 3.7.0This change ensures:
Validation
Validation details are attached in
Validations.docx
Which contains UI and CLI validation confirmed:
ibm_entitlement_keypipeline-additional-configscontainsIBM_ENTITLEMENT_KEYvalueFrom.secretKeyRefIBM_ENTITLEMENT_KEYin the validated newer-channel pathibm_entitlement_keyValidated on Fyre OCP cluster:
fvt-corecompleted successfullyfvt-slscompleted successfullyfvt-managecompleted successfullyFVT dashboard link: