Direct installation into k8s not working (since 0.39) about reloader

[evilr00t]

@steefdebruijn I've analyzed helm chart and this seems to work:

curl https://raw.githubusercontent.com/stakater/Reloader/master/deployments/kubernetes/reloader.yaml | sed 's/RELEASE-NAME-//g' |sed 's/RELEASE-NAME/v0.0.40/g'|kubectl apply -f -

[ralf-cestusio]

This is currently what we do.

[evilr00t]

That would be more "flexible" I would say, jq is dependency here tho.

LATEST_VERSION=$(curl -s https://api.github.com/repos/stakater/Reloader/releases | jq '.[0].name' | tr -d '"')
curl https://raw.githubusercontent.com/stakater/Reloader/master/deployments/kubernetes/reloader.yaml | sed 's/RELEASE-NAME-//g' |sed "s/RELEASE-NAME/${LATEST_VERSION}/g"|kubectl apply -f -

[aliartiza75]

@evilr00t thank you for your input but this issue will be fixed soon and after that, these kinds of temporary fixes will not used.

[aliartiza75]

@evilr00t README.md has been updated

[waseem-h]

It is due to a bug in our CI/CD pipeline for Reloader. I will ask someone to fix that. @kahootali can you please update the helm template command to have the name attribute set when running it? Like helm template . --name ${chartName}. It should be here: https://github.com/stakater/stakater-pipeline-library/blob/master/src/io/stakater/charts/Templates.groovy#L78
Then point the Jenkinsfile for this repo to use stakater-pipeline-library instead of the old one

[rasheedamir]

PR to clarify it in the README #93

[ralf-cestusio]

The sed approach does not really fix the issue.
If we are applying the configuration with gitops tooling it is not good to have to preprocess the files to make them applyable.

[aliartiza75]

hi @magicmoose,

what you can do is generate the reloader manifest using the sed approach then add that manifest in your gitops workflow.

Although this is not the best solution, I will try to resolve it asap.

[evilr00t]

@kahootali what about README? it hasn't been changed and still uses sed method.

[aliartiza75]

@evilr00t, I have created a PR that has the updated command.