[Compass-ci] Re: [PATCH v3 compass-ci] fix: sched: add 2 fields into cluster state

...

• cluster_state.each_value do |role_state|

•  if %(#{role_state["roles"]}) == role
  

•    return role_state
  end
  

  end

 if all condition is false, you should also have return value.
 otherwise, return value will be "cluster_state".

Hash[key] will raise an exception if key is nonexistent. Here raising the exception or returning 'role_state'.

Thanks, RenWen

 Thanks,
 Liushaofei

...

end

2.23.0