You are viewing a plain text version of this content. The canonical link for it is here.
Posted to notifications@superset.apache.org by GitBox <gi...@apache.org> on 2022/04/07 14:15:09 UTC

[GitHub] [superset] riazarbi opened a new issue, #19588: Helm upgrade from 0.5.8 to 0.5.9 fails

riazarbi opened a new issue, #19588:
URL: https://github.com/apache/superset/issues/19588

   When upgrading my helm release fro v 0.5.8 to 0.5.9 I get the following error - 
   
   ```
   Error: UPGRADE FAILED: cannot patch "superset-redis-master" with kind StatefulSet: StatefulSet.apps "superset-redis-master" is invalid: spec: Forbidden: updates to statefulset spec for fields other than 'replicas', 'template', and 'updateStrategy' are forbidden
   ```
   
   #### How to reproduce the bug
   
   1. Deploy a helm chart with a v 0.5.8 release
   2. Upgrade the release to v 0.5.9
   
   ### Expected results
   
   The upgrade to complete successfully.
   
   ### Actual results
   
   I get the above error
   
   ### Environment
   
   - browser type and version: Not relevant
   - superset version: `1.4.0`
   - python version: Not relevant
   - node.js version: Not relevant
   - any feature flags active: Not relevant
   
   ### Checklist
   
   Make sure to follow these steps before submitting your issue - thank you!
   
   - [X] I have checked the superset logs for python stacktraces and included it here as text if there are any.
   - [ ] I have reproduced the issue with at least the latest released version of superset. **Not possible**
   - [X] I have checked the issue tracker for the same issue and I haven't found one similar.
   
   ### Additional context
   I've tried to add `--reuse-values` to the helm upgrade command, and I get a different error
   
   ```
   ubuntu-admin@k3s-server-32:~/deployments$ sudo KUBECONFIG=/etc/rancher/k3s/k3s.yaml helm upgrade --install superset superset/superset --values superset2-values.yaml --version=0.5.9 --namespace=superset2 --create-namespace --reuse-values
   coalesce.go:163: warning: skipped value for extraEnv: Not a table.
   Error: UPGRADE FAILED: error validating "": error validating data: ValidationError(StatefulSet.spec.template.spec.containers[0].command): invalid type for io.k8s.api.core.v1.Container.command: got "string", expected "array"
   ```
   
   
   


-- 
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: notifications-unsubscribe@superset.apache.org.apache.org

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


---------------------------------------------------------------------
To unsubscribe, e-mail: notifications-unsubscribe@superset.apache.org
For additional commands, e-mail: notifications-help@superset.apache.org


[GitHub] [superset] riazarbi commented on issue #19588: Helm upgrade from 0.5.8 to 0.5.9 fails

Posted by GitBox <gi...@apache.org>.
riazarbi commented on issue #19588:
URL: https://github.com/apache/superset/issues/19588#issuecomment-1091817366

   Alright, I fixed my own problem in an unsatisfactory kind of way. 
   
   It appears the root cause is that the redis image tag was bumped. Statefulsets don't allow you to do this, so the upgrade fails. 
   
   I managed to get past it by setting 
   
   ```
   redis:
     enabled: false
   ```
   
   In order to upgrade from 0.5.8 to 0.5.9. 
   
   Since the statefulset was no longer part of the spec, the upgrade succeeded. But of course superset was broken because I don't have a separate redis endpoint. 
   
   _Then_ I changed the values.yaml back to
   
   ```
   redis:
     enabled: true
   ```
   
   
   And updated the values in the redis section as per this commit - https://github.com/apache/superset/commit/33d1c967649890500fb788678e8a10afc08bc99a
   
   And then redeployed for 0.5.9 _again_ s that this time the redis statefulset _is_ part f the deployment and hey presto, it's working. 
   
   Bear in mind this solution will result in some downtime if you try it. 


-- 
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: notifications-unsubscribe@superset.apache.org

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


---------------------------------------------------------------------
To unsubscribe, e-mail: notifications-unsubscribe@superset.apache.org
For additional commands, e-mail: notifications-help@superset.apache.org


[GitHub] [superset] riazarbi closed issue #19588: Helm upgrade from 0.5.8 to 0.5.9 fails

Posted by GitBox <gi...@apache.org>.
riazarbi closed issue #19588: Helm upgrade from 0.5.8 to 0.5.9 fails
URL: https://github.com/apache/superset/issues/19588


-- 
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: notifications-unsubscribe@superset.apache.org

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


---------------------------------------------------------------------
To unsubscribe, e-mail: notifications-unsubscribe@superset.apache.org
For additional commands, e-mail: notifications-help@superset.apache.org