merge from upstream#10
Merged
eric846 merged 6 commits intoeric846:adaptive-rps-protos2from Jul 16, 2020
Merged
Conversation
- First iteration on testing gcc support in CI. This restricts the task to just building Nighthawk, and not testing it yet, as that needs a change in Envoy to go in first. - Small cleanup, and a bugfix for those who use the script to run locally, which would be detected as running in CircleCI. Related issue: #382 (needs a follow up to close, left in as a code-level todo). Signed-off-by: Otto van der Schaaf <oschaaf@we-amp.com>
Signed-off-by: Otto van der Schaaf <oschaaf@we-amp.com>
#391) Add new NH statistic class CircllhistStatistic and SinkableCircllhistStatistic. CircllhistStatistic uses Circllhist under the hood to compute statistics where Circllhist is used in the implementation of Envoy Histograms. Compared to HdrHistogram Circllhist trades precision for fast performance in merge and insertion according to #115. SinkableCircllhistStatistic wraps the Envoy::Stats::Histogram interface and is used to flush histogram value to downstream Envoy stats Sinks. Linked Issues: #344 Testing: unit tests
Add an extension which implements dynamic delay injection based on the number of concurrent requests being handled, as well as static delays. Fixes #389 and also #82. We may consider removing the direct fault filter configs from our docs as well as emit a log warning when detecting usage to deprecate. It would be nice if we could add the fault filter extension in code instead of imposing configuration of that upon the end user, making that an implementation detail. The following shows a per-request config example. This header-provided configuration gets merged with any provided static configuration to determine a final configuration for each request: ``` curl -H "x-nighthawk-test-server-config:{static_delay:\"1s\"}" -vv 127.0.0.1:10000 ``` Full yaml config for the http_filters section: ```yaml http_filters: - name: dynamic-delay config: concurrency_based_delay: minimal_delay: 0.05s concurrency_delay_factor: 0.010s - name: test-server config: response_body_size: 10 response_headers: - { header: { key: "x-nh", value: "1" } } ``` Signed-off-by: Otto van der Schaaf <oschaaf@we-amp.com>
…essfully. This command will create the files needed for CLang intellisense to work. (#403) Signed-off-by: Peter Morgan <8431046+pamorgan@users.noreply.github.com>
Update Envoy + run gcc-built tests in CI because now we can. Fixes #382 Signed-off-by: Otto van der Schaaf <oschaaf@we-amp.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
4 participants
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.
No description provided.