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

pubsub: not working inside docker (no error output) #2733

Closed
victorperin opened this issue Nov 8, 2017 · 8 comments
Closed

pubsub: not working inside docker (no error output) #2733

victorperin opened this issue Nov 8, 2017 · 8 comments
Assignees

Comments

@victorperin
Copy link

victorperin commented Nov 8, 2017

Environment details

  • OS: Fedora 25 (Docker 17.09)
  • Node.js version: Using image node:9.0.0-alpine (same problem with older versions)
  • npm version: 5.5.1
  • google-cloud-node version: "@google-cloud/pubsub": "0.14.7"

Steps to reproduce

I created a repository reproducing the steps, to make it easy to everyone understand what is the problem.
https://github.com/victorperin/google-pubsub-node-docker-bug

Seems to be some kind of lib missing, but I don't know how to find it.

@victorperin
Copy link
Author

victorperin commented Nov 8, 2017

Tried to reproduce with all pubsub dependencies and found that the problem is with:

@google-cloud/common-grpc

@victorperin
Copy link
Author

Digging down the rabbit hole i found the responsible is the grpc package itself.

@stephenplusplus
Copy link
Contributor

This issue was moved to grpc/grpc-node#85

@BrunoQuaresma
Copy link

Any prediction of update the common-grpc package?

@victorperin
Copy link
Author

Waiting for it too.

@stephenplusplus
Copy link
Contributor

The latest release of grpc is 1.7.2. common-grpc depends on ^1.7.1, so 1.7.2 will be picked up for new installs.

@BrunoQuaresma
Copy link

I tried to make a new install but still not working 😞

@callmehiphop
Copy link
Contributor

@stephenplusplus pubsub no longer depends on common-grpc, so for it to pick up these changes up, we would need to update gax-nodejs which would also require gapics being regenerated.

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

No branches or pull requests

4 participants