ci: add label condition to integration-test-deployment.yml#35708
ci: add label condition to integration-test-deployment.yml#35708mergify[bot] merged 3 commits intomainfrom
Conversation
Also updates the name of the job to clarify its purpose.
kumvprat
left a comment
There was a problem hiding this comment.
The workflow is not present anymore in the list of workflows that ran for this PR. Is that expected ? I assumed it will be skipped but would still show up
The workflow by default skips PR's if they do not contain snapshot changes. So this is expected. |
|
Thank you for contributing! Your pull request will be updated from main and then merged automatically (do not update manually, and be sure to allow changes to be pushed to your fork). |
|
Thank you for contributing! Your pull request will be updated from main and then merged automatically (do not update manually, and be sure to allow changes to be pushed to your fork). |
|
Comments on closed issues and PRs are hard for our team to see. |
Reason for this change
The new integ test deployment workflow asks for a deployment in the UI. This has been confusing for contributors as it is similarly named to the general integ test available today.
Description of changes
request-deployment-integ-runDescribe any new or updated permissions being added
No new permissions added.
Checklist
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache-2.0 license