Fix update issue on where service doesn't roll back on failure

[awmirantis]

Signed-off-by: Adam Williams [email protected]
- What I did
When the updateConfig delay parameter is larger than the monitor parameter an error in updating can be missed and rollback not performed. This fix will check for the case where monitor < delay and use the larger of the two.

Fixes 37972
- How I did it

- How to test it
Manual testing that failed update will rollback.
- Description for the changelog

[awmirantis]

This test succeeds for me when ran locally. Could someone (@dperny @thaJeztah) kick off a replay for me?

[thaJeztah]

I restarted CI; in the meantime we also updated some vendoring, so perhaps good to rebase this PR.

Do you think it's possible to write a test for this, or complicated?

[codecov]

Codecov Report

Merging #3003 (3f633b2) into master (62d78f2) will decrease coverage by 0.03%.
The diff coverage is 100.00%.

❗ Current head 3f633b2 differs from pull request most recent head fc1338c. Consider uploading reports for the commit fc1338c to get more accurate results

@@            Coverage Diff             @@
##           master    #3003      +/-   ##
==========================================
- Coverage   60.82%   60.79%   -0.04%     
==========================================
  Files         142      142              
  Lines       20295    20297       +2     
==========================================
- Hits        12345    12340       -5     
+ Misses       6494     6493       -1     
- Partials     1456     1464       +8