Merge lp:~fu-wei/lava-scheduler/multinode into lp:lava-scheduler/multinode

Proposed by Fu Wei
Status: Merged
Approved by: Neil Williams
Approved revision: 282
Merged at revision: 282
Proposed branch: lp:~fu-wei/lava-scheduler/multinode
Merge into: lp:lava-scheduler/multinode
Diff against target: 12 lines (+1/-1)
1 file modified
lava_scheduler_app/utils.py (+1/-1)
To merge this branch: bzr merge lp:~fu-wei/lava-scheduler/multinode
Reviewer Review Type Date Requested Status
Neil Williams Approve
Linaro Automation & Validation Pending
Review via email: mp+181766@code.launchpad.net

Description of the change

Fix the hard code problem of 'logging_level', change 'DEBUG' back to the info from multinode json file

To post a comment you must log in.
Revision history for this message
Neil Williams (codehelp) wrote :

Thanks for spotting this. Approved.

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'lava_scheduler_app/utils.py'
2--- lava_scheduler_app/utils.py 2013-08-20 09:06:06 +0000
3+++ lava_scheduler_app/utils.py 2013-08-23 09:56:56 +0000
4@@ -82,7 +82,7 @@
5
6 node_json[role][c]["role"] = role
7 # multinode node stage 2
8- node_json[role][c]["logging_level"] = "DEBUG"
9+ node_json[role][c]["logging_level"] = json_jobdata["logging_level"]
10 node_json[role][c]["device_type"] = clients["device_type"]
11
12 return node_json

Subscribers

People subscribed via source and target branches

to all changes:
to status/vote changes: