Azure · mohsha-msft · Sep 2, 2021 · Jun 7, 2021 · Jun 7, 2021 · Jun 8, 2021
diff --git a/.github/CODEOWNERS b/.github/CODEOWNERS
@@ -7,16 +7,16 @@
 
 # Catch all for non-code project files and unowned files | folders
 *                        @rickwinter @jhendrixMSFT
-/sdk/                    @rickwinter @jhendrixMSFT @richardpark-msft @seankane-msft @chamons
+/sdk/                    @rickwinter @chlowell @jhendrixMSFT @richardpark-msft @seankane-msft
 
 # Samples
-/samples/                @rickwinter @jhendrixMSFT @richardpark-msft @seankane-msft @chamons
+/samples/                @rickwinter @chlowell @jhendrixMSFT @richardpark-msft @seankane-msft
 
 # PRLabel: %Azure.Core
-/sdk/azcore/             @rickwinter @jhendrixMSFT @richardpark-msft @seankane-msft @chamons
+/sdk/azcore/             @rickwinter @chlowell @jhendrixMSFT @richardpark-msft @seankane-msft
 
 # PRLabel: %Azure.Identity
-/sdk/azidentity/         @rickwinter @jhendrixMSFT
+/sdk/azidentity/         @rickwinter @chlowell
 
 # PRLabel: %KeyVault
 /sdk/keyvault/           @rickwinter @jhendrixMSFT
@@ -38,7 +38,7 @@
 # SDK (track1 and track1.5)
 #############
 
-/profiles/ @ArcturusZhang @lirenhe
-/services/ @ArcturusZhang @lirenhe
-/storage/ @jhendrixMSFT
-/version/ @ArcturusZhang @lirenhe
+/profiles/               @ArcturusZhang @lirenhe
+/services/               @ArcturusZhang @lirenhe
+/storage/                @jhendrixMSFT
+/version/                @ArcturusZhang @lirenhe
diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -1,5 +1,37 @@
 # CHANGELOG
 
+## `v57.1.0`
+
+### New Packages
+
+- `github.com/Azure/azure-sdk-for-go/services/mysql/mgmt/2021-05-01/mysqlflexibleservers`
+- `github.com/Azure/azure-sdk-for-go/services/servicefabric/mgmt/2021-06-01/servicefabric`
+
+## `v57.0.0`
+
+### New Packages
+
+- `github.com/Azure/azure-sdk-for-go/services/compute/mgmt/2021-07-01/compute`
+- `github.com/Azure/azure-sdk-for-go/services/dataprotection/mgmt/2021-07-01/dataprotection`
+- `github.com/Azure/azure-sdk-for-go/services/guestconfiguration/mgmt/2021-01-25/guestconfiguration`
+- `github.com/Azure/azure-sdk-for-go/services/netapp/mgmt/2021-06-01/netapp`
+- `github.com/Azure/azure-sdk-for-go/services/network/mgmt/2021-03-01/network`
+- `github.com/Azure/azure-sdk-for-go/services/preview/synapse/mgmt/2021-06-01-preview/synapse`
+
+### Updated Packages
+
+| Package Path | Changelog |
+| :--- | :---: |
+| `github.com/Azure/azure-sdk-for-go/services/datafactory/mgmt/2018-06-01/datafactory` | [details](https://github.com/Azure/azure-sdk-for-go/blob/v57.0.0/services/datafactory/mgmt/2018-06-01/datafactory/CHANGELOG.md) |
+
+### Breaking Changes
+
+| Package Path | Changelog |
+| :--- | :---: |
+| `github.com/Azure/azure-sdk-for-go/services/guestconfiguration/mgmt/2020-06-25/guestconfiguration` | [details](https://github.com/Azure/azure-sdk-for-go/blob/v57.0.0/services/guestconfiguration/mgmt/2020-06-25/guestconfiguration/CHANGELOG.md) |
+| `github.com/Azure/azure-sdk-for-go/services/postgresql/mgmt/2021-06-01/postgresqlflexibleservers` | [details](https://github.com/Azure/azure-sdk-for-go/blob/v57.0.0/services/postgresql/mgmt/2021-06-01/postgresqlflexibleservers/CHANGELOG.md) |
+| `github.com/Azure/azure-sdk-for-go/services/preview/cosmos-db/mgmt/2021-03-01-preview/documentdb` | [details](https://github.com/Azure/azure-sdk-for-go/blob/v57.0.0/services/preview/cosmos-db/mgmt/2021-03-01-preview/documentdb/CHANGELOG.md) |
+
 ## `v56.3.0`
 
 ### New Packages
@@ -10,7 +42,7 @@
 
 | Package Path | Changelog |
 | :--- | :---: |
-| `github.com/Azure/azure-sdk-for-go/services/preview/apimanagement/mgmt/2021-01-01-preview/apimanagement` | [details](https://github.com/Azure/azure-sdk-for-go/tree/main/services/preview/apimanagement/mgmt/2021-01-01-preview/apimanagement/CHANGELOG.md) |
+| `github.com/Azure/azure-sdk-for-go/services/preview/apimanagement/mgmt/2021-01-01-preview/apimanagement` | [details](https://github.com/Azure/azure-sdk-for-go/blob/v56.3.0/services/preview/apimanagement/mgmt/2021-01-01-preview/apimanagement/CHANGELOG.md) |
 
 ## `v56.2.0`
 

diff --git a/CODE_OF_CONDUCT.md b/CODE_OF_CONDUCT.md
@@ -0,0 +1,9 @@
+# Microsoft Open Source Code of Conduct
+
+This project has adopted the [Microsoft Open Source Code of Conduct](https://opensource.microsoft.com/codeofconduct/).
+
+Resources:
+
+- [Microsoft Open Source Code of Conduct](https://opensource.microsoft.com/codeofconduct/)
+- [Microsoft Code of Conduct FAQ](https://opensource.microsoft.com/codeofconduct/faq/)
+- Contact [opencode@microsoft.com](mailto:opencode@microsoft.com) with questions or concerns
diff --git a/CONTRIBUTING.md b/CONTRIBUTING.md
@@ -1,4 +1,47 @@
-# Contributing
+# Azure SDK for Go Contributing Guide
+
+Thank you for your interest in contributing to Azure SDK for Go.
+
+- For reporting bugs, requesting features, or asking for support, please file an issue in the [issues](https://github.com/Azure/azure-sdk-for-go/issues) section of the project.
+
+- If you would like to become an active contributor to this project please follow the instructions provided in [Microsoft Azure Projects Contribution Guidelines](https://azure.github.io/azure-sdk/policies_opensource.html).
+
+- To make code changes, or contribute something new, please follow the [GitHub Forks / Pull requests model](https://help.github.com/articles/fork-a-repo/): Fork the repo, make the change and propose it back by submitting a pull request.
+
+## Pull Requests
+
+- **DO** follow the API design and implementation [Go Guidelines](https://azure.github.io/azure-sdk/golang_introduction.html).
+  - When submitting large changes or features, **DO** have an issue or spec doc that describes the design, usage, and motivating scenario.
+- **DO** submit all code changes via pull requests (PRs) rather than through a direct commit. PRs will be reviewed and potentially merged by the repo maintainers after a peer review that includes at least one maintainer.
+- **DO** review your own PR to make sure there are no unintended changes or commits before submitting it.
+- **DO NOT** submit "work in progress" PRs. A PR should only be submitted when it is considered ready for review and subsequent merging by the contributor.
+  - If the change is work-in-progress or an experiment, **DO** start off as a temporary draft PR.
+- **DO** give PRs short-but-descriptive names (e.g. "Improve code coverage for Azure.Core by 10%", not "Fix #1234") and add a description which explains why the change is being made.
+- **DO** refer to any relevant issues, and include [keywords](https://help.github.com/articles/closing-issues-via-commit-messages/) that automatically close issues when the PR is merged.
+- **DO** tag any users that should know about and/or review the change.
+- **DO** ensure each commit successfully builds. The entire PR must pass all tests in the Continuous Integration (CI) system before it'll be merged.
+- **DO** address PR feedback in an additional commit(s) rather than amending the existing commits, and only rebase/squash them when necessary. This makes it easier for reviewers to track changes.
+- **DO** assume that ["Squash and Merge"](https://github.com/blog/2141-squash-your-commits) will be used to merge your commit unless you request otherwise in the PR.
+- **DO NOT** mix independent, unrelated changes in one PR. Separate real product/test code changes from larger code formatting/dead code removal changes. Separate unrelated fixes into separate PRs, especially if they are in different modules or files that otherwise wouldn't be changed.
+- **DO** comment your code focusing on "why", where necessary. Otherwise, aim to keep it self-documenting with appropriate names and style.
+- **DO** add [GoDoc style comments](https://azure.github.io/azure-sdk/golang_introduction.html#documentation-style) when adding new APIs or modifying header files.
+- **DO** make sure there are no typos or spelling errors, especially in user-facing documentation.
+- **DO** verify if your changes have impact elsewhere. For instance, do you need to update other docs or exiting markdown files that might be impacted?
+- **DO** add relevant unit tests to ensure CI will catch future regressions.
+
+## Merging Pull Requests (for project contributors with write access)
+
+- **DO** use ["Squash and Merge"](https://github.com/blog/2141-squash-your-commits) by default for individual contributions unless requested by the PR author.
+  Do so, even if the PR contains only one commit. It creates a simpler history than "Create a Merge Commit".
+  Reasons that PR authors may request "Merge and Commit" may include (but are not limited to):
+
+  - The change is easier to understand as a series of focused commits. Each commit in the series must be buildable so as not to break `git bisect`.
+  - Contributor is using an e-mail address other than the primary GitHub address and wants that preserved in the history. Contributor must be willing to squash
+    the commits manually before acceptance.
+
+## Developer Guide
+
+### Repo structure
 
 Most packages under the `services` directory in the SDK are generated from [Azure API specs][azure_rest_specs]
 using [Azure/autorest.go][] and [Azure/autorest][]. These generated packages depend on the HTTP client implemented at [Azure/go-autorest][]. Therefore when contributing, please make sure you do not change anything under the `services` directory.
@@ -10,7 +53,24 @@ using [Azure/autorest.go][] and [Azure/autorest][]. These generated packages dep
 
 For bugs or feature requests you can submit them using the [Github issues page][issues] and filling the appropriate template.
 
-## Building
+### Codespaces
+
+Codespaces is new technology that allows you to use a container as your development environment. This repo provides a Codespaces container which is supported by both GitHub Codespaces and VS Code Codespaces.
+
+#### GitHub Codespaces
+
+1. From the Azure SDK GitHub repo, click on the "Code -> Open with Codespaces" button.
+1. Open a Terminal. The development environment will be ready for you. Continue to [Building and Testing](https://github.com/Azure/azure-sdk-for-go/blob/main/CONTRIBUTING.md#building-and-testing).
+
+#### VS Code Codespaces
+
+1. Install the [VS Code Remote Extension Pack](https://marketplace.visualstudio.com/items?itemName=ms-vscode-remote.vscode-remote-extensionpack)
+1. When you open the Azure SDK for Go repo in VS Code, it will prompt you to open the project in the Dev Container. If it does not prompt you, then hit CTRL+P, and select "Remote-Containers: Open Folder in Container..."
+1. Open a Terminal. The development environment will be ready for you. Continue to [Building and Testing](https://github.com/Azure/azure-sdk-for-go/blob/main/CONTRIBUTING.md#building-and-testing).
+
+### Building and Testing
+
+#### Building
 
 SDKs are either old (track 1) or new (track 2):
 
@@ -23,14 +83,6 @@ To build, run `go build` from the respective SDK directory.
 
 There currently is not a repository wide way to build or regenerate code.
 
-## Guidelines and Code of Conduct
-
-Also please see these [guidelines][guidelines] about contributing to Azure projects.
-
-This project follows the [Microsoft Open Source Code of Conduct][coc]. For more information see the [Code of Conduct FAQ][cocfaq]. Contact [opencode@microsoft.com][cocmail] with questions and comments.
+#### Testing
 
-[guidelines]: https://opensource.microsoft.com/collaborate/
-[coc]: https://opensource.microsoft.com/codeofconduct/
-[cocfaq]: https://opensource.microsoft.com/codeofconduct/faq/
-[cocmail]: mailto:opencode@microsoft.com
-[issues]: https://github.com/Azure/Azure-sdk-for-go/issues
+To test, run 'go test' from the respective directory.
diff --git a/LICENSE.txt b/LICENSE.txt
@@ -1,6 +1,6 @@
 The MIT License (MIT)
 
-Copyright (c) 2021 Microsoft
+Copyright (c) Microsoft Corporation.
 
 Permission is hereby granted, free of charge, to any person obtaining a copy
 of this software and associated documentation files (the "Software"), to deal
@@ -18,4 +18,4 @@ FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT. IN NO EVENT SHALL THE
 AUTHORS OR COPYRIGHT HOLDERS BE LIABLE FOR ANY CLAIM, DAMAGES OR OTHER
 LIABILITY, WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING FROM,
 OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS IN THE
-SOFTWARE.
+SOFTWARE.
diff --git a/azure-pipelines.yml b/azure-pipelines.yml
@@ -4,40 +4,42 @@ trigger:
   paths:
     exclude:
     - sdk/
-    - tools/generator
+    - tools/
 
 pr:
   paths:
     exclude:
     - sdk/
-    - tools/generator
+    - tools/
 
 jobs:
   - job: Build_Test
     strategy:
       matrix:
-        Linux_Go113:
+        Linux_Go116:
           pool.name: azsdk-pool-mms-ubuntu-1804-general
-          go.version: '1.14'
-        Linux_Go114:
+          go.version: '1.16.7'
+        Linux_Go117:
           pool.name: azsdk-pool-mms-ubuntu-1804-general
-          go.version: '1.15'
+          go.version: '1.17'
 
     pool:
       name: $(pool.name)
 
     variables:
       GOPATH: '$(system.defaultWorkingDirectory)/work'
       sdkPath: '$(GOPATH)/src/github.com/$(build.repository.name)'
+      GO111MODULE: 'off'
       IGNORE_BREAKING_CHANGES: true
-      go.list.filter: '| grep -v vendor | grep -v azure-sdk-for-go/sdk | grep -v azure-sdk-for-go/tools/generator'
-      go.test.filter: '-path ./vendor -prune -o -path ./sdk -prune -o -path ./tools/generator -prune'
+      go.list.filter: '| grep -v vendor | grep -v azure-sdk-for-go/sdk | grep -v azure-sdk-for-go/tools'
+      go.test.filter: '-path ./vendor -prune -o -path ./sdk -prune -o -path ./tools -prune'
 
     steps:
     - task: GoTool@0
       inputs:
         version: '$(go.version)'
       displayName: "Select Go Version"
+
     - script: |
         set -e
         mkdir -p '$(GOPATH)/bin'
@@ -47,6 +49,7 @@ jobs:
         echo '##vso[task.prependpath]$(GOROOT)/bin'
         echo '##vso[task.prependpath]$(GOPATH)/bin'
       displayName: 'Create Go Workspace'
+
     - script: |
         set -e
         go version
@@ -55,37 +58,46 @@ jobs:
         go get -u golang.org/x/lint/golint
       workingDirectory: '$(sdkPath)'
       displayName: 'Install Dependencies'
+
     - script: go vet -v $(go list ./... $(go.list.filter))
       workingDirectory: '$(sdkPath)'
       displayName: 'Vet'
+
     - script: go build -v $(go list ./... $(go.list.filter))
       workingDirectory: '$(sdkPath)'
       displayName: 'Build'
+
     - script: go test $(dirname $(find . $(go.test.filter) -name '*_test.go' -print) | sort -u)
       workingDirectory: '$(sdkPath)'
       displayName: 'Run Tests'
+
     - template: /eng/common/pipelines/templates/steps/verify-links.yml
       parameters:
         Directory: '.'
         ScriptDirectory: '$(sdkPath)/eng/common/scripts'
         WorkingDirectory: '$(sdkPath)'
         Urls: $(Get-ChildItem -Path '$(sdkPath)/*.md' -Recurse | Where {$_.FullName -notlike "*/vendor/*" -and $_.FullName -notlike "*/sdk/*"})
+
     - script: go run ./tools/apidiff/main.go packages ./services FETCH_HEAD~1 FETCH_HEAD --copyrepo --breakingchanges || $IGNORE_BREAKING_CHANGES
       workingDirectory: '$(sdkPath)'
       displayName: 'Display Breaking Changes'
+
     - script: go run ./tools/pkgchk/main.go ./services --exceptions ./tools/pkgchk/exceptions.txt
       workingDirectory: '$(sdkPath)'
       displayName: 'Verify Package Directory'
+
     - script: grep -L -r --include *.go --exclude-dir vendor -P "Copyright (\d{4}|\(c\)) Microsoft" ./ | tee >&2
       workingDirectory: '$(sdkPath)'
       displayName: 'Copyright Header Check'
       failOnStderr: true
       condition: succeededOrFailed()
+
     - script: gofmt -s -l -d $(find . -path ./vendor -prune -o -name '*.go' -print) >&2
       workingDirectory: '$(sdkPath)'
       displayName: 'Format Check'
       failOnStderr: true
-      condition: succeededOrFailed()
+      condition: and(succeededOrFailed(), startsWith(variables['go.version'], '1.16'))
+
     - script: |
         golint ./storage/... >&2
       workingDirectory: '$(sdkPath)'

diff --git a/documentation/MIGRATION_GUIDE.md b/documentation/MIGRATION_GUIDE.md
@@ -2,7 +2,7 @@
 
 This document is intended for users that are familiar with an older version of the Azure SDK For Go for management modules (`services/**/mgmt/**`) and wish to migrate their application to the next version of Azure resource management libraries (`sdk/**/arm**`)
 
-**For users new to the Azure SDK For Go for resource management modules, please see the [README for 'sdk/armcore`](https://github.com/Azure/azure-sdk-for-go/tree/main/sdk/armcore) and the README for every individual package.**
+**For users new to the Azure SDK For Go for resource management modules, please see the [README for 'sdk/azcore`](https://github.com/Azure/azure-sdk-for-go/tree/main/sdk/azcore) and the README for every individual package.**
 
 ## Table of contents
 
@@ -40,7 +40,7 @@ To the show the code snippets for the change:
 authorizer, err := adal.NewServicePrincipalToken(oAuthToken, "<ClientId>", "<ClientSecret>", endpoint)
 client := resources.NewGroupsClient("<SubscriptionId>")
 client.Authorizer = authorizer
-```        
+```
 
 **Equivalent in new version (`sdk/**/arm**`)**
 
@@ -54,7 +54,7 @@ For detailed information on the benefits of using the new authentication classes
 
 ### Error Handling
 
-There are some minor changes in the error handling. 
+There are some minor changes in the error handling.
 
 - When there is an error in the SDK request, in the old version (`services/**/mgmt/**`), the return value will all be non-nil, and you can get the raw HTTP response from the response value. In the new version (`sdk/**/arm**`), the first return value will be empty and you need to cast the error to `HTTPResponse` interface to get the raw HTTP response. When the request is successful and there is no error returned, you will need to get the raw HTTP response in `RawResponse` property of the first return value.
 
@@ -180,7 +180,7 @@ In new version (`sdk/**/arm**`), we use `(armcore.ConnectionOptions).PerCallPoli
 
 Similar to the customized policy, there are changes regarding how the custom HTTP client is configured as well. You can now use the `(armcore.ConnectionOptions).HTTPClient` option in `github.com/Azure/azure-sdk-for-go/sdk/armcore` module to use your own implementation of HTTP client and plug in what they need into the configuration.
 
-**In old version (`services/**/mgmt/**`)** 
+**In old version (`services/**/mgmt/**`)**
 ```go
 httpClient := NewYourOwnHTTPClient{}
 client := resources.NewGroupsClient("<SubscriptionId>")
@@ -199,4 +199,4 @@ client := armresources.NewResourceGroupsClient(connection, "<SubscriptionId>")
 
 ## Need help?
 
-If you have encountered an issue during migration, please file an issue via [Github Issues](https://github.com/Azure/azure-sdk-for-go/issues) and make sure you add the "Preview" label to the issue
+If you have encountered an issue during migration, please file an issue via [Github Issues](https://github.com/Azure/azure-sdk-for-go/issues) and make sure you add the "Preview" label to the issue