Commit 13555e6
fix: Update
## Description
This PR resolves the following issues in the
`avm/res/net-app/net-app-account` module
* On the NetApp Volume submodule, `zone` has been renamed to
`availabilityZone`.
* The type for `availabilityZone` has been changed to `string` from
`int[]` and is now optional with null as the default. Previously the
module would try and deploy a volume into all three zones if no value is
defined.
* Changed: `protocolTypes` is now an optional array of strings with
three possible options `'NFSv3', 'NFSv4.1', and 'CIFS'`. If no value is
provided, the default value of `['NFSv3']` is used.
* Added to the description for `protocolTypes` indicating that the user
will need to specify `['NFSv3','CIFS']` or `['NFSv4.1','CIFS']` if
creating dual-stack volumes.
* Added in additional submodule README.md and main.json files generated
by Set-AVMModule.
<!--
>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 #4506
Closes #4534
-->
## Pipeline Reference
<!-- Insert your Pipeline Status Badge below -->
| Pipeline |
| -------- |
|
[](https://github.com/thecmdradama/bicep-registry-modules/actions/workflows/avm.res.net-app.net-app-account.yml)
|
## Type of Change
<!-- Use the checkboxes [x] on the options that are relevant. -->
- [ ] Update to CI Environment or utilities (Non-module affecting
changes)
- [x] Azure Verified Module updates:
- [ ] Bugfix containing backwards-compatible bug fixes, and I have NOT
bumped the MAJOR or MINOR version in `version.json`:
- [ ] Someone has opened a bug report issue, and I have included "Closes
#{bug_report_issue_number}" in the PR description.
- [ ] The bug was found by the module author, and no one has opened an
issue to report it yet.
- [x] 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`.
- [x] Update to documentation
## 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
<!-- Please keep up to date with the contribution guide at
https://aka.ms/avm/contribute/bicep -->
---------
Co-authored-by: Adam Ricket <adam.ricket@cybercx.com.au>
Co-authored-by: Alexander Sehr <ASehr@hotmail.de>avm/res/net-app/net-app-account - fix optional zone for volumes and specify protocolTypes options (#4469)1 parent 7732c3f commit 13555e6
File tree
16 files changed
+639
-223
lines changed- avm/res/net-app/net-app-account
- backup-policies
- backup-vault
- backup
- capacity-pool
- volume
- snapshot-policies
- tests/e2e
- max
- nfs3
16 files changed
+639
-223
lines changedLarge diffs are not rendered by default.
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
4 | 4 | | |
5 | 5 | | |
6 | 6 | | |
7 | | - | |
8 | | - | |
| 7 | + | |
| 8 | + | |
9 | 9 | | |
10 | 10 | | |
11 | 11 | | |
| |||
Lines changed: 2 additions & 2 deletions
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
5 | 5 | | |
6 | 6 | | |
7 | 7 | | |
8 | | - | |
9 | | - | |
| 8 | + | |
| 9 | + | |
10 | 10 | | |
11 | 11 | | |
12 | 12 | | |
| |||
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
5 | 5 | | |
6 | 6 | | |
7 | 7 | | |
8 | | - | |
9 | | - | |
| 8 | + | |
| 9 | + | |
10 | 10 | | |
11 | 11 | | |
12 | 12 | | |
| |||
144 | 144 | | |
145 | 145 | | |
146 | 146 | | |
147 | | - | |
148 | | - | |
| 147 | + | |
| 148 | + | |
149 | 149 | | |
150 | 150 | | |
151 | 151 | | |
| |||
0 commit comments