jcicchella commented on issue #21478:
URL: https://github.com/apache/superset/issues/21478#issuecomment-1284732979

   I found this error google searching "Additional property annotations is not 
allowed", so I apologize if this might not be the solution to your specific 
issue, but that specific error string is pretty unique and led me here. 
Hopefully, this can help someone else out. 
   
   The issue is that there is some schema file that is validating the helm 
chart. The message is coming from schema validation. I was using the airflow 
chart, but the same principle should apply. In the chart there is a schema file 
like `values.schema.json` that's what is blocking the modification. In the 
airflow example, I was trying to modify the airflow flower helm chart to add 
pod annotations to allow Prometheus to scrape flower. 
   
   If you look at the chart properties you see 
   https://airflow.apache.org/docs/helm-chart/stable/parameters-ref.html#flower
   
   `flower.podAnnotations` `Annotations to add to the Flower pods.`
   
   So you can't override the annotations like you normally would in the values 
file, you need to override the property config. So it looks like this in my 
example: 
   
   `airflow:
     flower:
       # Enable flower.
       # If True, and using CeleryExecutor/CeleryKubernetesExecutor, will 
deploy flower app.
       enabled: True
   
       podAnnotations: {
         prometheus.io/port: "5555",
         prometheus.io/path: "/metrics",
         sidecar.istio.io/inject: "false"    ,    
         prometheus.io/scrape: "true"
   
       }`
   
   It is simply just a change in syntax. I'm not familiar with the superset 
helm chart, but I'm hoping this concept can help you find a solution to your 
issue or someone else who stumbles upon this from a google search. 
   
   
   
   
   
   
   
   
   
   
   


-- 
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]


---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to