You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
{{ message }}
This repository has been archived by the owner on Jun 12, 2024. It is now read-only.
As far as I could see from the current codebase, draftd sends the whole project files as the build context to the docker daemon.
This, "in theory", slow down docker build by unnecessary sending unchanged files from draftd to docker daemon.
Similarly as I've noted in GoogleContainerTools/skaffold#195, I guess we could enhance draft to use streaming to send only diffs from draftd to docker daemon.
Just for clarification, this is for the "2" in the below diagram:
draft --1--> draftd --2--> docker daemon
Honestly, I'm not sure how the path "1" works today in draft. It would make sense to address it in an another github issue if necessary, too.
The text was updated successfully, but these errors were encountered:
Yes! We would love to have this in place. I know @fibonacci1729 has played with a few ideas in this space and might be able to share some of this thoughts.
@mumoshu the new draft .12 removes the daemon completely, removing the problem. Rerun tests, and see if there are still optimizations here we should work on.
While we have removed draftd from the system and thereby eliminated the draft --1--> draftd cycle, technically speaking we are still sending the entire build context to the docker daemon.
Docker has a relatively new feature that allows clients to send incremental build contexts to the daemon. We could take advantage of that during long-running sessions such as a draft up with the local file watcher feature enabled.
Other than that, you're completely right. For the most part this issue is moot.
Sign up for freeto subscribe to this conversation on GitHub.
Already have an account?
Sign in.
As far as I could see from the current codebase, draftd sends the whole project files as the build context to the docker daemon.
This, "in theory", slow down docker build by unnecessary sending unchanged files from draftd to docker daemon.
Similarly as I've noted in GoogleContainerTools/skaffold#195, I guess we could enhance draft to use streaming to send only diffs from draftd to docker daemon.
Just for clarification, this is for the "2" in the below diagram:
Honestly, I'm not sure how the path "1" works today in draft. It would make sense to address it in an another github issue if necessary, too.
The text was updated successfully, but these errors were encountered: