PreethamSanji commented on code in PR #69703:
URL: https://github.com/apache/airflow/pull/69703#discussion_r3585104943
##########
helm-tests/tests/helm_tests/other/test_git_sync_scheduler.py:
##########
@@ -126,6 +126,44 @@ def test_validate_the_git_sync_container_spec(self):
},
}
+ def test_metrics_port_added_when_metrics_enabled(self):
+ docs = render_chart(
+ values={
+ "executor": "LocalExecutor",
Review Comment:
Done, parametrized the test across the four deployments that can carry the
git-sync sidecar: scheduler (LocalExecutor), workers and triggerer
(CeleryExecutor), and dag-processor (KubernetesExecutor, since with that
executor the sidecar renders there rather than on the scheduler or workers).
--
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]
