AVM Issue Triage
“AVM Core Team Triage” Explained
This page provides guidance for members of the AVM Core Team on how to triage module proposals and generic issues filed in the AVM repository, as well as how to manage these GitHub issues throughout their lifecycle.
During the AVM Core Team Triage step, the following will be checked, completed and actioned by the AVM Core Team during their triage calls (which are currently twice per week).
Note
Every module needs a module proposal to be created in the AVM repository.
Tip
During the triage process, the AVM Core Team should also check the status of following queries:
- Open items that need triaging: ย Needs: Triage ๐ย
- Bicep items (that need triaging): ย Language: Bicep ๐ชย ย &ย ย Needs: Triage ๐ย
- Terraform items (that need triaging): ย Language: Terraform ๐ย ย &ย ย Needs: Triage ๐ย
- Open items that need triaging AND aren’t being triaged yet: ย Needs: Triage ๐ย ย & NOT ย ย Status: In Triage ๐ย
- Open items that need attention: ย Needs: Attention ๐ย
- Open items that need owners: ย Needs: Module Owner ๐ฃย
- Open items with no recent activity: ย Status: No Recent Activity ๐คย
- Open question/feedback items: ย Type: Question/Feedback ๐โโ๏ธย
- Open items with long term status: ย Status: long-term โณย
- Open items that are not in a project.
Module Proposal triage
An issue is considered to be a module proposal if
- it was opened through the “New AVM Module Proposal ๐” template,
- it has the labels of ย Needs: Triage ๐ย and ย Type: New Module Proposal ๐กย applied to it, and
- it is assigned to the “AVM - Module Triage” GitHub project.
Follow these steps to triage a module proposal:
Add the ย Status: In Triage ๐ย label to indicate you’re in the process of triaging the issue.
Check module proposal issue/form:
- Check the Bicep or Terraform module indexes for the proposed module to make sure it is not already available or being worked on.
- Ensure the module’s details are correct as per specifications - naming, classification (resource/pattern) etc.
- Check if the module is added to the “
Proposed
” column on the AVM - Modules Triage GitHub project board. - Check if the requestor is a Microsoft FTE.
- If there’s any additional clarification needed, contact the requestor through comments (using their GH handle) or internal channels - for Microsoft FTEs only! You can look them up by their name or using the Microsoft internal “1ES Open Source Assistant Browser Extension”. Make sure you capture any decisions regarding the module in the comments section.
- Make adjustments to the module’s name/classification as needed.
- Change the name of the issue to reflect the module’s name, i.e.,
- After the “[Module Proposal]:” prefix, change the issues name to the module’s approved name between backticks, i.e., ` and `, e.g.,
avm/res/sql/managed-instance
for a Bicep module, oravm-res-compute-virtualmachine
for a Terraform module. - Example:
- “[Module Proposal]:
avm/res/sql/managed-instance
” - “[Module Proposal]:
avm-res-sql-managedinstance
”
- “[Module Proposal]:
- After the “[Module Proposal]:” prefix, change the issues name to the module’s approved name between backticks, i.e., ` and `, e.g.,
- Check if the GitHub Policy Service Bot has correctly applied the module language label: ย Language: Bicep ๐ชย or ย Language: Terraform ๐ย
Apply relevant labels
- Module classification (resource/pattern): ย Class: Resource Module ๐ฆย or ย Class: Pattern Module ๐ฆย
Triaging pattern modules
As part of the triage of pattern modules, the following points need to be considered/clarified with the module requestor:
- Shouldn’t this be a resource module? What makes it a pattern - e.g., does it deploy multiple resources?
- What is it for? What problem does it fix or provides a solution for?
- What is/isn’t part of it? Which resource and/or pattern modules are planned to be leveraged in it? Provide a list of resources that would be part of the planned module.
- Where is it coming from/what’s backing it - e.g., Azure Architecture Center (AAC), community request, customer example. Provide an architectural diagram and related documentation if possible - or a pointer to these if they are publicly available.
- Don’t let the module’s scope to grow too big, split it up to multiple smaller ones that are more maintainable - e.g., hub & spoke networking should should be split to a generic hub networking and multiple workload specific spoke networking patterns.
- The module’s name should be as descriptive as possible.
- Adopt strict name-to-scope mapping - e.g., hub & spoke networking shouldn’t deploy monitoring.
Scenario 1: Requestor doesn’t want to / can’t be module owner
Note
If requestor is interested in becoming a module owner, but is not a Microsoft FTE, the AVM core team will try to find a Microsoft FTE to be the module owner whom the requestor can collaborate with.
- If the requestor indicated they didn’t want to or can’t become a module owner (or is not a Microsoft FTE), make sure the ย Needs: Module Owner ๐ฃย label is assigned to the issue. Note: the GitHub Policy Service Bot should automatically do this, based on how the issue author responded to the related question.
- Move the issue to the “
Looking for owners
” column on the AVM - Modules Triage GitHub project board. - Find module owners - if the requestor didn’t volunteer in the module proposal OR the requestor does not want or cannot be owner of the module:
- Try to find an owner from the AVM communities or await a module owner to comment and propose themselves on the proposal issue.
- When a new potential owner is identified, continue with the steps described as follows.
Scenario 2: Requestor wants to and can become module owner
If the requestor indicated they want to become the module owner, the GitHub Policy Service Bot will add the ย Status: Owners Identified ๐คย label and will assign the issue to the requestor.
You MUST still confirm that the requestor is a Microsoft FTE and that they understand the implications of becoming the owner! If any of these conditions aren’t met, remove the ย Status: Owners Identified ๐คย label and unassign the issue from the requestor.
- Make sure the requestor is a Microsoft FTE. You can look them up by their name or using the Microsoft internal “1ES Open Source Assistant Browser Extension”.
- Clarify the roles and responsibilities of the module owner:
- Clarify they understand and accept what “module ownership” means by replying in a comment to the requestor/proposed owner:
- Once module owner identified has confirmed they understand and accept their roles and responsibilities as an AVM module owner
- Make sure the issue is assigned to the confirmed module owner.
- Move the issue into the “
In development
” column on the AVM - Modules Triage GitHub Project board. - Make sure the ย Status: Owners Identified ๐คย label is added to the issue.
- If applied earlier, remove the ย Needs: Module Owner ๐ฃย label from the issue.
- Remove the labels of ย Needs: Triage ๐ย and ย Status: In Triage ๐ย to indicate you’re done with triaging the issue.
- Update the AVM Module Indexes, following the process documented internally.
- Use the following text to approve module development
Important
Although, it’s not directly part of the module proposal triage process, to begin development, module owners and contributors might need additional help from the AVM core team, such as:
- Update any Azure RBAC permissions for test tenants/subscription, if needed.
- In case of Bicep modules only:
- Look for the module owners confirmation on the related
[Module Proposal]
issue that they have created the required-module-owners-
and-module-contributors-
GitHub teams. - Ensure the
-module-owners-
and-module-contributors-
GitHub teams have been assigned to their respective parent teams as outlined here. - Ensure the
CODEOWNERS
file in the BRM repo has been updated. - Ensure the
AVM Module Issue template
file in the BRM repo has been updated.
- Look for the module owners confirmation on the related
Post-Development issue management
Once module is developed and v0.1.0
has been published to the relevant registry
- Assign the ย Status: Module Available ๐ขย label to the issue.
- Move the issue into “
Done
” column in AVM - Modules Triage GitHub Project. - Update the AVM Module Indexes, following the process documented internally.
- Close the issue.
Important
- The Module Proposal issue MUST remain open until the module is fully developed, tested and published to the relevant registry.
- Do NOT close the issue before the successful publication is confirmed!
- Once the module is fully developed, tested and published to the relevant registry, and the Module Proposal issue was closed, it MUST remain closed.
Orphaned modules
When a module becomes orphaned
If a module meets the criteria described in the “Orphaned AVM Modules” chapter, the modules is considered to be orphaned and the below steps must be performed.
An issue is considered to be an orphaned module if
- it was opened through the “Orphaned AVM Module ๐” template,
- it has the labels of ย Needs: Triage ๐ย , ย Needs: Module Owner ๐ฃย and the ย Status: Module Orphaned ๐ย applied to it, and
- it is assigned to the “AVM - Module Triage” GitHub project.
Note
The original Module Proposal issue related to the module in question MUST remain closed and intact.
Instead, a new Orphaned Module issue must be opened that MUST remain open until the ownership is fully confirmed!
Once the Orphaned Module issue was closed, it MUST remain closed. If the module will subsequently become orphaned again, a new Orphaned Module issue must be opened.
- Create a new issue using the “Orphaned AVM Module ๐” issue template.
- Make sure the ย Needs: Triage ๐ย , ย Needs: Module Owner ๐ฃย , and the ย Status: Module Orphaned ๐ย labels are assigned to the issue.
- Move the issue into the “
Orphaned
” column on the AVM - Modules Triage GitHub Project board. - Update the AVM Module Indexes, following the process documented internally.
- Place an information notice as per the below guidelines:
- In case of a Bicep module:
- Place the information notice - with the text below - in an
ORPHANED.md
file, in the module’s root. - Run the
utilities/tools/Set-AVMModule.ps1
utility with the module path as an input. This re-generates the moduleโsREADME.md
file, so that theREADME.md
file will also contain the same notice in its header. - Make sure the content of the
ORPHANED.md
file is displayed in theREADME.md
in its header (right after the title).
- Place the information notice - with the text below - in an
- In case of a Terraform module, place the information notice - with the text below - in the
README.md
file, in the module’s root. - Once the information notice is placed, submit a Pull Request.
- In case of a Bicep module:
Include the following text in the information notice:
- Try to find a new owner using the AVM communities or await a new module owner to comment and propose themselves on the issue.
When a new owner is identified
Tip
To look for Orphaned Modules:
- Click on the following link to use this saved query โก๏ธ ย Status: Module Orphaned ๐ย โฌ ๏ธ.
- Check the
Orphaned
swim lane on the Module Triage board.
- When a new potential owner is identified, clarify the roles and responsibilities of the module owner:
- Clarify they understand and accept what “module ownership” means by replying in a comment to the requestor/proposed owner:
- Once the new module owner candidate has confirmed they understand and accept their roles and responsibilities as an AVM module owner
- Assign the issue to the confirmed module owner.
- Remove the ย Status: Module Orphaned ๐ย and the ย Needs: Module Owner ๐ฃย labels from the issue.
- Add the ย Status: Module Available ๐ขย and ย Status: Owners Identified ๐คย labels to the issue.
- Move the issue into the “
Done
” column on the AVM - Modules Triage GitHub Project board.
- Update the AVM Module Indexes, following the process documented internally.
- Get the new owner(s) and any new contributor(s) added to the related
-module-owners-
or-module-contributors-
teams. See SNFR20 for more details. - Remove the information notice (i.e., the file that states that
โ ๏ธTHIS MODULE IS CURRENTLY ORPHANED.โ ๏ธ, etc.
):- In case of a Bicep module:
- Delete the
ORPHANED.md
file from the module’s root. - Run the
utilities/tools/Set-AVMModule.ps1
utility with the module path as an input. This re-generates the moduleโsREADME.md
file, so that it will no longer contain the orphaned module notice in its header. - Double check the previous steps was successful and the
README.md
file no longer has the information notice in its header (right after the title).
- Delete the
- In case of a Terraform module, remove the information notice from the
README.md
file in the module’s root. - Once the information notice is removed, submit a Pull Request.
- In case of a Bicep module:
- Use the following text to confirm the new ownership of an orphaned module:
- Close the Orphaned Module issue.
General feedback/question, documentation update and other standard issues
An issue is a “General Question/Feedback โ” if it was opened through the “General Question/Feedback โ” issue template, and has the labels of ย Type: Question/Feedback ๐โโ๏ธย and ย Needs: Triage ๐ย applied to it.
An issue is a “AVM Documentation Update ๐” if it was opened through the “AVM Documentation Update ๐” issue template, and has the labels of ย Type: Documentation ๐ย and ย Needs: Triage ๐ย applied to it.
An issue is considered to be a “standard issue” or “blank issue” if it was opened without using an issue template, and hence it does NOT have any labels assigned, OR only has the ย Needs: Triage ๐ย label assigned.
When triaging the issue, consider adding one of the following labels as fits:
- ย Type: Documentation ๐ย
- ย Type: Feature Request โย
- ย Type: Bug ๐ย
- ย Type: Security Bug ๐ย
To see the full list of available labels, please refer to the GitHub Repo Labels section.
Note
If an intended module proposal was mistakenly opened as a “General Question/Feedback โ” or other standard issue, and hence, it doesn’t have the ย Type: New Module Proposal ๐กย label associated to it, a new issue MUST be created using the “New AVM Module Proposal ๐” issue template. The mistakenly created “General Question/Feedback โ” or other standard issue MUST be closed.