Skip to content

ci: DO NOT MERGE - Test to showcase feasibility of validation on PR#6732

Draft
eriqua wants to merge 73 commits intoAzure:mainfrom
eriqua:main
Draft

ci: DO NOT MERGE - Test to showcase feasibility of validation on PR#6732
eriqua wants to merge 73 commits intoAzure:mainfrom
eriqua:main

Conversation

@eriqua
Copy link
Copy Markdown
Collaborator

@eriqua eriqua commented Mar 18, 2026

Description

This PR validates the feasibility of a workflow running on pull_request to check if the modules updated by the PR are onboarded to MAR. Workflow structure:

  1. Detects the list of changed modules

    Note: this is currently done by an external action, we may want to develop our logic

  2. Retrieves the list of onboarded modules

    Note: this is temporarily done using the list of modules in the avm index csv, it will change once the actual list of onboarded modules is available in AVM

  3. Checks the list of changed modules against the list of onboarded modules.
  4. Fails if one of the changed modules is not found in the list of onboarded modules

Test invalid PR eriqua#27
image

Test valid PR eriqua#26
image

Pipeline Reference

Pipeline

Type of Change

  • Azure Verified Module updates:
    • Bugfix containing backwards-compatible bug fixes, and I have NOT bumped the MAJOR or MINOR version in version.json:
    • Feature update backwards compatible feature updates, and I have bumped the MINOR version in version.json.
    • Breaking changes and I have bumped the MAJOR version in version.json.
    • Update to documentation
  • Update to CI Environment or utilities (Non-module affecting changes)

Checklist

  • I'm sure there are no other open Pull Requests for the same update/change
  • I have run Set-AVMModule locally to generate the supporting module files.
  • My corresponding pipelines / checks run clean and green without any errors or warnings
  • I have updated the module's CHANGELOG.md file with an entry for the next version

@microsoft-github-policy-service microsoft-github-policy-service Bot added Needs: Triage 🔍 Maintainers need to triage still Type: AVM 🅰️ ✌️ Ⓜ️ This is an AVM related issue labels Mar 18, 2026
@eriqua eriqua changed the title test: DO NOT MERGE - Test to showcase feasibility of validation on PR ci: DO NOT MERGE - Test to showcase feasibility of validation on PR Mar 18, 2026
@eriqua eriqua marked this pull request as ready for review March 18, 2026 16:42
@eriqua eriqua requested a review from a team as a code owner March 18, 2026 16:42
@eriqua eriqua added Awaiting Module Path Assignment and removed Needs: Triage 🔍 Maintainers need to triage still labels Mar 18, 2026
@eriqua eriqua marked this pull request as draft March 18, 2026 22:36
@jtracey93 jtracey93 added the Status: Do Not Merge ⛔ Do not merge PRs with this label attached as they are not ready or aligned to future direction etc. label Mar 19, 2026
@eriqua eriqua self-assigned this Mar 19, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Status: Do Not Merge ⛔ Do not merge PRs with this label attached as they are not ready or aligned to future direction etc. Type: AVM 🅰️ ✌️ Ⓜ️ This is an AVM related issue

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants