Skip to content
This repository has been archived by the owner on Mar 24, 2023. It is now read-only.

Error message in step 3 with Rook operator helm chart #931

Closed
Timoses opened this issue May 9, 2019 · 3 comments
Closed

Error message in step 3 with Rook operator helm chart #931

Timoses opened this issue May 9, 2019 · 3 comments

Comments

@Timoses
Copy link

Timoses commented May 9, 2019

Expected behavior

Ship continues to step 3

Actual behavior

Error message displayed:

execute render step: execute plan: execute templating: copy state value to tmp directory .ship/tmp/chartrendered: merge helm values: unmarshal base values: yaml: line 7: found character that cannot start any token

Information

  • Ship version:
"version": "0.41.0",
"git": "14db0e3",

Steps to reproduce the behavior

  1. Ship init (Cloning and renaming of values.yaml.tmpl because of Step 2 not displayed without warning when no values.yaml present #930):
$ git clone https://github.com/rook/rook.git
$ mv rook/cluster/charts/rook-ceph/values.yaml.tmpl rook/cluster/charts/rook-ceph/values.yaml
$ ship init rook/cluster/charts/rook-ceph
  1. open Browser
  2. Step 2 (values): Change image -> tag to e.g. master
  3. Step 3: Error message is displayed (see above)
@dexhorthy
Copy link
Member

@Timoses it looks like that values.yaml.tmpl might not quite be valid YAML just yet, we might need to run it through whatever preproccessor fills in those %%VERSION%% placeholders, before running ship. (Even though you've modified the values, we still store the original values for diffing, and those have to be valid yaml).

I think the best bet would be to just remove that %%VERSION%% from the values file before running ship init. You can leave it as a placeholder, or set it to the value you need.

As mentioned in #930 there are unfortunately a few manual steps here to get the Helm Chart to a valid state.

@dexhorthy
Copy link
Member

dexhorthy commented May 10, 2019

@Timoses another workaround is to fetch the released chart from the rook charts repo:

helm repo add rook-release https://charts.rook.io/release
helm fetch --untar rook-release/rook-ceph
ship init ./rook-ceph

This unfortunately will not track the upstream as new updates come in though.

@Timoses
Copy link
Author

Timoses commented May 23, 2019

Thanks for the info!

I'm closing this issue as the error message stated where the error appeared in a reasonable manner.

@Timoses Timoses closed this as completed May 23, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants