Skip to content
This repository has been archived by the owner on Nov 1, 2022. It is now read-only.

Reimplement --git-poll-interval in Helm operator #1910

Merged
merged 3 commits into from
Apr 9, 2019

Conversation

hiddeco
Copy link
Member

@hiddeco hiddeco commented Apr 9, 2019

This got removed a while ago, and was told to be replaced by --charts-sync-interval. The --charts-sync-interval flag is however not used to control the polling of git repositories, but to control how often we should look if any mutations have occurred to Helm releases.

Add it back to restore the feature and regain control over git polling again.

Copy link
Member

@squaremo squaremo left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think the new chart value warrants a mention in the chart README. Otherwise, looks good 🍍

This got removed a while ago, and was told to be replaced by
`--charts-sync-interval`. The `--charts-sync-interval` flag is however
_not_ used to control the polling of git repositories, but to control
how often we should look if any mutations have occurred to Helm
releases.

Add it back to restore the feature and regain control over git polling
again.
@hiddeco hiddeco changed the title Reimplement --git-poll-interval for Helm operator Reimplement --git-poll-interval in Helm operator Apr 9, 2019
@hiddeco hiddeco merged commit 43f4c53 into master Apr 9, 2019
@hiddeco hiddeco deleted the helm/git-poll-interval branch April 9, 2019 12:09
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants