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

Remove deprecated use of wait. functions #10546

Merged
merged 3 commits into from
Jul 29, 2024

Conversation

willdot
Copy link
Contributor

@willdot willdot commented Jul 18, 2024

Proposed Changes

The following functions have been deprecated (along with their replacements)

PollImmediateUntilWithContext is now PollUntilContextCancel

PollImmediateInfiniteWithContext is now PollUntilContextCancel

PollImmediateWithContext is now PollUntilContextTimeout

PollImmediate is now PollUntilContextTimeout

Since all of these functions that are being replaced as Immediate the immediate flag is being set as true.

Types of Changes

Verification

Testing

Linked Issues

#9321

User-Facing Change


Further Comments

@willdot willdot requested a review from a team as a code owner July 18, 2024 19:46
brandond
brandond previously approved these changes Jul 23, 2024
Copy link
Member

@brandond brandond left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you! lgtm, one minor nit!

Co-authored-by: Brad Davidson <[email protected]>
Signed-off-by: Will Andrews <[email protected]>
@willdot
Copy link
Contributor Author

willdot commented Jul 23, 2024

Thank you! lgtm, one minor nit!

Ah good catch. Committed!

@willdot willdot requested a review from brandond July 23, 2024 09:29
vitorsavian
vitorsavian previously approved these changes Jul 23, 2024
brandond
brandond previously approved these changes Jul 23, 2024
@dereknola
Copy link
Member

There is an error in the split role cluster config on this PR.

pkg/cluster/managed.go Outdated Show resolved Hide resolved
Co-authored-by: Derek Nola <[email protected]>
Signed-off-by: Will Andrews <[email protected]>
@willdot willdot dismissed stale reviews from brandond and vitorsavian via 2024e59 July 25, 2024 17:37
@willdot willdot requested a review from dereknola July 25, 2024 18:33
Copy link

codecov bot commented Jul 26, 2024

Codecov Report

Attention: Patch coverage is 52.38095% with 10 lines in your changes missing coverage. Please review.

Project coverage is 43.32%. Comparing base (37830fe) to head (2024e59).
Report is 5 commits behind head on master.

Files Patch % Lines
pkg/secretsencrypt/config.go 0.00% 5 Missing ⚠️
pkg/cluster/managed.go 0.00% 1 Missing ⚠️
pkg/cluster/storage.go 66.66% 1 Missing ⚠️
pkg/etcd/etcd.go 0.00% 1 Missing ⚠️
pkg/spegel/bootstrap.go 0.00% 1 Missing ⚠️
pkg/spegel/spegel.go 0.00% 1 Missing ⚠️

❗ There is a different number of reports uploaded between BASE (37830fe) and HEAD (2024e59). Click for more details.

HEAD has 3 uploads less than BASE
Flag BASE (37830fe) HEAD (2024e59)
e2etests 9 6
Additional details and impacted files
@@            Coverage Diff             @@
##           master   #10546      +/-   ##
==========================================
- Coverage   49.66%   43.32%   -6.34%     
==========================================
  Files         179      179              
  Lines       14936    14953      +17     
==========================================
- Hits         7418     6479     -939     
- Misses       6158     7277    +1119     
+ Partials     1360     1197     -163     
Flag Coverage Δ
e2etests 36.28% <47.61%> (-10.29%) ⬇️
inttests 19.70% <28.57%> (-0.02%) ⬇️
unittests 13.39% <0.00%> (+0.03%) ⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants