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 benchmark (code to be able to do it) #451

Closed
ldemailly opened this issue Jul 5, 2017 · 0 comments · Fixed by #519
Closed

grpc benchmark (code to be able to do it) #451

ldemailly opened this issue Jul 5, 2017 · 0 comments · Fixed by #519
Assignees

Comments

@ldemailly
Copy link
Contributor

No description provided.

@ldemailly ldemailly mentioned this issue Jul 5, 2017
19 tasks
@ldemailly ldemailly self-assigned this Aug 2, 2017
@ldemailly ldemailly changed the title grpc benchmark grpc benchmark (code to be able to do it) Aug 2, 2017
ldemailly added a commit that referenced this issue Aug 7, 2017
fortio now supports grpc testing as well
fixes #451

docker images are created for fortio, grpcping, echosrv

* Simple grpc cli/server

Gazelle doesn’t seem to generate build rule for proto

* Added payload flag

* User fortio logger, temp check in of .pb. file

* Renamed to grpcping

* Allow grpc_cli ls to work

* ping and clock skew calculations

* Add histograms to grpcping

* Make http runner accessible/reusable as library

So #517 can be better/easier

* Return actual QPS and indicate default runner options aren't needed for programatic calls

Also auto set the testing function for http by default

* Comment on the provenance of the code

* Adding gprc std health svc check (through -health)

* Grpc runner as library

* Fortio now has a -grpc flag to exercise grpc servers (that have health services)

For #451

* Minor fixes/updates

* Added comment per code review

* Opsa spell check

* Fix Dockerfile

Builds 3 binaries which is kind of big
Todo: consider 1 binary

* Move grpc in separate package to keep echosrv binary small

Move grpc in separate package to keep echosrv binary small and
dependencies smaller when not using grpc (so echosrv which doesn’t use
grpc stays 4mb instead of becoming 6mb)
(Seemingly a side effect of go’s init() that not using still pulls it
in)

Individual images fortio.fortio fortio.echosrv fortio.grpcping as well
as a combo images having all 3

* Adding images push

* Adding missing GIT_SHA

* Typo galore

* Debug docker version

* Got push failure, trying authorize

+ reduce the verbosity of http

* Code review comments

Thx doug!
rshriram pushed a commit that referenced this issue Oct 30, 2017
fortio now supports grpc testing as well
fixes #451

docker images are created for fortio, grpcping, echosrv

* Simple grpc cli/server

Gazelle doesn’t seem to generate build rule for proto

* Added payload flag

* User fortio logger, temp check in of .pb. file

* Renamed to grpcping

* Allow grpc_cli ls to work

* ping and clock skew calculations

* Add histograms to grpcping

* Make http runner accessible/reusable as library

So #517 can be better/easier

* Return actual QPS and indicate default runner options aren't needed for programatic calls

Also auto set the testing function for http by default

* Comment on the provenance of the code

* Adding gprc std health svc check (through -health)

* Grpc runner as library

* Fortio now has a -grpc flag to exercise grpc servers (that have health services)

For #451

* Minor fixes/updates

* Added comment per code review

* Opsa spell check

* Fix Dockerfile

Builds 3 binaries which is kind of big
Todo: consider 1 binary

* Move grpc in separate package to keep echosrv binary small

Move grpc in separate package to keep echosrv binary small and
dependencies smaller when not using grpc (so echosrv which doesn’t use
grpc stays 4mb instead of becoming 6mb)
(Seemingly a side effect of go’s init() that not using still pulls it
in)

Individual images fortio.fortio fortio.echosrv fortio.grpcping as well
as a combo images having all 3

* Adding images push

* Adding missing GIT_SHA

* Typo galore

* Debug docker version

* Got push failure, trying authorize

+ reduce the verbosity of http

* Code review comments

Thx doug!

Former-commit-id: 13ced35
vbatts pushed a commit to vbatts/istio that referenced this issue Oct 31, 2017
fortio now supports grpc testing as well
fixes istio#451

docker images are created for fortio, grpcping, echosrv

* Simple grpc cli/server

Gazelle doesn’t seem to generate build rule for proto

* Added payload flag

* User fortio logger, temp check in of .pb. file

* Renamed to grpcping

* Allow grpc_cli ls to work

* ping and clock skew calculations

* Add histograms to grpcping

* Make http runner accessible/reusable as library

So istio#517 can be better/easier

* Return actual QPS and indicate default runner options aren't needed for programatic calls

Also auto set the testing function for http by default

* Comment on the provenance of the code

* Adding gprc std health svc check (through -health)

* Grpc runner as library

* Fortio now has a -grpc flag to exercise grpc servers (that have health services)

For istio#451

* Minor fixes/updates

* Added comment per code review

* Opsa spell check

* Fix Dockerfile

Builds 3 binaries which is kind of big
Todo: consider 1 binary

* Move grpc in separate package to keep echosrv binary small

Move grpc in separate package to keep echosrv binary small and
dependencies smaller when not using grpc (so echosrv which doesn’t use
grpc stays 4mb instead of becoming 6mb)
(Seemingly a side effect of go’s init() that not using still pulls it
in)

Individual images fortio.fortio fortio.echosrv fortio.grpcping as well
as a combo images having all 3

* Adding images push

* Adding missing GIT_SHA

* Typo galore

* Debug docker version

* Got push failure, trying authorize

+ reduce the verbosity of http

* Code review comments

Thx doug!

Former-commit-id: 13ced35
mandarjog pushed a commit that referenced this issue Oct 31, 2017
Basic SSL/TLS for ingress wildcard host based on k8s ingress resource. Also includes code for more complex scenarios with multiple TLS and non-TLS hosts that will be exposed in a future commit.
mandarjog pushed a commit that referenced this issue Nov 2, 2017
fortio now supports grpc testing as well
fixes #451

docker images are created for fortio, grpcping, echosrv

* Simple grpc cli/server

Gazelle doesn’t seem to generate build rule for proto

* Added payload flag

* User fortio logger, temp check in of .pb. file

* Renamed to grpcping

* Allow grpc_cli ls to work

* ping and clock skew calculations

* Add histograms to grpcping

* Make http runner accessible/reusable as library

So #517 can be better/easier

* Return actual QPS and indicate default runner options aren't needed for programatic calls

Also auto set the testing function for http by default

* Comment on the provenance of the code

* Adding gprc std health svc check (through -health)

* Grpc runner as library

* Fortio now has a -grpc flag to exercise grpc servers (that have health services)

For #451

* Minor fixes/updates

* Added comment per code review

* Opsa spell check

* Fix Dockerfile

Builds 3 binaries which is kind of big
Todo: consider 1 binary

* Move grpc in separate package to keep echosrv binary small

Move grpc in separate package to keep echosrv binary small and
dependencies smaller when not using grpc (so echosrv which doesn’t use
grpc stays 4mb instead of becoming 6mb)
(Seemingly a side effect of go’s init() that not using still pulls it
in)

Individual images fortio.fortio fortio.echosrv fortio.grpcping as well
as a combo images having all 3

* Adding images push

* Adding missing GIT_SHA

* Typo galore

* Debug docker version

* Got push failure, trying authorize

+ reduce the verbosity of http

* Code review comments

Thx doug!

Former-commit-id: 13ced35
guptasu pushed a commit to guptasu/istio that referenced this issue Jun 11, 2018
* Update value to include data

* update generated code

* Fix doc

* update generated code
kyessenov pushed a commit to kyessenov/istio that referenced this issue Aug 13, 2018
antonioberben pushed a commit to antonioberben/istio that referenced this issue Jan 29, 2024
Signed-off-by: Pavel Nikolov <pavelnikolov@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant