astronomer / astro-cli / 38328640402

61%

main: 45%

push

github

Carry the project's files in the image a deploy or astro package generates (#2302)

* Carry the project's files in the image a deploy or astro package generates

A generated build's context held only the generated requirements.txt and
packages.txt, so the image a manifest deploy shipped had none of the
project's own code: no include/, no plugins/, nothing dags/ imports. A deploy
sends dags/ alone beside the image, so on Astro a DAG importing include
failed (here, blueprint's registry swallowed the ImportError and reported
"No blueprints found").

ForManifest (deploy, astro package) now sets Request.ProjectFiles, and a
generated build copies the project into its context, which the runtime
image's ONBUILD COPY . . lays down after both installs. Version control,
virtualenvs, .astro and caches are left out, the project's own
requirements.txt and packages.txt never replace the generated ones, and a
project .dockerignore applies to the rest without dropping them. Local
Docker mode (ForLocalManifest) is unchanged: it runs the project mounted.

Co-Authored-By: Claude <noreply@anthropic.com>

* Address review: keep secrets out, follow symlinks, hash the files into package tags

- The plaintext .env and the per-machine files Astro tools write
  (imagebuild.ShippedImageIgnores, mirroring scaffold's .dockerignore rules)
  never ship, and come after the project's .dockerignore so no ! line brings
  them back. A root-module test holds the list to cover scaffold's.
- The project's .dockerignore applies during the walk (moby/patternmatcher),
  so an excluded directory is never read or copied.
- Symlinks are carried as what they point at: a symlinked project root is
  walked, a link out of the project no longer dangles in the image, Windows
  needs no symlink privilege, loops and dangling links are skipped, and a file
  vanishing mid-walk is skipped.
- astro package's content-addressed tag includes a digest of the carried
  files (imagebuild.ProjectFilesDigest), so ed... (continued)

6 of 8 new or added lines in 1 file covered. (75.0%)

40187 of 69616 relevant lines covered (57.73%)

233.68 hits per line