-
Notifications
You must be signed in to change notification settings - Fork 23
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 spectatord #77
Merged
Merged
Use spectatord #77
Commits on Sep 28, 2020
-
Switches from implementing a stateful registry to a registry that sends all acitivity to a spectatord sidecar. The build only supports bazel now, and we add a dependency on abseil and asio (removing the curl/cares/rapidjson/boringssl, etc. dependencies)
Configuration menu - View commit details
-
Copy full SHA for 45ed1c0 - Browse repository at this point
Copy the full SHA 45ed1c0View commit details
Commits on Sep 29, 2020
-
Test: Print a stacktrace on exceptions/crashes
Currently the travis build is failing but not giving a lot of detail about what is causing an exception.
Configuration menu - View commit details
-
Copy full SHA for 5f2afd3 - Browse repository at this point
Copy the full SHA 5f2afd3View commit details -
The latest build shows the exception happening because we didn't join a thread which points to something throwing in one of the test servers (udp/local)
Configuration menu - View commit details
-
Copy full SHA for 1a668ad - Browse repository at this point
Copy the full SHA 1a668adView commit details -
Add some logging to test servers
Diagnosing the joinable thread throwing an exception during the destructor of the test server
Configuration menu - View commit details
-
Copy full SHA for 3cf8644 - Browse repository at this point
Copy the full SHA 3cf8644View commit details -
Configuration menu - View commit details
-
Copy full SHA for 0a5e1f9 - Browse repository at this point
Copy the full SHA 0a5e1f9View commit details -
Configuration menu - View commit details
-
Copy full SHA for 4384035 - Browse repository at this point
Copy the full SHA 4384035View commit details -
Configuration menu - View commit details
-
Copy full SHA for 19ad1d2 - Browse repository at this point
Copy the full SHA 19ad1d2View commit details -
Fix deleting TestPublisher as a Publisher
We were missing a virtual destructor for Publisher causing the TestPublisher instance in our tests to not be destructed properly (the dtor for Publisher was invoked through the pointer, and not TestPublisher)
Configuration menu - View commit details
-
Copy full SHA for 5b545ee - Browse repository at this point
Copy the full SHA 5b545eeView commit details
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.