Skip to content

Conversation

@meteorfox
Copy link
Owner

This adds the oss-benchmark.

Please review but don't merge, as I want to merge to upstream.

@meteorfox meteorfox requested a review from adhanotia May 10, 2018 05:27
@meteorfox
Copy link
Owner Author

meteorfox commented May 10, 2018

Since it builds on top of the nginx branch, it shows the nginx changes. You can ignore those as you already reviewed them.

New changes are from commit 70a034f forward.

This link shows only that subset:
https://github.com/meteorfox/fbkutils/pull/6/files/a12c277feef0e5f661fbdcf4e30de4ffdd126eba..75b0a5afdeedb579860fe72d2384f04ee919534d

@meteorfox meteorfox force-pushed the benchpress/oss-benchmark branch from 75b0a5a to e418ca8 Compare May 10, 2018 23:12
#!/bin/bash
set -e
set -x

Copy link
Collaborator

Choose a reason for hiding this comment

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

Add prerequisites here. Libraries we need to install on our test hosts before we can run this script.

# Installs HHVM's oss-performance under benchmarks/
#

HHVM_PATH=${HHVM_PATH:-'hhvm'}
Copy link
Collaborator

Choose a reason for hiding this comment

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

Does this assume hhvm is installed or we are installing it using benchpress before calling the oss-performance installer?

set -e
set -x

SIEGE_GIT_REPO='https://github.com/JoeDog/siege.git'
Copy link
Collaborator

Choose a reason for hiding this comment

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

Lets add prerequisite library section here.

@meteorfox
Copy link
Owner Author

Updates:

  • Allow it to run as root.
  • Fix bug where it was assuming hhvm was installed in the PATH
  • Allow run.sh in oss-performance to pass-thru any arguments to the perf.php

@meteorfox meteorfox force-pushed the benchpress/oss-benchmark branch from 3e48c29 to 223aed6 Compare June 19, 2018 19:39
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 this pull request may close these issues.

3 participants