Skip to content

Add Encryption Parameters in VNet #16129

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 13 commits into from
Oct 22, 2021
Merged

Add Encryption Parameters in VNet #16129

merged 13 commits into from
Oct 22, 2021

Conversation

dtuCloud
Copy link
Contributor

@dtuCloud dtuCloud commented Oct 20, 2021 •
edited
Loading

Description

  1. add 2 optional parameters (EnableEncryption, EncryptionEnforcementPolicy ) to virtual network so customer can enable encryption and specify the encryption policy.
  2. add 1 property "VnetEncryptionSupported", inside the PSNetworkInterface, it is read-only, and does not affect existing commands.
  3. add 1 property "RemoteVirtualNetworkEncryption" inside PSVirtualNetworkPeering, it is read-only, and does not affect existing commands.

Checklist

  • I have read the Submitting Changes section of CONTRIBUTING.md
  • The title of the PR is clear and informative
  • The appropriate ChangeLog.md file(s) has been updated:
    • For any service, the ChangeLog.md file can be found at src/{{SERVICE}}/{{SERVICE}}/ChangeLog.md
    • A snippet outlining the change(s) made in the PR should be written under the ## Upcoming Release header -- no new version header should be added
  • The PR does not introduce breaking changes
  • If applicable, the changes made in the PR have proper test coverage
  • For public API changes to cmdlets:

@dtuCloud dtuCloud marked this pull request as ready for review October 20, 2021 09:13
@BethanyZhou BethanyZhou merged commit 43c9c3b into Azure:network-may-21 Oct 22, 2021
isra-fel added a commit that referenced this pull request Nov 9, 2021
@MikhailTryakhov @shengliumsft
@tejasshah7 @dtuCloud @BethanyZhou @isra-fel
Azure Network May release (#16327)
40b58bc 
* [Network] Add Optional Parameter IsSecuritySite to New-AzVpnSite Cmdlet (#15912)

* add isSecuritySite to vpnSite

* change to switchParameter

* add record test

* add test result

* add test with vpnsitelink

* add test with vpnsitelink

* add test with vpnsitelink

* add tests for backward compatibility

* changeLog updated

* changeLog updated

* add help files

* Support for new WAF Exclusion Match Variables (#16126)

* Support for WAF Per Rule Exclusions (#16127)

* Support for WAF Per Rule Exclusions

* Including SDK

* Add Encryption Parameters in VNet  (#16129)

* draft: local debug able to send command

* fix the console output

* add tests

* change help md

* record and change log

* add assertion on peering

* try adding json file to csproj

* try wait for the job to be done

* change to liveonly

* address comment: update help message

Co-authored-by: Beisi Zhou <[email protected]>

* Added new nuget package (#16274)

* Update ChangeLog.md

* Removed conflict mark

* Moved bastion changes to the correct one

* fixed incorrect merging section

Co-authored-by: Sheng Liu <[email protected]>
Co-authored-by: tejasshah7 <[email protected]>
Co-authored-by: dtuCloud <[email protected]>
Co-authored-by: Beisi Zhou <[email protected]>
Co-authored-by: Yeming Liu <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@BethanyZhou BethanyZhou BethanyZhou approved these changes

@MikhailTryakhov MikhailTryakhov Awaiting requested review from MikhailTryakhov

Assignees
No one assigned
Labels
Waiting for CI :shipit:
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@dtuCloud @BethanyZhou