Skip to content

Conversation

gayatriramac
Copy link
Contributor

Added a powershell script to add the mentioned 8 network security rules (4 inbound and 4 outbound) to the network security group created for AzureBastionSubnet.

This script is tested.

Inbound Rules:
Via Script:
image

Via Documentation:
image

Outbound Rules:
Via Script:
image

Via Documentation:
image

Note: The protocol is "Ah" as the powershell code uses "Ah" for "Any" protocol type.
image

Added a powershell script to add the mentioned 8 network security rules (4 inbound and 4 outbound) to the network security group created for AzureBastionSubnet.

This script is tested.

Note: The protocol is "Ah" as the powershell code uses "Ah" for "Any" protocol type.
Copy link
Contributor

@gayatriramac : Thanks for your contribution! The author(s) and reviewer(s) have been notified to review your proposed change. @abell

Copy link
Contributor

Learn Build status updates of commit 6e319b8:

⚠️ Validation status: warnings

File Status Preview URL Details
articles/bastion/bastion-nsg.md ⚠️Warning Details

articles/bastion/bastion-nsg.md

  • Line 58, Column 1: [Warning: code-block-unclosed - See documentation] Unclosed code block. Code blocks must begin and end with triple backticks (```).

For more details, please refer to the build report.

Note: Your PR may contain errors or warnings or suggestions unrelated to the files you changed. This happens when external dependencies like GitHub alias, Microsoft alias, cross repo links are updated. Please use these instructions to resolve them.

@Court72
Copy link
Contributor

Court72 commented Aug 8, 2025

@AbdullahBell

Can you review the proposed changes? Note that the validation warning must be resolved before merging.

Important: When the changes are ready for publication, adding a #sign-off comment is the best way to signal that the PR is ready for the review team to merge.

#label:"aq-pr-triaged"
@MicrosoftDocs/public-repo-pr-review-team

@prmerger-automator prmerger-automator bot added the aq-pr-triaged tracking label for the PR review team label Aug 8, 2025
@isamorris99
Copy link
Contributor

#sign-off

Copy link
Contributor

Invalid command: '#sign-off'. Only the assigned author of one or more file in this PR can sign off. @abell

@Court72
Copy link
Contributor

Court72 commented Aug 11, 2025

@isamorris99,

PRMerger will only accept sign off from the article's listed author. However, if you are a Microsoft employee authorized to update this content, you can send a request to [email protected] for this PR to be merged. Include your role and an explanation of why the PR should be merged without approval from the listed author.

Important: The validation warning must be resolved before we can merge.

Copy link
Contributor

Learn Build status updates of commit 0f3e2ab:

✅ Validation status: passed

File Status Preview URL Details
articles/bastion/bastion-nsg.md ✅Succeeded

For more details, please refer to the build report.

Copy link
Contributor

PRMerger Results

Issue Description
File Change Percent This PR contains file(s) with more than 30% file change.

@Court72
Copy link
Contributor

Court72 commented Aug 18, 2025

Thanks, @AbdullahBell. I committed your suggestion. When the changes are ready for publication, adding a #sign-off comment is the best way to signal that the PR is ready for the review team to merge.

@AbdullahBell
Copy link
Contributor

#sign-off

Copy link
Contributor

Invalid command: '#sign-off'. Only the assigned author of one or more file in this PR can sign off. @abell

@v-dirichards v-dirichards merged commit 629404e into MicrosoftDocs:main Aug 19, 2025
2 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.

5 participants