Merge lp:~bladernr/checkbox/fix-config-file-vars into lp:~checkbox-dev/checkbox/ppa-packaging-plainbox-provider-canonical-certification

[Jeff Lane ]

fixes a couple of hte default variable entries so they aren't accidentally left blank.

[Daniel Manrique (roadmr)]

Blank values shouldn't cause the test to crash, if you have some time could you please file a bug about that? file it on the "plainbox-provider-checkbox" project (https://bugs.launchpad.net/plainbox-provider-checkbox/).

This is fine as a workaround and due to its simplicity I think it's OK to merge this.

review: Approve

=== modified file 'debian/extras/canonical-certification-config/canonical-certification.conf'
--- debian/extras/canonical-certification-config/canonical-certification.conf	2014-04-01 16:05:51 +0000
+++ debian/extras/canonical-certification-config/canonical-certification.conf	2014-04-17 21:28:44 +0000
@@ -16,7 +16,7 @@
 [environment]
 # For virtualization testing.
 # Uncomment the following 2 lines and set the proper parameters:
-#KVM_TIMEOUT = 
+#KVM_TIMEOUT = 3000
 #KVM_IMAGE =
 # For network testing.
 # WARNING: ALL four parameters need a value, don't leave anything
@@ -24,5 +24,5 @@
 # Uncomment the following 4 lines and set the proper parameters:
 #TEST_TARGET_FTP = your-ftp-server.example.com
 #TEST_USER = anonymous
-#TEST_PASS = 
+#TEST_PASS = password
 #TEST_TARGET_IPERF =  your-iperf-server.example.com