Skip to content

Commit

Permalink
Increase number of expected reconciles
Browse files Browse the repository at this point in the history
Signed-off-by: Jonathan West <[email protected]>
  • Loading branch information
jgwest committed Nov 1, 2024
1 parent f399ca6 commit bf50c41
Showing 1 changed file with 3 additions and 3 deletions.
6 changes: 3 additions & 3 deletions hack/run-rollouts-manager-e2e-tests.sh
Original file line number Diff line number Diff line change
Expand Up @@ -150,17 +150,17 @@ sanity_test_metrics_data() {

fi

if [[ "`expr $DELTA_ERROR_RECONCILES \> 30`" == "1" ]]; then
if [[ "`expr $DELTA_ERROR_RECONCILES \> 60`" == "1" ]]; then
# This value is arbitrary, and should be updated if at any point it becomes inaccurate (but first audit the test/code to make sure it is not an actual product/test issue, before increasing)
echo "Number of Reconcile calls that returned an error $DELTA_ERROR_RECONCILES was greater than the expected value"
echo "Number of Reconcile calls that returned an error '$DELTA_ERROR_RECONCILES' was greater than the expected value"
exit 1

fi

if [[ "`expr $DELTA_SUCCESS_RECONCILES \> 1200`" == "1" ]]; then
# This value is arbitrary, and should be updated if at any point it becomes inaccurate (but first audit the test/code to make sure it is not an actual product/test issue, before increasing)

echo "Number of Reconcile calls that returned success $DELTA_SUCCESS_RECONCILES was greater than the expected value"
echo "Number of Reconcile calls that returned success '$DELTA_SUCCESS_RECONCILES' was greater than the expected value"
exit 1

fi
Expand Down

0 comments on commit bf50c41

Please sign in to comment.