Skip to content

Commit 05df015

Browse files
Kingshuk-MicrosoftPrajwal-Microsoftsethsteenken
authored
feat: codemod Update virtual network and storage account modules (#6260)
- Deleted the old virtual network module and replaced it with a new implementation. - Updated storage account module to use a new SKU default and improved parameter handling. - Enhanced the test cases for enterprise-grade configurations, including new parameters for Azure AI services. - Bumped version to 0.2 to reflect the changes made across modules. ## Description <!-- >Thank you for your contribution ! > Please include a summary of the change and which issue is fixed. > Please also include the context. > List any dependencies that are required for this change. Fixes #123 Closes #123 --> ## Pipeline Reference <!-- Insert your Pipeline Status Badge below --> | Pipeline | | -------- | [![avm.ptn.sa.modernize-your-code](https://github.com/Ravikirana-Microsoft/bicep-registry-modules/actions/workflows/avm.ptn.sa.modernize-your-code.yml/badge.svg?branch=avm%2Fptn%2Fsa%2Fmodernize-yourcode)](https://github.com/Ravikirana-Microsoft/bicep-registry-modules/actions/workflows/avm.ptn.sa.modernize-your-code.yml) ## 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 - [ ] 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 <!-- Please keep up to date with the contribution guide at https://aka.ms/avm/contribute/bicep --> --------- Co-authored-by: Prajwal-Microsoft <[email protected]> Co-authored-by: Seth Steenken <[email protected]>
1 parent 335ad4d commit 05df015

28 files changed

+41113
-52006
lines changed

.github/workflows/avm.ptn.sa.modernize-your-code.yml

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -28,8 +28,8 @@ on:
2828
paths:
2929
- ".github/actions/templates/avm-**"
3030
- ".github/workflows/avm.template.module.yml"
31-
- ".github/workflows/avm.ptn.azd.ml-hub-dependencies.yml"
32-
- "avm/ptn/azd/ml-hub-dependencies/**"
31+
- ".github/workflows/avm.ptn.sa.modernize-your-code.yml"
32+
- "avm/ptn/sa/modernize-your-code/**"
3333
- "utilities/pipelines/**"
3434
- "!utilities/pipelines/platform/**"
3535
- "!*/**/child-module-publish-allowed-list.json"

avm/ptn/sa/modernize-your-code/CHANGELOG.md

Lines changed: 16 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -2,6 +2,21 @@
22

33
The latest version of the changelog can be found [here](https://github.com/Azure/bicep-registry-modules/blob/main/avm/ptn/sa/modernize-your-code/CHANGELOG.md).
44

5+
## 0.2.0
6+
7+
### Changes
8+
9+
- Fix Bicep linter warnings (`no-unused-params`, `use-recent-api-versions`)
10+
- Update parameter descriptions to comply with AVM standards (Required./Optional. prefix and trailing dot)
11+
- Remove support for existing Log Analytics and AI Project resource IDs
12+
- Update API versions for Key Vault and Cognitive Services
13+
14+
### Breaking Changes
15+
16+
- Removed parameters:
17+
- `azureExistingAIProjectResourceId`
18+
- `existingLogAnalyticsWorkspaceId`
19+
520
## 0.1.0
621

722
### Changes
@@ -11,4 +26,4 @@ The latest version of the changelog can be found [here](https://github.com/Azure
1126

1227
### Breaking Changes
1328

14-
- None
29+
- None

avm/ptn/sa/modernize-your-code/README.md

Lines changed: 256 additions & 124 deletions
Large diffs are not rendered by default.

0 commit comments

Comments
 (0)