yellow team mailing list archive

[Francesco Banconi <francesco.banconi@xxxxxxxxxxxxx>]

Francesco Banconi has proposed merging lp:~frankban/lpsetup/install-lxc-stops-lxc into lp:lpsetup.

Requested reviews:
  Yellow Squad (yellow)

For more details, see:
https://code.launchpad.net/~frankban/lpsetup/install-lxc-stops-lxc/+merge/116250

== Changes ==

Moved the *stop_lxc* step so that it is the last step executed by install-lxc.
-- 
https://code.launchpad.net/~frankban/lpsetup/install-lxc-stops-lxc/+merge/116250
Your team Yellow Squad is requested to review the proposed merge of lp:~frankban/lpsetup/install-lxc-stops-lxc into lp:lpsetup.

=== modified file 'lpsetup/subcommands/initlxc.py'
--- lpsetup/subcommands/initlxc.py	2012-07-18 10:53:15 +0000
+++ lpsetup/subcommands/initlxc.py	2012-07-23 11:30:30 +0000
@@ -226,7 +226,7 @@
     stop_lxc_step = (stop_lxc,
         'lxc_name', 'ssh_key_path')
 
-    steps = (
+    base_steps = (
         (initialize, 'user'),
         inithost.SubCommand.setup_home_step,
         create_lxc_step,
@@ -234,8 +234,8 @@
         wait_for_lxc_step,
         install_lpsetup_in_lxc_step,
         inithost_in_lxc_step,
-        stop_lxc_step,
         )
+    steps = base_steps + (stop_lxc_step,)
 
     help = __doc__
     needs_root = True

=== modified file 'lpsetup/subcommands/install_lxc.py'
--- lpsetup/subcommands/install_lxc.py	2012-07-20 14:51:13 +0000
+++ lpsetup/subcommands/install_lxc.py	2012-07-23 11:30:30 +0000
@@ -109,7 +109,7 @@
     development model.
     """
 
-    steps = initlxc.SubCommand.steps + (
+    steps = initlxc.SubCommand.base_steps + (
         # Run on host:
         (create_scripts, 'lxc_name', 'ssh_key_path', 'user'),
         # Run inside the container:
@@ -120,7 +120,7 @@
          'external_path', 'target_dir', 'use_http'),
         (finish_inithost_in_lxc, 'lxc_name', 'ssh_key_path', 'home_dir',
          'user', 'target_dir'),
-        )
+        ) + (initlxc.SubCommand.stop_lxc_step,)
 
     help = __doc__
 

=== modified file 'lpsetup/tests/subcommands/test_install_lxc.py'
--- lpsetup/tests/subcommands/test_install_lxc.py	2012-07-20 10:40:43 +0000
+++ lpsetup/tests/subcommands/test_install_lxc.py	2012-07-23 11:30:30 +0000
@@ -56,11 +56,11 @@
         handlers.handle_source,
         handlers.handle_target_from_repository,
         )
-    expected_steps = test_initlxc.InitLxcTest.expected_steps + (
+    expected_steps = test_initlxc.InitLxcTest.expected_steps[:-1] + (
         create_scripts_step,
         init_repo_in_lxc_step,
         update_in_lxc_step,
         finish_inithost_in_lxc_step,
-        )
+        ) + (test_initlxc.stop_lxc_step,)
     needs_root = True
     maxDiff = None