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

rules review endpoint for other user #11172

Merged
merged 5 commits into from
Oct 4, 2016

Conversation

mfojtik
Copy link
Contributor

@mfojtik mfojtik commented Sep 30, 2016

Original PR: #10611

@mfojtik
Copy link
Contributor Author

mfojtik commented Sep 30, 2016

@deads2k see the last three commits and feel free to cherry-pick to your branch or merge this ;-)

@mfojtik
Copy link
Contributor Author

mfojtik commented Sep 30, 2016

[test]

@deads2k
Copy link
Contributor

deads2k commented Sep 30, 2016

lgtm. free to merge on green

@mfojtik
Copy link
Contributor Author

mfojtik commented Sep 30, 2016

@deads2k can you check on your laptop the protobuf files? for some reason travis reports they are wrong

@mfojtik mfojtik force-pushed the deads-rules-review branch 2 times, most recently from 1bf146c to d296e70 Compare September 30, 2016 12:36
@mfojtik
Copy link
Contributor Author

mfojtik commented Sep 30, 2016

[test]

@openshift-bot openshift-bot added the needs-rebase Indicates a PR cannot be merged because it has merge conflicts with HEAD. label Sep 30, 2016
@mfojtik mfojtik force-pushed the deads-rules-review branch from d296e70 to 530d3a1 Compare October 4, 2016 08:11
@openshift-bot openshift-bot removed the needs-rebase Indicates a PR cannot be merged because it has merge conflicts with HEAD. label Oct 4, 2016
@mfojtik
Copy link
Contributor Author

mfojtik commented Oct 4, 2016

[test]

@openshift-bot
Copy link
Contributor

Evaluated for origin test up to 530d3a1

@mfojtik
Copy link
Contributor Author

mfojtik commented Oct 4, 2016

@deads2k I think this is now good to merge (the flake was device mapper in conformance, the other tests passed).

@deads2k
Copy link
Contributor

deads2k commented Oct 4, 2016

[merge]

@openshift-bot
Copy link
Contributor

Evaluated for origin merge up to 530d3a1

@openshift-bot
Copy link
Contributor

continuous-integration/openshift-jenkins/test FAILURE (https://ci.openshift.redhat.com/jenkins/job/test_pr_origin/9609/)

@openshift-bot openshift-bot merged commit 785f440 into openshift:master Oct 4, 2016
@openshift-bot
Copy link
Contributor

continuous-integration/openshift-jenkins/merge SUCCESS (https://ci.openshift.redhat.com/jenkins/job/test_pr_origin/9626/)

@mfojtik mfojtik deleted the deads-rules-review branch September 5, 2018 21:07
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