Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Conversation

@CaseyCarter
Copy link
Contributor

.... but set the UI to require a maintainer comment to trigger.

.... but set the UI to require a maintainer comment to trigger.
@CaseyCarter CaseyCarter added the infrastructure Related to repository automation label Jan 4, 2024
@CaseyCarter CaseyCarter requested a review from a team as a code owner January 4, 2024 02:44
@CaseyCarter
Copy link
Contributor Author

/azp run STL-ASan-CI

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@CaseyCarter
Copy link
Contributor Author

CaseyCarter commented Jan 4, 2024

If and when we merge this change, we need to update the Wiki page that describes how to recreate the pipelines: there's a manual UI step needed to require a maintainer comment before running a pipeline. Without that setting, STL-ASan-CI will now run automatically on all PRs.

As far as a can tell, there's no way to do this purely in YAML - which is why I didn't do so previously. I think the value of triggering the optional pipeline outweighs the annoyance of hand configuration. It's annoying to PR an ASan fix with no way to run the CI before merging.

@StephanTLavavej StephanTLavavej self-assigned this Jan 8, 2024
@StephanTLavavej
Copy link
Member

I'm mirroring this to the MSVC-internal repo - please notify me if any further changes are pushed.

@StephanTLavavej StephanTLavavej merged commit 5fd9165 into microsoft:main Jan 9, 2024
@StephanTLavavej
Copy link
Member

Thanks for improving the ASAN experience! 📈 🎉 😸

@CaseyCarter CaseyCarter deleted the triggered-asan branch January 9, 2024 16:16
@CaseyCarter
Copy link
Contributor Author

If and when we merge this change, we need to update the Wiki page that describes how to recreate the pipelines: there's a manual UI step needed to require a maintainer comment before running a pipeline. Without that setting, STL-ASan-CI will now run automatically on all PRs.

I have updated the Wiki instructions.

@MahmoudGSaleh MahmoudGSaleh added the ASan Address Sanitizer label Nov 1, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ASan Address Sanitizer infrastructure Related to repository automation

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants