[8.17](backport #7057) [flakiness] kubernetes_secrets provider unit tests #7156
+16
−17
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
What does this PR do?
This PR introduces the following fixes and improvements:
Loosens Time Constraints in Unit Tests (needed for certain type of CI runners):
TTLDelete
value in test configurations was increased from2s
to10s
, ensuring tests have more leeway in processing cache expiration.waitForSignal
value was adjusted from1s
to2s
to reduce test flakiness.Adds Debug Logging for Fetch When Cache is Disabled: relevant comment
Fixes a Typo in a Unit Test Name: relevant comment
"key in secret not found"
was incorrectly named and is now corrected to"key in secret found"
.Why is it important?
Fixes a test failure (
Test_Run/secret_no-update_and_no-signal
) that caused CI flakinessChecklist
./changelog/fragments
using the changelog tool.Disruptive User Impact
No expected impact on users. These changes are limited to debugging logs and test improvements.
How to test this PR locally
Related issues
This is an automatic backport of pull request #7057 done by [Mergify](https://mergify.com).