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

getting connection refused with Cart Service #140

Closed
puckpuck opened this issue Jun 15, 2022 · 6 comments · Fixed by #145
Closed

getting connection refused with Cart Service #140

puckpuck opened this issue Jun 15, 2022 · 6 comments · Fixed by #145
Assignees
Labels
bug Something isn't working

Comments

@puckpuck
Copy link
Contributor

Bug Report

Which version of the demo you are using?
982cefedaa504a52b658e32d45e4d5f5a8ca68bd - PR #105

Symptom

The cart service seems to be unreachable, and all upstream services return an error when attempting to connect

What is the expected behavior?

No errors

What is the actual behavior?

Getting this error:
transport: Error while dialing dial tcp 172.18.0.11:7070: connect: connection refused

Reproduce

git checkout 982cefedaa504a52b658e32d45e4d5f5a8ca68bd
docker compose build
docker compose up

look at any of the traces for /cart/checkout on the frontend service in Jaeger

We will close this issue if:

  • The steps you provided are complex.
  • If we can not reproduce the behavior you're reporting.

Additional Context

See image of broken trace
Screen Shot 2022-06-15 at 12 33 02 AM

@puckpuck puckpuck added the bug Something isn't working label Jun 15, 2022
@julianocosta89
Copy link
Member

@puckpuck can we revert the PR #105 in order to have a running sample again?

@robbkidd
Copy link
Member

My current workaround for this is to add the following to the cartservice environment stanza in docker-compose.yml:

- ASPNETCORE_URLS=http://*:${CART_SERVICE_PORT}

This gets me past the frontend service failing to connect to cartservice and to a new error, a 500 from cartservice itself:

The request matched multiple endpoints. Matches: 

gRPC - Unimplemented service
gRPC - Unimplemented service

@puckpuck
Copy link
Contributor Author

@puckpuck can we revert the PR #105 in order to have a running sample again?

I'm going to investigate this deeper to try and figure out what in the PR broke it. I'll know more by tomorrow.

@mic-max
Copy link
Contributor

mic-max commented Jun 20, 2022

@puckpuck can we revert the PR #105 in order to have a running sample again?

I'm going to investigate this deeper to try and figure out what in the PR broke it. I'll know more by tomorrow.

Ya I'd like to know why as well. I wanted to remove the generate files, which ./services/CartService.cs is one of, but obviously would prefer the example still works :D

@puckpuck
Copy link
Contributor Author

Ya I'd like to know why as well. I wanted to remove the generate files, which ./services/CartService.cs is one of, but obviously would prefer the example still works :D

Though the logic doesn't seem like much, I don't believe that file is generated. It acts as a thin wrapper around calling into the CartStore itself.

@mic-max
Copy link
Contributor

mic-max commented Jun 22, 2022

Ya I'd like to know why as well. I wanted to remove the generate files, which ./services/CartService.cs is one of, but obviously would prefer the example still works :D

Though the logic doesn't seem like much, I don't believe that file is generated. It acts as a thin wrapper around calling into the CartStore itself.

Ah that does seem plausible. I wonder if this wrapper pattern is typically used in C# protobuf projects..

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants