Merge lp://staging/~sinzui/juju-release-tools/one-cloud-publish into lp://staging/juju-release-tools

Proposed by Curtis Hovey
Status: Merged
Merged at revision: 211
Proposed branch: lp://staging/~sinzui/juju-release-tools/one-cloud-publish
Merge into: lp://staging/juju-release-tools
Diff against target: 263 lines (+254/-0)
2 files modified
publish_streams.py (+127/-0)
tests/test_publish_streams.py (+127/-0)
To merge this branch: bzr merge lp://staging/~sinzui/juju-release-tools/one-cloud-publish
Reviewer Review Type Date Requested Status
Martin Packman (community) Approve
Review via email: mp+268235@code.staging.launchpad.net

Description of the change

Verify local metadata to cloud metadata.

This branch introduces a rewrite of the publish-public-tools.bash script. The publish_streams.py script can verify a local stream with a remote stream just like the older script. eg.

    ../juju-release-tools/publish_streams.py -d -v -r testing released ./juju-dist azure

My next branch will do syncing.

To post a comment you must log in.
Revision history for this message
Martin Packman (gz) wrote :

Looks good, some comments inline.

review: Approve
217. By Curtis Hovey

simplify the CPCS dict.

218. By Curtis Hovey

Updated docstrings.

Revision history for this message
Curtis Hovey (sinzui) wrote :

Thank you for the review. I updated the code and commented inline regarding why "stream" is not used in the verification case.

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