-
Notifications
You must be signed in to change notification settings - Fork 56
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
added uperf to run from run_snafu --tool #138
Conversation
Can one of the admins verify this patch? |
/rerun all |
Results for SNAFU CI Test
|
uperf_wrapper/trigger_uperf.py
Outdated
|
||
class Trigger_uperf(): | ||
def __init__(self, args): | ||
stdout = self._run_uperf(args.workload[0]) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think all of this will need to move to a new function called emit_actions(), you can take a look at https://github.com/cloud-bulldozer/snafu/blob/master/run_snafu.py#L119 and that's why the comment below about yield documents and the index...
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Posted comments also CI will keep failing until we update how we run the test in ripsaw as the snafu ci uses ripsaw !
/rerun all |
Results for SNAFU CI Test
|
/rerun all |
Results for SNAFU CI Test
|
/rerun all |
Results for SNAFU CI Test
|
/rerun all |
1 similar comment
/rerun all |
Results for SNAFU CI Test
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The documents in elasticsearch are missing user and client_ips field
Looking at the code, I don't see how this can be? |
@jtaleric The user was exported in the environment variable but it wasn't found when fetched by the uperf. We tried to pass the user and client_ips as argument
Here the user was sent correctly but the client_ip wasn't sent.
|
/rerun all |
Results for SNAFU CI Test
|
/rerun all |
Results for SNAFU CI Test
|
/rerun all |
Results for SNAFU CI Test
|
/rerun all |
Results for SNAFU CI Test
|
/rerun all |
Results for SNAFU CI Test
|
/rerun all |
Results for SNAFU CI Test
|
/rerun all |
Results for SNAFU CI Test
|
/rerun all |
Results for SNAFU CI Test
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Everything looks good now with the PR
Depends-On: 283