Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix flake seen in pod_shutdown_test.go #1695

Merged
merged 1 commit into from
Sep 27, 2023
Merged

Conversation

cfryanr
Copy link
Member

@cfryanr cfryanr commented Sep 25, 2023

Trying to fix a flake in a recently added test seen once here https://hush-house.pivotal.io/builds/1879066113#L6480ce4a:48:59.

Release note:

NONE

@codecov
Copy link

codecov bot commented Sep 25, 2023

Codecov Report

Merging #1695 (87b7ea1) into main (7513092) will decrease coverage by 0.02%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##             main    #1695      +/-   ##
==========================================
- Coverage   79.16%   79.14%   -0.02%     
==========================================
  Files         163      163              
  Lines       15767    15767              
==========================================
- Hits        12482    12479       -3     
- Misses       2970     2972       +2     
- Partials      315      316       +1     

see 2 files with indirect coverage changes

@cfryanr cfryanr force-pushed the fix_pod_shutdown_test_flake branch from b32095f to eb1ac4f Compare September 26, 2023 17:11
@cfryanr
Copy link
Member Author

cfryanr commented Sep 26, 2023

This passed all CI jobs at 11 am on Tues Sept 26. I'm going to force push to run them all again to see if they pass again.

@cfryanr cfryanr force-pushed the fix_pod_shutdown_test_flake branch from eb1ac4f to 71c37a2 Compare September 26, 2023 18:00
@cfryanr
Copy link
Member Author

cfryanr commented Sep 26, 2023

Everything passed a second time. I will try running everything for a third time.

@cfryanr cfryanr force-pushed the fix_pod_shutdown_test_flake branch from 71c37a2 to 7ca590f Compare September 26, 2023 18:56
@cfryanr
Copy link
Member Author

cfryanr commented Sep 26, 2023

Everything passed a third time, so maybe this PR is helping.

@cfryanr cfryanr marked this pull request as ready for review September 26, 2023 20:22
@cfryanr cfryanr enabled auto-merge September 26, 2023 20:23
@cfryanr cfryanr force-pushed the fix_pod_shutdown_test_flake branch from 7ca590f to 87b7ea1 Compare September 26, 2023 21:06
@cfryanr cfryanr merged commit 24069b5 into main Sep 27, 2023
8 checks passed
@cfryanr cfryanr deleted the fix_pod_shutdown_test_flake branch September 27, 2023 14:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants