airbyte-platform icon indicating copy to clipboard operation
airbyte-platform copied to clipboard

Updated template to fix metrics helm issue

Open talha-naeem1 opened this issue 1 year ago • 4 comments
trafficstars

What:

While deploying airbyte metrics helm chart, I had to pass imagepullsection in values file like this:

global:
  imagePullSecrets:
    - name: "ghcr-secret"

While i was installing helm chart i was getting this issue:

Error: YAML parse error on airbyte/charts/metrics/templates/deployment.yaml: error converting YAML to JSON: yaml: line 25: did not find expected key
helm.go:84: [debug] error converting YAML to JSON: yaml: line 25: did not find expected key
YAML parse error on airbyte/charts/metrics/templates/deployment.yaml
helm.sh/helm/v3/pkg/releaseutil.(*manifestFile).sort
	helm.sh/helm/v3/pkg/releaseutil/manifest_sorter.go:146
helm.sh/helm/v3/pkg/releaseutil.SortManifests
	helm.sh/helm/v3/pkg/releaseutil/manifest_sorter.go:106
helm.sh/helm/v3/pkg/action.(*Configuration).renderResources
	helm.sh/helm/v3/pkg/action/action.go:168
helm.sh/helm/v3/pkg/action.(*Install).RunWithContext
	helm.sh/helm/v3/pkg/action/install.go:304
main.runInstall
	helm.sh/helm/v3/cmd/helm/install.go:306
main.newTemplateCmd.func2
	helm.sh/helm/v3/cmd/helm/template.go:95
github.com/spf13/cobra.(*Command).execute
	github.com/spf13/[email protected]/command.go:940
github.com/spf13/cobra.(*Command).ExecuteC
	github.com/spf13/[email protected]/command.go:1068
github.com/spf13/cobra.(*Command).Execute
	github.com/spf13/[email protected]/command.go:992
main.main
	helm.sh/helm/v3/cmd/helm/helm.go:83
runtime.main
	runtime/proc.go:250
runtime.goexit
	runtime/asm_amd64.s:1598

How:

I have fixed the identation which is causing the above mentioned error and blcoks the usage of airbyte-platform helm chart with any custom image.

talha-naeem1 avatar Aug 01 '24 10:08 talha-naeem1

CLA assistant check
Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.
1 out of 2 committers have signed the CLA.

:white_check_mark: talha-naeem1
:x: perangel
You have signed the CLA already but the status is still pending? Let us recheck it.

CLAassistant avatar Aug 02 '24 06:08 CLAassistant

Fixes https://github.com/airbytehq/airbyte/issues/42932

talha-naeem1 avatar Aug 02 '24 07:08 talha-naeem1

/assign @marcosmarxm

talha-naeem1 avatar Aug 02 '24 07:08 talha-naeem1

Thanks for the contribution @talha-naeem1 I'm going to ask the deployment team to take a look

marcosmarxm avatar Aug 02 '24 14:08 marcosmarxm

/create-oss-pr

perangel avatar Sep 24 '24 18:09 perangel

This has been merged, thank you for the contribution!

perangel avatar Sep 24 '24 19:09 perangel