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

[SWL-3678] Add a new line to trigger to update loxigen-artifacts #545

Closed
wants to merge 1 commit into from

Conversation

nick-bsn
Copy link
Contributor

@nick-bsn nick-bsn commented Apr 10, 2017

Reviewer: @andi-bigswitch
CC @kenchiang

  • Added a new blank line to trigger to update the loxigen-artifacts

This is to trigger a loxigen-artifacts update after #544, which was greenbuttoned. Loxigen-artifacts is updated at the end of the abat automerge process, so we didn't see an update for #544.

@andi-bigswitch
Copy link
Contributor

JENKINS-RETRY

@andi-bigswitch
Copy link
Contributor

ping

@bsn-abat
Copy link

Refer to this link for build results (access rights to CI server needed):
https://jenkins.bigswitch.com/job/loxigen_pull_req_artifacts/92/

@bsn-abat
Copy link

Refer to this link for build results (access rights to CI server needed):
https://jenkins.bigswitch.com/job/loxigen_pull_req_docker/112/

@andi-bigswitch
Copy link
Contributor

JENKINS-RETRY

@andi-bigswitch
Copy link
Contributor

ARTIFACTS-RETRY

1 similar comment
@andi-bigswitch
Copy link
Contributor

ARTIFACTS-RETRY

@bsn-abat
Copy link

Refer to this link for build results (access rights to CI server needed):
https://jenkins.bigswitch.com/job/loxigen_pull_req_artifacts/93/

@andi-bigswitch
Copy link
Contributor

ARTIFACTS-RETRY

@bsn-abat
Copy link

Refer to this link for build results (access rights to CI server needed):
https://jenkins.bigswitch.com/job/loxigen_pull_req_artifacts/94/

@andi-bigswitch
Copy link
Contributor

@nick-bsn - actually, we probably don't need this PR any more.

That is because I had to file a PR to fix the artifacts build ( #546 ) -- as a side effect of this PR being merged, loxigen-artifacts already has been updated:

floodlight/loxigen-artifacts@c00b526

should have the latest code...

@nick-bsn
Copy link
Contributor Author

@andi-bigswitch Thanks a lot for your help!

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.

None yet

3 participants