dimberman commented on a change in pull request #11533:
URL: https://github.com/apache/airflow/pull/11533#discussion_r504955240
##########
File path: chart/templates/create-user-job.yaml
##########
@@ -57,14 +57,6 @@ spec:
- name: {{ template "registry_secret" . }}
{{- end }}
containers:
- - name: run-airflow-migrations
Review comment:
@mik-laj this code is already tested. If the migrations don't happen,
the k8s tests never launch. Also we're not adding any code, just moving code
around. I'm glad to consider tests if you have an idea, but AFAIK this
deployment IS tested as we have system tests.
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
[email protected]
