-
Notifications
You must be signed in to change notification settings - Fork 14
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
Speedup container startup #456
Conversation
@unkcpz can you test if this image from this PR starts faster? (I would expect at least 5s faster, not solving your problem, but still nice) |
Thanks @danielhollas, I also notice the slow down of the unnecessary migration I think this is the one bring most improvement. Will give this a look! |
On my machine, I ran
and it took ~13s, while the latest image takes ~18s. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hi @danielhollas, let me know when this is ready for review.
@unkcpz ready for review. I got rid of one more verdi invocation. |
06210df
to
6f639aa
Compare
@superstar54 this one is ready to go. Would be great if you could have a look when you have a bit of time. Thanks! |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM!
The main strategy here is to remove all unneeded verdi invocations, because each one costs at least 0.5s.