Skip to content
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

Jump to bottom

x/vgo: import "github.com/Azure/azure-sdk-for-go": module source tree too big #25470

Closed
davidwalter0 opened this issue May 20, 2018 · 7 comments
Closed

x/vgo: import "github.com/Azure/azure-sdk-for-go": module source tree too big #25470

davidwalter0 opened this issue May 20, 2018 · 7 comments
Labels
FrozenDueToAge
Milestone
vgo

Comments

@davidwalter0
Copy link

davidwalter0 commented May 20, 2018 •
edited

Please answer these questions before submitting your issue. Thanks!

What version of Go are you using (go version)?

go version
go version go1.10.2 linux/amd64

Does this issue reproduce with the latest release?

Yes

What operating system and processor architecture are you using (go env)?

GOARCH="amd64"
GOBIN=""
GOCACHE="/home/david/.cache/go-build"
GOEXE=""
GOHOSTARCH="amd64"
GOHOSTOS="linux"
GOOS="linux"
GOPATH="/go"
GORACE=""
GOROOT="/usr/local/go"
GOTMPDIR=""
GOTOOLDIR="/usr/local/go/pkg/tool/linux_amd64"
GCCGO="gccgo"
CC="gcc"
CXX="g++"
CGO_ENABLED="1"
CGO_CFLAGS="-g -O2"
CGO_CPPFLAGS=""
CGO_CXXFLAGS="-g -O2"
CGO_FFLAGS="-g -O2"
CGO_LDFLAGS="-g -O2"
PKG_CONFIG="pkg-config"
GOGCCFLAGS="-fPIC -m64 -pthread -fmessage-length=0 -fdebug-prefix-map=/tmp/go-build297679675=/tmp/go-build -gno-record-gcc-switches"

What did you do?

vgo get github.com/Azure/azure-sdk-for-go@v16.2.0

If possible, provide a recipe for reproducing the error.

vgo get github.com/Azure/azure-sdk-for-go@v16.2.0
vgo: downloading github.com/Azure/azure-sdk-for-go v0.0.0-20180503200013-52423c14f13e
vgo: import "github.com/Azure/azure-sdk-for-go": module source tree too big

A complete runnable program is good.
A link on play.golang.org is best.

What did you expect to see?

vgo: resolving import "github.com/Azure/azure-sdk-for-go"
vgo: finding github.com/Azure/azure-sdk-for-go 
vgo: adding github.com/Azure/azure-sdk-for-go

What did you see instead?

vgo get github.com/Azure/azure-sdk-for-go@v16.2.0
vgo: downloading github.com/Azure/azure-sdk-for-go v0.0.0-20180503200013-52423c14f13e
vgo: import "github.com/Azure/azure-sdk-for-go": module source tree too big
@gopherbot gopherbot added this to the vgo milestone May 20, 2018
@kardianos
Copy link
Contributor

This looks like a duplicate (or perhaps a variant) of #25424 .

@kardianos
Copy link
Contributor

The total files in the azure sdk is > 100MiB , mostly in the services directory. Some of this size appears to be caused by versioning service APIs by date, so probably a lot of duplicate code.

The question this raises is: should there be a max module size? If so, what should this max size be? Can it be overridden (say from an env var)?

@ncw
Copy link
Contributor

ncw commented May 31, 2018

Is there a workaround for this? Unfortunately I've hit this brick wall in my first vgo adventure (with rclone)!

@kardianos
Copy link
Contributor

Right now I think the work around is to go get vgo, then adjust the 100 MiB limit to be higher.

@rsc What do you think should be done here? The options I see are to:

  • Make the limit higher and require packages to fall within it.
  • Keep the limit as-is, require the packages to pull down external resources or in Azure's case, actually version their API calls into separate modules.
  • Allow this limit to be set based on amount of RAM available.
  • Allow this limit to be overridden by an environment variable or similar.

@oiooj
Copy link
Member

oiooj commented Jun 6, 2018

Fixed by rsc.

@oiooj oiooj closed this as completed Jun 6, 2018
@ncw
Copy link
Contributor

ncw commented Jun 6, 2018

@oiooj do we have a commit or a CL so we can examine the fix?

@oiooj
Copy link
Member

oiooj commented Jun 6, 2018

@ncw , The change is CL 116355.

@agnivade agnivade mentioned this issue Dec 13, 2018
Closed
@paulquerna-okta paulquerna-okta mentioned this issue May 29, 2019
Closed
@golang golang locked and limited conversation to collaborators Jun 6, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees
No one assigned
Labels
FrozenDueToAge
Projects
None yet
Milestone
vgo
Development

No branches or pull requests
5 participants
@davidwalter0 @ncw @kardianos @oiooj @gopherbot