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

update pyloxi to floodlight/loxigen-artifacts @ bc7d73888551ad432b818420400aeb0b61c8d5d7 #190

Merged
merged 2 commits into from
Aug 20, 2015

Conversation

rlane
Copy link
Contributor

@rlane rlane commented Aug 20, 2015

Reviewer: trivial

@bsn-abat
Copy link

@bsn-abat
Copy link

ABAT: AUTO-REJECT: Your change failed pre-merge validation defined in oftest/.abat-automerge. See log for details.

INSTRUCTIONS: Please fix your problem by

  1. Reviewing the logs below and git commiting a fix.

  2. Use git push to add new commits to this same remote branch/pull request.

  3. Get another code review approval from your co-worker indicated by a line of "ACK" in comment, or add another comment yourself with a line of "R_E_T_R_Y (Remove the underscores)". That will cause abat to try to merge your code again.

Log file at http://vbs26.eng.bigswitch.com/abat/2015.08.20.0146-m.oftest.master/abat.log
/cc

These are obsolete and are causing pylint errors.
@rlane
Copy link
Contributor Author

rlane commented Aug 20, 2015

RETRY

@bsn-abat
Copy link

@bsn-abat
Copy link

ABAT: AUTO-REJECT: Your change failed pre-merge validation defined in oftest/.abat-automerge. See log for details.

INSTRUCTIONS: Please fix your problem by

  1. Reviewing the logs below and git commiting a fix.

  2. Use git push to add new commits to this same remote branch/pull request.

  3. Get another code review approval from your co-worker indicated by a line of "ACK" in comment, or add another comment yourself with a line of "R_E_T_R_Y (Remove the underscores)". That will cause abat to try to merge your code again.

Log file at http://vbs26.eng.bigswitch.com/abat/2015.08.20.0155-m.oftest.master/abat.log
/cc

@rlane
Copy link
Contributor Author

rlane commented Aug 20, 2015

RETRY

@bsn-abat
Copy link

@bsn-abat
Copy link

ABAT: ACCEPT: Successfully merged

In case you want to see the build log, check out :
Log file at http://vbs26.eng.bigswitch.com/abat/2015.08.20.0200-m.oftest.master/abat.log
/cc

@bsn-abat bsn-abat merged commit 877b9ed into floodlight:master Aug 20, 2015
bsn-abat pushed a commit that referenced this pull request Aug 20, 2015
update pyloxi to floodlight/loxigen-artifacts @ bc7d73888551ad432b818420400aeb0b61c8d5d7 (#190)
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.

2 participants