Skip to content
This repository has been archived by the owner on Sep 26, 2021. It is now read-only.

azure driver import packages which is not exist any more #1844

Closed
xiaohui opened this issue Sep 12, 2015 · 7 comments
Closed

azure driver import packages which is not exist any more #1844

xiaohui opened this issue Sep 12, 2015 · 7 comments

Comments

@xiaohui
Copy link
Contributor

xiaohui commented Sep 12, 2015

azure's driver import an package github.com/MSOpenTech/azure-sdk-for-go which is not exist anymore. it will make ci/circleci check failed.

go get -t -d -v ./...
github.com/MSOpenTech/azure-sdk-for-go (download)
package github.com/MSOpenTech/azure-sdk-for-go/clients/vmClient
imports github.com/MSOpenTech/azure-sdk-for-go/clients/vmClient
imports github.com/MSOpenTech/azure-sdk-for-go/clients/vmClient: cannot find package "github.com/MSOpenTech/azure-sdk-for-go/clients/vmClient" in any of:
/usr/local/go/src/github.com/MSOpenTech/azure-sdk-for-go/clients/vmClient (from $GOROOT)
/home/ubuntu/.go_workspace/src/github.com/MSOpenTech/azure-sdk-for-go/clients/vmClient (from $GOPATH)
/usr/local/go_workspace/src/github.com/MSOpenTech/azure-sdk-for-go/clients/vmClient
go get -t -d -v ./... returned exit code 1
/home/ubuntu/.go_project/src/github.com/MSOpenTech/azure-sdk-for-go/clients/vmClient Action failed: go get

@thaJeztah
Copy link
Member

/cc @ahmetalpbalkan

@ahmetb
Copy link
Contributor

ahmetb commented Sep 12, 2015

You are not using godep. There's a vendored old version of azure-sdk-for-go here which works for machine.

@dmp42
Copy link
Contributor

dmp42 commented Sep 13, 2015

Thanks @ahmetalpbalkan

Closing.

@dmp42 dmp42 closed this as completed Sep 13, 2015
@xiaohui
Copy link
Contributor Author

xiaohui commented Sep 14, 2015

@ahmetalpbalkan @dmp42 It happened at ci/circleci tests, I don't know why not using godep for cirleci, but it actually effect the PR's check, should we replace the github.com/MSOpenTech/azure-sdk-for-go package to the right one?

@ahmetb
Copy link
Contributor

ahmetb commented Sep 14, 2015

@xiaohui nope not now, the new repo is entirely different than what we got here. I already have an open work item in this repo assigned to me to do that sometime. :)

@dmp42
Copy link
Contributor

dmp42 commented Sep 14, 2015

@xiaohui circleci tests are not ready. Please ignore circle for now. Sorry for the inconvenience.

@xiaohui
Copy link
Contributor Author

xiaohui commented Sep 14, 2015

👌thanks, @ahmetalpbalkan @dmp42

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

4 participants