Merge lp:~bac/charmworld/fix-bundle-branch-delete-key into lp:~juju-jitsu/charmworld/trunk

[Brad Crittenden]

Fix ingest KeyError for 'branch_deleted'.

https://codereview.appspot.com/16240044/

[Brad Crittenden (bac)]

Reviewers: mp+192409_code.launchpad.net,

Message:
Please take a look.

Description:
Fix ingest KeyError for 'branch_deleted'.

https://code.launchpad.net/~bac/charmworld/show-bundle-changes/+merge/192409

(do not edit description out of merge proposal)

Please review this at https://codereview.appspot.com/16240044/
Index: [revision details]
=== added file '[revision details]'
--- [revision details] 2012-01-01 00:00:00 +0000
+++ [revision details] 2012-01-01 00:00:00 +0000
@@ -0,0 +1,2 @@
+Old revision: <email address hidden>
+New revision: <email address hidden>

Index: charmworld/jobs/lp.py
=== modified file 'charmworld/jobs/lp.py'
--- charmworld/jobs/lp.py 2013-10-17 20:19:55 +0000
+++ charmworld/jobs/lp.py 2013-10-23 21:53:17 +0000
@@ -140,6 +140,7 @@
                           charm_copy['branch_spec'],
                           charm_copy['store_data']['revision'])
      for basket in _filtered_payloads(baskets, import_filter):
+ basket['branch_deleted'] = False
          added = basket_queue.put(basket)
          if added:
              log.info('Queueing bundle %s', basket['branch_spec'])

Index: charmworld/jobs/tests/test_lp.py
=== modified file 'charmworld/jobs/tests/test_lp.py'
--- charmworld/jobs/tests/test_lp.py 2013-10-17 20:19:55 +0000
+++ charmworld/jobs/tests/test_lp.py 2013-10-23 21:53:17 +0000
@@ -296,6 +296,7 @@
      BASKET_PAYLOAD = {
          'foo': 'bar',
          'branch_spec': '~baz/bar/1/bundle',
+ 'branch_deleted': False,
      }

      def setUp(self):

[Brad Crittenden (bac)]

LGTM, yo

https://codereview.appspot.com/16240044/

[Brad Crittenden (bac)]

*** Submitted:

Fix ingest KeyError for 'branch_deleted'.

R=
CC=
https://codereview.appspot.com/16240044

https://codereview.appspot.com/16240044/

=== modified file 'charmworld/jobs/lp.py'
--- charmworld/jobs/lp.py	2013-10-17 20:19:55 +0000
+++ charmworld/jobs/lp.py	2013-10-23 21:56:14 +0000
@@ -140,6 +140,7 @@
                          charm_copy['branch_spec'],
                          charm_copy['store_data']['revision'])
     for basket in _filtered_payloads(baskets, import_filter):
+        basket['branch_deleted'] = False
         added = basket_queue.put(basket)
         if added:
             log.info('Queueing bundle %s', basket['branch_spec'])
 
=== modified file 'charmworld/jobs/tests/test_lp.py'
--- charmworld/jobs/tests/test_lp.py	2013-10-17 20:19:55 +0000
+++ charmworld/jobs/tests/test_lp.py	2013-10-23 21:56:14 +0000
@@ -296,6 +296,7 @@
     BASKET_PAYLOAD = {
         'foo': 'bar',
         'branch_spec': '~baz/bar/1/bundle',
+        'branch_deleted': False,
     }
 
     def setUp(self):