Join GitHub today
GitHub is home to over 20 million developers working together to host and review code, manage projects, and build software together.
Fixed lp:1425807 - functional-ha-backup CI job failure due to backup/restore changes #1693
Conversation
|
Self-approving, as this is the best and easiest fix to try unblocking the restore job |
|
Status: merge request accepted. Url: http://juju-ci.vapour.ws:8080/job/github-merge-juju |
added a commit
that referenced
this pull request
Feb 26, 2015
jujubot
merged commit 0872c2c
into
juju:master
Feb 26, 2015
dimitern
deleted the
dimitern:lp-1425807-fix-restore-ci
branch
Feb 26, 2015
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
dimitern commentedFeb 26, 2015
The culprit most likely is the combination of #1667 (redirecting juju
restore to juju backups restore) and mostly #1596 - causing an early
"exit 1" due to missing quotes around a bash if block, thus confusing
the CI job to think restore (expectedly) failed earlier.
(Review request: http://reviews.vapour.ws/r/1021/)