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

Update storage dep for aws-sdk-go 1.34.3 #727

Merged
merged 2 commits into from
Aug 14, 2020

Conversation

bunsenmcdubbs
Copy link
Contributor

@bunsenmcdubbs bunsenmcdubbs commented Aug 12, 2020

Updating to include aws-sdk-go bugfix. More info here: wal-g/storages#22

Since wal-g/storages isn't a go module yet, I ran the following commands:

go mod tidy # (to clean up ahead of time)
go get -u github.com/wal-g/storages
go get -u github.com/aws/aws-sdk-go

@@ -347,8 +343,6 @@ github.com/xiang90/probing v0.0.0-20190116061207-43a291ad63a2/go.mod h1:UETIi67q
github.com/xordataexchange/crypt v0.0.3-0.20170626215501-b2862e3d0a77/go.mod h1:aYKd//L2LvnjZzWKhF00oedf4jCCReLcmhLdhm1A27Q=
go.etcd.io/bbolt v1.3.2/go.mod h1:IbVyRI1SCnLcuJnV2u8VeU0CEYM7e686BmAb1XKL+uU=
go.mongodb.org/mongo-driver v1.0.3/go.mod h1:u7ryQJ+DOzQmeO7zB6MHyr8jkEQvC8vH7qLUO4lqsUM=
go.mongodb.org/mongo-driver v1.2.0 h1:6fhXjXSzzXRQdqtFKOI1CDw6Gw5x6VflovRpfbrlVi0=
go.mongodb.org/mongo-driver v1.2.0/go.mod h1:u7ryQJ+DOzQmeO7zB6MHyr8jkEQvC8vH7qLUO4lqsUM=
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Any idea why this is removing mongo drivers? I assume it is just some clean up from go mod tidy

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ya im not sure... cc @x4m or anyone else more familiar with mongo usage plz chime in here. This was output from go mod tidy

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks like just dropping old version of mongo driver as mongo-driver v1.3.4 is still here and go.mod file has v1.3.4 of mongo-driver

Copy link
Collaborator

@x4m x4m left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@x4m x4m merged commit 991c27b into wal-g:master Aug 14, 2020
@bunsenmcdubbs bunsenmcdubbs deleted the update-storage-dep-aws-sdk branch August 14, 2020 18:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants