Skip to content

[minor] Add support For Installation and FVT setup for Aiservice 9.1.x #79

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
wants to merge 12 commits into
base: stable
Choose a base branch
from
Open

[minor] Add support For Installation and FVT setup for Aiservice 9.1.x #79

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
12 commits
Select commit Hold shift + click to select a range
8a1480e
[patch] Added template for aibroker
HetviShah1 May 12, 2025
b078b66
[patch] Add support for aiservice pipelinerun
May 12, 2025
a2e4437
[patch] Add support for aiservice pipelinerun
May 12, 2025
ffccb9a
[patch] Change PVC name in aiservice install pipelinerun
May 13, 2025
8162a8b
[patch] add some extra params which required for aiservice
May 14, 2025
85485fc
[patch] Add new vars to pipelineRun
May 14, 2025
79675f2
[patch] add rsl params
May 20, 2025
292c103
[patch] reorder some s3 params
May 21, 2025
b1fa6e8
[patch] Add extra params
May 29, 2025
d0f8fe3
[patch] Add hold flag for aiservice-wait-suite
May 29, 2025
60f0e37
[patch] Merge stable into aibroker
Jun 3, 2025
ae77195
[patch] remove unused var from pipeline
Jun 5, 2025
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
4 changes: 3 additions & 1 deletion src/mas/devops/data/catalogs/v9-250403-amd64.yaml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -77,7 +77,9 @@ mas_visualinspection_version:
9.0.x: 9.0.8 # updated
8.10.x: 8.8.4 # No Update
8.11.x: 8.9.11 # updated

mas_aibroker_version:
Copy link
Contributor

Choose a reason for hiding this comment

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

aibroker or aiservice, I thought we were using the latter now?

9.0.x: 9.0.5

# Extra Images for UDS
# ------------------------------------------------------------------------------
uds_extras_version: 1.5.0
Expand Down
12 changes: 12 additions & 0 deletions src/mas/devops/tekton.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -402,6 +402,18 @@ def launchInstallPipeline(dynClient: DynamicClient, params: dict) -> str:
return pipelineURL


def launchInstallPipelineForAiservice(dynClient: DynamicClient, params: dict) -> str:
"""
Create a PipelineRun to install the Aiservice
"""
instanceId = params["mas_instance_id"]
namespace = f"mas-{instanceId}-pipelines"
Copy link
Contributor

Choose a reason for hiding this comment

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

Unless the aiservice is part of the mas instance, it should not be using the mas-instanceid-pipelines namespace. This is only for pipelines that are scoped to the mas instance.

e.g.

  • we use mas-pipelines namespace for the cluster-scoped update
  • we use mas-instanceid-pipelines for the instance-scoped upgrade, install, and uninstall

timestamp = launchPipelineRun(dynClient, namespace, "pipelinerun-aiservice-install", params)

pipelineURL = f"{getConsoleURL(dynClient)}/k8s/ns/mas-{instanceId}-pipelines/tekton.dev~v1beta1~PipelineRun/{instanceId}-install-{timestamp}"
return pipelineURL


def launchUpdatePipeline(dynClient: DynamicClient, params: dict) -> str:
"""
Create a PipelineRun to update the Maximo Operator Catalog
Expand Down
Loading
Loading