Skip to content

PR for merging PowerShell Script for Ingress and Egress Rules for AzureBastionSubnet NSG #127491

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

gayatriramac
Copy link

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.

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.

4 participants