ci: added workflow for telemetry template check in azure.yaml file while raising the PR to main #600
Conversation
…raising the PR to main
Dhruvkumar-Microsoft
requested review from
Avijit-Microsoft,
Prajwal-Microsoft,
Roopan-Microsoft,
Vinay-Microsoft and
aniaroramsft
as code owners
There was a problem hiding this comment.
Pull Request Overview
This PR adds a GitHub Actions workflow to validate that the azure.yaml file contains a properly formatted and uncommented template property for telemetry purposes. The workflow triggers on pull requests to the main branch when azure.yaml is modified.
- Introduces automated validation for telemetry template property in
azure.yaml - Ensures the template line is present and not commented out
- Provides clear error messages for missing or commented template lines
|
|
||
| steps: | ||
| - name: Checkout code | ||
| uses: actions/checkout@v3 |
There was a problem hiding this comment.
Consider updating to actions/checkout@v4 for better performance and latest features. The v3 version is older and may have security or compatibility issues.
| uses: actions/checkout@v3 | |
| uses: actions/checkout@v4 |
|
|
||
| jobs: | ||
| validate-template-property: | ||
| name : validate-template-property |
There was a problem hiding this comment.
Remove the extra space before the colon. YAML formatting should be consistent with 'name: validate-template-property'.
| name : validate-template-property | |
| name: validate-template-property |
|
🎉 This PR is included in version 1.6.1 🎉 The release is available on GitHub release Your semantic-release bot 📦🚀 |
Purpose
This pull request introduces a new GitHub Actions workflow to ensure the
azure.yamlfile includes a requiredtemplateproperty for telemetry validation. The workflow checks for both the presence and proper formatting of thetemplateline.New GitHub Actions Workflow:
.github/workflows/telemetry-template-check.yml: Added a workflow namedvalidate template property for telemetrythat triggers on pull requests to themainbranch when changes are made toazure.yaml. It verifies the presence and correct formatting of thetemplateproperty inazure.yaml, ensuring it is not commented out and matches the required value.Does this introduce a breaking change?
Golden Path Validation
Deployment Validation
What to Check
Verify that the following are valid
Other Information