chore: upgrade to 2020-06-30/compute

[andyzhangx]

What type of PR is this?
/kind api-change

What this PR does / why we need it:
chore: upgrade to 2020-06-30/compute

Which issue(s) this PR fixes:

Fixes #

Special notes for your reviewer:

sed -i 's/2019-12-01\/compute/2020-06-30\/compute/g' ./provider/*.*
sed -i 's/2019-12-01\/compute/2020-06-30\/compute/g' ./mockvmsets/*.*
sed -i 's/2019-12-01\/compute/2020-06-30\/compute/g' ./azureclients/diskclient/*.*
sed -i 's/2019-12-01\/compute/2020-06-30\/compute/g' ./azureclients/vmssclient/*.*
sed -i 's/2019-12-01\/compute/2020-06-30\/compute/g' ./azureclients/vmssvmclient/*.*
sed -i 's/2019-12-01\/compute/2020-06-30\/compute/g' ./azureclients/snapshotclient/*.*
sed -i 's/2019-12-01\/compute/2020-06-30\/compute/g' ./azureclients/vmsizeclient/*.*
sed -i 's/2019-12-01\/compute/2020-06-30\/compute/g' ./azureclients/vmclient/*.*
sed -i 's/2019-12-01\/compute/2020-06-30\/compute/g' ./azureclients/diskclient/mockdiskclient/*.*
sed -i 's/2019-12-01\/compute/2020-06-30\/compute/g' ./azureclients/vmssclient/mockvmssclient/*.*
sed -i 's/2019-12-01\/compute/2020-06-30\/compute/g' ./azureclients/snapshotclient/mocksnapshotclient/*.*
sed -i 's/2019-12-01\/compute/2020-06-30\/compute/g' ./azureclients/vmsizeclient/mockvmsizeclient/*.*
sed -i 's/2019-12-01\/compute/2020-06-30\/compute/g' ./azureclients/vmclient/mockvmclient/*.*
sed -i 's/2019-12-01\/compute/2020-06-30\/compute/g' ./azureclients/vmssvmclient/mockvmssvmclient/*.*

Release note:

none

[k8s-ci-robot]

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: andyzhangx

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Needs approval from an approver in each of these files:

• OWNERS [andyzhangx]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

[andyzhangx]

@nilo19 how to fix this issue, golangci-lint is just a code check tool, why it's included in vendor/modules.txt? We could remove this tool by replacing with github action.

level=error msg="Running error: context loading failed: failed to load program with go/packages: err: exit status 1: stderr: go: inconsistent vendoring in /home/prow/go/src/sigs.k8s.io/cloud-provider-azure:\n\tgithub.com/golangci/golangci-lint@v1.25.0: is explicitly required in go.mod, but not marked as explicit in vendor/modules.txt\n\nrun 'go mod vendor' to sync, or use -mod=mod or -mod=readonly to ignore the vendor directory\n"
make: *** [Makefile:164: test-lint] Error 3

also what's the following ut failure:

        	Test:       	TestListWithNextPage
    azure_containerserviceclient_test.go:402: missing call(s) to *mockarmclient.MockInterface.PrepareGetRequest(is anything, is anything) /home/prow/go/src/sigs.k8s.io/cloud-provider-azure/pkg/azureclients/containerserviceclient/azure_containerserviceclient_test.go:386
    azure_containerserviceclient_test.go:402: missing call(s) to *mockarmclient.MockInterface.Send(is anything, is anything) /home/prow/go/src/sigs.k8s.io/cloud-provider-azure/pkg/azureclients/containerserviceclient/azure_containerserviceclient_test.go:387
    azure_containerserviceclient_test.go:402: missing call(s) to *mockarmclient.MockInterface.CloseResponse(is anything, is anything) /home/prow/go/src/sigs.k8s.io/cloud-provider-azure/pkg/azureclients/containerserviceclient/azure_containerserviceclient_test.go:397
    azure_containerserviceclient_test.go:402: aborting test due to missing call(s)
--- FAIL: TestListWithNextPage (0.00s)

[nilo19]

@andyzhangx I think it is because of this line https://github.com/kubernetes-sigs/cloud-provider-azure/blob/master/hack/verify-golint.sh#L19

[andyzhangx]

@andyzhangx I think it is because of this line https://github.com/kubernetes-sigs/cloud-provider-azure/blob/master/hack/verify-golint.sh#L19

could switch to use https://github.com/kubernetes-sigs/azuredisk-csi-driver/blob/master/hack/verify-golint.sh

[andyzhangx]

/test pull-kubernetes-e2e-azure-disk-vmss-ccm

[feiskyer]

could switch to use https://github.com/kubernetes-sigs/azuredisk-csi-driver/blob/master/hack/verify-golint.sh

yes, let's switch to download the tool directly

[andyzhangx]

After upgrade to go sdk v49.1.0, only following ut failed, is it related to this PR(kubernetes/kubernetes#96211) ? @feiskyer

~/go/src/github.com/kubernetes-sigs/cloud-provider-azure/pkg/azureclients/containerserviceclient# go test
--- FAIL: TestListWithNextPage (0.00s)
    azure_containerserviceclient_test.go:401:
                Error Trace:    azure_containerserviceclient_test.go:401
                Error:          Not equal:
                                expected: 6
                                actual  : 3
                Test:           TestListWithNextPage
    azure_containerserviceclient_test.go:402: missing call(s) to *mockarmclient.MockInterface.PrepareGetRequest(is anything, is anything) /root/go/src/github.com/kubernetes-sigs/cloud-provider-azure/pkg/azureclients/containerserviceclient/azure_containerserviceclient_test.go:386
    azure_containerserviceclient_test.go:402: missing call(s) to *mockarmclient.MockInterface.Send(is anything, is anything) /root/go/src/github.com/kubernetes-sigs/cloud-provider-azure/pkg/azureclients/containerserviceclient/azure_containerserviceclient_test.go:387
    azure_containerserviceclient_test.go:402: missing call(s) to *mockarmclient.MockInterface.CloseResponse(is anything, is anything) /root/go/src/github.com/kubernetes-sigs/cloud-provider-azure/pkg/azureclients/containerserviceclient/azure_containerserviceclient_test.go:397
    azure_containerserviceclient_test.go:402: aborting test due to missing call(s)

[nilo19]

for the golangci-lint problem:
#446

[andyzhangx]

/retest

[nilo19]

/lgtm