You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@airflow.apache.org by GitBox <gi...@apache.org> on 2020/09/25 09:59:24 UTC

[GitHub] [airflow] alikhtag opened a new issue #11146: Cleanup CronJob fails

alikhtag opened a new issue #11146:
URL: https://github.com/apache/airflow/issues/11146


   **Apache Airflow version**: 1.10.12
   
   **Kubernetes version** : 1.17.9
   
   **Environment**:
   
   - **Cloud provider or hardware configuration**: Azure Kubernetes Service
   - **OS** (e.g. from /etc/os-release): Linux
   - **Install tools**: Helm chart (/chart in repo) 
   - **Others**:
   
   **What happened**:
   Pod cannot run since `airflow-cleanup-pods` is not in $PATH
   ```
   Error: failed to start container "airflow-cleanup-pods": Error response from daemon: OCI runtime create failed: container_linux.go:349: starting container process caused "exec: \"airflow-cleanup-pods\": executable file not found in $PATH": unknown
   ```
   
   **What you expected to happen**: 
   Cleanup job pod runs
   
   <!-- What do you think went wrong? -->
   
   **How to reproduce it**:
   * Deploy Helm Chart
   * Manually trigger `airflow-cleanup ` CronJob 
   * See it cannot run since `airflow-cleanup-pods` is not in $PATH


----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



[GitHub] [airflow] kaxil closed issue #11146: Cleanup CronJob fails

Posted by GitBox <gi...@apache.org>.
kaxil closed issue #11146:
URL: https://github.com/apache/airflow/issues/11146


   


----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



[GitHub] [airflow] alikhtag commented on issue #11146: Cleanup CronJob fails

Posted by GitBox <gi...@apache.org>.
alikhtag commented on issue #11146:
URL: https://github.com/apache/airflow/issues/11146#issuecomment-698843483


   Found something in `github`, I assume these scripts were not migrated when `Helm` chart was donated by `Astronomer` ? 
   https://github.com/astronomer/astronomer-airflow-scripts
   


----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



[GitHub] [airflow] boring-cyborg[bot] commented on issue #11146: Cleanup CronJob fails

Posted by GitBox <gi...@apache.org>.
boring-cyborg[bot] commented on issue #11146:
URL: https://github.com/apache/airflow/issues/11146#issuecomment-698841098


   Thanks for opening your first issue here! Be sure to follow the issue template!
   


----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



[GitHub] [airflow] kaxil commented on issue #11146: Cleanup CronJob fails

Posted by GitBox <gi...@apache.org>.
kaxil commented on issue #11146:
URL: https://github.com/apache/airflow/issues/11146#issuecomment-698845512


   thanks for raising the issue, we will take a look into this one.


----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



[GitHub] [airflow] kaxil commented on issue #11146: Cleanup CronJob fails

Posted by GitBox <gi...@apache.org>.
kaxil commented on issue #11146:
URL: https://github.com/apache/airflow/issues/11146#issuecomment-715642111


   PR: https://github.com/apache/airflow/pull/11802


----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



[GitHub] [airflow] kaxil commented on issue #11146: Cleanup CronJob fails

Posted by GitBox <gi...@apache.org>.
kaxil commented on issue #11146:
URL: https://github.com/apache/airflow/issues/11146#issuecomment-698845512


   thanks for raising the issue, we will take a look into this one.


----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



[GitHub] [airflow] mik-laj commented on issue #11146: Cleanup CronJob fails

Posted by GitBox <gi...@apache.org>.
mik-laj commented on issue #11146:
URL: https://github.com/apache/airflow/issues/11146#issuecomment-712544436


   @kaxil Any progress on this ticket? I can see Helm Chart still using this script which is not available in the official image.
   https://github.com/apache/airflow/blob/3970bfad4c878d99adce80a4bfd824a15132a161/chart/templates/cleanup/cleanup-cronjob.yaml#L64
   I think it won't be a big problem to add this script as a command to the CLI.
   https://github.com/apache/airflow/blob/3970bfad4c878d99adce80a4bfd824a15132a161/airflow/cli/cli_parser.py#L1276
   What do you think about it?


----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



[GitHub] [airflow] boring-cyborg[bot] commented on issue #11146: Cleanup CronJob fails

Posted by GitBox <gi...@apache.org>.
boring-cyborg[bot] commented on issue #11146:
URL: https://github.com/apache/airflow/issues/11146#issuecomment-698841098


   Thanks for opening your first issue here! Be sure to follow the issue template!
   


----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



[GitHub] [airflow] alikhtag edited a comment on issue #11146: Cleanup CronJob fails

Posted by GitBox <gi...@apache.org>.
alikhtag edited a comment on issue #11146:
URL: https://github.com/apache/airflow/issues/11146#issuecomment-698843483


   Found something in `github`, I assume these scripts were not migrated when `Helm` chart was donated by `Astronomer` ? 
   https://github.com/astronomer/astronomer-airflow-scripts
   Is it possible to add them? If not, then this `CronJob` may have to be removed. 
   


----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



[GitHub] [airflow] alikhtag commented on issue #11146: Cleanup CronJob fails

Posted by GitBox <gi...@apache.org>.
alikhtag commented on issue #11146:
URL: https://github.com/apache/airflow/issues/11146#issuecomment-698843483


   Found something in `github`, I assume these scripts were not migrated when `Helm` chart was donated by `Astronomer` ? 
   https://github.com/astronomer/astronomer-airflow-scripts
   


----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



[GitHub] [airflow] alikhtag edited a comment on issue #11146: Cleanup CronJob fails

Posted by GitBox <gi...@apache.org>.
alikhtag edited a comment on issue #11146:
URL: https://github.com/apache/airflow/issues/11146#issuecomment-698843483


   Found something in `github`, I assume these scripts were not migrated when `Helm` chart was donated by `Astronomer` ? 
   https://github.com/astronomer/astronomer-airflow-scripts
   Is it possible to add them? If not, then this `CronJob` may have to be removed. 
   


----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



[GitHub] [airflow] mik-laj commented on issue #11146: Cleanup CronJob fails

Posted by GitBox <gi...@apache.org>.
mik-laj commented on issue #11146:
URL: https://github.com/apache/airflow/issues/11146#issuecomment-699045201


   @kaxil I assigned you to this issue. Is it ok?


----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org