Merge lp://staging/~sinzui/juju-release-tools/ignroe-epoch into lp://staging/juju-release-tools

Proposed by Curtis Hovey
Status: Merged
Merged at revision: 344
Proposed branch: lp://staging/~sinzui/juju-release-tools/ignroe-epoch
Merge into: lp://staging/juju-release-tools
Diff against target: 13 lines (+2/-1)
1 file modified
generate_agents.py (+2/-1)
To merge this branch: bzr merge lp://staging/~sinzui/juju-release-tools/ignroe-epoch
Reviewer Review Type Date Requested Status
Nicholas Skaggs (community) Approve
Review via email: mp+308465@code.staging.launchpad.net

Description of the change

Strip the epoch from the base_version when making aagents

We saw this error after debs were download and they were extracted to convert the jujud into an agent

OSError: [Errno 2] No such file or directory: '/tmp/tmpNlQsjT/contents/usr/lib/juju-1:2.0.0/bin/jujud'

^ lib/juju-1:2.0.0 should be lib/juju-2.0

The epoch was not stripped, corrupting both the base_version and the major_minor.

To post a comment you must log in.
Revision history for this message
Curtis Hovey (sinzui) wrote :

This works with
    control_version = "1:2.0.0-0ubuntu1~16.10.1~juju1"
and
    control_version = "2.0.0-0ubuntu1~16.10.1~juju1"

Revision history for this message
Nicholas Skaggs (nskaggs) wrote :

Thank you!

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