Nary Yeh
Nary Yeh
Check the behaviour when main container OOM, the sidecar container will still get `SIGTERM` and upload output if exists 
Also removed the shared namespace watcher as this watcher does not work in the current approach. It will be terminated by `SIGTERM` 
Sorry for the late reply! I just updated the description to include the test script I used and the new result screenshot Thanks!
This is the default behaviour. You may use % instead to jump between both end of [].
Hi @dentiny , PTAL Thanks!
Hi @dentiny , I've modified to retry only on specific http status errors (list below) and added backoff. - 408 - Request Timeout - 429 - Too Many Requests -...
Hi @dentiny I found that my original retry logic is incorrect. The max attempt times should be maxRetry + 1. Originally, I use `
@dentiny I modified the `TestAPIServerClientRetry` to let it validate the `callCount` is `expectCallCount` and add the test case for `maxRetry` = 0 Thanks!
> Hi @machichima , I didn't see new tests added for "maxRetry = 0" case (no new commits either). Did I mis-understand? I am so sorry... I forgot to push...
cc @kevin85421 for final review The failing CI for autoscaler seems to be flaky test, while this PR only modify apiserver part