Results 96 comments of chaosi-zju

After numerous days of meticulous exploration, I finally found the most comprehensive solution. I added detailed comments and UT in the code, thanks for your precious time to review again~...

Hi @XiShanYongYe-Chang, all your comments have been resolved, do you have time to check again? Hi @ikaven1024, Do you have any further comments?

kindly ping @ikaven1024 ~ ٩(๑❛ᴗ❛๑)۶

> Hi @chaosi-zju, can you help fill out the release note? done

https://github.com/karmada-io/karmada/blob/f04c17f72749a8db253da7b2790b65d8e912ce24/pkg/resourceinterpreter/customized/declarative/luavm/lua_convert.go#L30-L104 In the `Retain` process of Custom ResourceInterpreter, after `Lua` script is executed, the empty filed like `affinity: {}` has been deleted, so the `Work` synchronized to member cluster is...

> Does it solve this issue #3617? it seems that it is the same problem~ I will test whether my PR #4656 can resolve the issue #3617 too

Hi @ikaven1024, I have tested that #4656 can also solve #3617 ! ```shell ➜ argo kubectl --context karmada-apiserver get rb NAME SCHEDULED FULLYAPPLIED AGE nginx-rollout-rollout True True 65s ```

Hi @Affan-7, sorry to delay the progress of the code reviewing due to our New year holiday. You have done a very good job, and can you help fix the...