Repository navigation
Liftr Commvault PowerShell cmdlets for stable version 2026-09-30 - #30352
Open
Saurabh Jaiswal (saurabh-jaiswal-333) wants to merge 2 commits into
Open
Saurabh Jaiswal (saurabh-jaiswal-333) wants to merge 2 commits into
Saurabh Jaiswal (saurabh-jaiswal-333) wants to merge 2 commits into
Conversation
… API 2026-09-30 Autorest-generated module (0.1.0) for the Azure Native ISV service Liftr-Commvault: 34 cmdlets for cloud accounts, plans, storages, protection groups, role mappings and protected items, with examples, help, UX metadata and test stubs. Generated from azure-rest-api-specs commit e91f0b95 (tag package-2026-09-30). Adds the module to tools/CreateMappings_rules.json. Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
Contributor
|
Azure Pipelines: There may be pipelines that require an authorized user to comment /azp run to run. |
Contributor
|
Thank you for your contribution Saurabh Jaiswal (@saurabh-jaiswal-333)! We will review the pull request and get back to you soon. |
Member
|
/azp run |
Contributor
|
Azure Pipelines: Successfully started running 3 pipeline(s). |
Contributor
There was a problem hiding this comment.
🟡 Changes recommended
One or more issues must be addressed before approval.
5 open issues
What changed in this PR
Introduces supporting infrastructure for the AutoRest-based Az.Commvault module targeting the stable 2026-09-30 API.
Changes:
- Adds module project metadata and Commvault documentation mappings.
- Documents resource management, backup/restore, SaaS activation, and compliance-lock workflows.
- Adds examples, UX metadata, shared helpers, and skipped scenario-test scaffolding.
Review validation included static inspection, comparison with pinned API examples, and a focused changelog-selection check; builds and scenario tests were not run.
| File | Description |
|---|---|
| tools/CreateMappings_rules.json | Updated as part of this pull request. |
| src/Commvault/Commvault/Properties/AssemblyInfo.cs | Updated as part of this pull request. |
| src/Commvault/Commvault/help/Update-AzCommvaultRoleMapping.md | Updated as part of this pull request. |
| src/Commvault/Commvault/help/Remove-AzCommvaultRoleMapping.md | Updated as part of this pull request. |
| src/Commvault/Commvault/help/Remove-AzCommvaultCloudAccount.md | Updated as part of this pull request. |
| src/Commvault/Commvault/help/Get-AzCommvaultStorage.md | Updated as part of this pull request. |
| src/Commvault/Commvault/help/Get-AzCommvaultRoleMapping.md | Updated as part of this pull request. |
| src/Commvault/Commvault/help/Get-AzCommvaultProtectionGroup.md | Updated as part of this pull request. |
| src/Commvault/Commvault/help/Get-AzCommvaultProtectedItemCount.md | Updated as part of this pull request. |
| src/Commvault/Commvault/help/Get-AzCommvaultPlan.md | Updated as part of this pull request. |
| src/Commvault/Commvault/help/Get-AzCommvaultCloudAccountLatestLinkedSaaS.md | Updated as part of this pull request. |
| src/Commvault/Commvault/help/Get-AzCommvaultCloudAccount.md | Updated as part of this pull request. |
| src/Commvault/Commvault/help/Enable-AzCommvaultStorageComplianceLock.md | Updated as part of this pull request. |
| src/Commvault/Commvault/help/Az.Commvault.md | Updated as part of this pull request. |
| src/Commvault/Commvault/Commvault.csproj | Updated as part of this pull request. |
| src/Commvault/Commvault/ChangeLog.md | Updated as part of this pull request. |
| src/Commvault/Commvault.Autorest/UX/Commvault.ContentStore/cloudAccounts-protectionGroups.json | Updated as part of this pull request. |
| src/Commvault/Commvault.Autorest/UX/Commvault.ContentStore/cloudAccounts-protectionGroups-protectedItems.json | Updated as part of this pull request. |
| src/Commvault/Commvault.Autorest/UX/Commvault.ContentStore/cloudAccounts-plans.json | Updated as part of this pull request. |
| src/Commvault/Commvault.Autorest/utils/Unprotect-SecureString.ps1 | Updated as part of this pull request. |
| src/Commvault/Commvault.Autorest/utils/Get-SubscriptionIdTestSafe.ps1 | Updated as part of this pull request. |
| src/Commvault/Commvault.Autorest/test/utils.ps1 | Updated as part of this pull request. |
| src/Commvault/Commvault.Autorest/test/Update-AzCommvaultStorage.Tests.ps1 | Updated as part of this pull request. |
| src/Commvault/Commvault.Autorest/test/Update-AzCommvaultRoleMapping.Tests.ps1 | Updated as part of this pull request. |
| src/Commvault/Commvault.Autorest/test/Update-AzCommvaultProtectionGroup.Tests.ps1 | Updated as part of this pull request. |
| src/Commvault/Commvault.Autorest/test/Update-AzCommvaultPlan.Tests.ps1 | Updated as part of this pull request. |
| src/Commvault/Commvault.Autorest/test/Update-AzCommvaultCloudAccount.Tests.ps1 | Updated as part of this pull request. |
| src/Commvault/Commvault.Autorest/test/Sync-AzCommvaultStorage.Tests.ps1 | Updated as part of this pull request. |
| src/Commvault/Commvault.Autorest/test/Stop-AzCommvaultProtectionGroupBackup.Tests.ps1 | Updated as part of this pull request. |
| src/Commvault/Commvault.Autorest/test/Resume-AzCommvaultProtectionGroupBackup.Tests.ps1 | Updated as part of this pull request. |
| src/Commvault/Commvault.Autorest/test/Restore-AzCommvaultProtectionGroup.Tests.ps1 | Updated as part of this pull request. |
| src/Commvault/Commvault.Autorest/test/Restore-AzCommvaultProtectedItem.Tests.ps1 | Updated as part of this pull request. |
| src/Commvault/Commvault.Autorest/test/Remove-AzCommvaultStorage.Tests.ps1 | Updated as part of this pull request. |
| src/Commvault/Commvault.Autorest/test/Remove-AzCommvaultRoleMapping.Tests.ps1 | Updated as part of this pull request. |
| src/Commvault/Commvault.Autorest/test/Remove-AzCommvaultProtectionGroup.Tests.ps1 | Updated as part of this pull request. |
| src/Commvault/Commvault.Autorest/test/Remove-AzCommvaultPlan.Tests.ps1 | Updated as part of this pull request. |
| src/Commvault/Commvault.Autorest/test/Remove-AzCommvaultCloudAccount.Tests.ps1 | Updated as part of this pull request. |
| src/Commvault/Commvault.Autorest/test/README.md | Updated as part of this pull request. |
| src/Commvault/Commvault.Autorest/test/New-AzCommvaultStorage.Tests.ps1 | Updated as part of this pull request. |
| src/Commvault/Commvault.Autorest/test/New-AzCommvaultRoleMapping.Tests.ps1 | Updated as part of this pull request. |
| src/Commvault/Commvault.Autorest/test/New-AzCommvaultProtectionGroup.Tests.ps1 | Updated as part of this pull request. |
| src/Commvault/Commvault.Autorest/test/New-AzCommvaultPlan.Tests.ps1 | Updated as part of this pull request. |
| src/Commvault/Commvault.Autorest/test/New-AzCommvaultCloudAccount.Tests.ps1 | Updated as part of this pull request. |
| src/Commvault/Commvault.Autorest/test/loadEnv.ps1 | Updated as part of this pull request. |
| src/Commvault/Commvault.Autorest/test/Initialize-AzCommvaultSaaS.Tests.ps1 | Updated as part of this pull request. |
| src/Commvault/Commvault.Autorest/test/Get-AzCommvaultStorage.Tests.ps1 | Updated as part of this pull request. |
| src/Commvault/Commvault.Autorest/test/Get-AzCommvaultRoleMapping.Tests.ps1 | Updated as part of this pull request. |
| src/Commvault/Commvault.Autorest/test/Get-AzCommvaultProtectionGroup.Tests.ps1 | Updated as part of this pull request. |
| src/Commvault/Commvault.Autorest/test/Get-AzCommvaultProtectedItemRestorePoint.Tests.ps1 | Updated as part of this pull request. |
| src/Commvault/Commvault.Autorest/test/Get-AzCommvaultProtectedItemCount.Tests.ps1 | Updated as part of this pull request. |
| src/Commvault/Commvault.Autorest/test/Get-AzCommvaultProtectedItem.Tests.ps1 | Updated as part of this pull request. |
| src/Commvault/Commvault.Autorest/test/Get-AzCommvaultPlan.Tests.ps1 | Updated as part of this pull request. |
| src/Commvault/Commvault.Autorest/test/Get-AzCommvaultCloudAccountLatestLinkedSaaS.Tests.ps1 | Updated as part of this pull request. |
| src/Commvault/Commvault.Autorest/test/Get-AzCommvaultCloudAccount.Tests.ps1 | Updated as part of this pull request. |
| src/Commvault/Commvault.Autorest/test/Enable-AzCommvaultStorageComplianceLock.Tests.ps1 | Updated as part of this pull request. |
| src/Commvault/Commvault.Autorest/test/Disable-AzCommvaultStorageComplianceLock.Tests.ps1 | Updated as part of this pull request. |
| src/Commvault/Commvault.Autorest/test/Connect-AzCommvaultCloudAccountSaaS.Tests.ps1 | Updated as part of this pull request. |
| src/Commvault/Commvault.Autorest/test/Backup-AzCommvaultProtectionGroup.Tests.ps1 | Updated as part of this pull request. |
| src/Commvault/Commvault.Autorest/resources/README.md | Updated as part of this pull request. |
| src/Commvault/Commvault.Autorest/Properties/AssemblyInfo.cs | Updated as part of this pull request. |
| src/Commvault/Commvault.Autorest/how-to.md | Updated as part of this pull request. |
| src/Commvault/Commvault.Autorest/generate-info.json | Updated as part of this pull request. |
| src/Commvault/Commvault.Autorest/examples/Update-AzCommvaultStorage.md | Updated as part of this pull request. |
| src/Commvault/Commvault.Autorest/examples/Update-AzCommvaultRoleMapping.md | Updated as part of this pull request. |
| src/Commvault/Commvault.Autorest/examples/Update-AzCommvaultProtectionGroup.md | Updated as part of this pull request. |
| src/Commvault/Commvault.Autorest/examples/Update-AzCommvaultPlan.md | Updated as part of this pull request. |
| src/Commvault/Commvault.Autorest/examples/Update-AzCommvaultCloudAccount.md | Updated as part of this pull request. |
| src/Commvault/Commvault.Autorest/examples/Sync-AzCommvaultStorage.md | Updated as part of this pull request. |
| src/Commvault/Commvault.Autorest/examples/Stop-AzCommvaultProtectionGroupBackup.md | Updated as part of this pull request. |
| src/Commvault/Commvault.Autorest/examples/Resume-AzCommvaultProtectionGroupBackup.md | Updated as part of this pull request. |
| src/Commvault/Commvault.Autorest/examples/Restore-AzCommvaultProtectionGroup.md | Updated as part of this pull request. |
| src/Commvault/Commvault.Autorest/examples/Restore-AzCommvaultProtectedItem.md | Updated as part of this pull request. |
| src/Commvault/Commvault.Autorest/examples/Remove-AzCommvaultStorage.md | Updated as part of this pull request. |
| src/Commvault/Commvault.Autorest/examples/Remove-AzCommvaultRoleMapping.md | Updated as part of this pull request. |
| src/Commvault/Commvault.Autorest/examples/Remove-AzCommvaultProtectionGroup.md | Updated as part of this pull request. |
| src/Commvault/Commvault.Autorest/examples/Remove-AzCommvaultPlan.md | Updated as part of this pull request. |
| src/Commvault/Commvault.Autorest/examples/Remove-AzCommvaultCloudAccount.md | Updated as part of this pull request. |
| src/Commvault/Commvault.Autorest/examples/New-AzCommvaultStorage.md | Updated as part of this pull request. |
| src/Commvault/Commvault.Autorest/examples/New-AzCommvaultRoleMapping.md | Updated as part of this pull request. |
| src/Commvault/Commvault.Autorest/examples/New-AzCommvaultProtectionGroup.md | Updated as part of this pull request. |
| src/Commvault/Commvault.Autorest/examples/New-AzCommvaultPlan.md | Updated as part of this pull request. |
| src/Commvault/Commvault.Autorest/examples/New-AzCommvaultCloudAccount.md | Updated as part of this pull request. |
| src/Commvault/Commvault.Autorest/examples/Initialize-AzCommvaultSaaS.md | Updated as part of this pull request. |
| src/Commvault/Commvault.Autorest/examples/Get-AzCommvaultStorage.md | Updated as part of this pull request. |
| src/Commvault/Commvault.Autorest/examples/Get-AzCommvaultRoleMapping.md | Updated as part of this pull request. |
| src/Commvault/Commvault.Autorest/examples/Get-AzCommvaultProtectionGroup.md | Updated as part of this pull request. |
| src/Commvault/Commvault.Autorest/examples/Get-AzCommvaultProtectedItemRestorePoint.md | Updated as part of this pull request. |
| src/Commvault/Commvault.Autorest/examples/Get-AzCommvaultProtectedItemCount.md | Updated as part of this pull request. |
| src/Commvault/Commvault.Autorest/examples/Get-AzCommvaultProtectedItem.md | Updated as part of this pull request. |
| src/Commvault/Commvault.Autorest/examples/Get-AzCommvaultPlan.md | Updated as part of this pull request. |
| src/Commvault/Commvault.Autorest/examples/Get-AzCommvaultCloudAccountLatestLinkedSaaS.md | Updated as part of this pull request. |
| src/Commvault/Commvault.Autorest/examples/Get-AzCommvaultCloudAccount.md | Updated as part of this pull request. |
| src/Commvault/Commvault.Autorest/examples/Enable-AzCommvaultStorageComplianceLock.md | Updated as part of this pull request. |
| src/Commvault/Commvault.Autorest/examples/Disable-AzCommvaultStorageComplianceLock.md | Updated as part of this pull request. |
| src/Commvault/Commvault.Autorest/examples/Connect-AzCommvaultCloudAccountSaaS.md | Updated as part of this pull request. |
| src/Commvault/Commvault.Autorest/examples/Backup-AzCommvaultProtectionGroup.md | Updated as part of this pull request. |
| src/Commvault/Commvault.Autorest/docs/Update-AzCommvaultRoleMapping.md | Updated as part of this pull request. |
| src/Commvault/Commvault.Autorest/docs/Remove-AzCommvaultRoleMapping.md | Updated as part of this pull request. |
| src/Commvault/Commvault.Autorest/docs/Remove-AzCommvaultCloudAccount.md | Updated as part of this pull request. |
| src/Commvault/Commvault.Autorest/docs/README.md | Updated as part of this pull request. |
| src/Commvault/Commvault.Autorest/docs/Get-AzCommvaultStorage.md | Updated as part of this pull request. |
| src/Commvault/Commvault.Autorest/docs/Get-AzCommvaultRoleMapping.md | Updated as part of this pull request. |
| src/Commvault/Commvault.Autorest/docs/Get-AzCommvaultProtectionGroup.md | Updated as part of this pull request. |
| src/Commvault/Commvault.Autorest/docs/Get-AzCommvaultProtectedItemCount.md | Updated as part of this pull request. |
| src/Commvault/Commvault.Autorest/docs/Get-AzCommvaultPlan.md | Updated as part of this pull request. |
| src/Commvault/Commvault.Autorest/docs/Get-AzCommvaultCloudAccountLatestLinkedSaaS.md | Updated as part of this pull request. |
| src/Commvault/Commvault.Autorest/docs/Get-AzCommvaultCloudAccount.md | Updated as part of this pull request. |
| src/Commvault/Commvault.Autorest/docs/Az.Commvault.md | Updated as part of this pull request. |
| src/Commvault/Commvault.Autorest/custom/README.md | Updated as part of this pull request. |
| src/Commvault/Commvault.Autorest/custom/Az.Commvault.custom.psm1 | Updated as part of this pull request. |
| src/Commvault/Commvault.Autorest/.gitignore | Updated as part of this pull request. |
| src/Commvault/Commvault.Autorest/.gitattributes | Updated as part of this pull request. |
Review effort: Balanced
- Remove the unexpanded Count parameter set of Get-AzCommvaultProtectedItemCount. A parameter set named Count makes ParameterSets.Count return the set instead of a number, which crashes the test coverage analysis and failed every Test job. - Move the initial release notes under Upcoming Release so the release tooling picks up the module. - Describe the protected item count as a total across the specified cloud accounts, matching the single count in the API response. Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
Member
|
/azp run |
Contributor
|
Azure Pipelines: Successfully started running 3 pipeline(s). |
Saurabh Jaiswal (saurabh-jaiswal-333)
added this to the
Az 16.5.0 (2026-11-03) milestone
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
🤖 PR Validation —⚠️ Review suggested
️✔️Az.Accounts
Description
New autorest-generated module Az.Commvault (
0.1.0) for the Azure Native ISV service Liftr-Commvault, generated from the stable Commvault.ContentStore API2026-09-30(azure-rest-api-specsspecification/liftrcommvault, tagpackage-2026-09-30, pinned to commite91f0b95c03a435a7d745e8c70930aadce552d39).Cmdlets (34): cloud accounts (
New/Get/Update/Remove-AzCommvaultCloudAccount), plans, storages, protection groups, role mapping, protected items, plus action cmdlets (Backup-,Restore-,Resume-/Stop-…ProtectionGroupBackup,Enable-/Disable-AzCommvaultStorageComplianceLock,Sync-AzCommvaultStorage,Get-AzCommvaultProtectedItemRestorePoint/ProtectedItemCount,Initialize-AzCommvaultSaaS,Connect-AzCommvaultCloudAccountSaaS,Get-AzCommvaultCloudAccountLatestLinkedSaaS).Design review: no cmdlet design review issue was filed for this module. The cmdlets are generated by autorest.powershell; the names that differ from the plain Get/New/Update/Remove pattern come from the operation names in the spec and are documented with a comment per directive in
Commvault.Autorest/README.md(for exampleSync-AzCommvaultStoragefor the storage refresh action and-StorageClassTypefor theclassproperty, which would otherwise end in a plural). Happy to open a design review if the team would like one.Validation done locally (same steps as the
powershell-corepipeline): Build, Analyze (static analysis exits 0; the Signature check reports warnings only, ConfirmImpact on theGet-list cmdlets, as for Az.Informatica) and Test. All 56 documented examples were also executed against a local mock ARM server generated from the swagger and its examples, with request bodies validated against the schema.Tests: the generated test stubs are skipped (
-skip). Creating a cloud account needs a real Commvault SaaS activation through the Azure Marketplace, so live recordings are not possible yet.Known service-side limitations (being raised with the Commvault RP team, not caused by the module): protection group and protected item operations require a user-context token that Azure PowerShell cannot obtain yet, and
Connect-AzCommvaultCloudAccountSaaS/Get-AzCommvaultCloudAccountLatestLinkedSaaScall actions that are declared in the spec but not yet implemented by the service.Mandatory Checklist
Please choose the target release of Azure PowerShell. (⚠️ Target release is a different concept from API readiness. Please click below links for details.)
Check this box to confirm: I have read the Submitting Changes section of
CONTRIBUTING.mdand reviewed the following information:ChangeLog.mdfile(s) appropriatelysrc/{{SERVICE}}/{{SERVICE}}/ChangeLog.md.## Upcoming Releaseheader in the past tense.ChangeLog.mdif no new release is required, such as fixing test case only.