- Committer: Tarmac
- Date: 2014-04-22 14:39:56 UTC
- mfrom: (503.1.1 exodus-026-fix)
- Revision ID: tarmac-20140422143956-z1e9c5ks7vp0cv11
Fix 026 exodus.
A basket, the thing ingested from Launchpad, contains one or more bundles. If
there are multiple bundles they may establish an inheritance relationship
between the bundles in that basket. The inheritance is done before the basket
is split into individual bundles and stored -- flattened in other words.
The 'inherits' line is not removed, however.
When the exodus happens, the re-use of 'store_bundles' attempts to flatten
again, but the relationship to the other bundle is invalid as the 'basket' now
just contains the one pre-processed bundle.
The fix for this exodus is to simply manually delete the 'inherits' key from
the bundle_data before proceeding.
The exodus has been tested against a full ingest of all charms and bundles.
The unit test has been modified to include an 'inherits' key in the data,
which is then ignored.
https://codereview.appspot.com/90300043/
R=jcsackett.
Approved by Juju Gui Bot.
A basket, the thing ingested from Launchpad, contains one or more bundles. If
there are multiple bundles they may establish an inheritance relationship
between the bundles in that basket. The inheritance is done before the basket
is split into individual bundles and stored -- flattened in other words.
The 'inherits' line is not removed, however.
When the exodus happens, the re-use of 'store_bundles' attempts to flatten
again, but the relationship to the other bundle is invalid as the 'basket' now
just contains the one pre-processed bundle.
The fix for this exodus is to simply manually delete the 'inherits' key from
the bundle_data before proceeding.
The exodus has been tested against a full ingest of all charms and bundles.
The unit test has been modified to include an 'inherits' key in the data,
which is then ignored.
https://codereview.appspot.com/90300043/
R=jcsackett.
Approved by Juju Gui Bot.
| Filename | Latest Rev | Last Changed | Committer | Comment | Size | ||
|---|---|---|---|---|---|---|---|
 .. |
|||||||
charmworld
|
1 | 13 years ago | kapil.foss at gmail | charmworld pyramid starter |
|
||
|
docs
|
33 | 13 years ago | kapil.foss at gmail | add install doc |
|
||
|
scripts
|
21 | 13 years ago | Kapil Thangavelu | add ctl scripts |
|
||
.bzrignore |
458.1.3 | 11 years ago | j.c.sackett | Bzr should ignore our conventional lp credentials | 430 bytes |
|
|
|
.lbox |
419.1.2 | 11 years ago | Brad Crittenden | lbox defaults to -cr and sets project to charmworl | 31 bytes |
|
|
.lbox.check |
491.1.2 | 11 years ago | Brad Crittenden | Move lint before test in lbox.check | 66 bytes |
|
|
|
CHANGES.txt |
67.1.10 | 12 years ago | Kapil Thangavelu | put back accidentally yanked changes | 28 bytes |
|
|
|
default.ini |
474.1.6 | 11 years ago | j.c.sackett | Changes to address commentar in review. | 2.9 KB |
|
|
|
distribute_setup.py |
71.1.2 | 12 years ago | Rick Harding | Add missing files | 15.3 KB |
|
|
|
LICENSE.txt |
61 | 12 years ago | kapil.foss at gmail | add license | 33.7 KB |
|
|
|
Makefile |
482.1.1 | 11 years ago | Brad Crittenden | Turn on lint for migrations and fix | 5.1 KB |
|
|
|
MANIFEST.in |
1 | 13 years ago | kapil.foss at gmail | charmworld pyramid starter | 133 bytes |
|
|
|
README.txt |
7 | 13 years ago | kapil.foss at gmail | work in progress on multi namespace support | 252 bytes |
|
|
|
requirements.txt |
487.2.1 | 11 years ago | Brad Crittenden | Check charm constraints when doing proof. Lots of | 1.3 KB |
|
|
|
setup.cfg |
304.4.26 | 11 years ago | Aaron Bentley | Move logging filters to nose config. | 569 bytes |
|
|
|
setup.py |
461.3.2 | 11 years ago | Benji York | checkpoint | 1.4 KB |
|
|
|
test-setup.ini |
474.1.6 | 11 years ago | j.c.sackett | Changes to address commentar in review. | 668 bytes |
|
|