[minor] Added configurable install plan approval support for AIService and AIService tenant #447
Open
Sakshi-Singhroha1 wants to merge 13 commits into
Open
[minor] Added configurable install plan approval support for AIService and AIService tenant #447Sakshi-Singhroha1 wants to merge 13 commits into
Sakshi-Singhroha1 wants to merge 13 commits into
Conversation
mnivedithaa
reviewed
Apr 30, 2026
Comment on lines
+72
to
+73
| aiservice_channel: "9.1.x" | ||
| aiservice_install_plan_approval: "Automatic" |
Contributor
There was a problem hiding this comment.
These needs to be removed from default values
Comment on lines
+259
to
+260
| aiservice_channel: "9.1.x" | ||
| aiservice_install_plan_approval: "Automatic" |
mnivedithaa
reviewed
May 8, 2026
Contributor
mnivedithaa
left a comment
There was a problem hiding this comment.
Aiservice tenant operator subscription also needs to be made configurable with this same option
mnivedithaa
approved these changes
May 15, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
JIRA:
MASCORE-12745 https://jsw.ibm.com/browse/MASCORE-12745
MASCORE-13410 https://jsw.ibm.com/browse/MASCORE-13410
MASCORE-13462 https://jsw.ibm.com/browse/MASCORE-13462
MASCORE-13884 https://jsw.ibm.com/browse/MASCORE-13884
Problem: The AIService and AIService tenant operator subscription
installPlanApprovalfields were hardcoded as"Automatic"in the GitOps templates. This prevented users from controlling whether operator updates require manual approval or install automatically.Requirement: Add Helm template support for configurable
aiservice_install_plan_approvalandaiservice_tenant_install_plan_approvalfields in GitOps, allowing the values to be passed from CLI and applied to the KubernetesSubscriptionresources.Why these changes are needed:
GitOps templates need to support dynamic
installPlanApprovalvalues from configurationEnables teams to control operator upgrade behavior per environment, such as
Manualfor production andAutomaticfor dev/testProvides default value
"Automatic"for backward compatibilityCompletes the parameter flow:
saas-envs → Tekton → CLI → GitOps → KubernetesExtends the same configurability to both AIService and AIService tenant operator subscriptions
What was changed:
This PR adds Helm template variable support for AIService and AIService tenant operator install plan approval settings across GitOps configuration files.
Changes made:
File 1:
instance-applications/113-ibm-aiservice/templates/04-aiservice-operator-subscription.yamlReplaced the hardcoded value with a Helm template variable:
File 2:
instance-applications/115-ibm-aiservice-tenant/templates/06-aiservice-tenant-operator-subscription.yamlReplaced the hardcoded value with a Helm template variable:
File 3:
root-applications/ibm-aiservice-instance-root/templates/040-ibm-aiservice-app.yamlAdded parameter passing for AIService install plan approval from root values into the instance application:
File 4:
root-applications/ibm-aiservice-instance-root/templates/070-aiservice-tenant-appset.yamlUpdated role parameter handling to safely default values when keys are not present:
File 5:
root-applications/ibm-aiservice-tenant-root/templates/100-ibm-aiservice-tenant-app.yamlAdded parameter passing for AIService tenant install plan approval:
Updated role handling to provide safe defaults when values are absent:
Adjusted sync option handling so namespace creation is enabled only when the cluster admin role is available
Impact:
✅ GitOps templates accept
aiservice_install_plan_approvalfrom CLI✅ GitOps templates accept
aiservice_tenant_install_plan_approvalfrom CLI✅ Kubernetes
Subscriptionresources are created with the correctinstallPlanApprovalvalues✅ Default value
"Automatic"ensures backward compatibility✅ Enables
ManualorAutomaticoperator upgrade control per environment✅ No breaking changes to existing deployments
✅ Improves root application safety by providing default handling for role parameters
Testing performed:
Template Validation:
✅ Validated Helm template syntax is correct
✅ Confirmed parameter flow from root applications to instance applications
✅ Verified default value
"Automatic"is set correctly✅ Verified tenant install plan approval value is passed correctly
Deployment Verification:
✅ Deployed to noble8 ArgoCD cluster
✅ Configuration with
aiservice_install_plan_approval: "Manual"deployed successfully✅ Configuration with
aiservice_install_plan_approval: "Automatic"deployed successfully✅ Configuration with
aiservice_tenant_install_plan_approvaldeployed successfully✅ Kubernetes
Subscriptionresources created with the correctinstallPlanApprovalfieldsTest Results:
✅ Templates render correctly with parameter values
✅ Accepts valid values
"Manual"and"Automatic"✅ Deployment to noble8 cluster successful
✅ Operator subscriptions created with the correct install plan approval settings
Related Pull Requests:
saas-envs-starter: PR #158 - Added prompts and template parameters for install plan configuration
saas-deploy-py: PR #266 - Added JSON schema validation for install plan approval fields
saas-tekton: PR #223 - Added Tekton pipeline support for install plan approval parameters
ibm-mas/cli: PR #2233 - Added CLI support for AIService and AIService tenant install plan approval parameters