Skip to content
This repository has been archived by the owner on Jan 13, 2022. It is now read-only.

Adding a command line option to specify a remote Siege client. #83

Merged
merged 1 commit into from
Jul 28, 2017

Conversation

kevand900
Copy link
Contributor

@kevand900 kevand900 commented Jun 27, 2017

Please note you must set up privileged SSH connection between the Server and Client.
It is also necessary to modify IP tables to allow INPUT and OUTPUT between Server and Client.
Remote Client is specified by --remote-siege=user@host, and is to be paried with --i-am-not-benchmarking.

Please note you must set up privileged SSH connection between the Server and Client.
It is also necessary to modify IP tables to allow INPUT and OUTPUT between Server and Client.
Remote Client is specified by --remote-siege=<user>@<host>, and is to be paried with --i-am-not-benchmarking.
@kevand900
Copy link
Contributor Author

Can someone please review? Thanks!

@mofarrell
Copy link
Contributor

@kevand900 This seems okay, but I don't know why it would be useful.

@kevand900
Copy link
Contributor Author

Having the Siege client able to run on a separate machine better represents a real-world scenario, as well as freeing up cycles for HHVM to be fully exercised.

@mofarrell
Copy link
Contributor

Have you done any experiments showing that the results aren't extremely noisy? I have had concerns with a lot of the recent changes moving things off the box. Yes is it more realistic in that your requests will be coming from else where, but it also means you might be measuring bottlenecks other than the webserver performance, and degrade the reproducibility of results.

@gdbentley
Copy link
Contributor

@mofarrell We're aware of the potential pitfalls. The intention is to perform measurements on an isolated high-speed subnet. If you'd like, we can schedule a meeting to discuss what we have in mind.

@mofarrell mofarrell merged commit e783511 into facebookarchive:master Jul 28, 2017
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants