Skip to content

Conversation

@jasmin-30
Copy link
Contributor

@jasmin-30 jasmin-30 commented Jan 22, 2026

Description

Add watsonx CA crt conditionally only when it is available in Secret on AWS SM.

Test Results

  • Tested changes by updating CLI version for aiservice tenant pipeline in saas-tekton.
  • Validated that generated GitOps-env params files contains watsonx_ca_crt parameter only when it exist

Test without CA Cert

Successful SPS pipeline:
Screenshot 2026-01-22 at 6 09 05 PM

Successful Argo Sync:
Screenshot 2026-01-22 at 6 12 02 PM

Kubernetes secret without watsonx ca crt:
Screenshot 2026-01-22 at 6 13 18 PM

Test with CA cert

Successful SPS pipeline:
Screenshot 2026-01-22 at 8 11 51 PM

Successful Argo Sync:
Screenshot 2026-01-22 at 8 12 19 PM

Kubernetes secret with watsonx ca crt:
Screenshot 2026-01-22 at 8 11 39 PM


⚠️ Notes for Reviewers

  • Ensure you have understood the PR guidelines in the Playbook before proceeding with a review.
  • Ensure all sections in the PR template are appropriately completed.

@jasmin-30 jasmin-30 self-assigned this Jan 22, 2026
@jasmin-30 jasmin-30 requested a review from a team as a code owner January 22, 2026 12:45
@jasmin-30 jasmin-30 closed this Jan 22, 2026
@jasmin-30 jasmin-30 reopened this Jan 22, 2026
Copy link
Contributor

@boonware boonware left a comment

Choose a reason for hiding this comment

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

Has the case where the CA cert is defined in SM been tested?

@jasmin-30
Copy link
Contributor Author

jasmin-30 commented Jan 22, 2026

Has the case where the CA cert is defined in SM been tested?

Yes, Updated test results in description.

However, CA cert needs to be included in SM when watsonx is configured with Self signed certificate. For SaaS deployments, only Watsonx SaaS is supported which uses CA signed certificate.
With that said, We need to clean up the on-prem watsonx configuration from gitops, which will be taken up as separate ticket.

@jasmin-30 jasmin-30 requested a review from boonware January 22, 2026 15:26
@whitfiea whitfiea merged commit 35707d8 into master Jan 23, 2026
13 of 14 checks passed
@whitfiea whitfiea deleted the masaib-1756 branch January 23, 2026 07:36
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.

4 participants