Merge lp:~bladernr/checkbox/fix-virt-no-config-warning into lp:checkbox

Proposed by Jeff Lane 
Status: Merged
Approved by: Po-Hsu Lin
Approved revision: 4360
Merged at revision: 4361
Proposed branch: lp:~bladernr/checkbox/fix-virt-no-config-warning
Merge into: lp:checkbox
Diff against target: 22 lines (+3/-2)
1 file modified
providers/plainbox-provider-checkbox/bin/virtualization (+3/-2)
To merge this branch: bzr merge lp:~bladernr/checkbox/fix-virt-no-config-warning
Reviewer Review Type Date Requested Status
Po-Hsu Lin Approve
Review via email: mp+295122@code.launchpad.net

Description of the change

Changed DEFAULT_CFG location to /etc/xdg since we no longer create /etc/checkbox.d.

Modified No Config File warnings to be more informative. People were confused by the messaging.

To post a comment you must log in.
Revision history for this message
Po-Hsu Lin (cypressyew) wrote :

LGTM, +1

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
=== modified file 'providers/plainbox-provider-checkbox/bin/virtualization'
--- providers/plainbox-provider-checkbox/bin/virtualization 2016-04-22 20:00:30 +0000
+++ providers/plainbox-provider-checkbox/bin/virtualization 2016-05-18 19:05:28 +0000
@@ -410,7 +410,7 @@
410def test_kvm(args):410def test_kvm(args):
411 print("Executing KVM Test", file=sys.stderr)411 print("Executing KVM Test", file=sys.stderr)
412412
413 DEFAULT_CFG = "/etc/checkbox.d/virtualization.cfg"413 DEFAULT_CFG = "/etc/xdg/virtualization.cfg"
414 image = ""414 image = ""
415 timeout = ""415 timeout = ""
416416
@@ -422,7 +422,8 @@
422 try:422 try:
423 config.readfp(open(config_file))423 config.readfp(open(config_file))
424 except IOError:424 except IOError:
425 logging.warn("No config file found")425 logging.warn("Config file %s was not found" % DEFAULT_CFG)
426 logging.warn("Now looking for ENV variables and command line arguments instead")
426 else:427 else:
427 try:428 try:
428 timeout = config.getfloat("KVM", "timeout")429 timeout = config.getfloat("KVM", "timeout")

Subscribers

People subscribed via source and target branches