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

Drop things from the queue when completed #3

Merged
merged 1 commit into from
Apr 16, 2020

Conversation

dramich
Copy link
Contributor

@dramich dramich commented Apr 16, 2020

@dramich
Copy link
Contributor Author

dramich commented Apr 16, 2020

@ibuildthecloud Can you take a look at this? I can't request reviewers in this repo.

@ibuildthecloud
Copy link
Contributor

Do you have more context on why the enqueueAfter change?

@dramich
Copy link
Contributor Author

dramich commented Apr 16, 2020

To try again, if you want to just mark the object as done regardless I can do that.

@ibuildthecloud
Copy link
Contributor

Ah, so the memory leak is that we don't ever call done? I don't think we want to reenque. I'd just always call done.

@ibuildthecloud ibuildthecloud merged commit 12cc38a into rancher:master Apr 16, 2020
ibuildthecloud pushed a commit to ibuildthecloud/steve that referenced this pull request Apr 4, 2022
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.

2 participants