google / google/adk-python

Dockerfile should first copy requirements.txt and install dependencies before copying agent code

Offen
#6,923 4 Kommentare 0 Reaktionen 2 zugewiesene Personen Beansprucht von @Jacksunwei Auf GitHub ansehen
needs review services
Vorherrschende Sprache
Python
Sterne
21.5k
Forks
4k
Ø Merge
1 T. 14 Std.
Gemergte PRs (30 T.)
37

Beschreibung

https://github.com/google/adk-python/blob/0dbc37d67e928d22acef937a8f441683c88af415/src/google/adk/cli/cli_deploy.py#L203

In the current structure of the Dockerfile, the build first creates a layer with the agent's code, and then creates another layer with the installed dependencies. This order of operations causes the build process to re-create the dependencies layer after every code change, even if the list of dependencies in the requirements.txt hasn't changed.

The correct order should be:
1. copy only requirements.txt file.
2. run pip install to download dependencies
3. copy agent code

This should reduce the amount of time it takes to build container images when redeploying updated agent code, as long as the dependencies list doesn't change.

Beitragsleitfaden

Beitragsleitfaden öffnen

Bewertung

Dieses Issue wurde noch nicht bewertet.

Neue Issues direkt in Ihr Postfach

Eine kurze Übersicht über anfängerfreundliche GitHub-Issues.