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 #10278

Closed
wants to merge 0 commits into from
Closed

Conversation

willdot
Copy link
Contributor

@willdot willdot commented Jun 3, 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

@vitorsavian
Copy link
Member

Hey @willdot, how are you? how its going with the PR?

@willdot
Copy link
Contributor Author

willdot commented Jul 16, 2024

Hey @willdot, how are you? how its going with the PR?

Hey @vitorsavian

Haven't had much time to look recently as I'm trying to find new work due to layoffs.

The status of this PR though, is that the continuous-integration/drone/pr stage is failing and I couldn't dig into the logs enough to work out why 🤷‍♂️

@vitorsavian
Copy link
Member

vitorsavian commented Jul 17, 2024

Hey @willdot

I'm sorry for your layoff.

if you could rebase your branch, since for what I saw it's only one test that is failing on our CI and could be fixed with the rebase.

@willdot
Copy link
Contributor Author

willdot commented Jul 17, 2024

Hey @willdot

I'm sorry for your layoff.

if you could rebase your branch, since for what I saw it's only one test that is failing on our CI and could be fixed with the rebase.

Thanks

I've rebased (after some voodoo magic and problems with DCO commit signing) but everything looks to be in place now.

Does make the commit history look messy so if it's a problem, I don't mind starting fresh and creating a new branch with the changes (there aren't many changes so shouldn't take too long)

@willdot willdot marked this pull request as ready for review July 17, 2024 19:51
@willdot willdot requested a review from a team as a code owner July 17, 2024 19:51
@vitorsavian
Copy link
Member

vitorsavian commented Jul 18, 2024

Thanks

I've rebased (after some voodoo magic and problems with DCO commit signing) but everything looks to be in place now.

Does make the commit history look messy so if it's a problem, I don't mind starting fresh and creating a new branch with the changes (there aren't many changes so shouldn't take too long)

I think it will be a good idea to create a new one with the changes to not change our commit history

@willdot
Copy link
Contributor Author

willdot commented Jul 18, 2024

@vitorsavian Please see #10546

I'm away now for the weekend so can continue with any changes / fixes next week 😄

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.

None yet

2 participants