-
Notifications
You must be signed in to change notification settings - Fork 3.9k
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
VeryEarly
merged 56 commits into
Azure:yabo/ResourceGraph-Engineerbuild
from
fenwickt:output
Jun 10, 2021
Merged
[Engineer build test] Fix for output type and null subscriptions collection in ResourceGraph #15165
VeryEarly
merged 56 commits into
Azure:yabo/ResourceGraph-Engineerbuild
from
fenwickt:output
Jun 10, 2021
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
* 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 <[email protected]>
* 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 <[email protected]> * 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 <[email protected]> Co-authored-by: wyunchi-ms <[email protected]>
* 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 <[email protected]>
* 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 <[email protected]>
Co-authored-by: Khushboo Baheti <[email protected]>
* 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 <[email protected]>
…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 <[email protected]>
…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 <[email protected]> Co-authored-by: Satya-anshu <[email protected]>
…zure#15108) * fix * fix Co-authored-by: matyang222 <[email protected]>
Upgrade Linux Image of CI pipeline to Ubuntu 20.04 Co-authored-by: wyunchi-ms <[email protected]>
Co-authored-by: wyunchi-ms <[email protected]>
* 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 <[email protected]> * Update ChangeLog.md Co-authored-by: LucasYao93 <[email protected]>
* Updating Az.SecurityInsights help files * Update Get-AzSentinelAlertRule.md
…in cmdlet (Azure#15155) Co-authored-by: wyunchi-ms <[email protected]>
* 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
* four cmdlets * location to follow resource group * add supportshouldprocess * update test * update key save location * capitalization * change * change * add two new parameter sets
* 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 <[email protected]>
* Use certificate file as input * Address review comments
…zure#15194) * fix day 1 behavior for route server * record and log
New-AzRoleAssignment Cmdlet needs graph token, so during Connect-AzAccount pass GraphAccessToken
…ultSharePermission (Azure#15190)
…re#15193)" (Azure#15201) This reverts commit e1f82c9.
* 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 <[email protected]>
…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 <[email protected]>
[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 <[email protected]>
Changes from PR Azure#15125
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Description
PR #15135
Checklist
CONTRIBUTING.md
ChangeLog.md
file(s) has been updated:ChangeLog.md
file can be found atsrc/{{SERVICE}}/{{SERVICE}}/ChangeLog.md
## Upcoming Release
header -- no new version header should be added