Merge lp://staging/~sinzui/juju-ci-tools/fix-var into lp://staging/juju-ci-tools

Proposed by Curtis Hovey
Status: Merged
Merged at revision: 960
Proposed branch: lp://staging/~sinzui/juju-ci-tools/fix-var
Merge into: lp://staging/juju-ci-tools
Diff against target: 14 lines (+2/-2)
1 file modified
run-deploy-job.bash (+2/-2)
To merge this branch: bzr merge lp://staging/~sinzui/juju-ci-tools/fix-var
Reviewer Review Type Date Requested Status
Aaron Bentley (community) Approve
Review via email: mp+259639@code.staging.launchpad.net

Description of the change

Use the lowercase $revision_build name for the var instead of uppercase.

The run-deploy-job script is failing
    /var/lib/jenkins/juju-ci-tools/run-deploy-job.bash: line 26: REVISION_BUILD: unbound variable

I used the uppercase name to be consistent with the revision-result job where we learned that Jenkins is not distinguishing between upper and lower cases, but in this situation lower is correct.

To post a comment you must log in.
Revision history for this message
Aaron Bentley (abentley) :
review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
The diff is not available at this time. You can reload the page or download it.

Subscribers

People subscribed via source and target branches