Merge lp://staging/~sinzui/juju-release-tools/agent-archive into lp://staging/juju-release-tools

Proposed by Curtis Hovey
Status: Merged
Merged at revision: 186
Proposed branch: lp://staging/~sinzui/juju-release-tools/agent-archive
Merge into: lp://staging/juju-release-tools
Diff against target: 332 lines (+51/-48)
4 files modified
agent_archive.py (+12/-10)
assemble-streams.bash (+2/-2)
get-revision-build-artefacts.bash (+2/-1)
tests/test_agent_archive.py (+35/-35)
To merge this branch: bzr merge lp://staging/~sinzui/juju-release-tools/agent-archive
Reviewer Review Type Date Requested Status
Aaron Bentley (community) Approve
Review via email: mp+260321@code.staging.launchpad.net

Description of the change

Rename win-agents to agent-archive, and win_agent_archive.py to agent_archive.py

This branch makes two mechanical changes and also adds the build-centos job to the list of jobs to get release binaries from.

Rename win_agent_archive.py to agent_archive.py and update everything that calls or imports it.

Rename juju-qa-data/win-agents to juju-qa-data/agent-archive. The agents are already at juju-qa-data/agent-archive. These commands used during releases work:

     ./juju-release-tools/agent_archive.py --config $JUJU_HOME/juju-qa.s3cfg add juju-1.24-beta5-centos7-amd64.tgz
     ./juju-release-tools/agent_archive.py --config $JUJU_HOME/juju-qa.s3cfg get 1.24-beta5 ./

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

Looks reasonable to me. If it runs, land it.

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