Skip to content

Register Microsoft.HardwareSecurityModules/paymentHsmClusters RP in Network cmdlets - #29839

Merged
VeryEarly merged 2 commits into
Azure:mainfrom
D-Alegria:add-paymenthsmclusters
Jul 19, 2026
Merged

Register Microsoft.HardwareSecurityModules/paymentHsmClusters RP in Network cmdlets#29839
VeryEarly merged 2 commits into
Azure:mainfrom
D-Alegria:add-paymenthsmclusters

Conversation

@D-Alegria

@D-Alegria D-Alegria commented Jul 13, 2026

Copy link
Copy Markdown
Contributor

…etwork cmdlets

Description

Adding Microsoft.HardwareSecurityModules/paymentHsmClusters RP in Network PrivateLinkService cmdlet

Microsoft.HardwareSecurityModules/paymentHsmClusters RP supports

List Private Link Resource API
Get Private Endpoint Connection API
List Private Endpoint Connection API
Approve Private Endpoint Connection
Deny Private Endpoint Connection
Remove Private Endpoint Connection

Mandatory Checklist

  • SHOULD update ChangeLog.md file(s) appropriately
    • Update src/Network/Network/ChangeLog.md.
      • A snippet outlining the change(s) made in the PR should be written under the ## Upcoming Release header in the past tense.
    • Should not change ChangeLog.md if no new release is required, such as fixing test case only.
  • SHOULD regenerate markdown help files if there is cmdlet API change. Instruction
  • SHOULD have proper test coverage for changes in pull request.
  • SHOULD NOT adjust version of module manually in pull request

Copilot AI review requested due to automatic review settings July 13, 2026 12:14
@azure-client-tools-bot-prd

Copy link
Copy Markdown
Thanks for your contribution! The pull request validation has started. Please revisit this comment for updated status.

@azure-pipelines

Copy link
Copy Markdown
Contributor
Azure Pipelines:
Successfully started running 2 pipeline(s).
There may be pipelines that require an authorized user to comment /azp run to run.

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

This PR extends the Network module’s Private Link common cmdlets support by registering the Microsoft.HardwareSecurityModules/paymentHsmClusters resource type in the PrivateLinkService provider configuration and documenting the addition in the module ChangeLog.

Changes:

  • Registered Microsoft.HardwareSecurityModules/paymentHsmClusters with API version 2025-12-01-preview in ProviderConfiguration.
  • Added a corresponding entry under ## Upcoming Release in the Network module ChangeLog.

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 1 comment.

File Description
src/Network/Network/PrivateLinkService/PrivateLinkServiceProvider/ProviderConfiguration.cs Adds a new provider configuration entry so Private Link cmdlets can recognize and operate on paymentHsmClusters.
src/Network/Network/ChangeLog.md Documents the newly onboarded resource type for the upcoming Network module release notes.

Comment thread src/Network/Network/ChangeLog.md Outdated
Copilot AI review requested due to automatic review settings July 15, 2026 11:07
@D-Alegria
D-Alegria marked this pull request as ready for review July 15, 2026 11:10
@azure-pipelines

Copy link
Copy Markdown
Contributor
Azure Pipelines:
There may be pipelines that require an authorized user to comment /azp run to run.

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

Copilot reviewed 2 out of 2 changed files in this pull request and generated no new comments.

@VeryEarly

Copy link
Copy Markdown
Collaborator

/azp run

@azure-pipelines

Copy link
Copy Markdown
Contributor
Azure Pipelines:
Successfully started running 3 pipeline(s).

@VeryEarly
VeryEarly merged commit 4ebb204 into Azure:main Jul 19, 2026
13 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants