[Compass-ci] Re: [PATCH compass-ci] scheduler: fix: alter processes of request_state

...

states = {"abort"       => "abort",

•          "started"     => "started",
          "finished"    => "finish",
  

maybe you can keep the same for started/finished: "started" => "started", "finished" => "finished",

"finish" used to do a response, lkp did a limit. can change "started" to "start".

...

          "failed"      => "abort",
          "wait_ready"  => "ready",
          "wait_finish" => "finish"}

case request_state

• when "abort", "finished", "failed"

• when "abort", "started", "failed" # update node state only update_cluster_state(cluster_id, job_id, {"state" => states[request_state]}) when "wait_ready"

@@ -30,7 +31,7 @@ class Sched }

  return cluster_state

• when "wait_finish"

• when "wait_finish", "finished"

the value you write above is finish, here is finished whether need to fix it?

"finished" is a request state, "finish" will be a node state and a response. You can refer 'lkp/lib/job.sh'.

Thanks, RenWen

Thanks, Luan Shengde

...

  update_cluster_state(cluster_id, job_id, {"state" => states[request_state]})
  while 1
    sleep(10)

-- 2.23.0