csi-addons / volume-replication-operator

Apache License 2.0
16 stars 23 forks source link

replication: set RequeueAfter to 30 secondary for requeue #123

Closed Madhu-1 closed 2 years ago

Madhu-1 commented 2 years ago

Setting Requeue time for 30 seconds as the resync can take time and having default Requeue exponential backoff time can affect the RTO time

closes: #120

Signed-off-by: Madhu Rajanna madhupr007@gmail.com

humblec commented 2 years ago

@Madhu-1 the referred issue says of requeing at 15seconds where this PR is on 30 seconds. so would be good to correct either of the place.

Also, ooc, do we have any of the controller references which does 30 second reque?

Madhu-1 commented 2 years ago

Also, ooc, do we have any of the controller references which does 30 second reque?

@humblec what controllers you are looking at here? the resync is something async operation and it does not fix the exponential backoff.