-
Notifications
You must be signed in to change notification settings - Fork 848
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
Rebased with main #15454
Merged
Merged
Rebased with main #15454
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
Contributor
mohsha-msft
commented
Sep 2, 2021
- The purpose of this PR is explained in this or a referenced issue.
- The PR does not update generated files.
- These files are managed by the codegen framework at Azure/autorest.go.
- Tests are included and/or updated for code changes.
- Updates to CHANGELOG.md are included.
- MIT license headers are included in each file.
* fix dependencies * fix the command graph * change the scripts in sdk automation * fix
…14741) * enable resourceID for MSI cred * update const and rename * improve docs, add tests * review feedback * update type alias name
* Update format of new Changelog Entry * Add parsing of changelog sections * Update ChangeLog Logic Co-authored-by: Chidozie Ononiwu <chononiw@microsoft.com>
Co-authored-by: Wes Haggard <Wes.Haggard@microsoft.com>
* Generated from specification/containerservice/resource-manager/readme.md tag package-2021-05 (commit hash: ef7ffaa1ae68ab17dba86d8e0be5fd1fb615ef64) * Generated from specification/cosmos-db/resource-manager/readme.md tag package-2021-05 (commit hash: ef7ffaa1ae68ab17dba86d8e0be5fd1fb615ef64) * Generated from specification/netapp/resource-manager/readme.md tag package-netapp-2021-04-01 (commit hash: ef7ffaa1ae68ab17dba86d8e0be5fd1fb615ef64) * Generated from specification/postgresql/resource-manager/readme.md tag package-2020-02-14-preview (commit hash: ef7ffaa1ae68ab17dba86d8e0be5fd1fb615ef64) * Generated from specification/web/resource-manager/readme.md tag package-2020-12 (commit hash: ef7ffaa1ae68ab17dba86d8e0be5fd1fb615ef64) * v55.2.0 * Update CHANGELOG.md
* LRO poller rewrite Simplified implementation of LRO pollers for ARM. Public surface area has been slightly changed, making it identical to the data-plane implementation. The different polling mechanisms have been split into internal packages, with an exported LROPoller that implements the overall polling algorithm. * Fix NopCloser package for earlier versions of Go * handle empty response when a model is provided log a message in this case * update location polling URL * fix final GET for POST * handle absense of provisioning state for initial response body polling * verify polling URLs * move checking of errors to Poll method * move logging of status to Poll() * export pipeline for pager-poller scenario * fail on a 202 DELETE/POST with no polling URL * differentiate between no response body and missing states this is important for body polling which can tolerate a missing response body for certain status codes. * relax provisioning state requirement on initial async PUT response * use provisioningState for loc polling when available * fix up token decoding * consolidate status constants * fix code comment * fix closing of response body * add comments to fields * simplify error handling for missing states * add check for empty Location header
* Add verification of changelog sections * Update eng/common/scripts/ChangeLog-Operations.ps1 Co-authored-by: Wes Haggard <weshaggard@users.noreply.github.com> Co-authored-by: Chidozie Ononiwu <chononiw@microsoft.com> Co-authored-by: Chidozie Ononiwu <31145988+chidozieononiwu@users.noreply.github.com> Co-authored-by: Wes Haggard <weshaggard@users.noreply.github.com>
* Generated from specification/trafficmanager/resource-manager/readme.md tag package-2018-08 (commit hash: 2b0b50e3e5bd467aa039a51cafa2a071593eca28) * Generated from specification/databricks/resource-manager/readme.md tag package-2021-04-01-preview (commit hash: 2b0b50e3e5bd467aa039a51cafa2a071593eca28) * Generated from specification/eventgrid/resource-manager/readme.md tag package-2021-06-preview (commit hash: 2b0b50e3e5bd467aa039a51cafa2a071593eca28) * Generated from specification/security/resource-manager/readme.md tag package-composite-v3 (commit hash: 2b0b50e3e5bd467aa039a51cafa2a071593eca28) * v55.3.0
* update with latest code generator * update changelog
* update with latest code generator * update changelog
Co-authored-by: Chidozie Ononiwu <chononiw@microsoft.com>
* update with latest code generator * update changelog
* update with latest code generator * update changelog
* update with latest code generator * fix test and update minor version instead * update changelog
* Update links with master to use main * Update ci.yml files Co-authored-by: Chidozie Ononiwu <chononiw@microsoft.com>
* Update pipeline generation tool version Consume latest changes from pipeline generation tool in Azure/azure-sdk-tools#1708 * Update tool version to include fix for public ci Co-authored-by: Wes Haggard <weshaggard@users.noreply.github.com>
* initial commit for private dns * fix copy/paste errors * gofmt
* Go recorded test framework
* add tenantID on TokenRequestOptions * bump azcore min version * improve comment
* adding service fabric support * update token handling * add test * fix expires_on handling * remove unused case in switch * add failure case test
* add service fabric program * update azidentity version * update comment * update azcore version
* initial commit for authorization RP * modify meta.json * gofmt * fix C+V issues
* Check for API review status only if release date is set in changelog * Change property name to ReleaseStatus Co-authored-by: Praveen Kuttappan <praveen.kuttappan@gmail.com>
Co-authored-by: Praveen Kuttappan <praveen.kuttappan@gmail.com>
Co-authored-by: Wes Haggard <Wes.Haggard@microsoft.com>
* Generated from specification/azurestackhci/resource-manager/readme.md tag package-preview-2021-01 (commit hash: 8dff86df71bee429af84ea4713288ef3cdd1db2f) * Generated from specification/cosmos-db/resource-manager/readme.md tag package-2021-06 (commit hash: 8dff86df71bee429af84ea4713288ef3cdd1db2f) * Generated from specification/hybridkubernetes/resource-manager/readme.md tag package-2021-04-01-preview (commit hash: 8dff86df71bee429af84ea4713288ef3cdd1db2f) * Generated from specification/monitor/resource-manager/readme.md tag package-2021-04 (commit hash: 8dff86df71bee429af84ea4713288ef3cdd1db2f) * Generated from specification/postgresql/resource-manager/readme.md tag package-flexibleserver-2021-06 (commit hash: 8dff86df71bee429af84ea4713288ef3cdd1db2f) * Generated from specification/sql/resource-manager/readme.md tag package-composite-v5 (commit hash: 8dff86df71bee429af84ea4713288ef3cdd1db2f) * v55.4.0
* Bug fix: Storing .env file for bicep file at service dir * Styling Co-authored-by: Albert Cheng <albertcheng@microsoft.com>
Add additional sections for setting up development environments. Add checklists for best practices for submitting PRs
Fixed some linter warnings. Renamed UserAgent to component per new telemetry policy. Version const is no longer exported.
Co-authored-by: Chidozie Ononiwu <chononiw@microsoft.com>
* Add random elements to default base name * Fix principal * tab Co-authored-by: JoshLove-msft <54595583+JoshLove-msft@users.noreply.github.com>
<!-- Thank you for contributing to the Azure SDK for Go. Please verify the following before submitting your PR, thank you! --> - [ ] The purpose of this PR is explained in this or a referenced issue. - [ ] The PR does not update generated files. - These files are managed by the codegen framework at [Azure/autorest.go][]. - [ ] Tests are included and/or updated for code changes. - [ ] Updates to [CHANGELOG.md][] are included. - [ ] MIT license headers are included in each file. [Azure/autorest.go]: https://github.com/Azure/autorest.go [CHANGELOG.md]: https://github.com/Azure/azure-sdk-for-go/blob/main/CHANGELOG.md
Co-authored-by: Mike Harder <mharder@microsoft.com>
* upgrade autorest.go version to 2.1.187 * Generated from specification/mysql/resource-manager/readme.md tag package-flexibleserver-2021-05-01 (commit hash: 51b37b069ecbb9d2fcd300eabd4b10b7911b7d7d) * Generated from specification/servicefabric/resource-manager/readme.md tag package-2021-06 (commit hash: 51b37b069ecbb9d2fcd300eabd4b10b7911b7d7d) * v57.1.0 * fix CHANGELOG.md
* [Release] sdk/compute/armcompute/0.2.1 generation from spec commit: d5ad9431bbc19c34090d97209eb497167db32879 * fix exmaple * goimports * fix * fix * fix * fix CHANGELOG release date * fix
Co-authored-by: Wes Haggard <Wes.Haggard@microsoft.com>
…15259) * moving into a separate PR to merge with main * adding yml and script to start server * renaming examples for go vet * fixing stop command * skipping problematic tests * bens feedback * testproxy changes from track2-tables * simplification * fix * getting main build-test.yml files * whitespace * adding tests for test proxy infra * setting azure_record_mode to playback * setting proxy cert variable * prepend ./ to testId, use testId to determine full path from repo root * workaround for magically dissappearing files * more tests * dropping coverage * removing azcore dependent code * undoing changes to eng files * unexporting a couple things * fixed issue with deleting recordings * adding additional tests * dropping code coverage * cleanup * richards comments * improving coverage * fixing recording options test * inching closer * a * dropping coverage 1%, have every inch of new code tested * adding test case for backwards slash path * no newline * fixing test * allowing parallel tests * removing snippet * undoing change to config * fixing armcore link * undoing changes to migration guide * Update sdk/internal/recording/recordings/TestUriSanitizer.json Co-authored-by: Rick Winter <rick.winter@microsoft.com> * Update sdk/internal/recording/recordings/TestStartStop.json Co-authored-by: Rick Winter <rick.winter@microsoft.com> * richards comments Co-authored-by: Rick Winter <rick.winter@microsoft.com>
mohsha-msft
requested review from
ArcturusZhang,
benbp,
chamons,
jhendrixMSFT,
lirenhe,
richardpark-msft,
RickWinter,
seankane-msft and
a team
as code owners
September 2, 2021 20:11
This repository is protected by Check Enforcer. The check-enforcer check-run will not pass until there is at least one more check-run successfully passing. Check Enforcer supports the following comment commands:
|
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.