Merge lp://staging/~abentley/juju-release-tools/lowercase-azure-regions into lp://staging/juju-release-tools

Proposed by Aaron Bentley
Status: Merged
Merged at revision: 326
Proposed branch: lp://staging/~abentley/juju-release-tools/lowercase-azure-regions
Merge into: lp://staging/juju-release-tools
Diff against target: 180 lines (+41/-40)
2 files modified
azure_image_streams.py (+30/-30)
tests/test_azure_image_streams.py (+11/-10)
To merge this branch: bzr merge lp://staging/~abentley/juju-release-tools/lowercase-azure-regions
Reviewer Review Type Date Requested Status
Curtis Hovey (community) code Approve
Review via email: mp+300946@code.staging.launchpad.net

Commit message

Use location name as region instead of display name.

Description of the change

This branch changes the region for azure-ARM to be the location name.

Old Azure's simplestreams used the display name, a value with capital letters and spaces. Andrew Wilkins has asked us to use the actual location name, which is the same value users might enter at the commandline when bootstrapping.

Given that old-Azure and Azure-ARM seem to be completely different when it comes to simplestreams, this seems like a reasonable request to me.

To post a comment you must log in.
326. By Aaron Bentley

Merge trunk.

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