Skip to content

[patch] Remove the rsl kubernetes secret creation and remove rls_url and rsl_org_id prarams#456

Open
Bhautik-Vala wants to merge 13 commits into
mainfrom
masaib-1915
Open

[patch] Remove the rsl kubernetes secret creation and remove rls_url and rsl_org_id prarams#456
Bhautik-Vala wants to merge 13 commits into
mainfrom
masaib-1915

Conversation

@Bhautik-Vala
Copy link
Copy Markdown

@Bhautik-Vala Bhautik-Vala commented May 11, 2026

Issue: MASAIB-2257

Description:
Update gitops helm chart for remove the rsl secret ( kubernetes secret ) creation as this is handled by aiservice operator and also remove the rsl related params - only rls_ca_crt will retain.

Testing:
Testing was done by deployment of aiservice through gitops.
Screenshot 2026-05-11 at 10 03 58 AM
Screenshot 2026-05-11 at 10 04 09 AM
Screenshot 2026-05-11 at 10 04 30 AM
Screenshot 2026-05-11 at 10 05 03 AM

@Bhautik-Vala Bhautik-Vala changed the title [patch] [patch] Remove the rsl kubernetes secret creation and remove rls_url and rsl_token prarams May 11, 2026
@Bhautik-Vala Bhautik-Vala changed the title [patch] Remove the rsl kubernetes secret creation and remove rls_url and rsl_token prarams [patch] Remove the rsl kubernetes secret creation and remove rls_url and rsl_org_id prarams May 11, 2026
@Bhautik-Vala Bhautik-Vala marked this pull request as draft May 11, 2026 07:18
@Bhautik-Vala Bhautik-Vala marked this pull request as ready for review May 12, 2026 10:19
@Bhautik-Vala Bhautik-Vala requested a review from whitfiea May 12, 2026 10:20
@Bhautik-Vala Bhautik-Vala requested a review from jasmin-30 May 13, 2026 08:33
url: "{{ .Values.slscfg_url }}"
secretName: "{{ .Values.tenantNamespace }}----sls-secret"
ca: "{{ .Values.slscfg_ca_b64enc }}"
{{- if .Values.rsl_ca_crt }}
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

This will add empty ca field even when .Values.rsl_ca_crt is empty. Can we improve condition to add only when it is not empty.

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