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

Rename `--registry-poll-interval` #2284

Merged
merged 1 commit into from Jul 26, 2019

Conversation

@hiddeco
Copy link
Member

commented Jul 23, 2019

The flag --registry-poll-interval used to control how often fluxd
would scan image repos for new images, and consequently update any
automated workloads. This has not been true for some time: it now
just controls the second part, while images are scanned on a
different schedule. The name is a persistent source of confusion.

Therefore: introduce a new flag --automation-interval, and make
--registry-poll-interval a synonym so as not to break things
suddenly, and deprecate the latter so as to encourage people to
change it.

Fixes #2281

@hiddeco hiddeco added the UX label Jul 23, 2019

@hiddeco hiddeco added this to the 1.14.0 milestone Jul 23, 2019

Rename `--registry-poll-interval`
The flag `--registry-poll-interval` used to control how often fluxd
would scan image repos for new images, and consequently update any
automated workloads. This has not been true for some time: it now
just controls the second part, while images are scanned on a
different schedule. The name is a persistent source of confusion.

Therefore: introduce a new flag `--automation-interval`, and make
`--registry-poll-interval` a synonym so as not to break things
suddenly, and deprecate the latter so as to encourage people to
change it.

@hiddeco hiddeco force-pushed the ux/2281-automation-interval branch from 807bf76 to d670a5a Jul 26, 2019

@hiddeco hiddeco merged commit 1796ba1 into master Jul 26, 2019

1 check passed

ci/circleci: build Your tests passed on CircleCI!
Details

@hiddeco hiddeco deleted the ux/2281-automation-interval branch Jul 26, 2019

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
2 participants
You can’t perform that action at this time.