[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