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.

Sign up for GitHub

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

Jump to bottom

Fixing ambiguous master error #118

Merged
merged 1 commit into from
Jan 16, 2020
Merged

Fixing ambiguous master error #118

merged 1 commit into from
Jan 16, 2020

Conversation

dry923
Copy link
Member

@dry923 dry923 commented Jan 14, 2020

CI is getting the following error when doing a git diff. As a result it is always running against all the tests. This is to resolve it.

...
++ git diff master --name-only
fatal: ambiguous argument 'master': unknown revision or path not in the working tree.
...

@dry923 dry923 added the ok to test Kick off our CI framework label Jan 14, 2020
@dry923
Copy link
Member Author

dry923 commented Jan 14, 2020

This now works appropriately. From CI:

...
++ git diff origin/master --name-only
+ diff_list=ci/run_ci.sh
...

@rht-perf-ci
Copy link

Results for SNAFU CI Test

Test Result Runtime
fio_wrapper PASS 00:07:21
fs_drift_wrapper PASS 00:07:28
hammerdb FAIL 00:14:11
iperf PASS 00:05:55
pgbench-wrapper PASS 00:07:27
smallfile_wrapper PASS 00:06:37
sysbench PASS 00:05:46
uperf-wrapper PASS 00:07:06
ycsb-wrapper PASS 00:05:56

aakarshg
aakarshg approved these changes Jan 16, 2020
View reviewed changes
Copy link
Contributor

@aakarshg aakarshg left a comment

Choose a reason for hiding this comment

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

LGTM

@aakarshg aakarshg merged commit 4c4deec into cloud-bulldozer:master Jan 16, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@aakarshg aakarshg aakarshg approved these changes

@jtaleric jtaleric Awaiting requested review from jtaleric

Assignees
No one assigned
Labels
ok to test Kick off our CI framework
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@dry923 @rht-perf-ci @aakarshg