Skip to content

Conversation

@hundredacres
Copy link
Contributor

Description

Deprecating module.

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

@hundredacres hundredacres requested a review from a team July 28, 2025 16:06
@hundredacres hundredacres requested a review from a team as a code owner July 28, 2025 16:06
@avm-organizer avm-organizer bot added the Needs: Core Team 🧞 This item needs the AVM Core Team to review it label Jul 28, 2025
@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 Jul 28, 2025
Copy link
Collaborator

@AlexanderSehr AlexanderSehr left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think there are 3 more TODOs:

  1. Remove the workflow file (it will fail now anyways)
  2. Remove the entry from the issue template & CODEOWNERS
  3. Remove the Teams

Thoughts, @matebarabas ?

@AlexanderSehr AlexanderSehr linked an issue Jul 28, 2025 that may be closed by this pull request
2 tasks
@hundredacres
Copy link
Contributor Author

@AlexanderSehr @matebarabas the docs indicate a 2 PR process, where the additional items (remove the workflow file, etc.) are done in the 2nd PR.

@AlexanderSehr
Copy link
Collaborator

@AlexanderSehr @matebarabas the docs indicate a 2 PR process, where the additional items (remove the workflow file, etc.) are done in the 2nd PR.

Hey @hundredacres,
you're correct. We merged these guidelines in just the other day and had another meeting about it today. There's a fair chance we'll simplify the process to on PR that takes care of it all, as publishing a new version of the module just to get the 'deprecated' note into the description seems to be a bit of a overkill if you ask me 😄 . We'll settle this latest this Thursday and can then resolve this accordingly.

@AlexanderSehr
Copy link
Collaborator

AlexanderSehr commented Aug 4, 2025

Hey @hundredacres,
the PR with the updated guidelines just went in.
Just a few more things left to do

  • Add the statement NOTE: This is the last published version and the module has since been deprecated. to the top-most entry of the CHANGELOG.md ### Changes header
  • Remove the module's workflow
  • Remove the module from the Github issue template (avm_module_issue.yml)
  • Remove the module from the CODEOWNERS file (CODEOWNERS)
  • Undo the change to the version.json file. We won't publish another version for the deprecation
  • Delete the module's Contributor & Owner teams

@hundredacres hundredacres requested review from a team as code owners August 4, 2025 17:56
@hundredacres
Copy link
Contributor Author

@AlexanderSehr completed the remaining steps

AlexanderSehr
AlexanderSehr previously approved these changes Aug 4, 2025
@AlexanderSehr AlexanderSehr enabled auto-merge (squash) August 4, 2025 18:33
Copy link
Contributor

@ReneHezser ReneHezser left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@AlexanderSehr AlexanderSehr merged commit 9fcc464 into Azure:main Aug 4, 2025
4 checks passed
@karpikpl
Copy link
Contributor

module shows as active in AVM catalog
image

@matebarabas
Copy link
Collaborator

@karpikpl, this PR will update the status to deprecated. Thanks for calling this out!

yffok pushed a commit to yffok/bicep-registry-modules that referenced this pull request Jan 15, 2026
## Description

Deprecating module.

## Pipeline Reference

<!-- Insert your Pipeline Status Badge below -->

| Pipeline |
| -------- |


## Type of Change

<!-- Use the checkboxes [x] on the options that are relevant. -->

- 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

- [X] I'm sure there are no other open Pull Requests for the same
update/change
- [X] I have run `Set-AVMModule` locally to generate the supporting
module files.
- [X] 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

<!-- Please keep up to date with the contribution guide at
https://aka.ms/avm/contribute/bicep -->

---------

Co-authored-by: Alexander Sehr <ASehr@hotmail.de>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Needs: Core Team 🧞 This item needs the AVM Core Team to review it Needs: Triage 🔍 Maintainers need to triage still Type: AVM 🅰️ ✌️ Ⓜ️ This is an AVM related issue

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Module Deprecation]: avm/res/network/front-door

5 participants