EX-5406 lipe: fix some compatibility issue in lpcc

- use absolute path of program in lpcc.service
- replace yaml.load() with yaml.safe_load()

Signed-off-by: Lei Feng <flei@whamcloud.com>
Test-Parameters: trivial testlist=sanity-pcc
Change-Id: Iad3fa408f5b19e7370c0a5b069ad7637611118a7
Reviewed-on: https://review.whamcloud.com/47678
Tested-by: jenkins <devops@whamcloud.com>
Tested-by: Maloo <maloo@whamcloud.com>
Reviewed-by: Jian Yu <yujian@whamcloud.com>
Reviewed-by: Li Xi <lixi@ddn.com>
Reviewed-by: Andreas Dilger <adilger@whamcloud.com>

diff --git a/lipe/lpcc b/lipe/lpcc
index 462efff..bd856b8 100755 (executable)
--- a/lipe/lpcc
+++ b/lipe/lpcc
@@ -224,7 +224,7 @@ class LpccService:
             return result
 
         result['status'] = "stopped"
-        pcclist = yaml.load(output)
+        pcclist = yaml.safe_load(output)
         if pcclist is not None and 'pcc' in pcclist:
             for pcc in pcclist['pcc']:
                 if pcc['pccpath'] == self.lpcc_cache:

diff --git a/lipe/systemd/lpcc.service b/lipe/systemd/lpcc.service
index 1d7a6c8..f791d45 100644 (file)
--- a/lipe/systemd/lpcc.service
+++ b/lipe/systemd/lpcc.service
@@ -8,7 +8,7 @@ ConditionPathExists=/etc/lpcc.conf
 
 [Service]
 Type=simple
-ExecStart=lpcc monitor
+ExecStart=/usr/bin/lpcc monitor
 ExecStop=kill $MAINPID
 
 [Install]