Skip to content

[minor] add the support for path based routing in the gitops#405

Merged
whitfiea merged 11 commits intomainfrom
ds.path
Mar 3, 2026
Merged

[minor] add the support for path based routing in the gitops#405
whitfiea merged 11 commits intomainfrom
ds.path

Conversation

@dixitgsathwara
Copy link
Contributor

@dixitgsathwara dixitgsathwara commented Feb 26, 2026

Description

  • enable the path based routing in the gitops used by the sre team to provision mas saas clusters. these changes are already being implemented in ansible devops and the MAS CLI and corresponding updates are now required in the GitOps pipeline to ensure consistent configuration management across all provisioning workflows.

Jira Issue

Testing

  • tested on noble8
image image

@dixitgsathwara dixitgsathwara changed the title [patch] add the suppor for path based routing in the gitops [patch] add the support for path based routing in the gitops Mar 2, 2026
@whitfiea whitfiea changed the title [patch] add the support for path based routing in the gitops [minor] add the support for path based routing in the gitops Mar 2, 2026
{{ .Values.suite_spec_settings_additional_properties | toYaml | indent 4 }}
{{- end }}
manualCertMgmt: {{ .Values.mas_manual_cert_mgmt | default false }}
routingMode: {{ .Values.routing_mode | default "subdomain" }}
Copy link
Member

Choose a reason for hiding this comment

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

Can we add this in an if condition like the ingressControllerName? Then if this is 9.0 or 9.1 then we won't set it, we don't need to set the default then. I asume that in 9.2 if it is not set at all then it default to subdomain still?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

yes, in MAS 9.2, if routing mode is not set then it defaults to subdomain. this default routing mode is configured in the saas-envs-starter (where MAS_ROUTING_MODE set during environment setup), and then this value is added to the mas-instance-params.yaml file.

@dixitgsathwara dixitgsathwara requested a review from whitfiea March 3, 2026 09:13
@whitfiea whitfiea merged commit 97dfb22 into main Mar 3, 2026
2 checks passed
@whitfiea whitfiea deleted the ds.path branch March 3, 2026 09:52
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