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

grpc-bridge example fails to get keys from the Python service #326

Closed
drobertduke opened this issue Jan 5, 2017 · 3 comments
Closed

grpc-bridge example fails to get keys from the Python service #326

drobertduke opened this issue Jan 5, 2017 · 3 comments
Labels
Milestone

Comments

@drobertduke
Copy link

After getting docker-compose build to run using the #325 branch, I am able to set keys via the Python service but not get them

$> docker-compose exec python /client/client.py set foo bar
setf foo to bar
$> docker-compose exec python /client/client.py get foo
Traceback (most recent call last):
  File "/client/client.py", line 78, in <module>
    run()
  File "/client/client.py", line 57, in run
    response = client.get(key)
  File "/client/client.py", line 27, in get
    return kv.GetResponse().FromString(resp.content[5:])
  File "/usr/local/lib/python2.7/dist-packages/google/protobuf/internal/python_message.py", line 780, in FromString
    message.MergeFromString(s)
  File "/usr/local/lib/python2.7/dist-packages/google/protobuf/internal/python_message.py", line 1087, in MergeFromString
    if self._InternalParse(serialized, 0, length) != length:
  File "/usr/local/lib/python2.7/dist-packages/google/protobuf/internal/python_message.py", line 1113, in InternalParse
    new_pos = local_SkipField(buffer, new_pos, end, tag_bytes)
  File "/usr/local/lib/python2.7/dist-packages/google/protobuf/internal/decoder.py", line 850, in SkipField
    return WIRETYPE_TO_SKIPPER[wire_type](buffer, pos, end)
  File "/usr/local/lib/python2.7/dist-packages/google/protobuf/internal/decoder.py", line 820, in _RaiseInvalidWireType
    raise _DecodeError('Tag had invalid wire type.')
google.protobuf.message.DecodeError: Tag had invalid wire type.

Printing resp.content yields upstream connect error or disconnect/reset before headers

https://github.com/lyft/envoy/blob/master/source/common/router/router.cc#L383

It looks like the case is that the downstream response has started, but there's an upstream reset which is not a GlobalTimeout or a PerTryTimeout.

@mattklein123
Copy link
Member

Related to #155. We need to automate building/testing the example containers.

@drobertduke
Copy link
Author

I've confirmed that this happens because the tutorial instructions are out of order (you need to build the go service before building the containers), and the failure to start /usr/local/bin/srv is swallowed because it starts in the background.

@junr03
Copy link
Member

junr03 commented Jan 18, 2017

fixed by #357

@junr03 junr03 closed this as completed Jan 18, 2017
rshriram pushed a commit to rshriram/envoy that referenced this issue Oct 30, 2018
* Create one mixerclient for each worker thread.

* Use std::thread
rshriram pushed a commit to rshriram/envoy that referenced this issue Oct 30, 2018
Automatic merge from submit-queue.

[DO NOT MERGE] Auto PR to update dependencies of mixerclient

This PR will be merged automatically once checks are successful.
```release-note
none
```
jpsim pushed a commit that referenced this issue Nov 28, 2022
Updating java library layer to conform to the streaming interfaces changes. Also, tweaks to make the java layer more consistent with the ObjC layer in envoyproxy/envoy-mobile#327

Signed-off-by: Alan Chiu <achiu@lyft.com>

For an explanation of how to fill out the fields, please see the relevant section
in [PULL_REQUESTS.md](https://github.com/envoyproxy/envoy/blob/master/PULL_REQUESTS.md)

Description: Updating java library interfaces
Risk Level: low
Testing: n/a
Docs Changes: n/a
Release Notes: n/a
[Optional Fixes #Issue]
[Optional Deprecated:]

Signed-off-by: JP Simard <jp@jpsim.com>
jpsim pushed a commit that referenced this issue Nov 29, 2022
Updating java library layer to conform to the streaming interfaces changes. Also, tweaks to make the java layer more consistent with the ObjC layer in envoyproxy/envoy-mobile#327

Signed-off-by: Alan Chiu <achiu@lyft.com>

For an explanation of how to fill out the fields, please see the relevant section
in [PULL_REQUESTS.md](https://github.com/envoyproxy/envoy/blob/master/PULL_REQUESTS.md)

Description: Updating java library interfaces
Risk Level: low
Testing: n/a
Docs Changes: n/a
Release Notes: n/a
[Optional Fixes #Issue]
[Optional Deprecated:]

Signed-off-by: JP Simard <jp@jpsim.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

3 participants