Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Unable to deploy feast chart in release 0.17.0 #2232

Closed
psheorangithub opened this issue Jan 21, 2022 · 2 comments · Fixed by #2234
Closed

Unable to deploy feast chart in release 0.17.0 #2232

psheorangithub opened this issue Jan 21, 2022 · 2 comments · Fixed by #2234

Comments

@psheorangithub
Copy link

Expected Behavior

Should be able to deploy chart seemlessly

Current Behavior

Looks like some issue in YAML files.I get below error when trying the install the base chart.
history.go:56: [debug] getting history for release feast-feast
upgrade.go:123: [debug] preparing upgrade for feast-feast
Error: UPGRADE FAILED: YAML parse error on feast/charts/feature-server/templates/deployment.yaml: error converting YAML to JSON: yaml: line 70: could not find expected ':'
helm.go:81: [debug] error converting YAML to JSON: yaml: line 70: could not find expected ':'
YAML parse error on feast/charts/feature-server/templates/deployment.yaml
helm.sh/helm/v3/pkg/releaseutil.(*manifestFile).sort
/home/circleci/helm.sh/helm/pkg/releaseutil/manifest_sorter.go:146
helm.sh/helm/v3/pkg/releaseutil.SortManifests
/home/circleci/helm.sh/helm/pkg/releaseutil/manifest_sorter.go:106
helm.sh/helm/v3/pkg/action.(*Configuration).renderResources
/home/circleci/helm.sh/helm/pkg/action/action.go:165
helm.sh/helm/v3/pkg/action.(*Upgrade).prepareUpgrade
/home/circleci/helm.sh/helm/pkg/action/upgrade.go:215
helm.sh/helm/v3/pkg/action.(*Upgrade).Run
/home/circleci/helm.sh/helm/pkg/action/upgrade.go:124
main.newUpgradeCmd.func2
/home/circleci/helm.sh/helm/cmd/helm/upgrade.go:155
github.com/spf13/cobra.(*Command).execute
/go/pkg/mod/github.com/spf13/cobra@v1.1.1/command.go:850
github.com/spf13/cobra.(*Command).ExecuteC
/go/pkg/mod/github.com/spf13/cobra@v1.1.1/command.go:958
github.com/spf13/cobra.(*Command).Execute
/go/pkg/mod/github.com/spf13/cobra@v1.1.1/command.go:895
main.main
/home/circleci/helm.sh/helm/cmd/helm/helm.go:80
runtime.main
/usr/local/go/src/runtime/proc.go:204
runtime.goexit
/usr/local/go/src/runtime/asm_amd64.s:1374
UPGRADE FAILED
main.newUpgradeCmd.func2
/home/circleci/helm.sh/helm/cmd/helm/upgrade.go:157
github.com/spf13/cobra.(*Command).execute
/go/pkg/mod/github.com/spf13/cobra@v1.1.1/command.go:850
github.com/spf13/cobra.(*Command).ExecuteC
/go/pkg/mod/github.com/spf13/cobra@v1.1.1/command.go:958
github.com/spf13/cobra.(*Command).Execute
/go/pkg/mod/github.com/spf13/cobra@v1.1.1/command.go:895
main.main
/home/circleci/helm.sh/helm/cmd/helm/helm.go:80
runtime.main
/usr/local/go/src/runtime/proc.go:204
runtime.goexit
/usr/local/go/src/runtime/asm_amd64.s:1374

Steps to reproduce

git clone https://github.com/feast-dev/feast.git
cd feast/infra/charts/feast
update registry path in values.yaml of the main chart
perform helm install

Specifications

  • Kube Version: 1.21.5
  • Helm version: 3.5.4
  • Platform:
  • Subsystem:

Possible Solution

@pyalex
Copy link
Collaborator

pyalex commented Jan 24, 2022

Hi @psheorangithub, thanks for reporting this. I have already created PR with fix for this issue.

@psheorangithub
Copy link
Author

Thanks @pyalex . Let me know once PR is approved

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants