Skip to content

Commit

Permalink
Fix .astro folder not present in few images causing failure for parse…
Browse files Browse the repository at this point in the history
… command (#1375)

* Fix .astro folder not present in few images causing failure for parse command

* Fix typo in comments
  • Loading branch information
neel-astro committed Aug 30, 2023
1 parent 986850a commit 6ec6411
Showing 1 changed file with 13 additions and 0 deletions.
13 changes: 13 additions & 0 deletions airflow/docker_image.go
Original file line number Diff line number Diff line change
Expand Up @@ -135,6 +135,19 @@ func (d *DockerImage) Pytest(pytestFile, airflowHome, envFile, testHomeDirectory
if docErr != nil {
return "", docErr
}
// cp .astro folder
// on some machine .astro is being docker ignored, but not
// on every machine, hence to keep behavior consistent
// copying the .astro folder explicitly
args = []string{
"cp",
airflowHome + "/.astro",
"astro-pytest:/usr/local/airflow/",
}
docErr = cmdExec(dockerCommand, stdout, stderr, args...)
if docErr != nil {
return "", docErr
}
// start pytest container
docErr = cmdExec(dockerCommand, stdout, stderr, []string{"start", "astro-pytest", "-a"}...)
if docErr != nil {
Expand Down

0 comments on commit 6ec6411

Please sign in to comment.