Merge lp://staging/~danilo/charms/trusty/glance-simplestreams-sync/normalize-swift-endpoint into lp://staging/~landscape/charms/trusty/glance-simplestreams-sync/landscape
Proposed by
Данило Шеган
| Status: | Rejected |
|---|---|
| Rejected by: | Данило Шеган |
| Proposed branch: | lp://staging/~danilo/charms/trusty/glance-simplestreams-sync/normalize-swift-endpoint |
| Merge into: | lp://staging/~landscape/charms/trusty/glance-simplestreams-sync/landscape |
| Diff against target: |
23 lines (+1/-12) 1 file modified
scripts/glance-simplestreams-sync.py (+1/-12) |
| To merge this branch: | bzr merge lp://staging/~danilo/charms/trusty/glance-simplestreams-sync/normalize-swift-endpoint |
| Related bugs: |
| Reviewer | Review Type | Date Requested | Status |
|---|---|---|---|
| Данило Шеган (community) | Disapprove | ||
| Adam Collard (community) | Approve | ||
| Geoff Teale | Pending | ||
|
Review via email:
|
|||
Description of the change
Swift endpoints being used for uploading images and setting the product streams endpoint are different, thus juju bootstrap cannot find images.
This makes them both use the same URL which has been tested to work.
Testing instructions:
1. Modify OPL to point to cs:~danilo/
https:/
2. Enable openstack-
3. Deploy a cloud
4. Fetch Juju env details and paste them into your ~/.juju/
5. juju bootstrap --debug -e YOUR-OPL-JUJU-ENV
To post a comment you must log in.
Unmerged revisions
- 64. By Данило Шеган
-
Clean-up all the useless method calls.
- 63. By Данило Шеган
-
Drop AUTH_<tenant-id> bucket from the swift path.
- 62. By Данило Шеган
-
Pass ksc in do_sync.
- 61. By Данило Шеган
-
Use the AUTH_* path when pushing images to swift store as well.
Preview Diff
[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
Looks good, +1, nice cleanup