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

Clean up engines/builds that were created incompletely #374

Open
ddaspit opened this issue Apr 20, 2024 · 3 comments · May be fixed by #510
Open

Clean up engines/builds that were created incompletely #374

ddaspit opened this issue Apr 20, 2024 · 3 comments · May be fixed by #510
Assignees
Labels
bug Something isn't working

Comments

@ddaspit
Copy link
Contributor

ddaspit commented Apr 20, 2024

It is possible for engines/builds to be in a state where the model exists in the Serval database, but the engine/build was not successfully created by the engine service. I'm not sure how frequently this occurs. I believe that it is fairly unlikely, but still possible.

One possible way to deal with this case is to add a flag to the engine/build model that indicates if the engine/build was successfully created by the engine service. When the engine/build is initially created the flag will be set to false. Once the engine/build is created by the engine service, the flag is set to true. A worker service could periodically check for engines/builds that have not been successfully created after a specified timeout (could be short) and clean up the engine/build.

@ddaspit ddaspit added the bug Something isn't working label Apr 20, 2024
@ddaspit ddaspit added this to Serval Apr 20, 2024
@github-project-automation github-project-automation bot moved this to 🆕 New in Serval Apr 20, 2024
@johnml1135
Copy link
Collaborator

The incomplete engines should all be resolved enough by 1.5. This is to delete the old data from the hard drive.

@johnml1135 johnml1135 self-assigned this Jun 17, 2024
@johnml1135
Copy link
Collaborator

I don't see this as a major concern at this point. There are at max only a few Gigs of space that are being used and no more that would be projected. The time is likely not worth the savings at this point.

@github-project-automation github-project-automation bot moved this from 🆕 New to ✅ Done in Serval Aug 12, 2024
@ddaspit
Copy link
Contributor Author

ddaspit commented Aug 13, 2024

We still haven't implemented proper support for this, so I would like to keep this open. It is still possible for an engine to get out of sync between Serval and the engine service.

@ddaspit ddaspit reopened this Aug 13, 2024
@johnml1135 johnml1135 moved this from ✅ Done to 🔖 Ready in Serval Aug 29, 2024
@johnml1135 johnml1135 assigned Enkidu93 and unassigned johnml1135 Oct 7, 2024
@Enkidu93 Enkidu93 linked a pull request Oct 15, 2024 that will close this issue
@johnml1135 johnml1135 moved this from 🔖 Ready to 🏗 In progress in Serval Nov 1, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
Status: 🏗 In progress
Development

Successfully merging a pull request may close this issue.

3 participants