Skip to content

Conversation

@GoWang
Copy link
Member

@GoWang GoWang commented Apr 13, 2021

Description:

What

We want to have the ability to trigger live tests separately for different packages

How

Set runtime parameters Services at the test YAML file.
Based on recommendations from sys engineering team, add in test filtering logic based on package name so that developers can override run parameters from the pipeline . By default, all packages will be built and tested.

Test:

@ghost ghost added the Communication label Apr 13, 2021
@GoWang
Copy link
Member Author

GoWang commented Apr 13, 2021

/azp run python -communication - tests

@azure-pipelines
Copy link

No pipelines are associated with this pull request.

@Azure Azure deleted a comment from azure-pipelines bot Apr 13, 2021
@GoWang
Copy link
Member Author

GoWang commented Apr 13, 2021

/azp run python - communication - tests

@azure-pipelines
Copy link

Azure Pipelines failed to run 1 pipeline(s).

@GoWang
Copy link
Member Author

GoWang commented Apr 13, 2021

/azp run python - communication - tests

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@GoWang GoWang marked this pull request as ready for review April 13, 2021 18:43
@GoWang GoWang requested a review from danieljurek as a code owner April 13, 2021 18:43
@GoWang
Copy link
Member Author

GoWang commented Apr 14, 2021

/azp run python - communication - tests

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@GoWang
Copy link
Member Author

GoWang commented Apr 14, 2021

/azp run python - communication - tests

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@GoWang
Copy link
Member Author

GoWang commented Apr 14, 2021

/azp run python - communication - tests

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

Copy link
Member

@beltr0n beltr0n left a comment

Choose a reason for hiding this comment

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

lgtm, but i'd probably wait for someone from sys engineering to chime in as well

Copy link
Member

@lsundaralingam lsundaralingam left a comment

Choose a reason for hiding this comment

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

Will this mean that if I have a PR with just SMS changes that only the SMS tests run?

@GoWang
Copy link
Member Author

GoWang commented Apr 15, 2021

Will this mean that if I have a PR with just SMS changes that only the SMS tests run?

you cannot trigger it directly from pr using /azp because azure cli doesn't support this, but you can always trigger it from UI
Screenshot 2021-04-14 172726

the above example is for phone numbers and the same apply to java and .net.

@benbp benbp self-requested a review April 15, 2021 17:28
@ghost
Copy link

ghost commented Apr 15, 2021

Hello @RezaJooyandeh!

Because this pull request has the auto-merge label, I will be glad to assist with helping to merge this pull request once all check-in policies pass.

p.s. you can customize the way I help with merging this pull request, such as holding this pull request until a specific person approves. Simply @mention me (@msftbot) and give me an instruction to get started! Learn more here.

@ghost ghost merged commit 0629108 into Azure:master Apr 15, 2021
This pull request was closed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants