Simplified Realloc and Refined Misc

[jschwinger233]

inherited from #260

[tonicmuroq]

... 这个 MR even more terrible than the original one .....

[CMGS]

return c.doDeployOneWorkload(ctx, node, opts, createMsg, seq+idx, deploy-1-idx)

[CMGS]

我认为应该先尝试修改容器资源，再做资源写入。。。

[jschwinger233]

吼

[jschwinger233]

其实我安排顺序主要关注的是“哪个操作更容易回滚”. 试想一个无法回滚的操作, 比如 ContainerRemove, 那这个操作最好就安排在 then 里面而不是 if:

util.Txn(
// if
func(ctx context.Context) error {
    return store.UpdateNode()
},
// then
func(ctx context.Context) error {
    return engine.RemoveContainer()
},
// else 
func(ctx context.Context) error {
    return store.UpdateNode()
}
)

就比下面的顺序好:

util.Txn(
// if
func(ctx context.Context) error {
    return engine.RemoveContainer()
},
// then
func(ctx context.Context) error {
    return store.UpdateNode()
},
// else
nil,
)

[jschwinger233]

这个顺序不能改, 我改回去了.
否则对于 cpu limit >0 && cpu request =0 && cpu bind =true 的请求会报错 "unlimited request with bind"

[jschwinger233]

realloc 测试1600行, 由于改了接口之后很多测试用例都没意义了, 我一个一个改吧....太难了

[jschwinger233]

改完 cli 之后还需测试一番