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
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.
Revision history for this message
Alvaro Uria (aluria) wrote :

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"

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

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