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.

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

Sync Compute folder from Master, and updated tests for 2021-03 #21209

Merged
merged 841 commits into from
May 19, 2021
Merged

Sync Compute folder from Master, and updated tests for 2021-03 #21209

merged 841 commits into from
May 19, 2021

Conversation

Sandido
Copy link
Member

@Sandido Sandido commented May 19, 2021

All SDK Contribution checklist:

This checklist is used to make sure that common guidelines for a pull request are followed.

  • Please open PR in Draft mode if it is:
    • Work in progress or not intended to be merged.
    • Encountering multiple pipeline failures and working on fixes.
  • If an SDK is being regenerated based on a new swagger spec, a link to the pull request containing these swagger spec changes has been included above.
  • I have read the contribution guidelines.
  • The pull request does not introduce breaking changes.

General Guidelines and Best Practices

  • Title of the pull request is clear and informative.
  • There are a small number of commits, each of which have an informative message. This means that previously merged commits do not appear in the history of the PR. For more information on cleaning up the commits in your PR, see this page.

Testing Guidelines

  • Pull request includes test coverage for the included changes.

SDK Generation Guidelines

  • The generate.cmd file for the SDK has been updated with the version of AutoRest, as well as the commitid of your swagger spec or link to the swagger spec, used to generate the code. (Track 2 only)
  • The *.csproj and AssemblyInfo.cs files have been updated with the new version of the SDK. Please double check nuget.org current release version.

Additional management plane SDK specific contribution checklist:

Note: Only applies to Microsoft.Azure.Management.[RP] or Azure.ResourceManager.[RP]

  • Include updated management metadata.
  • Update AzureRP.props to add/remove version info to maintain up to date API versions.

Management plane SDK Troubleshooting

  • If this is very first SDK for a services and you are adding new service folders directly under /SDK, please add new service label and/or contact assigned reviewer.

  • If the check fails at the Verify Code Generation step, please ensure:

    • Do not modify any code in generated folders.
    • Do not selectively include/remove generated files in the PR.
    • Do use generate.ps1/cmd to generate this PR instead of calling autorest directly.
      Please pay attention to the @microsoft.csharp version output after running generate.ps1. If it is lower than current released version (2.3.82), please run it again as it should pull down the latest version.

    Note: We have recently updated the PSH module called by generate.ps1 to emit additional data. This would help reduce/eliminate the Code Verification check error. Please run following command:

      `dotnet msbuild eng/mgmt.proj /t:Util /p:UtilityName=InstallPsModules`

Old outstanding PR cleanup

Please note:
If PRs (including draft) has been out for more than 60 days and there are no responses from our query or followups, they will be closed to maintain a concise list for our reviewers.

bikamani and others added 30 commits April 29, 2021 10:51
@bikamani
[TSI] Bring sample changes to master (#20755)
339eaaf
@LarryOsterman @jsquire
Revisions from the MAA API Review (#20558)
c099967 
Significant revisions to the API surface to reflect API review comments.

Co-authored-by: Jesse Squire <[email protected]>
@minnieliu
[Communication] - SMS - Re-enabling Unauthorized Number SMS Test (#20642
1b559a9 
)

* Re-enabling Unauthorized Number SMS Test

* Update recordings

Co-authored-by: Minnie Liu <[email protected]>
@pakrym
Readme and xmlk docs for Azure.Monitor.Query (#20731)
ed55a68
@JoshLove-msft
Add DLQ sample (#20739)
4763c30 
* Add DLQ sample

* Fix broken links

* PR FB
@LarryOsterman
Prepare Attestation SDK for GA release (#20760)
09de85a
@Mohit-Chakraborty
Search indexer knowledge store (#20691)
4d3fa9e 
* Update to rest spec version d95c18e2
Introduce SearchIndexerKnowledgeStore

* Add tests for KnowledgeStore in skillsets

* Update sample in markdown

* Fix code that sanitizes storage account key from session records
@pakrym
Adjust metric APIs and add tests (#20756)
0069b17
[TSI] Query Samples (#20744)
b39b6c1
@christothes
Refactor batch to a pure model (#20723)
1e60698 
* refactor batch
@azure-sdk @weshaggard
Sync Devops helper scripts from azure-sdk repo (#20763)
73d8f9b 
DevOps helper scripts were updated in azure-sdk repo
so syncing those changes into the tools repo so they
can be used prepare-release script.

Co-authored-by: Wes Haggard <[email protected]>
@LarryOsterman
Renamed ModelFactory methods to align with guidelines. (#20767)
89814e9 
* Renamed ModelFactory methods to align with Azure naming guidelines.
@pakrym
Remove net5.0 from TestFramework explicit TargetFrameworks list (#20771)
f1d1a68
@jsquire
[Event Hubs Processor] Stop attempting to steal inappropriately (#20775)
70a38f7 
The focus of these changes is to fix a bug in which load balancing
incorrectly identifies a need to attempt to steal a partition (even from
itself) and then fails to find an eligible candidate partition after running
unnecessary logic.  This manifests only as a log message for host applications,
as partitions were never actually stolen.
@luxiong-ms @ShivangiReja
add multivariate test sample (#20517)
c917e3a 
* add multivariate test sample

* add multivariate in README

* add md file for multivariate detect

* fix

* Add broken links to ingore-links.txt

* add snippet

* align snippet name in md and cs file;
skip line after main method

* update after snippet generation and code generation

* remove trailing whitespace;
correct links

* remove the added ignore links for anomaly detector

Co-authored-by: Shivangi Reja <[email protected]>
@tg-msft
Use sample snippets for WPS (#20780)
6f8da74
@jsquire
[Service Bus Client] README Link Fix (#20776)
e2cc675 
The focus of these changes is to fix the "Source code" link in the README
to reference the correct location.
@annelo-msft
uncomment nuget and docs links (#20782)
f5bb3b9
@Mohit-Chakraborty
Allow null value for PatternTokenizer["flags"] (#20789)
5ac2fa9 
When there are no Flags set, we should pass null to the service
@maririos
separate endpoints into test classes and apply api-version to class (#…
cb60f49 
…20757)
@Mohit-Chakraborty
Pass default value (1) of query count to service if it is not set exp…
68bfcd4 
…licitly (#20790)

Update tests to validate change in behavior
@minnieliu
[Communication] - ALL - Add subscription configuration to tests.yml (#…
d14007e 
…20781)

* Adding Subscription Configurations to tests.yml

* Fix formatting

* Updating tset-resources.json

* Fix formatting

* Fix indent

* Add extra line

* Revert test-resources.json

* Change back test-resources.json

* Change back test-resources.json

* Cleanup test-resources

* Adressed review comments

* Fix pipeline

Co-authored-by: Minnie Liu <[email protected]>
@JoshLove-msft
Respect webproxy from config (#20743)
9a4c3ac 
* Respect webproxy from config

* PR FB

* Rename to ClientRetryOptions for consistency with EH

* Fix test

* Fix test

* PR FB

* Export API

* Update dependency

* Update reference

* Use ToString for URI

* Revert ToString

* Fix JSON deserialization issue

* Remove unused code
@azure-sdk @mitchdenny
Sync eng/common directory with azure-sdk-tools for PR 1579 (#20740)
4eaed83 
* Allow propogation of SetDevVersion.

* Allow use of SetDevVersion locally and across stages.

Co-authored-by: Mitch Denny <[email protected]>
@mitchdenny
Propogate SetDevVersion.
793f822
@KrzysztofCwalina @pakrym
Added ContentType Extensible Enum (#20768)
ad52f8e 
* Added ContentType Extensible Enum

* Update sdk/core/Azure.Core.Experimental/src/ContentType.cs

Co-authored-by: Pavel Krymets <[email protected]>

* Updated API file

Co-authored-by: Pavel Krymets <[email protected]>
@maririos
recognizeContentTests (#20794)
4037707
@azure-sdk @chidozieononiwu @weshaggard
Sync eng/common directory with azure-sdk-tools for PR 1524 (#20769)
4dc0560 
* Make RELEASE_TITLE_REGEX more specific

* Update eng/common/scripts/ChangeLog-Operations.ps1

Co-authored-by: Wes Haggard <[email protected]>

* Update release REGEX

* tighten up changelog verification

* Ensure releaseDate is in the appriopriate format.

* Update error messages.

* Retrun false after error

Co-authored-by: Chidozie Ononiwu <[email protected]>
Co-authored-by: Chidozie Ononiwu <[email protected]>
Co-authored-by: Wes Haggard <[email protected]>
@nichatur
SDK Tests for Cosmos DB API Version 2021-04-15 (#20699)
d9201df
@KalpeshChavan12
Regenerate Microsoft.Maintenance from API version 2021-05-01. (#20732)
2cf9fbc 
Co-authored-by: Kalpesh Chavan <[email protected]>
maririos and others added 18 commits May 18, 2021 12:32
@maririos
update new swagger (#21158)
4213ed5
@azure-sdk
Increment version for eventhub releases (#21152)
c74b70f 
Increment package version after release of Microsoft.Azure.WebJobs.Extensions.EventHubs
@maririos
[TA] Rename analyze to remove batch from the name (#21134)
aed8abe 
* rename analyze to remove batch

* add changelog
@viparek
Resource Identity fixes for SQL Server (#21136)
97515da 
* Resource Identity fixes for SQL Server

* Modified assemblyinfo and csproj
@azure-sdk
Increment version for storage releases (#21168)
fb3caae 
Increment package version after release of Microsoft.Azure.WebJobs.Extensions.Storage.Blobs
@azure-sdk
Increment version for storage releases (#21169)
3533e97 
Increment package version after release of Microsoft.Azure.WebJobs.Extensions.Storage.Queues
@kasobol-msft
Merge branch 'master' into hotfix/storage/webjobs_release_2021_05_18
36267f6
@Kotasudhakarreddy
Generated a .NET SDK based on new api-version 2020-12-01 (#20300)
e439762 
* Generated based on new api-version 2020-12-01

* Addressed Code review comments.

* 2020-12 version is reverted/

* Generated with 2020-12 version

* Revert "Generated with 2020-12 version"

This reverts commit e6178a7cb54aec24d7b9547cd6752b4d4978d41d.

* .Net SDK generated for version "2020-12-01"

* Latest changes for apiversion:2020-12
@pakrym
Move DateTimeRange to Azure.Core.Experimental (#21149)
55357eb
@pakrym
Statistics as BinaryData property and parameter renames (#21164)
e8db319
@pakrym
Merge AppConfig 1.0.3 path branch (#21115)
d25efe3
@pakrym
Add metric query sample, cleanup logs samples (#21171)
a5afb86
@azure-sdk @benbp
Sync eng/common directory with azure-sdk-tools for PR 1585 (#21161)
53bed52 
* Sparse checkout docs repository in release step

* Detect default branch for custom sparse checkout repository

* Use sparse checkout paths parameter in docs metadata release

Co-authored-by: Ben Broderick Phillips <[email protected]>
@JoshLove-msft
SB extension API updates (#21165)
91f9eb3 
* SB extension API updates

* PR FB
@hivyas
[AVA] Updating links in ReadMe (#21176)
558de77 
* updating links and changelog

* updating links
@heaths
Fix ShareLink sample to be able to build again (#21170)
2103c38 
* Fix ShareLink sample to be able to build again

Relates to #21074

* Build Key Vault samples in CI
@ajpeacock0
[Communication] - Azure.Communication.NetworkTraversal - Adding the A…
2032102 
…CS NetworkTraversal SDK (#21094)

* Adding the Azure.Communication.NetworkTraversal SDK

* Adding the Azure.Communication.NetworkTraversal SDK - update
@grizzlytheodore
only Compute changes
446b322
@ghost ghost added the Compute label May 19, 2021
@Sandido Sandido marked this pull request as ready for review May 19, 2021 22:38
@Sandido Sandido merged commit 46d1d18 into cplat_2021-03-01 May 19, 2021
@Sandido Sandido deleted the cplat202103sync2 branch May 19, 2021 22:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@allenjzhang allenjzhang Awaiting requested review from allenjzhang

@bquantump bquantump Awaiting requested review from bquantump

@KrzysztofCwalina KrzysztofCwalina Awaiting requested review from KrzysztofCwalina

@m-nash m-nash Awaiting requested review from m-nash

@markcowl markcowl Awaiting requested review from markcowl

@nisha-bhatia nisha-bhatia Awaiting requested review from nisha-bhatia

@tg-msft tg-msft Awaiting requested review from tg-msft

@YalinLi0312 YalinLi0312 Awaiting requested review from YalinLi0312

Assignees
No one assigned
Labels
Compute
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.