-
Notifications
You must be signed in to change notification settings - Fork 850
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
[azcore] JoinPaths will preserve query parameters #15258
Merged
Merged
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
seankane-msft
requested review from
chamons,
richardpark-msft and
RickWinter
as code owners
August 10, 2021 16:39
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:
|
jhendrixMSFT
approved these changes
Aug 13, 2021
Fixes #15243 |
mohsha-msft
added a commit
that referenced
this pull request
Sep 2, 2021
* Fix dependency (2nd round) (#14745) * fix dependencies * fix the command graph * change the scripts in sdk automation * fix * azidentity: Adding Resource ID support in ManagedIdentityCredential (#14741) * enable resourceID for MSI cred * update const and rename * improve docs, add tests * review feedback * update type alias name * Sync eng/common directory with azure-sdk-tools for PR 1633 (#14757) * Update format of new Changelog Entry * Add parsing of changelog sections * Update ChangeLog Logic Co-authored-by: Chidozie Ononiwu <chononiw@microsoft.com> * Strict mode needs the variables initialized (#14762) Co-authored-by: Wes Haggard <Wes.Haggard@microsoft.com> * Release v55.2.0 1623309052 2 (#14766) * 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 * update references for credential types table (#14749) * LRO poller rewrite (#14752) * 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 * Sync eng/common directory with azure-sdk-tools for PR 1688 (#14780) * 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> * Release v55.3.0 1623834595 (#14782) * 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 (#14776) * update with latest code generator * update changelog * update with latest code generator (#14773) * update with latest code generator * update changelog * Fix hard-coded master (#14797) Co-authored-by: Chidozie Ononiwu <chononiw@microsoft.com> * update with latest code generator (#14775) * update with latest code generator * update changelog * update with latest code generator (#14774) * update with latest code generator * update changelog * update with latest code generator (#14772) * update with latest code generator * fix test and update minor version instead * update changelog * initial change to squash the package files (#14853) * Sync eng/common directory with azure-sdk-tools for PR 1716 (#14870) * Update links with master to use main * Update ci.yml files Co-authored-by: Chidozie Ononiwu <chononiw@microsoft.com> * Sync eng/common directory with azure-sdk-tools for PR 1719 (#14871) * 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> * Track 2 `privatedns` module release (#14793) * initial commit for private dns * fix copy/paste errors * gofmt * Go recorded test framework (#14767) * Go recorded test framework * Add tenantID on TokenRequestOptions (#14879) * add tenantID on TokenRequestOptions * bump azcore min version * improve comment * azidentity: add support for service fabric MSI environment (#14783) * adding service fabric support * update token handling * add test * fix expires_on handling * remove unused case in switch * add failure case test * update version.go (#14886) * azidentity: Add service fabric test program (#14788) * add service fabric program * update azidentity version * update comment * update azcore version * Release track 2 authorization module (#14867) * initial commit for authorization RP * modify meta.json * gofmt * fix C+V issues * Sync eng/common directory with azure-sdk-tools for PR 1729 (#14887) * 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> * Add Ubuntu 20 to local dns bypass template (#14896) Co-authored-by: Praveen Kuttappan <praveen.kuttappan@gmail.com> * Update master to main (#14909) * Update references from master to main (#14911) Co-authored-by: Wes Haggard <Wes.Haggard@microsoft.com> * Release v55.4.0 (#14914) * 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 * rename armmonitor to arminsights (#14725) * Update CodeOwners and Notice.txt (#14919) * Update CODEOWNERS file Update Notices.txt to standard microsoft format * Remove old files * change package name back to armmonitor (#14935) * Release v55.5.0 1625035291 (#14942) * Generated from specification/appconfiguration/resource-manager/readme.md tag package-2021-03-01-preview (commit hash: b548283ea6f4b3d797e85432fcc6e3ff2a2ac224) * Generated from specification/applicationinsights/resource-manager/readme.md tag package-2020-02-02 (commit hash: b548283ea6f4b3d797e85432fcc6e3ff2a2ac224) * Generated from specification/signalr/resource-manager/readme.md tag package-2021-06-01-preview (commit hash: b548283ea6f4b3d797e85432fcc6e3ff2a2ac224) * Generated from specification/webpubsub/resource-manager/readme.md tag package-2021-06-01-preview (commit hash: b548283ea6f4b3d797e85432fcc6e3ff2a2ac224) * Generated from specification/apimanagement/resource-manager/readme.md tag package-preview-2021-01 (commit hash: b548283ea6f4b3d797e85432fcc6e3ff2a2ac224) * v55.5.0 * Sync eng/common directory with azure-sdk-tools for PR 1725 (#14933) * Bring changes from JS docs metadata * Move business logic inside Update-DocsMsMetadata.ps1 * Update with the latest changes in JS PR * Update from latest PR feedback * Add check for empty path to prevent crashes when creating relative paths Co-authored-by: Daniel Jurek <djurek@microsoft.com> * Update devops workitem helpers (#14934) - Switch to using rest instead of cli for querying work items as cli is limited to 1000 items only. - Fix issues with Generated fields not being set. - Correctly sort older workitems by version isntead of string. Co-authored-by: Wes Haggard <Wes.Haggard@microsoft.com> * Add prepare pipelines pipeline (#14947) * Sync eng/common directory with azure-sdk-tools for PR 1763 (#14938) * Disable release date check * Release check is not finding release date Co-authored-by: Praveen Kuttappan <praveen.kuttappan@gmail.com> * Update docs metadata mutation logic (#14950) Co-authored-by: Daniel Jurek <djurek@microsoft.com> * Sync eng/common directory with azure-sdk-tools for PR 1772 (#14962) * Update change log headers based on guideline update Updates based on Azure/azure-sdk#3103 - Renamed "Key Bugs Fixed" to "Bugs Fixed" - Renamed "Fixed" to "Other Changes" Added a warning in validation if at lease one of the recommended headers aren't used. * Update eng/common/scripts/ChangeLog-Operations.ps1 Co-authored-by: JoshLove-msft <54595583+JoshLove-msft@users.noreply.github.com> Co-authored-by: Wes Haggard <Wes.Haggard@microsoft.com> Co-authored-by: Wes Haggard <weshaggard@users.noreply.github.com> Co-authored-by: JoshLove-msft <54595583+JoshLove-msft@users.noreply.github.com> * Skip live test generation in prepare-pipelines (#14958) * onboard track2: containerregistry (#14988) * Switch ubuntu 18 to ubuntu 20 (#14995) Co-authored-by: Chidozie Ononiwu <chononiw@microsoft.com> * Sync eng/common directory with azure-sdk-tools for PR 1767 (#14970) * Support building and deploying bicep templates * Add bicep powershell install aka link to deployment error message * Write bicep compiled arm templates to temp directory * Simplify bicep building code/function usage * Use bicep location for compiled arm templates, and remove them on success Co-authored-by: Ben Broderick Phillips <bebroder@microsoft.com> * fix readme in recording (#14981) * Update Ubuntu 18 to 20 (#14986) * Add build.ps1 script to build all SDKs (#15014) - Add build script and convert T2 CI to use new build.ps1 * Add root directory to list of sparse checkout paths (#15021) Explicitly add '/*' '!/*/' to the list of paths so we include the files in the root of the repo. Added some extra command logging so we can easily repro locally without needing to find the code that ran. Update to '/eng' so that we only get paths that start with eng instead of every path that has eng as a folder. Co-authored-by: Wes Haggard <Wes.Haggard@microsoft.com> * Release v55.6.0 1625727542 (#14999) * Generated from specification/resources/resource-manager/readme.md tag package-features-2021-07 (commit hash: f9883569df850be792a37df17a1a28a5bdcdfc39) * Generated from specification/security/resource-manager/readme.md tag package-composite-v3 (commit hash: f9883569df850be792a37df17a1a28a5bdcdfc39) * Generated from specification/vmware/resource-manager/readme.md tag package-2021-06-01 (commit hash: f9883569df850be792a37df17a1a28a5bdcdfc39) * Generated from specification/web/resource-manager/readme.md tag package-2021-01-15 (commit hash: f9883569df850be792a37df17a1a28a5bdcdfc39) * Generated from specification/appplatform/resource-manager/readme.md tag package-preview-2021-06 (commit hash: f9883569df850be792a37df17a1a28a5bdcdfc39) * Generated from specification/edgeorder/resource-manager/readme.md tag package-2020-12-preview (commit hash: f9883569df850be792a37df17a1a28a5bdcdfc39) * Generated from specification/quantum/resource-manager/readme.md tag package-2019-11-04-preview (commit hash: f9883569df850be792a37df17a1a28a5bdcdfc39) * Generated from specification/redis/resource-manager/readme.md tag package-2020-12 (commit hash: f9883569df850be792a37df17a1a28a5bdcdfc39) * v55.6.0 * azidentity: Add environment variable for resource ID (#14872) Fixes #14742 * Unexport HTTP headers to reduce public surface area (#15038) * Update CODEOWNERS (#15040) * Start documentation on build (#15012) * Sync eng/common directory with azure-sdk-tools for PR 1808 (#15044) * Add FilterPoliCheckResult.ps1 * Add description to FilterPoliCheckResults.ps1 Co-authored-by: Chidozie Ononiwu <chononiw@microsoft.com> * Release Monitor track 2 module (#15031) * initial commit of monitor track 2 module * some attempt to generate monitor, not finished * update readme * regenerate and refresh the metadata * Track2 release: containerregistry (#14992) * onboard track2: containerregistry * release track2 for containerregistry * track2 containerregistry release * release containerregistry track2 * Update sdk/containerregistry/armcontainerregistry/README.md Co-authored-by: Arcturus <dapzhang@microsoft.com> * Update sdk/containerregistry/armcontainerregistry/README.md Co-authored-by: Arcturus <dapzhang@microsoft.com> * Update sdk/containerregistry/armcontainerregistry/README.md Co-authored-by: Arcturus <dapzhang@microsoft.com> * Update sdk/containerregistry/armcontainerregistry/README.md Co-authored-by: Arcturus <dapzhang@microsoft.com> * Update sdk/containerregistry/armcontainerregistry/README.md Co-authored-by: Arcturus <dapzhang@microsoft.com> Co-authored-by: Arcturus <dapzhang@microsoft.com> * web track 2 module onboarding (#14984) * introduce web RP * Update sdk/web/armweb/ci.yml Co-authored-by: Wes Haggard <weshaggard@users.noreply.github.com> Co-authored-by: Wes Haggard <weshaggard@users.noreply.github.com> * Initial release of track 2 module `agrifood` (#14799) * initial commit of agrifood * fix bad path * Update sdk/agrifood/armagrifood/ci.yml Co-authored-by: Wes Haggard <weshaggard@users.noreply.github.com> Co-authored-by: Wes Haggard <weshaggard@users.noreply.github.com> * Sync eng/common directory with azure-sdk-tools for PR 1748 (#14930) * Simplify Ubuntu image name check for dns bypass * Fix typo Co-authored-by: Praveen Kuttappan <praveen.kuttappan@gmail.com> * Release v55.7.0 1626332017 (#15069) * Generated from specification/mediaservices/resource-manager/readme.md tag package-2021-06 (commit hash: e3980786418c280ecd7ed2e141407ed0f63cd493) * Generated from specification/security/resource-manager/readme.md tag package-composite-v3 (commit hash: e3980786418c280ecd7ed2e141407ed0f63cd493) * Generated from specification/mysql/resource-manager/readme.md tag package-2020-01-01 (commit hash: e3980786418c280ecd7ed2e141407ed0f63cd493) * Generated from specification/postgresql/resource-manager/readme.md tag package-2020-01-01 (commit hash: e3980786418c280ecd7ed2e141407ed0f63cd493) * v55.7.0 * remove exit 0 portion (#15035) * Build.ps1 improvements (#15017) - Fixes build.ps1 to work from relative locations (and in remove parallel logic that was breaking CI). * [Core] Renamings for AuthPolicies (#15103) * renamings * fixing lint * [Core] Rename AnonymousCredential (#15104) * renaming to New... * fixing lint * catalinas comments * fixing a bad merge * fixing lint step for azidentity (#15109) * [EngSys] Combine Build Build and Build Test pipelines (#15085) * only finds tests with a Test method * adding conditional to ci.yml files * removing the need for build.yml file * changing files to use one archetype-sdk-client.yml file * bad conditional * passing parameter through * checking for explicit true * moving coverage installation into test section * had to fix profiles ci.yml file * fixing hang up in gofmt when there are no go files in dir * changing echo to fmt * error check on samples * removing build.yml file * bens comments and lint * undoing change * simplifying * Release v55.8.0 1626925362 (#15118) * Generated from specification/containerinstance/resource-manager/readme.md tag package-2021-03 (commit hash: 4e6cea592e12915061beb893134e1a70d7e5c495) * Generated from specification/mysql/resource-manager/readme.md tag package-flexibleserver-2021-05-01-preview (commit hash: 4e6cea592e12915061beb893134e1a70d7e5c495) * Generated from specification/compute/resource-manager/readme.md tag package-2021-04-01 (commit hash: 4e6cea592e12915061beb893134e1a70d7e5c495) * v55.8.0 * Sync eng/common directory with azure-sdk-tools for PR 1827 (#15126) * update article type appropriately Co-authored-by: scbedd <45376673+scbedd@users.noreply.github.com> * track2 add servicebus ci.yml and LICENSE.txt file (#15132) * add track2 servicebus ci.yml and LICENSE.txt file * modify * Add sql ci.yml and LICENSE.txt file (#15133) * [build.ps1] Add support for converting _meta.json to autorest.md during build (#15111) * update the code owner to include renhe (#15131) * [EngSys] adding coverage tools (#15076) * adding coverage tools * forgot to include 'go run' * need to find where the script is running from * fixing script * fixing passing of coverage goal value * removing flag from build-test * fixing tools script for multiple files * updating ci.yml files, polishing script * removing coverage goals * undoing changes * dropping core down, fixing to.go lint * creating a config file for the SDK and using that instead of putting values in ci.yml * putting coverage in a script * fixing location of script * fixing scope * fixing powershell portions, float instead of &float64 * forcing pipelines * Update tools/internal/coverage/main.go Co-authored-by: Ben Broderick Phillips <ben@benbp.net> Co-authored-by: Ben Broderick Phillips <ben@benbp.net> * Many build.ps1 improvements (#15135) - Fixes #15128 - Fixes #15127 - Part of #15102 * add basic ga tag w/ settable campaignid (#15157) Co-authored-by: scbedd <45376673+scbedd@users.noreply.github.com> * add auxiliary tenants for multi-tenant scenario (#15123) * Introduce the migration guide to track 2 modules (#14980) * migration guide * resolve comments * move the migration guide to documentation directory * fix typo * resolve comments * resolve comments * Add Documentation for Track 2 Management SDK (#15096) * Update docs to reflect Track 2 status * Update README.md * Update README.md * Create previous-versions-quickstart.md * Update README.md * Update README.md * Update README.md * Update README.md * Create new-version-quickstart.md * Update new-version-quickstart.md * Update README.md * Update new-version-quickstart.md * Update new-version-quickstart.md * Update new-version-quickstart.md * Update new-version-quickstart.md * Update new-version-quickstart.md * add code snipet for resource groups * update code for vnet * update * Update new-version-quickstart.md * Update new-version-quickstart.md * Update new-version-quickstart.md * Update new-version-quickstart.md * Update new-version-quickstart.md * Update new-version-quickstart.md * Update new-version-quickstart.md * Update new-version-quickstart.md * Update new-version-quickstart.md * Update README.md Co-authored-by: Arcturus <dapzhang@microsoft.com> * Update documentation/new-version-quickstart.md Co-authored-by: Arcturus <dapzhang@microsoft.com> * Update documentation/new-version-quickstart.md Co-authored-by: Arcturus <dapzhang@microsoft.com> * Update documentation/new-version-quickstart.md Co-authored-by: Arcturus <dapzhang@microsoft.com> * Update README.md * Update README.md * Update README.md * Update documentation/new-version-quickstart.md Co-authored-by: Arcturus <dapzhang@microsoft.com> * Update documentation/new-version-quickstart.md Co-authored-by: Arcturus <dapzhang@microsoft.com> * Update documentation/new-version-quickstart.md Co-authored-by: Arcturus <dapzhang@microsoft.com> * Update documentation/new-version-quickstart.md Co-authored-by: Arcturus <dapzhang@microsoft.com> * Update documentation/new-version-quickstart.md Co-authored-by: Arcturus <dapzhang@microsoft.com> * Update documentation/new-version-quickstart.md Co-authored-by: Arcturus <dapzhang@microsoft.com> * Update documentation/new-version-quickstart.md Co-authored-by: Arcturus <dapzhang@microsoft.com> * Update documentation/new-version-quickstart.md Co-authored-by: Arcturus <dapzhang@microsoft.com> * Update documentation/new-version-quickstart.md Co-authored-by: Arcturus <dapzhang@microsoft.com> * Update new-version-quickstart.md * Update new-version-quickstart.md * Update README.md * Update README.md * Update new-version-quickstart.md * Update new-version-quickstart.md * Update README.md * Update previous-versions-quickstart.md * update code snippet Co-authored-by: ArcturusZhang <dapzhang@microsoft.com> * Verify API surface level against all approved revisions for GA releases (#15159) Co-authored-by: praveenkuttappan <prmarott@microsoft.com> * [Internal] fixing lint issues (#15162) * [EngSys] adding triggers on eng for internal and azcore (#15161) * adding eng for internal and azcore * adding a parameter for specifying the service directory * fixing script * fixing pipeline * do not run golangci on tests * adding false flag silencer * run lint on tests * [armcore] Add auxiliary tenants to connection options (#15124) * add auxiliary tenants to options * update based on latest azcore * update to latest azcore * lint updates * update nolint * [azidentity] Update code to use the latest azcore (#15137) * update code to latest azcore * update go.mod * remove unused header * type assertions * [azidentity] Expiring resource rework w/ multi-tenant support (#15138) * expiring resource rework * add auxiliary auth header * adding default logger functionality to internal (#15182) * Release v56.0.0 1627549243 (#15186) * Generated from specification/applicationinsights/resource-manager/readme.md tag package-2020-02-02 (commit hash: b47572ab2069865484c493a4f679d4e72f8b303d) * Generated from specification/automation/resource-manager/readme.md tag package-2019-06 (commit hash: b47572ab2069865484c493a4f679d4e72f8b303d) * Generated from specification/storagecache/resource-manager/readme.md tag package-2021-05 (commit hash: b47572ab2069865484c493a4f679d4e72f8b303d) * Generated from specification/batch/resource-manager/readme.md tag package-2021-06 (commit hash: b47572ab2069865484c493a4f679d4e72f8b303d) * Generated from specification/cost-management/resource-manager/readme.md tag package-2019-11 (commit hash: b47572ab2069865484c493a4f679d4e72f8b303d) * Generated from specification/machinelearningservices/resource-manager/readme.md tag package-2021-04-01 (commit hash: b47572ab2069865484c493a4f679d4e72f8b303d) * Generated from specification/consumption/resource-manager/readme.md tag package-2019-10 (commit hash: b47572ab2069865484c493a4f679d4e72f8b303d) * Generated from specification/datafactory/resource-manager/readme.md tag package-2018-06 (commit hash: b47572ab2069865484c493a4f679d4e72f8b303d) * Generated from specification/postgresql/resource-manager/readme.md tag package-flexibleserver-2021-06 (commit hash: b47572ab2069865484c493a4f679d4e72f8b303d) * v56.0.0 * [azcore] Remove logger from azcore (#15160) * initial work, issues with using internal package * fixing up errors in logger file * adding logger test file * removed an example * feedback and removing circular dependency * moving logclassification to be in logger, type alias in azcore * update internal version * fixes in azcore * fixes to azcore, works now * linter false positive * catalina's comments * more feedback * update version * Update sdk/azcore/version.go Co-authored-by: catalinaperalta <catalinaperaltah@hotmail.com> Co-authored-by: catalinaperalta <catalinaperaltah@hotmail.com> * [build.ps1] Early return error if autorest fails (#15179) * Move stress testing scripts to eng/common (#15189) Co-authored-by: Ben Broderick Phillips <bebroder@microsoft.com> * Update README.md (#15177) * remove race and covermode flags from test statement (#15200) * [Azcore] simplify progress objects (#15194) * started combining * single progress for request and response * adding docstrings * modify build-test to find silent error * undoing eng change * removing race and covermode from the test command * enhance build.ps1 script (#15191) * enhance build.ps1 script * fix typo * update autorest.go version (#15208) * [tools/generator] add the template command (#15206) * adds the template command * fix package name of readDir * add a new comment * resolve linter issues * add autorest.md template * update the template accordingly * fix typo in template (#15213) * fix typo in template * fix gofmt * Release v56.1.0 1628127975 (#15212) * Generated from specification/monitor/resource-manager/readme.md tag package-2021-07 (commit hash: 111fbceeeb7c2c4905b80fe2ab760e5564b3458a) * v56.1.0 * Add ability to leave SourceBranch empty when queueing a build (builds using the default branch) (#15195) Co-authored-by: Daniel Jurek <djurek@microsoft.com> * move verify link to the last step (#15220) * redirect the output to stderr in gofmt checking so that we can give errors when the code is not formatted (#15217) * redirect the output to stderr in gofmt checking so that we can give errors when the code is not formatted * fix redirect * switch to script as PS doesn't like redirecting to stderr Co-authored-by: Joel Hendrix <jhendrix@microsoft.com> * Fix build status and code coverage badges (#15224) * [azcore] Add ETag type to Core (#15207) * add type def for ETag * adding etag specific files and more functionality * testing weak * testing for etag equality * weak tag comparisons * adding docstrings * addressing richard and jeffs feedback. split equals to strong and weak, using more HasValue throughout * fixing one test * adding EOL * adding note about the difference between nil and empty etags * more feedback * fix spelling, remove comment about nil * format check * [azidentity] adding a changelog to azidentity (#15240) closes #15235 * [azcore] adding changelog (#15239) Adds a changelog with previous release information. closes #15234 * Release the track 2 `containerservice` module (#14792) * initial commit of the container service * go mod tidy * regen * regenerate * fix readme * track2 generator service bus (#15214) * Track2 generator cosmos db (#15218) * track2 generator Azure Cosmos DB * fix newclient * track2 generator postgresql (#15219) * track2 generator postgresql * delete postgresqlflexibleservers package * track2 generator postgresql flexible servers (#15221) * track2 generator postgresql flexible servers * gofmt Co-authored-by: arcturusZhang <ufo54153@gmail.com> * [EngSys] display output of go test (#15257) * track2 generator event grid (#15249) * track2 generator event grid * build.go * Fixed small bugs (#15266) <!-- 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 * Sync eng/common directory with azure-sdk-tools for PR 1866 (#15261) * Deploy for stress tests at resource group scope * Automatically compile bicep files in stress test deploy script Co-authored-by: Ben Broderick Phillips <bebroder@microsoft.com> * Refactor internal log package (#15265) Shortened package name to log. Renamed types to remove stuttering. Made logger implementation unexported. Moved logger methods to funcs to simplify API. * track2 release `armapimanagement` (#15250) * track2 generator apimanagement * fix build.go * track2 generator operationalinsights (#15251) * track2 release `armeventhub` (#15252) * track2 generator eventhub * fix build.go * track2 release `armredis` (#15253) * track2 gennerator redis * fix build.go * Web track2 release (#15193) * release web track 2 module * regenerate * regenerate * regenerate * gofmt * Release v56.2.0 1628747491 (#15270) * Generated from specification/machinelearningservices/resource-manager/readme.md tag package-2021-07-01 (commit hash: 47545c5f0842f5be29529674a4609d75a973d783) * Generated from specification/network/resource-manager/readme.md tag package-2021-02 (commit hash: 47545c5f0842f5be29529674a4609d75a973d783) * Generated from specification/web/resource-manager/readme.md tag package-2021-02 (commit hash: 47545c5f0842f5be29529674a4609d75a973d783) * Generated from specification/containerservice/resource-manager/readme.md tag package-2021-07 (commit hash: 47545c5f0842f5be29529674a4609d75a973d783) * Generated from specification/datafactory/resource-manager/readme.md tag package-2018-06 (commit hash: 47545c5f0842f5be29529674a4609d75a973d783) * v56.2.0 * Sync eng/common directory with azure-sdk-tools for PR 1894 (#15281) * Purge Key Vaults after deleting resource group * Resolve PR feedback * Purge Key Vaults and Managed HSMs * Clean up all purgeable KVs and MHSMs * Ignore null/empty purgeable resources * Resolve PR feedback * Resolve PR feedback * Fix logging, process each item when collection piped Co-authored-by: Heath Stewart <heaths@microsoft.com> * Renamed package runtime to diag (#15278) * Renamed package runtime to diag Since it deals with diagnostics and we want to use the name runtime for a package in azcore. Deleted FrameError as it's not very useful. Added Caller() for times when a full stack trace isn't needed. * Remove log.Listener * [azcore] JoinPaths will preserve query parameters (#15258) * adding join paths changes * fixing test * small fix and adding test scenario * jeffs feedback from another PR * forgot to remove the func * cleaner return * Sync eng/common directory with azure-sdk-tools for PR 1876 (#15279) * rebase * Resolve conflict Co-authored-by: Albert Cheng <albertcheng@microsoft.com> * Removed and renamed some content (#15285) Unexported policyFunc Renamed Transport to Transporter Removed TransportFunc, Pager, and Poller interfaces Added responseError (removed from internal) Fixed logging type aliases Updated logging usage based on internal/log refactor Removed StatusCodesForRetry, ProgressReceiver, ErrNoMorePolicies Renamed LROPoller to Poller * Add Rest Method checks to Prepare-Release (#15290) Co-authored-by: Wes Haggard <Wes.Haggard@microsoft.com> * Move logging to log package (#15287) * Fix resource clean-up script (#15289) - Fix rg.Name to rg.ResourceGroupName - Add more verbose logging for better debugging - Handle deleted resource groups when gathering puragable resource - Remove coerce now that we are collecting in functions Co-authored-by: Wes Haggard <Wes.Haggard@microsoft.com> * Remove Response type (#15294) Just return an *http.Response instead. Changed Response methods to functions. * Add support for 'files' configuration (#15288) Co-authored-by: Daniel Jurek <djurek@microsoft.com> * Prevent wildcard expansion in git sparse checkout add (#15286) Co-authored-by: Ben Broderick Phillips <bebroder@microsoft.com> * Sync eng/common directory with azure-sdk-tools for PR 1909 (#15302) * add test-proxy invocations to eng/common folder Co-authored-by: scbedd <45376673+scbedd@users.noreply.github.com> * Add errinfo package with error types (#15300) * Add errinfo package with error types * rename based on feedback * Enable API review approval check for Java spring packages (#15305) Co-authored-by: praveenkuttappan <prmarott@microsoft.com> * Update to latest versions of Go (#15304) * Update to latest versions of Go * update internal * update azcore * run code coverage tools from module directory * use explicit path to test and code coverage reports * fix 1.16 patch version * Update to crypto secure random number generation (#15190) * Removing C:\Git\azure-sdk-tools\eng\common\scripts\FilterPoliCheckResults.ps1 as it was no longer used (#15311) Co-authored-by: Chidozie Ononiwu <chononiw@microsoft.com> * Fixed bugs in Resource Group demo code (#15301) <!-- 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 * Fix some misbehaving tests (#15309) * Fix some misbehaving tests Enable tests in CI * add missing tests and lower CC threshold * add a bit more coverage * increase code coverage goal for azcore added move coverage to azcore and fixed bug in retry policy. fixed test publishing to include sub-directories. * add missing EOL * adding dates to existing changelogs (#15306) * Sync eng/common directory with azure-sdk-tools for PR 1912 (#15314) * Attempt to purge all vaults, managed HSMs Reverts #1910. Vaults and managed HSMs are automatically purged on their purge date. The point was to purge them daily to preserve capacity. The default purge date is +90 days. * Add timeout and more logging * Pass required -Resource * Fix log message * Ensure the $Resource is correctly captured Added comment to new code explaining why, since ScriptBlock.GetNewClosure() is not working as expected. * Add -ErrorAction to Receive-Job Worked without terminating when run locally, but failed on the first error in the AzDO agent. * Use $using:r instead of creating ScriptBlock More idiomatic for passing ScriptBlocks to jobs. * Resolve PR feedback * Change default DeleteAfterHours to 120 Resolves #1917 * Use the Az cmdlets built-in -AsJob Co-authored-by: Heath Stewart <heaths@microsoft.com> * Fix query batch processing (#15319) Co-authored-by: Wes Haggard <Wes.Haggard@microsoft.com> * fix: three dots path not work in linux in build script (#15297) * Release v56.3.0 1629350585 (#15323) * Generated from specification/apimanagement/resource-manager/readme.md tag package-preview-2021-01 (commit hash: 30d1537f0c9aa49e6e04401b1d16f612b31231e7) * Generated from specification/purview/resource-manager/readme.md tag package-2021-07-01 (commit hash: 30d1537f0c9aa49e6e04401b1d16f612b31231e7) * v56.3.0 * fix * Remove old unused update changelog script (#15326) Co-authored-by: Wes Haggard <Wes.Haggard@microsoft.com> * add clear output flag to build.ps1 (#15255) * add clear output flag * remove the path parameter and make sure the process-sdk function only runs in current directory * change flag name * [tools/generator] add automation and release command for track2 (#15271) * [tools/generator] add automation and release command for track2 * add copyright and do gofmt * Use semver to pump version, move git operation to repo package and fix some typo * Add specfic version to release cmd, seperate package for automation and remove test config file * add automation script and config for track2 * fix: three dots path not work in linux in build script * fix: ln pwsh.exe to pwsh to run in linux * fix: ignore dep check for go-git * chore: label name change for track2 * fix: wrong package path result in empty git commit * fix: repo addr replacement in autorest.md * chore: move script to eng folder and change name * fix: onboard generation need to replace NewClientMethod * chore: change all naming from track2 to v2 * fix: add new build constraint * fix: template compat with go 1.17 and change commit msg for release * chore: add v2 to readme begin regex to support future change of `track2` naming * chore: remove track2 naming * fix: duplicated and wrong expression * fix: add package title param to release cmd * fix: ci * Prevent ManagedIdentityCredential mutating GetToken arguments (#15331) * [Azcore] bumps version of internal (#15347) * bump internal to latest * upgrading internal * not the correct one * changelog update * [azidentity] bump azcore and internal dependencies (#15348) * Update CODEOWNERS (#15357) - Thanks for all the fish * introduce pipelines to the modules in tools directory (#14926) * Sync eng/common directory with azure-sdk-tools for PR 1918 (#15317) * Stress test deployment script resiliency fixes * Use PSModule-Helpers to install powershell-yaml Co-authored-by: Ben Broderick Phillips <bebroder@microsoft.com> * bumping Build_Test to 1.16 and 1.17 (#15370) * bumping Build_Test to 1.16 and 1.17 * turning off go111module * putting go111module in variables section * only running gofmt on 1.16 versions * Refactor azcore (#15349) * Refactor azcore See the CHANGELOG for details. * add additional test coverage move early header check back to Pipeline.Do * add nil body check * added more test coverage removed spurious error return from NewPoller() sig. fixed bug in FinalResponse() to perform the final GET if the URL is specified but the terminal response contains no content. * fix tests that don't return a final payload * remove bad examples * move constants to make version info easier to find * moved Poller constructors out of root packages for core, moved to azcore/runtime for arm, moved to azcore/arm/runtime also moved ARM RP registration policy to runtime In order to do this, it was necessary to move all pipeline related content to its own internal package azcore/internal/pipeline. This was to avoid problems with circular dependencies. It also groups things that are logically related and cleaned up the code. * Fix armresources documentation link (#15380) * Deprecate to module (#15387) Update build constraints format. * Deprecate armcore module (#15386) Update build constraints per Go 1.17. Bump patch version. * Delete armcore and to modules (#15388) They've been deprecated and no longer required. * Add live test pipeline templates (#15382) * Add live test pipeline templates * Remove coverage goal from ci stage template * Stip empty changelog sections before release (#15397) Co-authored-by: Chidozie Ononiwu <chononiw@microsoft.com> * Update link after package refactor (#15399) * Release v57.0.0 1630045276 (#15400) * delete services\securityinsight\mgmt\v1.0\securityinsight * Generated from specification/network/resource-manager/readme.md tag package-2021-03 (commit hash: 6e95cd4c6086d61bb3de67cf6e4209a7b84efa23) * Generated from specification/cosmos-db/resource-manager/readme.md tag package-2021-03-preview (commit hash: 6e95cd4c6086d61bb3de67cf6e4209a7b84efa23) * Generated from specification/datafactory/resource-manager/readme.md tag package-2018-06 (commit hash: 6e95cd4c6086d61bb3de67cf6e4209a7b84efa23) * Generated from specification/dataprotection/resource-manager/readme.md tag package-2021-07 (commit hash: 6e95cd4c6086d61bb3de67cf6e4209a7b84efa23) * Generated from specification/netapp/resource-manager/readme.md tag package-netapp-2021-06-01 (commit hash: 6e95cd4c6086d61bb3de67cf6e4209a7b84efa23) * Generated from specification/synapse/resource-manager/readme.md tag package-preview-2021-06 (commit hash: 6e95cd4c6086d61bb3de67cf6e4209a7b84efa23) * Generated from specification/compute/resource-manager/readme.md tag package-2021-07-01 (commit hash: 6e95cd4c6086d61bb3de67cf6e4209a7b84efa23) * Generated from specification/guestconfiguration/resource-manager/readme.md tag package-2021-01-25 (commit hash: 6e95cd4c6086d61bb3de67cf6e4209a7b84efa23) * Generated from specification/postgresql/resource-manager/readme.md tag package-flexibleserver-2021-06 (commit hash: 6e95cd4c6086d61bb3de67cf6e4209a7b84efa23) * Generated from specification/guestconfiguration/resource-manager/readme.md tag package-2020-06-25 (commit hash: 6e95cd4c6086d61bb3de67cf6e4209a7b84efa23) * v57.0.0 * Remove coverage goal parameter reference (#15398) * ensure test principal creation succeeds properly (#15402) Co-authored-by: scbedd <45376673+scbedd@users.noreply.github.com> * Update CODEOWNERS (#15389) * Update LICENSE (#15390) * Sync eng/common directory with azure-sdk-tools for PR 1943 (#15406) * Bug fix: Storing .env file for bicep file at service dir * Styling Co-authored-by: Albert Cheng <albertcheng@microsoft.com> * Update Contributing documentation (#15393) Add additional sections for setting up development environments. Add checklists for best practices for submitting PRs * Add CODE_OF_CONDUCT (#15391) * Fix casing in CONTRIBUTING.md (#15410) * Update based on azcore refactor (#15383) Fixed some linter warnings. Renamed UserAgent to component per new telemetry policy. Version const is no longer exported. * Skip eng common workflow enforcer for private repos (#15411) Co-authored-by: Chidozie Ononiwu <chononiw@microsoft.com> * Sync eng/common directory with azure-sdk-tools for PR 1953 (#15413) * Add random elements to default base name * Fix principal * tab Co-authored-by: JoshLove-msft <54595583+JoshLove-msft@users.noreply.github.com> * upgrade autorest version 25->27 (#15417) * Return CredentialUnavailableError when IMDS has no assigned identity (#15377) * update template accordingly (#15419) * Updating azcore & azidentity changelog (#15423) <!-- 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 * Unexport AzureCLITokenProvider (#15427) * Make "Setup Versioning Properties" cancellable (#15430) Co-authored-by: Mike Harder <mharder@microsoft.com> * Release v57.1.0 (#15437) * 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 track 2 `armcompute` v0.3.0 (#15426) * [Release] sdk/compute/armcompute/0.2.1 generation from spec commit: d5ad9431bbc19c34090d97209eb497167db32879 * fix exmaple * goimports * fix * fix * fix * fix CHANGELOG release date * fix * Support v0 versions as prerelease beta versions (#15431) Co-authored-by: Wes Haggard <Wes.Haggard@microsoft.com> * [Internal] Add methods for configuring tests to work with test proxy (#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> Co-authored-by: Arcturus <dapzhang@microsoft.com> Co-authored-by: catalinaperalta <catalinaperaltah@hotmail.com> Co-authored-by: Azure SDK Bot <53356347+azure-sdk@users.noreply.github.com> Co-authored-by: Chidozie Ononiwu <chononiw@microsoft.com> Co-authored-by: Wes Haggard <Wes.Haggard@microsoft.com> Co-authored-by: 陈箭飞 <32889038+chenjianfei2017@users.noreply.github.com> Co-authored-by: Joel Hendrix <jhendrix@microsoft.com> Co-authored-by: Chidozie Ononiwu <31145988+chidozieononiwu@users.noreply.github.com> Co-authored-by: Wes Haggard <weshaggard@users.noreply.github.com> Co-authored-by: Christopher Scott <chriss@microsoft.com> Co-authored-by: Praveen Kuttappan <praveen.kuttappan@gmail.com> Co-authored-by: Rick Winter <rick.winter@microsoft.com> Co-authored-by: Daniel Jurek <djurek@microsoft.com> Co-authored-by: Ben Broderick Phillips <bebroder@microsoft.com> Co-authored-by: JoshLove-msft <54595583+JoshLove-msft@users.noreply.github.com> Co-authored-by: Chris Hamons <chris.hamons@microsoft.com> Co-authored-by: Sean Kane <68240067+seankane-msft@users.noreply.github.com> Co-authored-by: 804873052 <46921893+804873052@users.noreply.github.com> Co-authored-by: scbedd <45376673+scbedd@users.noreply.github.com> Co-authored-by: Ben Broderick Phillips <ben@benbp.net> Co-authored-by: nickzhums <56864335+nickzhums@users.noreply.github.com> Co-authored-by: praveenkuttappan <prmarott@microsoft.com> Co-authored-by: arcturusZhang <ufo54153@gmail.com> Co-authored-by: Tom Archer <tarcher@microsoft.com> Co-authored-by: Heath Stewart <heaths@microsoft.com> Co-authored-by: Albert Cheng <albertcheng@microsoft.com> Co-authored-by: Tom Archer <tom.archer@microsoft.com> Co-authored-by: tadelesh <tadelesh.shi@live.cn> Co-authored-by: Charles Lowell <chlowe@microsoft.com> Co-authored-by: Mike Harder <mharder@microsoft.com>
mohsha-msft
added a commit
that referenced
this pull request
Sep 2, 2021
* Fix dependency (2nd round) (#14745) * fix dependencies * fix the command graph * change the scripts in sdk automation * fix * azidentity: Adding Resource ID support in ManagedIdentityCredential (#14741) * enable resourceID for MSI cred * update const and rename * improve docs, add tests * review feedback * update type alias name * Sync eng/common directory with azure-sdk-tools for PR 1633 (#14757) * Update format of new Changelog Entry * Add parsing of changelog sections * Update ChangeLog Logic Co-authored-by: Chidozie Ononiwu <chononiw@microsoft.com> * Strict mode needs the variables initialized (#14762) Co-authored-by: Wes Haggard <Wes.Haggard@microsoft.com> * Release v55.2.0 1623309052 2 (#14766) * 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 * update references for credential types table (#14749) * LRO poller rewrite (#14752) * 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 * Sync eng/common directory with azure-sdk-tools for PR 1688 (#14780) * 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> * Release v55.3.0 1623834595 (#14782) * 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 (#14776) * update with latest code generator * update changelog * update with latest code generator (#14773) * update with latest code generator * update changelog * Fix hard-coded master (#14797) Co-authored-by: Chidozie Ononiwu <chononiw@microsoft.com> * update with latest code generator (#14775) * update with latest code generator * update changelog * update with latest code generator (#14774) * update with latest code generator * update changelog * update with latest code generator (#14772) * update with latest code generator * fix test and update minor version instead * update changelog * initial change to squash the package files (#14853) * Sync eng/common directory with azure-sdk-tools for PR 1716 (#14870) * Update links with master to use main * Update ci.yml files Co-authored-by: Chidozie Ononiwu <chononiw@microsoft.com> * Sync eng/common directory with azure-sdk-tools for PR 1719 (#14871) * 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> * Track 2 `privatedns` module release (#14793) * initial commit for private dns * fix copy/paste errors * gofmt * Go recorded test framework (#14767) * Go recorded test framework * Add tenantID on TokenRequestOptions (#14879) * add tenantID on TokenRequestOptions * bump azcore min version * improve comment * azidentity: add support for service fabric MSI environment (#14783) * adding service fabric support * update token handling * add test * fix expires_on handling * remove unused case in switch * add failure case test * update version.go (#14886) * azidentity: Add service fabric test program (#14788) * add service fabric program * update azidentity version * update comment * update azcore version * Release track 2 authorization module (#14867) * initial commit for authorization RP * modify meta.json * gofmt * fix C+V issues * Sync eng/common directory with azure-sdk-tools for PR 1729 (#14887) * 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> * Add Ubuntu 20 to local dns bypass template (#14896) Co-authored-by: Praveen Kuttappan <praveen.kuttappan@gmail.com> * Update master to main (#14909) * Update references from master to main (#14911) Co-authored-by: Wes Haggard <Wes.Haggard@microsoft.com> * Release v55.4.0 (#14914) * 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 * rename armmonitor to arminsights (#14725) * Update CodeOwners and Notice.txt (#14919) * Update CODEOWNERS file Update Notices.txt to standard microsoft format * Remove old files * change package name back to armmonitor (#14935) * Release v55.5.0 1625035291 (#14942) * Generated from specification/appconfiguration/resource-manager/readme.md tag package-2021-03-01-preview (commit hash: b548283ea6f4b3d797e85432fcc6e3ff2a2ac224) * Generated from specification/applicationinsights/resource-manager/readme.md tag package-2020-02-02 (commit hash: b548283ea6f4b3d797e85432fcc6e3ff2a2ac224) * Generated from specification/signalr/resource-manager/readme.md tag package-2021-06-01-preview (commit hash: b548283ea6f4b3d797e85432fcc6e3ff2a2ac224) * Generated from specification/webpubsub/resource-manager/readme.md tag package-2021-06-01-preview (commit hash: b548283ea6f4b3d797e85432fcc6e3ff2a2ac224) * Generated from specification/apimanagement/resource-manager/readme.md tag package-preview-2021-01 (commit hash: b548283ea6f4b3d797e85432fcc6e3ff2a2ac224) * v55.5.0 * Sync eng/common directory with azure-sdk-tools for PR 1725 (#14933) * Bring changes from JS docs metadata * Move business logic inside Update-DocsMsMetadata.ps1 * Update with the latest changes in JS PR * Update from latest PR feedback * Add check for empty path to prevent crashes when creating relative paths Co-authored-by: Daniel Jurek <djurek@microsoft.com> * Update devops workitem helpers (#14934) - Switch to using rest instead of cli for querying work items as cli is limited to 1000 items only. - Fix issues with Generated fields not being set. - Correctly sort older workitems by version isntead of string. Co-authored-by: Wes Haggard <Wes.Haggard@microsoft.com> * Add prepare pipelines pipeline (#14947) * Sync eng/common directory with azure-sdk-tools for PR 1763 (#14938) * Disable release date check * Release check is not finding release date Co-authored-by: Praveen Kuttappan <praveen.kuttappan@gmail.com> * Update docs metadata mutation logic (#14950) Co-authored-by: Daniel Jurek <djurek@microsoft.com> * Sync eng/common directory with azure-sdk-tools for PR 1772 (#14962) * Update change log headers based on guideline update Updates based on Azure/azure-sdk#3103 - Renamed "Key Bugs Fixed" to "Bugs Fixed" - Renamed "Fixed" to "Other Changes" Added a warning in validation if at lease one of the recommended headers aren't used. * Update eng/common/scripts/ChangeLog-Operations.ps1 Co-authored-by: JoshLove-msft <54595583+JoshLove-msft@users.noreply.github.com> Co-authored-by: Wes Haggard <Wes.Haggard@microsoft.com> Co-authored-by: Wes Haggard <weshaggard@users.noreply.github.com> Co-authored-by: JoshLove-msft <54595583+JoshLove-msft@users.noreply.github.com> * Skip live test generation in prepare-pipelines (#14958) * onboard track2: containerregistry (#14988) * Switch ubuntu 18 to ubuntu 20 (#14995) Co-authored-by: Chidozie Ononiwu <chononiw@microsoft.com> * Sync eng/common directory with azure-sdk-tools for PR 1767 (#14970) * Support building and deploying bicep templates * Add bicep powershell install aka link to deployment error message * Write bicep compiled arm templates to temp directory * Simplify bicep building code/function usage * Use bicep location for compiled arm templates, and remove them on success Co-authored-by: Ben Broderick Phillips <bebroder@microsoft.com> * fix readme in recording (#14981) * Update Ubuntu 18 to 20 (#14986) * Add build.ps1 script to build all SDKs (#15014) - Add build script and convert T2 CI to use new build.ps1 * Add root directory to list of sparse checkout paths (#15021) Explicitly add '/*' '!/*/' to the list of paths so we include the files in the root of the repo. Added some extra command logging so we can easily repro locally without needing to find the code that ran. Update to '/eng' so that we only get paths that start with eng instead of every path that has eng as a folder. Co-authored-by: Wes Haggard <Wes.Haggard@microsoft.com> * Release v55.6.0 1625727542 (#14999) * Generated from specification/resources/resource-manager/readme.md tag package-features-2021-07 (commit hash: f9883569df850be792a37df17a1a28a5bdcdfc39) * Generated from specification/security/resource-manager/readme.md tag package-composite-v3 (commit hash: f9883569df850be792a37df17a1a28a5bdcdfc39) * Generated from specification/vmware/resource-manager/readme.md tag package-2021-06-01 (commit hash: f9883569df850be792a37df17a1a28a5bdcdfc39) * Generated from specification/web/resource-manager/readme.md tag package-2021-01-15 (commit hash: f9883569df850be792a37df17a1a28a5bdcdfc39) * Generated from specification/appplatform/resource-manager/readme.md tag package-preview-2021-06 (commit hash: f9883569df850be792a37df17a1a28a5bdcdfc39) * Generated from specification/edgeorder/resource-manager/readme.md tag package-2020-12-preview (commit hash: f9883569df850be792a37df17a1a28a5bdcdfc39) * Generated from specification/quantum/resource-manager/readme.md tag package-2019-11-04-preview (commit hash: f9883569df850be792a37df17a1a28a5bdcdfc39) * Generated from specification/redis/resource-manager/readme.md tag package-2020-12 (commit hash: f9883569df850be792a37df17a1a28a5bdcdfc39) * v55.6.0 * azidentity: Add environment variable for resource ID (#14872) Fixes #14742 * Unexport HTTP headers to reduce public surface area (#15038) * Update CODEOWNERS (#15040) * Start documentation on build (#15012) * Sync eng/common directory with azure-sdk-tools for PR 1808 (#15044) * Add FilterPoliCheckResult.ps1 * Add description to FilterPoliCheckResults.ps1 Co-authored-by: Chidozie Ononiwu <chononiw@microsoft.com> * Release Monitor track 2 module (#15031) * initial commit of monitor track 2 module * some attempt to generate monitor, not finished * update readme * regenerate and refresh the metadata * Track2 release: containerregistry (#14992) * onboard track2: containerregistry * release track2 for containerregistry * track2 containerregistry release * release containerregistry track2 * Update sdk/containerregistry/armcontainerregistry/README.md Co-authored-by: Arcturus <dapzhang@microsoft.com> * Update sdk/containerregistry/armcontainerregistry/README.md Co-authored-by: Arcturus <dapzhang@microsoft.com> * Update sdk/containerregistry/armcontainerregistry/README.md Co-authored-by: Arcturus <dapzhang@microsoft.com> * Update sdk/containerregistry/armcontainerregistry/README.md Co-authored-by: Arcturus <dapzhang@microsoft.com> * Update sdk/containerregistry/armcontainerregistry/README.md Co-authored-by: Arcturus <dapzhang@microsoft.com> Co-authored-by: Arcturus <dapzhang@microsoft.com> * web track 2 module onboarding (#14984) * introduce web RP * Update sdk/web/armweb/ci.yml Co-authored-by: Wes Haggard <weshaggard@users.noreply.github.com> Co-authored-by: Wes Haggard <weshaggard@users.noreply.github.com> * Initial release of track 2 module `agrifood` (#14799) * initial commit of agrifood * fix bad path * Update sdk/agrifood/armagrifood/ci.yml Co-authored-by: Wes Haggard <weshaggard@users.noreply.github.com> Co-authored-by: Wes Haggard <weshaggard@users.noreply.github.com> * Sync eng/common directory with azure-sdk-tools for PR 1748 (#14930) * Simplify Ubuntu image name check for dns bypass * Fix typo Co-authored-by: Praveen Kuttappan <praveen.kuttappan@gmail.com> * Release v55.7.0 1626332017 (#15069) * Generated from specification/mediaservices/resource-manager/readme.md tag package-2021-06 (commit hash: e3980786418c280ecd7ed2e141407ed0f63cd493) * Generated from specification/security/resource-manager/readme.md tag package-composite-v3 (commit hash: e3980786418c280ecd7ed2e141407ed0f63cd493) * Generated from specification/mysql/resource-manager/readme.md tag package-2020-01-01 (commit hash: e3980786418c280ecd7ed2e141407ed0f63cd493) * Generated from specification/postgresql/resource-manager/readme.md tag package-2020-01-01 (commit hash: e3980786418c280ecd7ed2e141407ed0f63cd493) * v55.7.0 * remove exit 0 portion (#15035) * Build.ps1 improvements (#15017) - Fixes build.ps1 to work from relative locations (and in remove parallel logic that was breaking CI). * [Core] Renamings for AuthPolicies (#15103) * renamings * fixing lint * [Core] Rename AnonymousCredential (#15104) * renaming to New... * fixing lint * catalinas comments * fixing a bad merge * fixing lint step for azidentity (#15109) * [EngSys] Combine Build Build and Build Test pipelines (#15085) * only finds tests with a Test method * adding conditional to ci.yml files * removing the need for build.yml file * changing files to use one archetype-sdk-client.yml file * bad conditional * passing parameter through * checking for explicit true * moving coverage installation into test section * had to fix profiles ci.yml file * fixing hang up in gofmt when there are no go files in dir * changing echo to fmt * error check on samples * removing build.yml file * bens comments and lint * undoing change * simplifying * Release v55.8.0 1626925362 (#15118) * Generated from specification/containerinstance/resource-manager/readme.md tag package-2021-03 (commit hash: 4e6cea592e12915061beb893134e1a70d7e5c495) * Generated from specification/mysql/resource-manager/readme.md tag package-flexibleserver-2021-05-01-preview (commit hash: 4e6cea592e12915061beb893134e1a70d7e5c495) * Generated from specification/compute/resource-manager/readme.md tag package-2021-04-01 (commit hash: 4e6cea592e12915061beb893134e1a70d7e5c495) * v55.8.0 * Sync eng/common directory with azure-sdk-tools for PR 1827 (#15126) * update article type appropriately Co-authored-by: scbedd <45376673+scbedd@users.noreply.github.com> * track2 add servicebus ci.yml and LICENSE.txt file (#15132) * add track2 servicebus ci.yml and LICENSE.txt file * modify * Add sql ci.yml and LICENSE.txt file (#15133) * [build.ps1] Add support for converting _meta.json to autorest.md during build (#15111) * update the code owner to include renhe (#15131) * [EngSys] adding coverage tools (#15076) * adding coverage tools * forgot to include 'go run' * need to find where the script is running from * fixing script * fixing passing of coverage goal value * removing flag from build-test * fixing tools script for multiple files * updating ci.yml files, polishing script * removing coverage goals * undoing changes * dropping core down, fixing to.go lint * creating a config file for the SDK and using that instead of putting values in ci.yml * putting coverage in a script * fixing location of script * fixing scope * fixing powershell portions, float instead of &float64 * forcing pipelines * Update tools/internal/coverage/main.go Co-authored-by: Ben Broderick Phillips <ben@benbp.net> Co-authored-by: Ben Broderick Phillips <ben@benbp.net> * Many build.ps1 improvements (#15135) - Fixes #15128 - Fixes #15127 - Part of #15102 * add basic ga tag w/ settable campaignid (#15157) Co-authored-by: scbedd <45376673+scbedd@users.noreply.github.com> * add auxiliary tenants for multi-tenant scenario (#15123) * Introduce the migration guide to track 2 modules (#14980) * migration guide * resolve comments * move the migration guide to documentation directory * fix typo * resolve comments * resolve comments * Add Documentation for Track 2 Management SDK (#15096) * Update docs to reflect Track 2 status * Update README.md * Update README.md * Create previous-versions-quickstart.md * Update README.md * Update README.md * Update README.md * Update README.md * Create new-version-quickstart.md * Update new-version-quickstart.md * Update README.md * Update new-version-quickstart.md * Update new-version-quickstart.md * Update new-version-quickstart.md * Update new-version-quickstart.md * Update new-version-quickstart.md * add code snipet for resource groups * update code for vnet * update * Update new-version-quickstart.md * Update new-version-quickstart.md * Update new-version-quickstart.md * Update new-version-quickstart.md * Update new-version-quickstart.md * Update new-version-quickstart.md * Update new-version-quickstart.md * Update new-version-quickstart.md * Update new-version-quickstart.md * Update README.md Co-authored-by: Arcturus <dapzhang@microsoft.com> * Update documentation/new-version-quickstart.md Co-authored-by: Arcturus <dapzhang@microsoft.com> * Update documentation/new-version-quickstart.md Co-authored-by: Arcturus <dapzhang@microsoft.com> * Update documentation/new-version-quickstart.md Co-authored-by: Arcturus <dapzhang@microsoft.com> * Update README.md * Update README.md * Update README.md * Update documentation/new-version-quickstart.md Co-authored-by: Arcturus <dapzhang@microsoft.com> * Update documentation/new-version-quickstart.md Co-authored-by: Arcturus <dapzhang@microsoft.com> * Update documentation/new-version-quickstart.md Co-authored-by: Arcturus <dapzhang@microsoft.com> * Update documentation/new-version-quickstart.md Co-authored-by: Arcturus <dapzhang@microsoft.com> * Update documentation/new-version-quickstart.md Co-authored-by: Arcturus <dapzhang@microsoft.com> * Update documentation/new-version-quickstart.md Co-authored-by: Arcturus <dapzhang@microsoft.com> * Update documentation/new-version-quickstart.md Co-authored-by: Arcturus <dapzhang@microsoft.com> * Update documentation/new-version-quickstart.md Co-authored-by: Arcturus <dapzhang@microsoft.com> * Update documentation/new-version-quickstart.md Co-authored-by: Arcturus <dapzhang@microsoft.com> * Update new-version-quickstart.md * Update new-version-quickstart.md * Update README.md * Update README.md * Update new-version-quickstart.md * Update new-version-quickstart.md * Update README.md * Update previous-versions-quickstart.md * update code snippet Co-authored-by: ArcturusZhang <dapzhang@microsoft.com> * Verify API surface level against all approved revisions for GA releases (#15159) Co-authored-by: praveenkuttappan <prmarott@microsoft.com> * [Internal] fixing lint issues (#15162) * [EngSys] adding triggers on eng for internal and azcore (#15161) * adding eng for internal and azcore * adding a parameter for specifying the service directory * fixing script * fixing pipeline * do not run golangci on tests * adding false flag silencer * run lint on tests * [armcore] Add auxiliary tenants to connection options (#15124) * add auxiliary tenants to options * update based on latest azcore * update to latest azcore * lint updates * update nolint * [azidentity] Update code to use the latest azcore (#15137) * update code to latest azcore * update go.mod * remove unused header * type assertions * [azidentity] Expiring resource rework w/ multi-tenant support (#15138) * expiring resource rework * add auxiliary auth header * adding default logger functionality to internal (#15182) * Release v56.0.0 1627549243 (#15186) * Generated from specification/applicationinsights/resource-manager/readme.md tag package-2020-02-02 (commit hash: b47572ab2069865484c493a4f679d4e72f8b303d) * Generated from specification/automation/resource-manager/readme.md tag package-2019-06 (commit hash: b47572ab2069865484c493a4f679d4e72f8b303d) * Generated from specification/storagecache/resource-manager/readme.md tag package-2021-05 (commit hash: b47572ab2069865484c493a4f679d4e72f8b303d) * Generated from specification/batch/resource-manager/readme.md tag package-2021-06 (commit hash: b47572ab2069865484c493a4f679d4e72f8b303d) * Generated from specification/cost-management/resource-manager/readme.md tag package-2019-11 (commit hash: b47572ab2069865484c493a4f679d4e72f8b303d) * Generated from specification/machinelearningservices/resource-manager/readme.md tag package-2021-04-01 (commit hash: b47572ab2069865484c493a4f679d4e72f8b303d) * Generated from specification/consumption/resource-manager/readme.md tag package-2019-10 (commit hash: b47572ab2069865484c493a4f679d4e72f8b303d) * Generated from specification/datafactory/resource-manager/readme.md tag package-2018-06 (commit hash: b47572ab2069865484c493a4f679d4e72f8b303d) * Generated from specification/postgresql/resource-manager/readme.md tag package-flexibleserver-2021-06 (commit hash: b47572ab2069865484c493a4f679d4e72f8b303d) * v56.0.0 * [azcore] Remove logger from azcore (#15160) * initial work, issues with using internal package * fixing up errors in logger file * adding logger test file * removed an example * feedback and removing circular dependency * moving logclassification to be in logger, type alias in azcore * update internal version * fixes in azcore * fixes to azcore, works now * linter false positive * catalina's comments * more feedback * update version * Update sdk/azcore/version.go Co-authored-by: catalinaperalta <catalinaperaltah@hotmail.com> Co-authored-by: catalinaperalta <catalinaperaltah@hotmail.com> * [build.ps1] Early return error if autorest fails (#15179) * Move stress testing scripts to eng/common (#15189) Co-authored-by: Ben Broderick Phillips <bebroder@microsoft.com> * Update README.md (#15177) * remove race and covermode flags from test statement (#15200) * [Azcore] simplify progress objects (#15194) * started combining * single progress for request and response * adding docstrings * modify build-test to find silent error * undoing eng change * removing race and covermode from the test command * enhance build.ps1 script (#15191) * enhance build.ps1 script * fix typo * update autorest.go version (#15208) * [tools/generator] add the template command (#15206) * adds the template command * fix package name of readDir * add a new comment * resolve linter issues * add autorest.md template * update the template accordingly * fix typo in template (#15213) * fix typo in template * fix gofmt * Release v56.1.0 1628127975 (#15212) * Generated from specification/monitor/resource-manager/readme.md tag package-2021-07 (commit hash: 111fbceeeb7c2c4905b80fe2ab760e5564b3458a) * v56.1.0 * Add ability to leave SourceBranch empty when queueing a build (builds using the default branch) (#15195) Co-authored-by: Daniel Jurek <djurek@microsoft.com> * move verify link to the last step (#15220) * redirect the output to stderr in gofmt checking so that we can give errors when the code is not formatted (#15217) * redirect the output to stderr in gofmt checking so that we can give errors when the code is not formatted * fix redirect * switch to script as PS doesn't like redirecting to stderr Co-authored-by: Joel Hendrix <jhendrix@microsoft.com> * Fix build status and code coverage badges (#15224) * [azcore] Add ETag type to Core (#15207) * add type def for ETag * adding etag specific files and more functionality * testing weak * testing for etag equality * weak tag comparisons * adding docstrings * addressing richard and jeffs feedback. split equals to strong and weak, using more HasValue throughout * fixing one test * adding EOL * adding note about the difference between nil and empty etags * more feedback * fix spelling, remove comment about nil * format check * [azidentity] adding a changelog to azidentity (#15240) closes #15235 * [azcore] adding changelog (#15239) Adds a changelog with previous release information. closes #15234 * Release the track 2 `containerservice` module (#14792) * initial commit of the container service * go mod tidy * regen * regenerate * fix readme * track2 generator service bus (#15214) * Track2 generator cosmos db (#15218) * track2 generator Azure Cosmos DB * fix newclient * track2 generator postgresql (#15219) * track2 generator postgresql * delete postgresqlflexibleservers package * track2 generator postgresql flexible servers (#15221) * track2 generator postgresql flexible servers * gofmt Co-authored-by: arcturusZhang <ufo54153@gmail.com> * [EngSys] display output of go test (#15257) * track2 generator event grid (#15249) * track2 generator event grid * build.go * Fixed small bugs (#15266) <!-- 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 * Sync eng/common directory with azure-sdk-tools for PR 1866 (#15261) * Deploy for stress tests at resource group scope * Automatically compile bicep files in stress test deploy script Co-authored-by: Ben Broderick Phillips <bebroder@microsoft.com> * Refactor internal log package (#15265) Shortened package name to log. Renamed types to remove stuttering. Made logger implementation unexported. Moved logger methods to funcs to simplify API. * track2 release `armapimanagement` (#15250) * track2 generator apimanagement * fix build.go * track2 generator operationalinsights (#15251) * track2 release `armeventhub` (#15252) * track2 generator eventhub * fix build.go * track2 release `armredis` (#15253) * track2 gennerator redis * fix build.go * Web track2 release (#15193) * release web track 2 module * regenerate * regenerate * regenerate * gofmt * Release v56.2.0 1628747491 (#15270) * Generated from specification/machinelearningservices/resource-manager/readme.md tag package-2021-07-01 (commit hash: 47545c5f0842f5be29529674a4609d75a973d783) * Generated from specification/network/resource-manager/readme.md tag package-2021-02 (commit hash: 47545c5f0842f5be29529674a4609d75a973d783) * Generated from specification/web/resource-manager/readme.md tag package-2021-02 (commit hash: 47545c5f0842f5be29529674a4609d75a973d783) * Generated from specification/containerservice/resource-manager/readme.md tag package-2021-07 (commit hash: 47545c5f0842f5be29529674a4609d75a973d783) * Generated from specification/datafactory/resource-manager/readme.md tag package-2018-06 (commit hash: 47545c5f0842f5be29529674a4609d75a973d783) * v56.2.0 * Sync eng/common directory with azure-sdk-tools for PR 1894 (#15281) * Purge Key Vaults after deleting resource group * Resolve PR feedback * Purge Key Vaults and Managed HSMs * Clean up all purgeable KVs and MHSMs * Ignore null/empty purgeable resources * Resolve PR feedback * Resolve PR feedback * Fix logging, process each item when collection piped Co-authored-by: Heath Stewart <heaths@microsoft.com> * Renamed package runtime to diag (#15278) * Renamed package runtime to diag Since it deals with diagnostics and we want to use the name runtime for a package in azcore. Deleted FrameError as it's not very useful. Added Caller() for times when a full stack trace isn't needed. * Remove log.Listener * [azcore] JoinPaths will preserve query parameters (#15258) * adding join paths changes * fixing test * small fix and adding test scenario * jeffs feedback from another PR * forgot to remove the func * cleaner return * Sync eng/common directory with azure-sdk-tools for PR 1876 (#15279) * rebase * Resolve conflict Co-authored-by: Albert Cheng <albertcheng@microsoft.com> * Removed and renamed some content (#15285) Unexported policyFunc Renamed Transport to Transporter Removed TransportFunc, Pager, and Poller interfaces Added responseError (removed from internal) Fixed logging type aliases Updated logging usage based on internal/log refactor Removed StatusCodesForRetry, ProgressReceiver, ErrNoMorePolicies Renamed LROPoller to Poller * Add Rest Method checks to Prepare-Release (#15290) Co-authored-by: Wes Haggard <Wes.Haggard@microsoft.com> * Move logging to log package (#15287) * Fix resource clean-up script (#15289) - Fix rg.Name to rg.ResourceGroupName - Add more verbose logging for better debugging - Handle deleted resource groups when gathering puragable resource - Remove coerce now that we are collecting in functions Co-authored-by: Wes Haggard <Wes.Haggard@microsoft.com> * Remove Response type (#15294) Just return an *http.Response instead. Changed Response methods to functions. * Add support for 'files' configuration (#15288) Co-authored-by: Daniel Jurek <djurek@microsoft.com> * Prevent wildcard expansion in git sparse checkout add (#15286) Co-authored-by: Ben Broderick Phillips <bebroder@microsoft.com> * Sync eng/common directory with azure-sdk-tools for PR 1909 (#15302) * add test-proxy invocations to eng/common folder Co-authored-by: scbedd <45376673+scbedd@users.noreply.github.com> * Add errinfo package with error types (#15300) * Add errinfo package with error types * rename based on feedback * Enable API review approval check for Java spring packages (#15305) Co-authored-by: praveenkuttappan <prmarott@microsoft.com> * Update to latest versions of Go (#15304) * Update to latest versions of Go * update internal * update azcore * run code coverage tools from module directory * use explicit path to test and code coverage reports * fix 1.16 patch version * Update to crypto secure random number generation (#15190) * Removing C:\Git\azure-sdk-tools\eng\common\scripts\FilterPoliCheckResults.ps1 as it was no longer used (#15311) Co-authored-by: Chidozie Ononiwu <chononiw@microsoft.com> * Fixed bugs in Resource Group demo code (#15301) <!-- 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 * Fix some misbehaving tests (#15309) * Fix some misbehaving tests Enable tests in CI * add missing tests and lower CC threshold * add a bit more coverage * increase code coverage goal for azcore added move coverage to azcore and fixed bug in retry policy. fixed test publishing to include sub-directories. * add missing EOL * adding dates to existing changelogs (#15306) * Sync eng/common directory with azure-sdk-tools for PR 1912 (#15314) * Attempt to purge all vaults, managed HSMs Reverts #1910. Vaults and managed HSMs are automatically purged on their purge date. The point was to purge them daily to preserve capacity. The default purge date is +90 days. * Add timeout and more logging * Pass required -Resource * Fix log message * Ensure the $Resource is correctly captured Added comment to new code explaining why, since ScriptBlock.GetNewClosure() is not working as expected. * Add -ErrorAction to Receive-Job Worked without terminating when run locally, but failed on the first error in the AzDO agent. * Use $using:r instead of creating ScriptBlock More idiomatic for passing ScriptBlocks to jobs. * Resolve PR feedback * Change default DeleteAfterHours to 120 Resolves #1917 * Use the Az cmdlets built-in -AsJob Co-authored-by: Heath Stewart <heaths@microsoft.com> * Fix query batch processing (#15319) Co-authored-by: Wes Haggard <Wes.Haggard@microsoft.com> * fix: three dots path not work in linux in build script (#15297) * Release v56.3.0 1629350585 (#15323) * Generated from specification/apimanagement/resource-manager/readme.md tag package-preview-2021-01 (commit hash: 30d1537f0c9aa49e6e04401b1d16f612b31231e7) * Generated from specification/purview/resource-manager/readme.md tag package-2021-07-01 (commit hash: 30d1537f0c9aa49e6e04401b1d16f612b31231e7) * v56.3.0 * fix * Remove old unused update changelog script (#15326) Co-authored-by: Wes Haggard <Wes.Haggard@microsoft.com> * add clear output flag to build.ps1 (#15255) * add clear output flag * remove the path parameter and make sure the process-sdk function only runs in current directory * change flag name * [tools/generator] add automation and release command for track2 (#15271) * [tools/generator] add automation and release command for track2 * add copyright and do gofmt * Use semver to pump version, move git operation to repo package and fix some typo * Add specfic version to release cmd, seperate package for automation and remove test config file * add automation script and config for track2 * fix: three dots path not work in linux in build script * fix: ln pwsh.exe to pwsh to run in linux * fix: ignore dep check for go-git * chore: label name change for track2 * fix: wrong package path result in empty git commit * fix: repo addr replacement in autorest.md * chore: move script to eng folder and change name * fix: onboard generation need to replace NewClientMethod * chore: change all naming from track2 to v2 * fix: add new build constraint * fix: template compat with go 1.17 and change commit msg for release * chore: add v2 to readme begin regex to support future change of `track2` naming * chore: remove track2 naming * fix: duplicated and wrong expression * fix: add package title param to release cmd * fix: ci * Prevent ManagedIdentityCredential mutating GetToken arguments (#15331) * [Azcore] bumps version of internal (#15347) * bump internal to latest * upgrading internal * not the correct one * changelog update * [azidentity] bump azcore and internal dependencies (#15348) * Update CODEOWNERS (#15357) - Thanks for all the fish * introduce pipelines to the modules in tools directory (#14926) * Sync eng/common directory with azure-sdk-tools for PR 1918 (#15317) * Stress test deployment script resiliency fixes * Use PSModule-Helpers to install powershell-yaml Co-authored-by: Ben Broderick Phillips <bebroder@microsoft.com> * bumping Build_Test to 1.16 and 1.17 (#15370) * bumping Build_Test to 1.16 and 1.17 * turning off go111module * putting go111module in variables section * only running gofmt on 1.16 versions * Refactor azcore (#15349) * Refactor azcore See the CHANGELOG for details. * add additional test coverage move early header check back to Pipeline.Do * add nil body check * added more test coverage removed spurious error return from NewPoller() sig. fixed bug in FinalResponse() to perform the final GET if the URL is specified but the terminal response contains no content. * fix tests that don't return a final payload * remove bad examples * move constants to make version info easier to find * moved Poller constructors out of root packages for core, moved to azcore/runtime for arm, moved to azcore/arm/runtime also moved ARM RP registration policy to runtime In order to do this, it was necessary to move all pipeline related content to its own internal package azcore/internal/pipeline. This was to avoid problems with circular dependencies. It also groups things that are logically related and cleaned up the code. * Fix armresources documentation link (#15380) * Deprecate to module (#15387) Update build constraints format. * Deprecate armcore module (#15386) Update build constraints per Go 1.17. Bump patch version. * Delete armcore and to modules (#15388) They've been deprecated and no longer required. * Add live test pipeline templates (#15382) * Add live test pipeline templates * Remove coverage goal from ci stage template * Stip empty changelog sections before release (#15397) Co-authored-by: Chidozie Ononiwu <chononiw@microsoft.com> * Update link after package refactor (#15399) * Release v57.0.0 1630045276 (#15400) * delete services\securityinsight\mgmt\v1.0\securityinsight * Generated from specification/network/resource-manager/readme.md tag package-2021-03 (commit hash: 6e95cd4c6086d61bb3de67cf6e4209a7b84efa23) * Generated from specification/cosmos-db/resource-manager/readme.md tag package-2021-03-preview (commit hash: 6e95cd4c6086d61bb3de67cf6e4209a7b84efa23) * Generated from specification/datafactory/resource-manager/readme.md tag package-2018-06 (commit hash: 6e95cd4c6086d61bb3de67cf6e4209a7b84efa23) * Generated from specification/dataprotection/resource-manager/readme.md tag package-2021-07 (commit hash: 6e95cd4c6086d61bb3de67cf6e4209a7b84efa23) * Generated from specification/netapp/resource-manager/readme.md tag package-netapp-2021-06-01 (commit hash: 6e95cd4c6086d61bb3de67cf6e4209a7b84efa23) * Generated from specification/synapse/resource-manager/readme.md tag package-preview-2021-06 (commit hash: 6e95cd4c6086d61bb3de67cf6e4209a7b84efa23) * Generated from specification/compute/resource-manager/readme.md tag package-2021-07-01 (commit hash: 6e95cd4c6086d61bb3de67cf6e4209a7b84efa23) * Generated from specification/guestconfiguration/resource-manager/readme.md tag package-2021-01-25 (commit hash: 6e95cd4c6086d61bb3de67cf6e4209a7b84efa23) * Generated from specification/postgresql/resource-manager/readme.md tag package-flexibleserver-2021-06 (commit hash: 6e95cd4c6086d61bb3de67cf6e4209a7b84efa23) * Generated from specification/guestconfiguration/resource-manager/readme.md tag package-2020-06-25 (commit hash: 6e95cd4c6086d61bb3de67cf6e4209a7b84efa23) * v57.0.0 * Remove coverage goal parameter reference (#15398) * ensure test principal creation succeeds properly (#15402) Co-authored-by: scbedd <45376673+scbedd@users.noreply.github.com> * Update CODEOWNERS (#15389) * Update LICENSE (#15390) * Sync eng/common directory with azure-sdk-tools for PR 1943 (#15406) * Bug fix: Storing .env file for bicep file at service dir * Styling Co-authored-by: Albert Cheng <albertcheng@microsoft.com> * Update Contributing documentation (#15393) Add additional sections for setting up development environments. Add checklists for best practices for submitting PRs * Add CODE_OF_CONDUCT (#15391) * Fix casing in CONTRIBUTING.md (#15410) * Update based on azcore refactor (#15383) Fixed some linter warnings. Renamed UserAgent to component per new telemetry policy. Version const is no longer exported. * Skip eng common workflow enforcer for private repos (#15411) Co-authored-by: Chidozie Ononiwu <chononiw@microsoft.com> * Sync eng/common directory with azure-sdk-tools for PR 1953 (#15413) * Add random elements to default base name * Fix principal * tab Co-authored-by: JoshLove-msft <54595583+JoshLove-msft@users.noreply.github.com> * upgrade autorest version 25->27 (#15417) * Return CredentialUnavailableError when IMDS has no assigned identity (#15377) * update template accordingly (#15419) * Updating azcore & azidentity changelog (#15423) <!-- 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 * Unexport AzureCLITokenProvider (#15427) * Make "Setup Versioning Properties" cancellable (#15430) Co-authored-by: Mike Harder <mharder@microsoft.com> * Release v57.1.0 (#15437) * 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 track 2 `armcompute` v0.3.0 (#15426) * [Release] sdk/compute/armcompute/0.2.1 generation from spec commit: d5ad9431bbc19c34090d97209eb497167db32879 * fix exmaple * goimports * fix * fix * fix * fix CHANGELOG release date * fix * Support v0 versions as prerelease beta versions (#15431) Co-authored-by: Wes Haggard <Wes.Haggard@microsoft.com> * [Internal] Add methods for configuring tests to work with test proxy (#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> * Test CI Trigger Co-authored-by: Arcturus <dapzhang@microsoft.com> Co-authored-by: catalinaperalta <catalinaperaltah@hotmail.com> Co-authored-by: Azure SDK Bot <53356347+azure-sdk@users.noreply.github.com> Co-authored-by: Chidozie Ononiwu <chononiw@microsoft.com> Co-authored-by: Wes Haggard <Wes.Haggard@microsoft.com> Co-authored-by: 陈箭飞 <32889038+chenjianfei2017@users.noreply.github.com> Co-authored-by: Joel Hendrix <jhendrix@microsoft.com> Co-authored-by: Chidozie Ononiwu <31145988+chidozieononiwu@users.noreply.github.com> Co-authored-by: Wes Haggard <weshaggard@users.noreply.github.com> Co-authored-by: Christopher Scott <chriss@microsoft.com> Co-authored-by: Praveen Kuttappan <praveen.kuttappan@gmail.com> Co-authored-by: Rick Winter <rick.winter@microsoft.com> Co-authored-by: Daniel Jurek <djurek@microsoft.com> Co-authored-by: Ben Broderick Phillips <bebroder@microsoft.com> Co-authored-by: JoshLove-msft <54595583+JoshLove-msft@users.noreply.github.com> Co-authored-by: Chris Hamons <chris.hamons@microsoft.com> Co-authored-by: Sean Kane <68240067+seankane-msft@users.noreply.github.com> Co-authored-by: 804873052 <46921893+804873052@users.noreply.github.com> Co-authored-by: scbedd <45376673+scbedd@users.noreply.github.com> Co-authored-by: Ben Broderick Phillips <ben@benbp.net> Co-authored-by: nickzhums <56864335+nickzhums@users.noreply.github.com> Co-authored-by: praveenkuttappan <prmarott@microsoft.com> Co-authored-by: arcturusZhang <ufo54153@gmail.com> Co-authored-by: Tom Archer <tarcher@microsoft.com> Co-authored-by: Heath Stewart <heaths@microsoft.com> Co-authored-by: Albert Cheng <albertcheng@microsoft.com> Co-authored-by: Tom Archer <tom.archer@microsoft.com> Co-authored-by: tadelesh <tadelesh.shi@live.cn> Co-authored-by: Charles Lowell <chlowe@microsoft.com> Co-authored-by: Mike Harder <mharder@microsoft.com>
jay-most
pushed a commit
to jay-most/azure-sdk-for-go
that referenced
this pull request
Sep 18, 2021
* adding join paths changes * fixing test * small fix and adding test scenario * jeffs feedback from another PR * forgot to remove the func * cleaner return
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.
Also updates the changelog with previous updates