Skip to content

Conversation

@Divyesh-Khokhar
Copy link
Contributor

@Divyesh-Khokhar Divyesh-Khokhar commented Jan 19, 2026

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.
Screenshot 2026-01-20 at 6 25 42 PM
Screenshot 2026-01-20 at 6 26 23 PM
Screenshot 2026-01-20 at 6 26 36 PM

Evidence: Validation of the existing FVT Manage pipeline confirms that AI service tests are not triggered as part of the current execution flow.
Screenshot 2026-01-21 at 12 30 13 PM
Screenshot 2026-01-21 at 12 30 30 PM

Validate locally:
Screenshot 2026-01-22 at 1 48 08 PM
Screenshot 2026-01-22 at 2 51 16 PM
Screenshot 2026-01-22 at 2 58 53 PM

@aniketchoudhary03 aniketchoudhary03 marked this pull request as ready for review January 21, 2026 13:28
@aniketchoudhary03 aniketchoudhary03 requested a review from a team as a code owner January 21, 2026 13:28
- name: MAX_RETRIES
value: "50"
- name: IGNORE_FAILURE
value: "True"
Copy link
Contributor

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

Copy link
Contributor Author

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
Copy link
Contributor

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

Copy link
Contributor Author

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
Copy link
Contributor

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.

Copy link
Contributor Author

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.

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.

3 participants