mihirduvedi commented on PR #71987: URL: https://github.com/apache/airflow/pull/71987#issuecomment-6427387112
Refreshed this PR against current `main` and addressed the four review threads: removed the separate label test module and news fragment, reverted the unrelated test changes, and included the recommended name, instance, and version labels. The version assertions now read `Chart.yaml` rather than hardcoding the application version. The [fork CI run](https://github.com/mihirduvedi/airflow/actions/runs/40392890113) passed all 2,280 Helm tests across nine groups with Kubernetes 1.31.12, Helm release checks, static checks, the Helm documentation build, and the normal and special core test matrices. The overall fork run is still red because of integration setup failures: [Kubernetes tests](https://github.com/mihirduvedi/airflow/actions/runs/40392890113/job/121836493285) cannot find the expected base PROD image before exercising the chart, and [PROD e2e tests](https://github.com/mihirduvedi/airflow/actions/runs/40392890113/job/121836493757) fail during image-pull setup. These are not reported as passing integration tests. This is ready for another review. Could a maintainer approve the upstream [Tests (AMD)](https://github.com/apache/airflow/actions/runs/40392811537) and [CodeQL](https://github.com/apache/airflow/actions/runs/40392810292) runs so project CI can complete validation? --- Drafted-by: Codex (GPT-6) (no human review before posting) -- 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]
