Skip to content

[chart/v1-2x-test] Add optional OTel service to the Airflow Helm Chart (#64902)#66465

Merged
jscheffl merged 1 commit into
apache:chart/v1-2x-testfrom
xBis7:backport-535e3cc-chart/v1-2x-test
May 7, 2026
Merged

[chart/v1-2x-test] Add optional OTel service to the Airflow Helm Chart (#64902)#66465
jscheffl merged 1 commit into
apache:chart/v1-2x-testfrom
xBis7:backport-535e3cc-chart/v1-2x-test

Conversation

@xBis7

@xBis7 xBis7 commented May 6, 2026 •
edited
Loading

Copy link
Copy Markdown
Contributor

Manual backport because of conflicts.

I tested it manually and everything still works.

Was generative AI tooling used to co-author this PR?
  • Yes (please specify the tool below)
  • No
  • Read the Pull Request Guidelines for more information. Note: commit author/co-author name and email in commits become permanently public when merged.
  • For fundamental code changes, an Airflow Improvement Proposal (AIP) is needed.
  • When adding dependency, check compliance with the ASF 3rd Party License Policy.
  • For significant user-facing changes create newsfragment: {pr_number}.significant.rst, in airflow-core/newsfragments. You can add this file in a follow-up commit after the PR is created so you know the PR number.

@xBis7

xBis7 commented May 6, 2026

Copy link
Copy Markdown
Contributor Author

@Miretpl @jscheffl This is the PR for the manual backport. Can you take a look?

@jscheffl

jscheffl commented May 6, 2026

Copy link
Copy Markdown
Contributor

Oh, CI is not (correctly) configured for the maintenance branch, so ran tests manually. Unfortunaltely helm tests fail:

breeze testing helm-tests:

________________________________________________ TestBaseChartTest.test_basic_deployment_with_standalone_dag_processor[2.11.0] ________________________________________________

self = <helm_tests.airflow_aux.test_basic_helm_chart.TestBaseChartTest object at 0x7b01d940d2a0>, version = '2.11.0'

    @pytest.mark.parametrize("version", ["2.11.0", "3.0.0", "default"])
    def test_basic_deployment_with_standalone_dag_processor(self, version):
        k8s_objects = render_chart(
            "test-basic",
            self._get_values_with_version(
                values={
                    "chart": {
                        "metadata": "AA",
                    },
                    "labels": {"test-label": "TEST-VALUE"},
                    "fullnameOverride": "test-basic",
                    "dagProcessor": {"enabled": True},
                },
                version=version,
            ),
        )
        list_of_kind_names_tuples = {
            (k8s_object["kind"], k8s_object["metadata"]["name"]) for k8s_object in k8s_objects
        }
        expected = {
            ("ServiceAccount", "test-basic-create-user-job"),
            ("ServiceAccount", "test-basic-migrate-database-job"),
            ("ServiceAccount", "test-basic-redis"),
            ("ServiceAccount", "test-basic-scheduler"),
            ("ServiceAccount", "test-basic-statsd"),
            ("ServiceAccount", "test-basic-triggerer"),
            ("ServiceAccount", "test-basic-dag-processor"),
            ("ServiceAccount", "test-basic-worker"),
            ("Secret", "test-basic-metadata"),
            ("Secret", "test-basic-broker-url"),
            ("Secret", "test-basic-fernet-key"),
            ("Secret", "test-basic-postgresql"),
            ("Secret", "test-basic-redis-password"),
            ("ConfigMap", "test-basic-config"),
            ("ConfigMap", "test-basic-statsd"),
            ("Role", "test-basic-pod-launcher-role"),
            ("Role", "test-basic-pod-log-reader-role"),
            ("RoleBinding", "test-basic-pod-launcher-rolebinding"),
            ("RoleBinding", "test-basic-pod-log-reader-rolebinding"),
            ("Service", "test-basic-postgresql-hl"),
            ("Service", "test-basic-postgresql"),
            ("Service", "test-basic-redis"),
            ("Service", "test-basic-statsd"),
            ("Service", "test-basic-triggerer"),
            ("Service", "test-basic-worker"),
            ("Deployment", "test-basic-scheduler"),
            ("Deployment", "test-basic-statsd"),
            ("StatefulSet", "test-basic-triggerer"),
            ("Deployment", "test-basic-dag-processor"),
            ("StatefulSet", "test-basic-postgresql"),
            ("StatefulSet", "test-basic-redis"),
            ("StatefulSet", "test-basic-worker"),
            ("Job", "test-basic-create-user"),
            ("Job", "test-basic-run-airflow-migrations"),
        }
        if self._is_airflow_3_or_above(version):
            expected.update(
                {
                    ("Deployment", "test-basic-api-server"),
                    ("Service", "test-basic-api-server"),
                    ("ServiceAccount", "test-basic-api-server"),
                    ("Secret", "test-basic-api-secret-key"),
                    ("Secret", "test-basic-jwt-secret"),
                }
            )
        else:
            expected.update(
                {
                    ("Service", "test-basic-webserver"),
                    ("Deployment", "test-basic-webserver"),
                    ("ServiceAccount", "test-basic-webserver"),
                    ("Secret", "test-basic-webserver-secret-key"),
                }
            )
>       assert list_of_kind_names_tuples == expected
E       AssertionError: assert equals failed
E         set([                                                                            set([                                                                           
E           ('ConfigMap', 'test-basic-config'),                                              ('ConfigMap', 'test-basic-config'),                                           
E           ('ConfigMap', 'test-basic-otel-collector'),                                                                                                                    
E           ('ConfigMap', 'test-basic-statsd'),                                              ('ConfigMap', 'test-basic-statsd'),                                           
E           ('Deployment', 'test-basic-dag-processor'),                                      ('Deployment', 'test-basic-dag-processor'),                                   
E           ('Deployment', 'test-basic-otel-collector'),                                                                                                                   
E           ('Deployment', 'test-basic-scheduler'),                                          ('Deployment', 'test-basic-scheduler'),                                       
E           ('Deployment', 'test-basic-statsd'),                                             ('Deployment', 'test-basic-statsd'),                                          
E           ('Deployment', 'test-basic-webserver'),                                          ('Deployment', 'test-basic-webserver'),                                       
E           ('Job', 'test-basic-create-user'),                                               ('Job', 'test-basic-create-user'),                                            
E           ('Job', 'test-basic-run-airflow-migrations'),                                    ('Job', 'test-basic-run-airflow-migrations'),                                 
E           ('Role', 'test-basic-pod-launcher-role'),                                        ('Role', 'test-basic-pod-launcher-role'),                                     
E           ('Role', 'test-basic-pod-log-reader-role'),                                      ('Role', 'test-basic-pod-log-reader-role'),                                   
E           ('RoleBinding', 'test-basic-pod-launcher-rolebinding'),                          ('RoleBinding', 'test-basic-pod-launcher-rolebinding'),                       
E           ('RoleBinding', 'test-basic-pod-log-reader-rolebinding'),                        ('RoleBinding', 'test-basic-pod-log-reader-rolebinding'),                     
E           ('Secret', 'test-basic-broker-url'),                                             ('Secret', 'test-basic-broker-url'),                                          
E           ('Secret', 'test-basic-fernet-key'),                                             ('Secret', 'test-basic-fernet-key'),                                          
E           ('Secret', 'test-basic-metadata'),                                               ('Secret', 'test-basic-metadata'),                                            
E           ('Secret', 'test-basic-postgresql'),                                             ('Secret', 'test-basic-postgresql'),                                          
E           ('Secret', 'test-basic-redis-password'),                                         ('Secret', 'test-basic-redis-password'),                                      
E           ('Secret', 'test-basic-webserver-secret-key'),                                   ('Secret', 'test-basic-webserver-secret-key'),                                
E           ('Service', 'test-basic-otel-collector'),                                                                                                                      
E           ('Service', 'test-basic-postgresql'),                                            ('Service', 'test-basic-postgresql'),                                         
E           ('Service', 'test-basic-postgresql-hl'),                                         ('Service', 'test-basic-postgresql-hl'),                                      
E           ('Service', 'test-basic-redis'),                                                 ('Service', 'test-basic-redis'),                                              
E           ('Service', 'test-basic-statsd'),                                                ('Service', 'test-basic-statsd'),                                             
E           ('Service', 'test-basic-triggerer'),                                             ('Service', 'test-basic-triggerer'),                                          
E           ('Service', 'test-basic-webserver'),                                             ('Service', 'test-basic-webserver'),                                          
E           ('Service', 'test-basic-worker'),                                                ('Service', 'test-basic-worker'),                                             
E           ('ServiceAccount', 'test-basic-create-user-job'),                                ('ServiceAccount', 'test-basic-create-user-job'),                             
E           ('ServiceAccount', 'test-basic-dag-processor'),                                  ('ServiceAccount', 'test-basic-dag-processor'),                               
E           ('ServiceAccount', 'test-basic-migrate-database-job'),                           ('ServiceAccount', 'test-basic-migrate-database-job'),                        
E           ('ServiceAccount', 'test-basic-otel-collector'),                                                                                                               
E           ('ServiceAccount', 'test-basic-redis'),                                          ('ServiceAccount', 'test-basic-redis'),                                       
E           ('ServiceAccount', 'test-basic-scheduler'),                                      ('ServiceAccount', 'test-basic-scheduler'),                                   
E           ('ServiceAccount', 'test-basic-statsd'),                                         ('ServiceAccount', 'test-basic-statsd'),                                      
E           ('ServiceAccount', 'test-basic-triggerer'),                                      ('ServiceAccount', 'test-basic-triggerer'),                                   
E           ('ServiceAccount', 'test-basic-webserver'),                                      ('ServiceAccount', 'test-basic-webserver'),                                   
E           ('ServiceAccount', 'test-basic-worker'),                                         ('ServiceAccount', 'test-basic-worker'),                                      
E           ('StatefulSet', 'test-basic-postgresql'),                                        ('StatefulSet', 'test-basic-postgresql'),                                     
E           ('StatefulSet', 'test-basic-redis'),                                             ('StatefulSet', 'test-basic-redis'),                                          
E           ('StatefulSet', 'test-basic-triggerer'),                                         ('StatefulSet', 'test-basic-triggerer'),                                      
E           ('StatefulSet', 'test-basic-worker'),                                            ('StatefulSet', 'test-basic-worker'),                                         
E         ])                                                                               ])

helm-tests/tests/helm_tests/airflow_aux/test_basic_helm_chart.py:289: AssertionError
________________________________________________ TestBaseChartTest.test_basic_deployment_with_standalone_dag_processor[3.0.0] _________________________________________________

self = <helm_tests.airflow_aux.test_basic_helm_chart.TestBaseChartTest object at 0x7b01d940d300>, version = '3.0.0'

    @pytest.mark.parametrize("version", ["2.11.0", "3.0.0", "default"])
    def test_basic_deployment_with_standalone_dag_processor(self, version):
        k8s_objects = render_chart(
            "test-basic",
            self._get_values_with_version(
                values={
                    "chart": {
                        "metadata": "AA",
                    },
                    "labels": {"test-label": "TEST-VALUE"},
                    "fullnameOverride": "test-basic",
                    "dagProcessor": {"enabled": True},
                },
                version=version,
            ),
        )
        list_of_kind_names_tuples = {
            (k8s_object["kind"], k8s_object["metadata"]["name"]) for k8s_object in k8s_objects
        }
        expected = {
            ("ServiceAccount", "test-basic-create-user-job"),
            ("ServiceAccount", "test-basic-migrate-database-job"),
            ("ServiceAccount", "test-basic-redis"),
            ("ServiceAccount", "test-basic-scheduler"),
            ("ServiceAccount", "test-basic-statsd"),
            ("ServiceAccount", "test-basic-triggerer"),
            ("ServiceAccount", "test-basic-dag-processor"),
            ("ServiceAccount", "test-basic-worker"),
            ("Secret", "test-basic-metadata"),
            ("Secret", "test-basic-broker-url"),
            ("Secret", "test-basic-fernet-key"),
            ("Secret", "test-basic-postgresql"),
            ("Secret", "test-basic-redis-password"),
            ("ConfigMap", "test-basic-config"),
            ("ConfigMap", "test-basic-statsd"),
            ("Role", "test-basic-pod-launcher-role"),
            ("Role", "test-basic-pod-log-reader-role"),
            ("RoleBinding", "test-basic-pod-launcher-rolebinding"),
            ("RoleBinding", "test-basic-pod-log-reader-rolebinding"),
            ("Service", "test-basic-postgresql-hl"),
            ("Service", "test-basic-postgresql"),
            ("Service", "test-basic-redis"),
            ("Service", "test-basic-statsd"),
            ("Service", "test-basic-triggerer"),
            ("Service", "test-basic-worker"),
            ("Deployment", "test-basic-scheduler"),
            ("Deployment", "test-basic-statsd"),
            ("StatefulSet", "test-basic-triggerer"),
            ("Deployment", "test-basic-dag-processor"),
            ("StatefulSet", "test-basic-postgresql"),
            ("StatefulSet", "test-basic-redis"),
            ("StatefulSet", "test-basic-worker"),
            ("Job", "test-basic-create-user"),
            ("Job", "test-basic-run-airflow-migrations"),
        }
        if self._is_airflow_3_or_above(version):
            expected.update(
                {
                    ("Deployment", "test-basic-api-server"),
                    ("Service", "test-basic-api-server"),
                    ("ServiceAccount", "test-basic-api-server"),
                    ("Secret", "test-basic-api-secret-key"),
                    ("Secret", "test-basic-jwt-secret"),
                }
            )
        else:
            expected.update(
                {
                    ("Service", "test-basic-webserver"),
                    ("Deployment", "test-basic-webserver"),
                    ("ServiceAccount", "test-basic-webserver"),
                    ("Secret", "test-basic-webserver-secret-key"),
                }
            )
>       assert list_of_kind_names_tuples == expected
E       AssertionError: assert equals failed
E         set([                                                                            set([                                                                           
E           ('ConfigMap', 'test-basic-config'),                                              ('ConfigMap', 'test-basic-config'),                                           
E           ('ConfigMap', 'test-basic-otel-collector'),                                                                                                                    
E           ('ConfigMap', 'test-basic-statsd'),                                              ('ConfigMap', 'test-basic-statsd'),                                           
E           ('Deployment', 'test-basic-api-server'),                                         ('Deployment', 'test-basic-api-server'),                                      
E           ('Deployment', 'test-basic-dag-processor'),                                      ('Deployment', 'test-basic-dag-processor'),                                   
E           ('Deployment', 'test-basic-otel-collector'),                                                                                                                   
E           ('Deployment', 'test-basic-scheduler'),                                          ('Deployment', 'test-basic-scheduler'),                                       
E           ('Deployment', 'test-basic-statsd'),                                             ('Deployment', 'test-basic-statsd'),                                          
E           ('Job', 'test-basic-create-user'),                                               ('Job', 'test-basic-create-user'),                                            
E           ('Job', 'test-basic-run-airflow-migrations'),                                    ('Job', 'test-basic-run-airflow-migrations'),                                 
E           ('Role', 'test-basic-pod-launcher-role'),                                        ('Role', 'test-basic-pod-launcher-role'),                                     
E           ('Role', 'test-basic-pod-log-reader-role'),                                      ('Role', 'test-basic-pod-log-reader-role'),                                   
E           ('RoleBinding', 'test-basic-pod-launcher-rolebinding'),                          ('RoleBinding', 'test-basic-pod-launcher-rolebinding'),                       
E           ('RoleBinding', 'test-basic-pod-log-reader-rolebinding'),                        ('RoleBinding', 'test-basic-pod-log-reader-rolebinding'),                     
E           ('Secret', 'test-basic-api-secret-key'),                                         ('Secret', 'test-basic-api-secret-key'),                                      
E           ('Secret', 'test-basic-broker-url'),                                             ('Secret', 'test-basic-broker-url'),                                          
E           ('Secret', 'test-basic-fernet-key'),                                             ('Secret', 'test-basic-fernet-key'),                                          
E           ('Secret', 'test-basic-jwt-secret'),                                             ('Secret', 'test-basic-jwt-secret'),                                          
E           ('Secret', 'test-basic-metadata'),                                               ('Secret', 'test-basic-metadata'),                                            
E           ('Secret', 'test-basic-postgresql'),                                             ('Secret', 'test-basic-postgresql'),                                          
E           ('Secret', 'test-basic-redis-password'),                                         ('Secret', 'test-basic-redis-password'),                                      
E           ('Service', 'test-basic-api-server'),                                            ('Service', 'test-basic-api-server'),                                         
E           ('Service', 'test-basic-otel-collector'),                                                                                                                      
E           ('Service', 'test-basic-postgresql'),                                            ('Service', 'test-basic-postgresql'),                                         
E           ('Service', 'test-basic-postgresql-hl'),                                         ('Service', 'test-basic-postgresql-hl'),                                      
E           ('Service', 'test-basic-redis'),                                                 ('Service', 'test-basic-redis'),                                              
E           ('Service', 'test-basic-statsd'),                                                ('Service', 'test-basic-statsd'),                                             
E           ('Service', 'test-basic-triggerer'),                                             ('Service', 'test-basic-triggerer'),                                          
E           ('Service', 'test-basic-worker'),                                                ('Service', 'test-basic-worker'),                                             
E           ('ServiceAccount', 'test-basic-api-server'),                                     ('ServiceAccount', 'test-basic-api-server'),                                  
E           ('ServiceAccount', 'test-basic-create-user-job'),                                ('ServiceAccount', 'test-basic-create-user-job'),                             
E           ('ServiceAccount', 'test-basic-dag-processor'),                                  ('ServiceAccount', 'test-basic-dag-processor'),                               
E           ('ServiceAccount', 'test-basic-migrate-database-job'),                           ('ServiceAccount', 'test-basic-migrate-database-job'),                        
E           ('ServiceAccount', 'test-basic-otel-collector'),                                                                                                               
E           ('ServiceAccount', 'test-basic-redis'),                                          ('ServiceAccount', 'test-basic-redis'),                                       
E           ('ServiceAccount', 'test-basic-scheduler'),                                      ('ServiceAccount', 'test-basic-scheduler'),                                   
E           ('ServiceAccount', 'test-basic-statsd'),                                         ('ServiceAccount', 'test-basic-statsd'),                                      
E           ('ServiceAccount', 'test-basic-triggerer'),                                      ('ServiceAccount', 'test-basic-triggerer'),                                   
E           ('ServiceAccount', 'test-basic-worker'),                                         ('ServiceAccount', 'test-basic-worker'),                                      
E           ('StatefulSet', 'test-basic-postgresql'),                                        ('StatefulSet', 'test-basic-postgresql'),                                     
E           ('StatefulSet', 'test-basic-redis'),                                             ('StatefulSet', 'test-basic-redis'),                                          
E           ('StatefulSet', 'test-basic-triggerer'),                                         ('StatefulSet', 'test-basic-triggerer'),                                      
E           ('StatefulSet', 'test-basic-worker'),                                            ('StatefulSet', 'test-basic-worker'),                                         
E         ])                                                                               ])

helm-tests/tests/helm_tests/airflow_aux/test_basic_helm_chart.py:289: AssertionError
_______________________________________________ TestBaseChartTest.test_basic_deployment_with_standalone_dag_processor[default] ________________________________________________

self = <helm_tests.airflow_aux.test_basic_helm_chart.TestBaseChartTest object at 0x7b01d940d5a0>, version = 'default'

    @pytest.mark.parametrize("version", ["2.11.0", "3.0.0", "default"])
    def test_basic_deployment_with_standalone_dag_processor(self, version):
        k8s_objects = render_chart(
            "test-basic",
            self._get_values_with_version(
                values={
                    "chart": {
                        "metadata": "AA",
                    },
                    "labels": {"test-label": "TEST-VALUE"},
                    "fullnameOverride": "test-basic",
                    "dagProcessor": {"enabled": True},
                },
                version=version,
            ),
        )
        list_of_kind_names_tuples = {
            (k8s_object["kind"], k8s_object["metadata"]["name"]) for k8s_object in k8s_objects
        }
        expected = {
            ("ServiceAccount", "test-basic-create-user-job"),
            ("ServiceAccount", "test-basic-migrate-database-job"),
            ("ServiceAccount", "test-basic-redis"),
            ("ServiceAccount", "test-basic-scheduler"),
            ("ServiceAccount", "test-basic-statsd"),
            ("ServiceAccount", "test-basic-triggerer"),
            ("ServiceAccount", "test-basic-dag-processor"),
            ("ServiceAccount", "test-basic-worker"),
            ("Secret", "test-basic-metadata"),
            ("Secret", "test-basic-broker-url"),
            ("Secret", "test-basic-fernet-key"),
            ("Secret", "test-basic-postgresql"),
            ("Secret", "test-basic-redis-password"),
            ("ConfigMap", "test-basic-config"),
            ("ConfigMap", "test-basic-statsd"),
            ("Role", "test-basic-pod-launcher-role"),
            ("Role", "test-basic-pod-log-reader-role"),
            ("RoleBinding", "test-basic-pod-launcher-rolebinding"),
            ("RoleBinding", "test-basic-pod-log-reader-rolebinding"),
            ("Service", "test-basic-postgresql-hl"),
            ("Service", "test-basic-postgresql"),
            ("Service", "test-basic-redis"),
            ("Service", "test-basic-statsd"),
            ("Service", "test-basic-triggerer"),
            ("Service", "test-basic-worker"),
            ("Deployment", "test-basic-scheduler"),
            ("Deployment", "test-basic-statsd"),
            ("StatefulSet", "test-basic-triggerer"),
            ("Deployment", "test-basic-dag-processor"),
            ("StatefulSet", "test-basic-postgresql"),
            ("StatefulSet", "test-basic-redis"),
            ("StatefulSet", "test-basic-worker"),
            ("Job", "test-basic-create-user"),
            ("Job", "test-basic-run-airflow-migrations"),
        }
        if self._is_airflow_3_or_above(version):
            expected.update(
                {
                    ("Deployment", "test-basic-api-server"),
                    ("Service", "test-basic-api-server"),
                    ("ServiceAccount", "test-basic-api-server"),
                    ("Secret", "test-basic-api-secret-key"),
                    ("Secret", "test-basic-jwt-secret"),
                }
            )
        else:
            expected.update(
                {
                    ("Service", "test-basic-webserver"),
                    ("Deployment", "test-basic-webserver"),
                    ("ServiceAccount", "test-basic-webserver"),
                    ("Secret", "test-basic-webserver-secret-key"),
                }
            )
>       assert list_of_kind_names_tuples == expected
E       AssertionError: assert equals failed
E         set([                                                                            set([                                                                           
E           ('ConfigMap', 'test-basic-config'),                                              ('ConfigMap', 'test-basic-config'),                                           
E           ('ConfigMap', 'test-basic-otel-collector'),                                                                                                                    
E           ('ConfigMap', 'test-basic-statsd'),                                              ('ConfigMap', 'test-basic-statsd'),                                           
E           ('Deployment', 'test-basic-api-server'),                                         ('Deployment', 'test-basic-api-server'),                                      
E           ('Deployment', 'test-basic-dag-processor'),                                      ('Deployment', 'test-basic-dag-processor'),                                   
E           ('Deployment', 'test-basic-otel-collector'),                                                                                                                   
E           ('Deployment', 'test-basic-scheduler'),                                          ('Deployment', 'test-basic-scheduler'),                                       
E           ('Deployment', 'test-basic-statsd'),                                             ('Deployment', 'test-basic-statsd'),                                          
E           ('Job', 'test-basic-create-user'),                                               ('Job', 'test-basic-create-user'),                                            
E           ('Job', 'test-basic-run-airflow-migrations'),                                    ('Job', 'test-basic-run-airflow-migrations'),                                 
E           ('Role', 'test-basic-pod-launcher-role'),                                        ('Role', 'test-basic-pod-launcher-role'),                                     
E           ('Role', 'test-basic-pod-log-reader-role'),                                      ('Role', 'test-basic-pod-log-reader-role'),                                   
E           ('RoleBinding', 'test-basic-pod-launcher-rolebinding'),                          ('RoleBinding', 'test-basic-pod-launcher-rolebinding'),                       
E           ('RoleBinding', 'test-basic-pod-log-reader-rolebinding'),                        ('RoleBinding', 'test-basic-pod-log-reader-rolebinding'),                     
E           ('Secret', 'test-basic-api-secret-key'),                                         ('Secret', 'test-basic-api-secret-key'),                                      
E           ('Secret', 'test-basic-broker-url'),                                             ('Secret', 'test-basic-broker-url'),                                          
E           ('Secret', 'test-basic-fernet-key'),                                             ('Secret', 'test-basic-fernet-key'),                                          
E           ('Secret', 'test-basic-jwt-secret'),                                             ('Secret', 'test-basic-jwt-secret'),                                          
E           ('Secret', 'test-basic-metadata'),                                               ('Secret', 'test-basic-metadata'),                                            
E           ('Secret', 'test-basic-postgresql'),                                             ('Secret', 'test-basic-postgresql'),                                          
E           ('Secret', 'test-basic-redis-password'),                                         ('Secret', 'test-basic-redis-password'),                                      
E           ('Service', 'test-basic-api-server'),                                            ('Service', 'test-basic-api-server'),                                         
E           ('Service', 'test-basic-otel-collector'),                                                                                                                      
E           ('Service', 'test-basic-postgresql'),                                            ('Service', 'test-basic-postgresql'),                                         
E           ('Service', 'test-basic-postgresql-hl'),                                         ('Service', 'test-basic-postgresql-hl'),                                      
E           ('Service', 'test-basic-redis'),                                                 ('Service', 'test-basic-redis'),                                              
E           ('Service', 'test-basic-statsd'),                                                ('Service', 'test-basic-statsd'),                                             
E           ('Service', 'test-basic-triggerer'),                                             ('Service', 'test-basic-triggerer'),                                          
E           ('Service', 'test-basic-worker'),                                                ('Service', 'test-basic-worker'),                                             
E           ('ServiceAccount', 'test-basic-api-server'),                                     ('ServiceAccount', 'test-basic-api-server'),                                  
E           ('ServiceAccount', 'test-basic-create-user-job'),                                ('ServiceAccount', 'test-basic-create-user-job'),                             
E           ('ServiceAccount', 'test-basic-dag-processor'),                                  ('ServiceAccount', 'test-basic-dag-processor'),                               
E           ('ServiceAccount', 'test-basic-migrate-database-job'),                           ('ServiceAccount', 'test-basic-migrate-database-job'),                        
E           ('ServiceAccount', 'test-basic-otel-collector'),                                                                                                               
E           ('ServiceAccount', 'test-basic-redis'),                                          ('ServiceAccount', 'test-basic-redis'),                                       
E           ('ServiceAccount', 'test-basic-scheduler'),                                      ('ServiceAccount', 'test-basic-scheduler'),                                   
E           ('ServiceAccount', 'test-basic-statsd'),                                         ('ServiceAccount', 'test-basic-statsd'),                                      
E           ('ServiceAccount', 'test-basic-triggerer'),                                      ('ServiceAccount', 'test-basic-triggerer'),                                   
E           ('ServiceAccount', 'test-basic-worker'),                                         ('ServiceAccount', 'test-basic-worker'),                                      
E           ('StatefulSet', 'test-basic-postgresql'),                                        ('StatefulSet', 'test-basic-postgresql'),                                     
E           ('StatefulSet', 'test-basic-redis'),                                             ('StatefulSet', 'test-basic-redis'),                                          
E           ('StatefulSet', 'test-basic-triggerer'),                                         ('StatefulSet', 'test-basic-triggerer'),                                      
E           ('StatefulSet', 'test-basic-worker'),                                            ('StatefulSet', 'test-basic-worker'),                                         
E         ])                                                                               ])

helm-tests/tests/helm_tests/airflow_aux/test_basic_helm_chart.py:289: AssertionError

@xBis7
Add optional OTel service to the Airflow Helm Chart (apache#64902)
a2cea95 
* add otel to helm chart

* use Kustomize for grafana, jaeger, prometheus

* enable specific service per flag + unit test

* remove grafana, jaeger and prometheus kustomization logic

* traces enabled and metrics disabled, by default

* remove otelCollector.enabled flag

* add statsd comments about otel metrics overriding the config

* make OTEL_METRIC_EXPORT_INTERVAL configurable and provide default value + entry in the values.schema.json

* remove hardcoded value for metrics otel_port in values.yaml

* add option to override the configmap

* add otelCollector.args and make the config.yml file as the default argument

* rename extraAnnotations to annotations in otel-collector-service.yaml

* parameterize the readiness and liveness probe values

* remove prometheus from the configmap

* update the default value for OTEL_TRACES_EXPORTER

* fix tests in airflow_aux + otel-collector-serviceaccount.yaml

* fix spellcheck errors in docs

* fix tests in security

* otel collector unit tests + networkpolicy file

* values.schema.json cleanup

* add a minimum to all integer configs in values.schema.json

* fix heading comments

* change config default to ~ from empty string

* fix static check error
@xBis7 xBis7 force-pushed the backport-535e3cc-chart/v1-2x-test branch from b2630a0 to a2cea95 Compare May 7, 2026 06:46
@xBis7

xBis7 commented May 7, 2026

Copy link
Copy Markdown
Contributor Author

@jscheffl Can you check again? It should have been fixed now.

@jscheffl

jscheffl commented May 7, 2026

Copy link
Copy Markdown
Contributor

@jscheffl Can you check again? It should have been fixed now.

Thanks! Tests pass now!

@jscheffl jscheffl merged commit 2175eb3 into apache:chart/v1-2x-test May 7, 2026
2 checks passed
@xBis7 xBis7 deleted the backport-535e3cc-chart/v1-2x-test branch May 8, 2026 11:57
@potiuk potiuk mentioned this pull request Jun 1, 2026
Closed
29 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@Miretpl Miretpl Miretpl approved these changes
@jscheffl jscheffl jscheffl approved these changes
@dstandish dstandish Awaiting requested review from dstandish
@jedcunningham jedcunningham Awaiting requested review from jedcunningham jedcunningham is a code owner
@potiuk potiuk Awaiting requested review from potiuk potiuk is a code owner
@ashb ashb Awaiting requested review from ashb ashb is a code owner
@gopidesupavan gopidesupavan Awaiting requested review from gopidesupavan gopidesupavan is a code owner
@amoghrajesh amoghrajesh Awaiting requested review from amoghrajesh amoghrajesh is a code owner
@bugraoz93 bugraoz93 Awaiting requested review from bugraoz93 bugraoz93 is a code owner
@jason810496 jason810496 Awaiting requested review from jason810496 jason810496 is a code owner
@ephraimbuddy ephraimbuddy Awaiting requested review from ephraimbuddy ephraimbuddy is a code owner
@choo121600 choo121600 Awaiting requested review from choo121600 choo121600 is a code owner
@hussein-awala hussein-awala Awaiting requested review from hussein-awala hussein-awala is a code owner

Assignees

No one assigned

Labels

area:dev-tools area:helm-chart Airflow Helm Chart

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@xBis7 @jscheffl @Miretpl