Skip to content

[patch] Remove entitlement key and artifactory token as param of tekton pipeline - #2617

Open
Bhautik-Vala wants to merge 15 commits into
masterfrom
entitlement
Open

Bhautik-Vala wants to merge 15 commits into
masterfrom
entitlement

Conversation

@Bhautik-Vala

@Bhautik-Vala Bhautik-Vala commented Sep 28, 2026 •

Copy link
Copy Markdown
Contributor

Description:
This changes made for remove the ibm_entitlement_key, artifactory_username and artifactory_token from pipeline params list so it will not shown as plain text in the pipelins' parameter section and yaml file,
instead we will create secret with name similar to mas actions like:

for mas install we will create mas-install-secret , for mas update mas-update-secrets etc..

and then while passing ENV vars ( IBM_ENTITLEMENT_KEY, ARTIFACTORY_TOKEN, ARTIFACTORY_USERNAME ) to Task we will read it from secrets.

Related PR: ibm-mas/python-devops#515

JIRA: https://jsw.ibm.com/browse/MASCORE-15201
slack Discussion: https://ibm-mas.slack.com/archives/CNDB5GZ3P/p1783070622831499?thread_ts=1783001636.283929&cid=CNDB5GZ3P

Testing:
Tested with pfvt also with all applications except MVI - https://dashboard.ibmmas.com/tests/pfvtbhautik7/9603

Tested mas install, mas aiservice-install, mas update, mas backup, mas restore

if secret not created because we haven't passed the value of ibm entitlement key or artifactory token like in mas update then without that secret also update pipeline completed successfully.

Screenshot 2026-09-29 at 3 15 21 PM Screenshot 2026-09-25 at 7 14 36 PM Screenshot 2026-09-25 at 7 14 52 PM Screenshot 2026-09-25 at 7 15 21 PM Screenshot 2026-09-25 at 7 14 20 PM

@Bhautik-Vala
Bhautik-Vala marked this pull request as ready for review October 7, 2026 08:42
@Bhautik-Vala
Bhautik-Vala requested a review from a team as a code owner October 7, 2026 08:42
@Bhautik-Vala
Bhautik-Vala requested a review from durera October 7, 2026 08:42
@Bhautik-Vala
Bhautik-Vala requested a review from whitfiea October 7, 2026 11:10

This branch has not been deployed

No deployments
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