pgvishnuram commented on PR #46003:
URL: https://github.com/apache/airflow/pull/46003#issuecomment-2615982556

   > Tests are failing:
   > 
   > ```
   > 
   > =========================== short test summary info 
============================
   > FAILED 
helm_tests/airflow_core/test_dag_processor.py::TestDagProcessorLogGroomer::test_log_groomer_collector_custom_env
 - tests.charts.helm_template_generator.HelmFailedError: Helm command failed. 
Args: (1, ['helm', 'template', 'release-name', '/opt/airflow/chart', 
'--values', '/tmp/tmpyt3ly0zi', '--kube-version', '1.29.1', '--namespace', 
'default', '--show-only', 
'templates/dag-processor/dag-processor-deployment.yaml'], b'', b"Error: values 
don't meet the specifications of the schema(s) in the following 
chart(s):\nairflow:\n- dagProcessor.logGroomerSidecar.env.0: name is 
required\n- dagProcessor.logGroomerSidecar.env.0: Additional property name: is 
not allowed\n- dagProcessor.logGroomerSidecar.env.1: name is required\n- 
dagProcessor.logGroomerSidecar.env.1: Additional property name: is not 
allowed\n\n")
   > Stderr: 
   > Error: values don't meet the specifications of the schema(s) in the 
following chart(s):
   > airflow:
   > - dagProcessor.logGroomerSidecar.env.0: name is required
   > - dagProcessor.logGroomerSidecar.env.0: Additional property name: is not 
allowed
   > - dagProcessor.logGroomerSidecar.env.1: name is required
   > - dagProcessor.logGroomerSidecar.env.1: Additional property name: is not 
allowed
   > FAILED 
helm_tests/airflow_core/test_dag_processor.py::TestDagProcessorLogGroomer::test_log_groomer_retention_days_overrides[None-None]
 - AssertionError
   > FAILED 
helm_tests/airflow_core/test_scheduler.py::TestSchedulerLogGroomer::test_log_groomer_collector_custom_env
 - AssertionError
   > FAILED 
helm_tests/airflow_core/test_scheduler.py::TestSchedulerLogGroomer::test_log_groomer_retention_days_overrides[None-None]
 - AssertionError
   > FAILED 
helm_tests/airflow_core/test_triggerer.py::TestTriggererLogGroomer::test_log_groomer_retention_days_overrides[None-None]
 - AssertionError
   > FAILED 
helm_tests/airflow_core/test_triggerer.py::TestTriggererLogGroomer::test_log_groomer_collector_custom_env
 - AssertionError
   > FAILED 
helm_tests/airflow_core/test_worker.py::TestWorkerLogGroomer::test_log_groomer_collector_custom_env
 - AssertionError
   > FAILED 
helm_tests/airflow_core/test_worker.py::TestWorkerLogGroomer::test_log_groomer_retention_days_overrides[None-None]
 - AssertionError
   > ================== 8 failed, 469 passed, 5 warnings in 57.85s 
==================
   > ```
   
   @eladkal there seems to be a typo in the pytest it will fixed in the new 
commits thanks


-- 
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: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to