Merge lp://staging/~ec0/charms/trusty/glance-sync-slave/layer-metadata-fixes into lp://staging/~canonical-bootstack/charms/trusty/glance-sync-slave/layer
Proposed by
James Hebden
Status: | Needs review |
---|---|
Proposed branch: | lp://staging/~ec0/charms/trusty/glance-sync-slave/layer-metadata-fixes |
Merge into: | lp://staging/~canonical-bootstack/charms/trusty/glance-sync-slave/layer |
Diff against target: |
942 lines (+587/-241) 6 files modified
config.yaml (+16/-0) files/glance_sync_slave.py (+542/-239) reactive/glance-sync-slave.py (+21/-0) templates/glance_sync_slave_cron.j2 (+1/-1) templates/glancesync.novarc.j2 (+5/-0) templates/novarc.j2 (+2/-1) |
To merge this branch: | bzr merge lp://staging/~ec0/charms/trusty/glance-sync-slave/layer-metadata-fixes |
Related bugs: |
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
Jill Rouleau | Pending | ||
Review via email: mp+328495@code.staging.launchpad.net |
Description of the change
Add changes to exclude glance and additional custom metadata from sync, as well as readonly attributes.
To post a comment you must log in.
Unmerged revisions
- 52. By James Hebden
-
[jhebden,r=jillr] Added sajoupa's changes to ignore syncing of glance and extra properties in metadata
- 51. By Alvaro Uria
-
added extra params to config.yaml and reactive logic (config-changed not calling @when)
- 50. By Alvaro Uria
-
fixes on project mapping and errors capture. First real tests ok
- 49. By Alvaro Uria
-
minor fixes: hardcoded glance master endpoint, temp_image format
- 48. By Alvaro Uria
-
glance-sync-slave redesign
1) My layer/branch (lp:~aluria/canonical-bootstack/bootstack-tele2-glance-sync-slave-layer) had issues on "juju set" a few params (see cRT#100292). This is the reason it was not pushed via ~canonical- bootstack user (but ~aluria).
2) glance-sync-slave python script on such layer (in ~aluria) has some modifications thought for tele2 (ie tenant mapping by replacing REGION_MASTER by REGION_SLAVE). I think we should keep a fork of the main "glance- sync-slave/ layer"