Skip to content
This repository was archived by the owner on Sep 9, 2020. It is now read-only.

dep prune: lock hash doesn't match #685

Closed
tomwilkie opened this issue May 31, 2017 · 4 comments
Closed

dep prune: lock hash doesn't match #685

tomwilkie opened this issue May 31, 2017 · 4 comments

Comments

@tomwilkie
Copy link
Contributor

Happens when I update a single dependancy:

Toms-MacBook-Pro:cortex twilkie$ dep ensure github.com/weaveworks/common@grpc_http_metrics
Toms-MacBook-Pro:cortex twilkie$ dep prune
lock hash doesn't match

Goes away if I do a dep ensure.

@tomwilkie
Copy link
Contributor Author

Also of note, I had to go and manually update the cached repo for github.com/weaveworks/common, as I was getting:

$ dep ensure github.com/weaveworks/common@grpc_http_metrics
ensure Solve(): No versions of github.com/weaveworks/common met constraints:
	master: Could not introduce github.com/weaveworks/common@master, as it is not allowed by constraint grpc_http_metrics from project github.com/weaveworks/cortex.
	context: Could not introduce github.com/weaveworks/common@context, as it is not allowed by constraint grpc_http_metrics from project github.com/weaveworks/cortex.
	grpc_http_metrics: Unable to update checked out version: fatal: reference is not a tree: 37bdded08835f193231acccdc7b18cbb748bcd73

	log-users: Could not introduce github.com/weaveworks/common@log-users, as it is not allowed by constraint grpc_http_metrics from project github.com/weaveworks/cortex.

Which looks like #639 again.

@sdboyer
Copy link
Member

sdboyer commented May 31, 2017

Happens when I update a single dependancy:

Hmm, seems like a regression on ensure. We should probably update our test harness to enforce the invariant that commands exit with the same lock inputs hash that it wrote out to the lockfile.

Also of note, I had to go and manually update the cached repo for github.com/weaveworks/common, as I was getting:

I'm gonna try to find time today to track this down personally, we can't leave it sit.

@tomwilkie
Copy link
Contributor Author

Thanks. I should have some more time to spend on dep next week I hope.

@sdboyer
Copy link
Member

sdboyer commented Aug 4, 2017

the underlying issue with dep ensure is now fixed; closing this.

@sdboyer sdboyer closed this as completed Aug 4, 2017
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

2 participants