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 interop: compute_engine_creds test fails #895

Open
stevej opened this issue Jan 5, 2017 · 1 comment
Open

gRPC interop: compute_engine_creds test fails #895

stevej opened this issue Jan 5, 2017 · 1 comment

Comments

@stevej
Copy link

stevej commented Jan 5, 2017

Jan 05, 2017 8:48:33 PM io.grpc.internal.ManagedChannelImpl <init>
INFO: [io.grpc.internal.ManagedChannelImpl-1] Created with target directaddress:https:///proxy-test-4b/10.240.0.10:
4141
Running test compute_engine_creds
Exception in thread "main" java.lang.AssertionError: expected:<null> but was:<>
        at org.junit.Assert.fail(Assert.java:88)
        at org.junit.Assert.failNotEquals(Assert.java:743)
        at org.junit.Assert.assertEquals(Assert.java:118)
        at org.junit.Assert.assertEquals(Assert.java:144)
        at io.grpc.testing.integration.AbstractInteropTest.computeEngineCreds(AbstractInteropTest.java:1091)
        at io.grpc.testing.integration.TestServiceClient.runTest(TestServiceClient.java:236)
        at io.grpc.testing.integration.TestServiceClient.run(TestServiceClient.java:200)
        at io.grpc.testing.integration.TestServiceClient.main(TestServiceClient.java:84)
Shutting down
@stevej
Copy link
Author

stevej commented Jan 5, 2017

it's worth noting that gRPC won't send credentials over an unencrypted channel and this test was done without TLS so I will re-run all of these credential failure test cases with TLS.

@stevej stevej self-assigned this Jan 5, 2017
@olix0r olix0r added the gRPC label Jan 6, 2017
Tim-Brooks pushed a commit to Tim-Brooks/linkerd that referenced this issue Dec 20, 2018
…pods (linkerd#895)

* Modify the Stat endpoint to also return the count of failed pods
* Add comments explaining pod count stats
* Rename total pod count to running pod count

This is to support the service mesh overview page, as I'd like to include an indicator of
failed pods there.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
No open projects
Development

No branches or pull requests

2 participants