Skip to content
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

[Engineer build test] Fix for output type and null subscriptions collection in ResourceGraph #15165

Merged
merged 56 commits into from
Jun 10, 2021

Conversation

fenwickt
Copy link
Contributor

@fenwickt fenwickt commented Jun 2, 2021

Description

PR #15135

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:
    • a cmdlet design review was approved for the changes in this repository (Microsoft internal only)
      • {Please put the link here}
    • the markdown help files have been regenerated using the commands listed here

anakarparida and others added 24 commits March 26, 2021 10:48
* Onboard DigitalTwins to Private Link Common Cmdlets

* Removed the change log

* Added DigitalTwins in lexicographical order
)

* Add Azure Route Server CRUD cmdlets to deprecate Virtual Router cmdlets (Azure#14036)

* change name from virtual router to route server

* add test records

* fix warning

* delete unused import

* no snat private ranges

* adding private range in firewall policies

* using AzPSArgumentException instead of PSArgumentException

* using Resources.resx for AzPSArgumentException

* reverting changes in Resources.Designer.cs that is not related to my code

* adding help instruction for set-AzFirewallPolicy

* updated changeLog

Co-authored-by: litchiyangMSFT <64560090+litchiyangMSFT@users.noreply.github.com>
* Adding new property to Virtual Network FlowTimeoutInMinutes

* Reverting local SDK changes

* Added recorded test and updated help files, changed FlowTimeoutInMinutes to FlowTimeout

* Update src/Network/Network/VirtualNetwork/NewAzureVirtualNetworkCommand.cs

Removing Abbreviation of mins and expanding to minutes

Co-authored-by: Beisi Zhou <zhoubeisi@gmail.com>

* Updated FlowTimeout Description

* Generated New Test Recording and updated PS Test script

* Adding local SDK for FlowTimeout

* Update the Network sdk version in test projects

* Update the Network sdk version in test projects

* Updated FlowTimeout Description

* Update the Network sdk version in test projects

Co-authored-by: Beisi Zhou <zhoubeisi@gmail.com>
Co-authored-by: wyunchi-ms <yunwang@microsoft.com>
* workingCode

* VNGCommands

* ConnChanges

* Format

* helpFiles

* tests

* changeLog

* Revert "changeLog"

This reverts commit 8dbb8a2.

* ChangeLogMd

* fixing csProj

* Revert "Revert "changeLog""

This reverts commit 7f69613.

* Fixing nupkg

* TestCsProj

* testRecords

* fixing comments

* Fixing Comments

Co-authored-by: Khushboo Baheti <khbaheti@microsoft.com>
* Fixed Azure Vnet peering build fix for new SDK

* Added support for VPN or ER selection

* Added default display

* Bug fixes

* Updated change log

* Reverting VNET Peering change mismatch with SDK

* Fixed help file
…viceConnection (Azure#14786)

Co-authored-by: Beisi Zhou <bez@microsoft.com>
Co-authored-by: Khushboo Baheti <khbaheti@microsoft.com>
* Added support for getting a single private link resource

* update help file

* Update src/Network/Network/help/Get-AzPrivateLinkResource.md

* Update src/Network/Network/help/Get-AzPrivateLinkResource.md

Co-authored-by: Beisi Zhou <bez@microsoft.com>
…ficate (Azure#14784)

* expose two read-only client cert properties

* update appgw senario test to cover two new read-only properties of client cert

* create zonal public ip referred by appgw with zone redundancy

* update session record

* update ChangeLog

* use cert chain name from appgw

* check client cert properties from appgw output

* update test case and record
* new branch

* fix

* fix

* fix

* fix

* fix

* try restart json

* replace file

Co-authored-by: matyang222 <matthew.yang@microsoft.com>
…te in peered vNets (Azure#14730)

* Changes for adding new commandlet for sync operation

* Reverting the unintended change related to flow timeout

* Updating with the SDK that has the newer fields

Updating with the SDK that has the newer fields

* Updating the networking SDK for all references

Updating the networking SDK for all references

* Added Parameter Set & Updated Changelog.md & Recorded PS Test

* Minor update to changelog.md file and removed comment from VirtualNetworkTests.cs

* Added default parameter set and ShouldSupportProcess=false

* Added default parameter set and ShouldSupportProcess=false

* Sync-AzVirtualNetworkPeering.md Help file

* Updated links in help files

* Minor spacing issue in help files hyperlink

* Changed synopsis and description from comment to actual string

* Added Sync-AzVirtualNetworkPeering to StaticAnalysis exception to ignore the shouldProcess error

* Updating build steps

* Setting supportsshouldprocess = true in sync-azvnetpeering

* Reverting the Signature Issues

Co-authored-by: Hari Prasad Perabattula <haperaba@microsoft.com>
Co-authored-by: Satya-anshu <70507845+Satya-anshu@users.noreply.github.com>
…zure#15108)

* fix

* fix

Co-authored-by: matyang222 <matthew.yang@microsoft.com>
Upgrade Linux Image of CI pipeline to Ubuntu 20.04

Co-authored-by: wyunchi-ms <yunwang@microsoft.com>
Co-authored-by: wyunchi-ms <yunwang@microsoft.com>
* Release Az.Resources 4.1.1 (Azure#15149)

* [Resource] Fixed issue that property  is cleaned by  [Azure#15134]. (Azure#15136)

* Az.Resources 4.1.1 release

* Update version of Az.Accounts in psd1

Co-authored-by: LucasYao93 <53558334+LucasYao93@users.noreply.github.com>

* Update ChangeLog.md

Co-authored-by: LucasYao93 <53558334+LucasYao93@users.noreply.github.com>
* Updating Az.SecurityInsights help files

* Update Get-AzSentinelAlertRule.md
…in cmdlet (Azure#15155)

Co-authored-by: wyunchi-ms <yunwang@microsoft.com>
* First availability of Az.DiskPool

* supress error

* remove unnecessary suppress
…uthentication & External Administrator Properties (Azure#15151)

* Rebase master

* Update Changelog

* Switch to singular noun

* Fix bug

* Check for administrator object

* Fix test
@fenwickt fenwickt requested a review from VeryEarly June 2, 2021 06:49
viparek and others added 20 commits June 8, 2021 09:45
* Adding CMK changes for PS cmdlets

* Add UMI. Address comments

* Added UMI related config

* Fix build error

* Fix static analysis errors

* UMI related fix.

* Fix test and build issues

* Update with UMI scenarios

* Fix generated files

* Fix identity issue

* Fix failing test

* Modified help files. Modified changelog.md

* Update AzureSqlDatabaseTransparentDataEncryptionCommunicator.cs

Remove legacy codes

Co-authored-by: Jin Lei <54836179+msJinLei@users.noreply.github.com>
* Use certificate file as input

* Address review comments
New-AzRoleAssignment Cmdlet needs graph token, so during Connect-AzAccount pass GraphAccessToken
* DeleteOptions for Disks and NetworkInterfaces

* DeleteOption fixes

* Upgrade to Microsoft.Azure.Management.Compute Version 47.0.0

* Updated help files

* temporyr change to move Compute package to version 47

Co-authored-by: Bashar Gharaibeh <basharghar@live.com>
…RUD command that complain missing resourcegroupName when using InputObject (Azure#15166)

* [ApplicationInsights] ApplicationInsightsLinkedStorageAccount CRUD command that complain missing resourcegroupName when using InputObject [Azure#14848].

* Update ChangeLog.md

* [ApplicationInsights] Add CRUD test cases of the LinkedStorageAccount for pipeline.

Co-authored-by: Dingmeng Xue <dixue@microsoft.com>
[Do not squash] merge network features to master
Azure#15203)

New-AzRoleAssignment Cmdlet needs graph token, so during Connect-AzAccount pass GraphAccessToken

Co-authored-by: digot007 <digot@microsoft.com>
@VeryEarly VeryEarly merged commit 4d1f891 into Azure:yabo/ResourceGraph-Engineerbuild Jun 10, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.