Merge lp://staging/~abentley/juju-release-tools/make-agent-json-tests into lp://staging/juju-release-tools

Proposed by Aaron Bentley
Status: Merged
Merged at revision: 261
Proposed branch: lp://staging/~abentley/juju-release-tools/make-agent-json-tests
Merge into: lp://staging/juju-release-tools
Diff against target: 286 lines (+205/-28)
2 files modified
make_agent_json.py (+54/-28)
tests/test_make_agent_json.py (+151/-0)
To merge this branch: bzr merge lp://staging/~abentley/juju-release-tools/make-agent-json-tests
Reviewer Review Type Date Requested Status
Curtis Hovey (community) code Approve
Review via email: mp+285536@code.staging.launchpad.net

Commit message

Support generating agent json for devel/proposed/released streams.

Description of the change

This branch tweaks make_agent_json to support arbitrary streams.

This is to support a follow-on, which extends generate_agents.py to generate agents and their json fragments.

In contrast to revision-build-streams, this new support assumes that the paths for various streams can coexist-- e.g. proposed, released and devel will all use the same binary for a given stable version. Concretely, the path a revision build is of the form agent/revision-build-3565/tarfile.tar.gz, but the path for an arbitrary stream is agent/1.27/tarfile.tar.gz, which mentions the version but not the stream name.

This branch also adds tests.

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

Thank you.

review: Approve (code)

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