canonical-hw-cert team mailing list archive

[Paul Larson <mp+426300@xxxxxxxxxxxxxxxxxx>]

Paul Larson has proposed merging ~pwlars/cc-lab-manager:listi-typo into cc-lab-manager:master.

Requested reviews:
  Canonical Hardware Certification (canonical-hw-cert)

For more details, see:
https://code.launchpad.net/~pwlars/cc-lab-manager/+git/cc-lab-manager/+merge/426300

I'm pretty sure this is what caused the typos in https://code.launchpad.net/~pwlars/hwcert-jenkins-jobs/+git/cert-juju/+merge/426299
-- 
Your team Canonical Hardware Certification is requested to review the proposed merge of ~pwlars/cc-lab-manager:listi-typo into cc-lab-manager:master.

diff --git a/cc_lab_manager/gen_config/gen_agent_tf_config.py b/cc_lab_manager/gen_config/gen_agent_tf_config.py
index 8acefe5..af86d82 100644
--- a/cc_lab_manager/gen_config/gen_agent_tf_config.py
+++ b/cc_lab_manager/gen_config/gen_agent_tf_config.py
@@ -220,7 +220,7 @@ def generate_agent_config(db_machine_list, cfg_path):
                 'sudo sd-mux-ctrl --device-serial=$(sudo sd-mux-ctrl --list'
                 '| grep Serial| cut -d "," -f 3| cut -d " " -f 3) --ts')
             agent_conf['control_switch_device_cmd'] = (
-                'sudo sd-mux-ctrl --device-serial=$(sudo sd-mux-ctrl --listi'
+                'sudo sd-mux-ctrl --device-serial=$(sudo sd-mux-ctrl --list'
                 '| grep Serial| cut -d "," -f 3| cut -d " " -f 3) --dut')
 
         # Update reboot script for known type.