command in datahub-ingestion-cron values.yaml is incorrect

[jimsmith]

This command is incorrect

datahub-helm/charts/datahub/subcharts/datahub-ingestion-cron/values.yaml

Line 26 in ee2bc53

#command: ["/bin/sh", "-c", "datahub ingest -c <recipe.fileName>"]

This error occurs:

client.go:142: [debug] creating 1 resource(s)
Error: INSTALLATION FAILED: 1 error occurred:
        * CronJob in version "v1" cannot be handled as a CronJob: json: cannot unmarshal array into Go struct field Container.spec.jobTemplate.spec.template.spec.containers.command of type string
        * 

The command is already duplicated:
["/bin/sh", "-c", is already in the helm template:

datahub-helm/charts/datahub/subcharts/datahub-ingestion-cron/templates/cron.yaml

Line 52 in ee2bc53

command: ["/bin/sh", "-c", {{ default $defaultCommand .command }} ]

This caused a few problems tracking this down, however it is a simple misconfiguration in values.yaml
command: "datahub ingest -c <recipe.fileName>"

After that it deploys into cluster as expected.

[sachinsaju]

@jimsmith Thanks for reporting and figuring out the issue. I have raised a PR for fixing the issue.

[jimsmith]

@sachinsaju I looked at the PR and noticed you have the square brackets they did not work for me in the workaround

[sachinsaju]

@jimsmith I have updated the PR.

[github-actions]

This issue is stale because it has been open for 30 days with no activity. If you believe this is still an issue on the latest DataHub release please leave a comment with the version that you tested it with. If this is a question/discussion please head to https://slack.datahubproject.io. For feature requests please use https://feature-requests.datahubproject.io

[jimsmith]

Issue closed as I can see this has been merged #391