Skip to content

Add a missing include of api/client/output.pb.h#655

Merged
dubious90 merged 69 commits intoenvoyproxy:mainfrom
eric846:include1
Mar 18, 2021
Merged

Add a missing include of api/client/output.pb.h#655
dubious90 merged 69 commits intoenvoyproxy:mainfrom
eric846:include1

Conversation

@eric846
Copy link
Copy Markdown
Contributor

@eric846 eric846 commented Mar 18, 2021

No description provided.

eric846 added 30 commits June 1, 2020 17:23
@eric846
Merge pull request #5 from envoyproxy/master
8ea442d 
update from master
@eric846
Merge pull request #6 from envoyproxy/master
5ac755a 
merge from upstream
@eric846
Merge pull request #7 from envoyproxy/master
b8c25a5 
merge from upstream
@eric846
initial commit
1c19c68 
Signed-off-by: eric846 <56563761+eric846@users.noreply.github.com>
@eric846
fix comments
7050686 
Signed-off-by: eric846 <56563761+eric846@users.noreply.github.com>
@eric846
fix format
0776563 
Signed-off-by: eric846 <56563761+eric846@users.noreply.github.com>
@eric846
rename adaptive_rps to adaptive_load
16fd8f6 
Signed-off-by: eric846 <56563761+eric846@users.noreply.github.com>
@eric846
add field_selector in example
c383010 
Signed-off-by: eric846 <56563761+eric846@users.noreply.github.com>
@eric846
fix example comment
6e1a483 
Signed-off-by: eric846 <56563761+eric846@users.noreply.github.com>
@eric846
fix format
4ef1140 
Signed-off-by: eric846 <56563761+eric846@users.noreply.github.com>
@eric846
add support for fault injection headers
4111bf4 
Signed-off-by: eric846 <56563761+eric846@users.noreply.github.com>
@eric846
replace linear and binary search with exponential search
871a959 
Signed-off-by: eric846 <56563761+eric846@users.noreply.github.com>
@eric846
add InputVariableSetter mechanism
1fd77c1 
Signed-off-by: eric846 <56563761+eric846@users.noreply.github.com>
@eric846
add input variable setter to build file
edc36b2 
Signed-off-by: eric846 <56563761+eric846@users.noreply.github.com>
@eric846
fix syntax errors
4d0364e 
Signed-off-by: eric846 <56563761+eric846@users.noreply.github.com>
@eric846
rename samples/adaptive_rps
aed6d94 
Signed-off-by: eric846 <56563761+eric846@users.noreply.github.com>
@eric846
improve comments, change step controller initial value from int64 to …
d9ae87d 
…double

Signed-off-by: eric846 <56563761+eric846@users.noreply.github.com>
@eric846
add proto validation rules, fix comments, make rps the default input_…
a05a6f5 
…variable_setter

Signed-off-by: eric846 <56563761+eric846@users.noreply.github.com>
@eric846
fix comment wording
8cd4d21 
Signed-off-by: eric846 <56563761+eric846@users.noreply.github.com>
@eric846
simplify protos, add defaults, specify required or optional
d814a96 
Signed-off-by: eric846 <56563761+eric846@users.noreply.github.com>
@eric846
add missing newline
5f5a885 
Signed-off-by: eric846 <56563761+eric846@users.noreply.github.com>
@eric846
Kick CI
7e20a78 
Signed-off-by: eric846 <56563761+eric846@users.noreply.github.com>
@eric846
simplify protos
9048267 
Signed-off-by: eric846 <56563761+eric846@users.noreply.github.com>
@eric846
fix format
306c0ec 
Signed-off-by: eric846 <56563761+eric846@users.noreply.github.com>
@eric846
fix some optional field comments and rules
d33f543 
Signed-off-by: eric846 <56563761+eric846@users.noreply.github.com>
@eric846
Merge pull request #10 from envoyproxy/master
442cca9 
merge from upstream
@eric846
add Nighthawk status field in BenchmarkResult as nested nighthawk.cli…
677b783 
…ent.Output turns out not to include the status

Signed-off-by: eric846 <56563761+eric846@users.noreply.github.com>
@eric846
switch to standard Envoy plugin config proto, add prefix to internal …
cefb366 
…plugin names, log thresholds only once per session

Signed-off-by: eric846 <56563761+eric846@users.noreply.github.com>
@eric846
Merge remote-tracking branch 'upstream/master' into adaptive-rps-protos2
f3684df 
Signed-off-by: eric846 <56563761+eric846@users.noreply.github.com>
@eric846
create headers
5463051 
Signed-off-by: eric846 <56563761+eric846@users.noreply.github.com>
eric846 added 26 commits September 3, 2020 16:29
@eric846
Merge remote-tracking branch 'upstream/master' into master2
ed32856 
Signed-off-by: eric846 <56563761+eric846@users.noreply.github.com>
@eric846
Merge remote-tracking branch 'upstream/master' into master2
eecf00d 
Signed-off-by: eric846 <56563761+eric846@users.noreply.github.com>
@eric846
Merge remote-tracking branch 'upstream/master' into master2
13179fb 
Signed-off-by: eric846 <56563761+eric846@users.noreply.github.com>
@eric846
Merge remote-tracking branch 'upstream/master' into master2
3eb89e6 
Signed-off-by: eric846 <56563761+eric846@users.noreply.github.com>
@eric846
Merge remote-tracking branch 'upstream/master' into master2
fae60ad 
Signed-off-by: eric846 <56563761+eric846@users.noreply.github.com>
@eric846
Merge remote-tracking branch 'upstream/master' into master2
9fec200 
Signed-off-by: eric846 <56563761+eric846@users.noreply.github.com>
@eric846
Merge remote-tracking branch 'upstream/master' into master2
9d1cf53 
Signed-off-by: eric846 <56563761+eric846@users.noreply.github.com>
@eric846
Merge remote-tracking branch 'upstream/master' into master2
cfad844 
Signed-off-by: eric846 <56563761+eric846@users.noreply.github.com>
@eric846
Merge remote-tracking branch 'upstream/master' into master2
bdde910 
Signed-off-by: eric846 <56563761+eric846@users.noreply.github.com>
@eric846
Merge remote-tracking branch 'upstream/master' into master2
8100376 
Signed-off-by: eric846 <56563761+eric846@users.noreply.github.com>
@eric846
Merge remote-tracking branch 'upstream/master' into master2
694fda6 
Signed-off-by: eric846 <56563761+eric846@users.noreply.github.com>
@eric846
Merge remote-tracking branch 'upstream/main'
1df6137 
Signed-off-by: eric846 <56563761+eric846@users.noreply.github.com>
@eric846
Merge remote-tracking branch 'upstream/main'
2b9e1e8 
Signed-off-by: eric846 <56563761+eric846@users.noreply.github.com>
@eric846
Merge remote-tracking branch 'upstream/main'
a6da150 
Signed-off-by: eric846 <56563761+eric846@users.noreply.github.com>
@eric846
Merge remote-tracking branch 'upstream/main'
4b8ba84 
Signed-off-by: eric846 <56563761+eric846@users.noreply.github.com>
@eric846
Merge remote-tracking branch 'upstream/main'
eaa20de 
Signed-off-by: eric846 <56563761+eric846@users.noreply.github.com>
@eric846
Merge remote-tracking branch 'upstream/main'
aedd520 
Signed-off-by: eric846 <56563761+eric846@users.noreply.github.com>
@eric846
Merge remote-tracking branch 'upstream/main'
4ec202a 
Signed-off-by: eric846 <56563761+eric846@users.noreply.github.com>
@eric846
Merge remote-tracking branch 'upstream/main'
f42c374 
Signed-off-by: eric846 <56563761+eric846@users.noreply.github.com>
@eric846
Merge remote-tracking branch 'upstream/main'
be5c0fa 
Signed-off-by: eric846 <56563761+eric846@users.noreply.github.com>
@eric846
Merge remote-tracking branch 'upstream/main'
7da182a 
Signed-off-by: eric846 <56563761+eric846@users.noreply.github.com>
@eric846
Merge remote-tracking branch 'upstream/main'
d3e5545 
Signed-off-by: eric846 <56563761+eric846@users.noreply.github.com>
@eric846
Merge remote-tracking branch 'upstream/main'
2f44065 
Signed-off-by: eric846 <56563761+eric846@users.noreply.github.com>
@eric846
Merge remote-tracking branch 'upstream/main'
190a691 
Signed-off-by: eric846 <56563761+eric846@users.noreply.github.com>
@eric846
Merge remote-tracking branch 'upstream/main'
74de764 
Signed-off-by: eric846 <56563761+eric846@users.noreply.github.com>
@eric846
add missing #include
c99585a 
Signed-off-by: eric846 <56563761+eric846@users.noreply.github.com>
@eric846 eric846 added the waiting-for-review A PR waiting for a review. label Mar 18, 2021
oschaaf
oschaaf approved these changes Mar 18, 2021
View reviewed changes
Copy link
Copy Markdown
Member

@oschaaf oschaaf left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks like I cleaned up too much, LGTM.
How did you catch this? It would be good to know if this is something that can be caught in CI.

@dubious90 dubious90 merged commit 490e0a9 into envoyproxy:main Mar 18, 2021
@dubious90 dubious90 self-assigned this Mar 18, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@oschaaf oschaaf oschaaf approved these changes

Assignees

@dubious90 dubious90

Labels

waiting-for-review A PR waiting for a review.

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@eric846 @oschaaf @dubious90