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

Add missing es indices to tests for ycsb/pgbench #149

Merged
merged 1 commit into from
Mar 2, 2020

Conversation

aakarshg
Copy link
Contributor

@aakarshg aakarshg commented Mar 2, 2020

This will increase code coverage for ycsb and pgbench.

@aakarshg aakarshg added the ok to test Kick off our CI framework label Mar 2, 2020
@aakarshg aakarshg requested a review from dry923 March 2, 2020 19:41
@dry923
Copy link
Member

dry923 commented Mar 2, 2020

/rerun all

@rht-perf-ci
Copy link

Results for SNAFU CI Test

Test Result Runtime
pgbench-wrappe FAIL 00:08:10

@dry923
Copy link
Member

dry923 commented Mar 2, 2020

@aakarshg looks like ripsaw-pgbench-results doesn't exist in marquez

@aakarshg
Copy link
Contributor Author

aakarshg commented Mar 2, 2020

@dry923 it's also odd that only pgbench test ran despite the change also impacting ycsb?

@rht-perf-ci
Copy link

Results for SNAFU CI Test

Test Result Runtime
pgbench-wrappe PASS 00:08:03

Copy link
Member

@dry923 dry923 left a comment

Choose a reason for hiding this comment

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

LGTM!

@dry923 dry923 merged commit 26a2630 into cloud-bulldozer:master Mar 2, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ok to test Kick off our CI framework
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants