-
Notifications
You must be signed in to change notification settings - Fork 43
[patch] Create mas-fvt-manage-ai pipeline #2009
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.
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
base: master
Are you sure you want to change the base?
Conversation
| - name: MAX_RETRIES | ||
| value: "50" | ||
| - name: IGNORE_FAILURE | ||
| value: "True" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Have a new line at the end of the task/yaml, to avoid parsing issue.
Example: #2000
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
New line is there at end of the file.
| params: | ||
| {{ lookup('template', pipeline_src_dir ~ '/taskdefs/fvt-manage/api/params.yml.j2') | indent(4) }} | ||
| - name: fvt_test_suite | ||
| value: base-api-setup |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Why base-api-setup is part of this pipeline? Its already part of fvt launcher
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We have removed base-api-setup as its already part of fvt-launcher. Thanks!
| name: mas-fvt-manage | ||
| key: MAS_APP_CHANNEL_MANAGE | ||
| optional: false | ||
| - name: MAS_WORKSPACE_ID |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We might need AI INSTANCE ID as well and restrict the test when AI isn't installed.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We have added changes to check if aiservice_instance_id available then only we will run manage-ai tests/pipeline.
This PR adds a new fvt-manage-ai pipeline, which enables running Manage service tests exclusively with the AI service, without triggering the full manage test suite.
Evidence: The fvt-manage-ai pipeline runs successfully without triggering any Manage service tests.



Evidence: Validation of the existing FVT Manage pipeline confirms that AI service tests are not triggered as part of the current execution flow.


Validate locally:


