What does this PR give to us:
During CNI ADD/DEL operations DANM waits 30 seconds to finish all
the delegated CNI tasks and DanmEp processing. In case the webhook
is too slow to respond, DANM timeouts without knowing the reason.
The webhook REST call timeout is lowered to 25 seconds (by default
it is 30 sec), so its timeout event can arrive back to DANM, which
can log that error message.
Which issue(s) this PR fixes:
Related to #144, which is still a valid issue.
Levovar
commented
3 years ago
What type of PR is this? bug
What does this PR give to us: During CNI ADD/DEL operations DANM waits 30 seconds to finish all the delegated CNI tasks and DanmEp processing. In case the webhook is too slow to respond, DANM timeouts without knowing the reason. The webhook REST call timeout is lowered to 25 seconds (by default it is 30 sec), so its timeout event can arrive back to DANM, which can log that error message.
Which issue(s) this PR fixes: Related to #144, which is still a valid issue.
Special notes for your reviewer:
Does this PR introduce a user-facing change?: No.