You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@airflow.apache.org by "ikholodkov (via GitHub)" <gi...@apache.org> on 2024/02/28 16:46:14 UTC

[I] migrateDatabaseJob: Additional property labels is not allowed [airflow]

ikholodkov opened a new issue, #37776:
URL: https://github.com/apache/airflow/issues/37776

   ### Official Helm Chart version
   
   1.12.0 (latest released)
   
   ### Apache Airflow version
   
   -
   
   ### Kubernetes Version
   
   -
   
   ### Helm Chart configuration
   
   _No response_
   
   ### Docker Image customizations
   
   _No response_
   
   ### What happened
   
   Trying to check template via _helm template airflow apache-airflow/airflow -f values.yml -f override.yml_
   I got an error `migrateDatabaseJob: Additional property labels is not allowed`.
   I didn't describe migrateDatabaseJob in override.yml
   I guess it may be due to this [PR](https://github.com/apache/airflow/pull/37490)
   
   Error occurs even I have labels section in migrateDatabaseJob, 
   
   ```
   migrateDatabaseJob:
     labels:
       tier: airflow
       component: run-airflow-migrations
   ```
   but disappear when labels described without whitespaces
   ```
   migrateDatabaseJob:
   labels:
     tier: airflow
     component: run-airflow-migrations
   ```
   
   ### What you think should happen instead
   
   _No response_
   
   ### How to reproduce
   
   helm repo add apache-airflow https://airflow.apache.org
   
   Copy [values](https://github.com/apache/airflow/blob/main/chart/values.yaml)
   
   Create override.yml and try to check template helm template airflow apache-airflow/airflow -f values.yml -f override.yml
   
   ### Anything else
   
   _No response_
   
   ### Are you willing to submit PR?
   
   - [ ] Yes I am willing to submit a PR!
   
   ### Code of Conduct
   
   - [X] I agree to follow this project's [Code of Conduct](https://github.com/apache/airflow/blob/main/CODE_OF_CONDUCT.md)
   


-- 
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: commits-unsubscribe@airflow.apache.org.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

Re: [I] migrateDatabaseJob: Additional property labels is not allowed [airflow]

Posted by "ikholodkov (via GitHub)" <gi...@apache.org>.
ikholodkov closed issue #37776: migrateDatabaseJob: Additional property labels is not allowed
URL: https://github.com/apache/airflow/issues/37776


-- 
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: commits-unsubscribe@airflow.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

Re: [I] migrateDatabaseJob: Additional property labels is not allowed [airflow]

Posted by "SamWheating (via GitHub)" <gi...@apache.org>.
SamWheating commented on issue #37776:
URL: https://github.com/apache/airflow/issues/37776#issuecomment-1970134612

   I think that this is because you're using `chart/values.yaml` from `main` while using the latest release of the chart, which is ~3 weeks old and doesn't include those fields.
   
   If you use [the values file from the `helm-chart/1.12.0` release ](https://github.com/apache/airflow/blob/helm-chart/1.12.0/chart/values.yaml) then the issue will resolve itself.
   
   Reproduced and validated the fix like so:
   ```
   > git checkout main
   > helm template airflow apache-airflow/airflow -f chart/values.yaml 
   Error: values don't meet the specifications of the schema(s) in the following chart(s):
   airflow:
   - migrateDatabaseJob: Additional property labels is not allowed
   - flower: Additional property readinessProbe is not allowed
   - flower: Additional property livenessProbe is not allowed
   
   > git checkout helm-chart/1.12.0
   > helm template airflow apache-airflow/airflow -f chart/values.yaml 
   
   ---
   # Source: airflow/templates/jobs/create-user-job-serviceaccount.yaml
   ###########################################
   ## Airflow Create User Job ServiceAccount
   ###########################################
   apiVersion: v1
   kind: ServiceAccount
   automountServiceAccountToken: true
   metadata:
     name: airflow-create-user-job
     ....
   ```


-- 
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: commits-unsubscribe@airflow.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

Re: [I] migrateDatabaseJob: Additional property labels is not allowed [airflow]

Posted by "ikholodkov (via GitHub)" <gi...@apache.org>.
ikholodkov commented on issue #37776:
URL: https://github.com/apache/airflow/issues/37776#issuecomment-1970555598

   @SamWheating sorry, it's my fault. 
   I've checked and values from release (obviously) is OK.
   Thank you for help!
   


-- 
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: commits-unsubscribe@airflow.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org