Skip to content
This repository has been archived by the owner on Feb 22, 2022. It is now read-only.

RollingUpdate: readObjectStart: expect { or n, but found t, error found in #10 byte of #11448

Closed
xrobeva opened this issue Feb 15, 2019 · 6 comments
Labels
lifecycle/stale Denotes an issue or PR has remained open with no activity and has become stale.

Comments

@xrobeva
Copy link

xrobeva commented Feb 15, 2019

Is this a request for help?: No


Is this a BUG REPORT or FEATURE REQUEST? (choose one): BUG REPORT

Version of Helm and Kubernetes:
helm version
Client: &version.Version{SemVer:"v2.10.0", GitCommit:"9ad53aac42165a5fadc6c87be0dea6b115f93090", GitTreeState:"clean"}
Server: &version.Version{SemVer:"v2.12.3", GitCommit:"eecf22f77df5f65c823aacd2dbd30ae6c65f186e", GitTreeState:"clean"}

Which chart:
charts/stable/jenkins
version: 0.32.1

What happened:
When Jenkins is deployed with .Values.Persistence.Enabled set to false, the deployment fails with this error message:
Error: release testing-udm-jenkins-66-dn failed: Deployment in version "v1" cannot be handled as a Deployment: v1.Deployment: Spec: v1.DeploymentSpec: Strategy: v1.DeploymentStrategy: RollingUpdate: readObjectStart: expect { or n, but found t, error found in #10 byte of ...|gUpdate":true,"type"|..., bigger context ...|dn-jenkins-master"}},"strategy":{"rollingUpdate":true,"type":"RollingUpdate"},"template":{"metadata"|...

What you expected to happen:
The deployment finishs SUCCESS

How to reproduce it (as minimally and precisely as possible):
Setting the .Values.Persistence.Enabled to false

Anything else we need to know:
This change "07fc9db" breaks the deployment.
rollingUpdate: {{ if .Values.Persistence.Enabled }}{{ else }}true{{ end }}
.spec.strategy.rollingUpdate is a dictionary and the change sets a boolean.

@maorfr
Copy link
Member

maorfr commented Feb 20, 2019

Duplicate of #11423

@maorfr
Copy link
Member

maorfr commented Mar 7, 2019

Hey @xrobeva,

Can this issue be closed?

@xrobeva
Copy link
Author

xrobeva commented Mar 7, 2019

Yes, you can.
Our deployment is already working
Thanks

@maorfr
Copy link
Member

maorfr commented Mar 7, 2019

Go ahead and close it :)

@stale
Copy link

stale bot commented Apr 6, 2019

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Any further update will cause the issue/pull request to no longer be considered stale. Thank you for your contributions.

@stale stale bot added the lifecycle/stale Denotes an issue or PR has remained open with no activity and has become stale. label Apr 6, 2019
@stale
Copy link

stale bot commented Apr 20, 2019

This issue is being automatically closed due to inactivity.

@stale stale bot closed this as completed Apr 20, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
lifecycle/stale Denotes an issue or PR has remained open with no activity and has become stale.
Projects
None yet
Development

No branches or pull requests

2 participants