huawei-cloudnative / firmament

The Firmament cluster scheduling platform
http://www.firmament.io
Apache License 2.0
19 stars 9 forks source link

Fix for few crash cases #9

Closed shivramsrivastava closed 6 years ago

shivramsrivastava commented 6 years ago

This PR addressed the following issues

1) The reply response to the gRPC call was not set prior to this change, this PR addresses that issue by setting the correct response to the TaskUpdate gRPC call.

2) Fixed a bug in remove task method, when a task is not scheduled and is called for deletion, the task removal was not done properly, this was causing a crash. This PR addresses that issue as well.

3) With the introduction of the simple solver for the pod affinity case, there were some issues/crashes, which are addressed in this PR.

4) For Toleration feature if no operator is specified then 'Equal' operator is assumed, this was missing prior to this fix, and is now addresed in this PR.