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

Use google.golang.org/grpc v1.29.1 to fix a dependency issue #394

Merged
merged 2 commits into from Dec 1, 2020

Conversation

maraino
Copy link
Collaborator

@maraino maraino commented Dec 1, 2020

Description

go.etcd.io/etcd depends on deprecated packages of gRPC that have been removed. This causes that go mod cannot build the dependency tree, and utilities like go mod tidy or go mod vendor does not work properly.

This fix uses a gRPC version that supports old and new interfaces. So it can be used by etcd and google cloud packages.

Fixes #377

go.etcd.io/etcd depends on deprecated packages of gRPC that have
been removed. This causes that go mod cannot build the dependency
tree and utilities like `go mod tidy` or `go mod vendor` does not
work properly.

This fix uses a gRPC version that supports old and new interfaces.
So it can be used by etcd and google cloud packages.

Fixes #377
@maraino maraino requested a review from dopey December 1, 2020 02:23
go.mod Outdated
gopkg.in/square/go-jose.v2 v2.5.1
software.sslmate.com/src/go-pkcs12 v0.0.0-20201103104416-57fc603b7f52
)

// This is a temporal workaround to fix a dependency problem between etcd and
// gRPC. The gRPC v1.29.1 supports old a new interfaces, so it can be used by
Copy link
Contributor

Choose a reason for hiding this comment

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

old and new interfaces

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Fixed in ca7d62d

Copy link
Contributor

@dopey dopey left a comment

Choose a reason for hiding this comment

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

approved, but just fix the one grammar thing I noted.

@codecov
Copy link

codecov bot commented Dec 1, 2020

Codecov Report

Merging #394 (ca7d62d) into master (7951724) will decrease coverage by 0.97%.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #394      +/-   ##
==========================================
- Coverage   63.89%   62.92%   -0.98%     
==========================================
  Files          56       56              
  Lines        6692     6993     +301     
==========================================
+ Hits         4276     4400     +124     
- Misses       2190     2367     +177     
  Partials      226      226              
Impacted Files Coverage Δ
command/certificate/sign.go 38.57% <0.00%> (-3.23%) ⬇️
command/certificate/create.go 52.45% <0.00%> (-2.84%) ⬇️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 7951724...ca7d62d. Read the comment docs.

@maraino maraino merged commit 02de9b4 into master Dec 1, 2020
@maraino maraino deleted the dependency-workaround branch December 1, 2020 02:34
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.

Error during "go mod vendor" to build in an offline environment
2 participants