Skip to content

Commit

Permalink
fix(shell_params): prevent generating ,celery in extra when there i…
Browse files Browse the repository at this point in the history
…s no other extra items (#42709)
  • Loading branch information
josix authored Oct 3, 2024
1 parent 2baca06 commit 59294c1
Showing 1 changed file with 3 additions and 1 deletion.
4 changes: 3 additions & 1 deletion dev/breeze/src/airflow_breeze/params/shell_params.py
Original file line number Diff line number Diff line change
Expand Up @@ -332,7 +332,9 @@ def compose_file(self) -> str:
get_console().print(
"[warning]Adding `celery` extras as it is implicitly needed by celery executor"
)
self.airflow_extras = ",".join(current_extras.split(",") + ["celery"])
self.airflow_extras = (
",".join(current_extras.split(",") + ["celery"]) if current_extras else "celery"
)

compose_file_list.append(DOCKER_COMPOSE_DIR / "base.yml")
self.add_docker_in_docker(compose_file_list)
Expand Down

0 comments on commit 59294c1

Please sign in to comment.