[GitHub] [airflow] Taragolis commented on pull request #29087: Unquarantine receive SIGTERM on Task Runner test (second attempt)

2023-01-24 Thread via GitHub


Taragolis commented on PR #29087:
URL: https://github.com/apache/airflow/pull/29087#issuecomment-1402820966

   Yet another rebase


-- 
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.

To unsubscribe, e-mail: commits-unsubscr...@airflow.apache.org

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



[GitHub] [airflow] Taragolis commented on pull request #29087: Unquarantine receive SIGTERM on Task Runner test (second attempt)

2023-01-23 Thread via GitHub


Taragolis commented on PR #29087:
URL: https://github.com/apache/airflow/pull/29087#issuecomment-1400282901

   Rebase again


-- 
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.

To unsubscribe, e-mail: commits-unsubscr...@airflow.apache.org

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



[GitHub] [airflow] Taragolis commented on pull request #29087: Unquarantine receive SIGTERM on Task Runner test (second attempt)

2023-01-22 Thread via GitHub


Taragolis commented on PR #29087:
URL: https://github.com/apache/airflow/pull/29087#issuecomment-1399520928

   Current failure not relevant to this PR, rather than to AIP-44
   
   ```console
   tests/cli/commands/test_internal_api_command.py:181: 
   _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ 
_ _ 
   
   self = 
   ignore_running = False
   
   def _check_processes(self, ignore_running=False):
   # Confirm that internal-api hasn't been launched.
   # pgrep returns exit status 1 if no process matched.
   # Use more specific regexps (^) to avoid matching pytest run when 
running specific method.
   # For instance, we want to be able to do: pytest -k 'gunicorn'
   exit_code_pgrep_internal_api = subprocess.Popen(["pgrep", "-c", 
"-f", "airflow internal-api"]).wait()
   exit_code_pgrep_gunicorn = subprocess.Popen(["pgrep", "-c", "-f", 
"^gunicorn"]).wait()
   if exit_code_pgrep_internal_api != 1 or exit_code_pgrep_gunicorn != 
1:
   subprocess.Popen(["ps", "-ax"]).wait()
   if exit_code_pgrep_internal_api != 1:
   subprocess.Popen(["pkill", "-9", "-f", 
"airflow-internal-api"]).wait()
   if exit_code_pgrep_gunicorn != 1:
   subprocess.Popen(["pkill", "-9", "-f", "^gunicorn"]).wait()
   if not ignore_running:
   >   raise AssertionError(
   "Background processes are running that prevent the test 
from passing successfully."
   )
   E   AssertionError: Background processes are running that 
prevent the test from passing successfully.
   ```
   
   @mhenc if you have a time could you have a look, maybe it is not important 
because I see this error first time
   
   ---
   
   And also see this Warning
   
   ```console
   - Captured stdout call 
-
    _
|__( )_  __/__  /  __
     /| |_  /__  ___/_  /_ __  /_  __ \_ | /| / /
   ___  ___ |  / _  /   _  __/ _  / / /_/ /_ |/ |/ /
_/_/  |_/_/  /_//_//_/  \//|__/
   [2023-01-22T14:10:19.082+] {internal_api_command.py:87} INFO - Running 
the Gunicorn Server with:
   Workers: 4 sync
   Host: 0.0.0.0:9080
   Timeout: 120
   Logfiles: - -
   Access Logformat: None
   =
   [2023-01-22T14:10:19.086+] {dagbag.py:538} INFO - Filling up the DagBag 
from /dev/null
   Please make sure to build the frontend in static/ directory and restart the 
server
   [2023-01-22T14:10:19.456+] {options.py:83} WARNING - The swagger_ui 
directory could not be found.
   Please install connexion with extra install: pip install 
connexion[swagger-ui]
   or provide the path to your local installation by passing 
swagger_path=
   
   [2023-01-22T14:10:19.457+] {options.py:83} WARNING - The swagger_ui 
directory could not be found.
   Please install connexion with extra install: pip install 
connexion[swagger-ui]
   or provide the path to your local installation by passing 
swagger_path=
   ```
   
   @JGoldman110 if you have a time could you have a look maybe we forget to 
change something else?


-- 
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.

To unsubscribe, e-mail: commits-unsubscr...@airflow.apache.org

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