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

Add support for optional package lists for feeds #99

Merged
merged 1 commit into from
May 7, 2021

Conversation

Qinusty
Copy link
Contributor

@Qinusty Qinusty commented Apr 30, 2021

This PR allows feeds to be configured with a set of packages to poll. In the case of certain feeds, this can help avoid lossy behaviour by ensuring the latest updates from those specific packages are checked during polling.

Support for this option is currently implemented in pypi and an error is reported when attempting to configure other feeds with this unsupported option.

Partially resolves #83

config/config_test.go Outdated Show resolved Hide resolved
config/config_test.go Outdated Show resolved Hide resolved
config/config_test.go Outdated Show resolved Hide resolved
config/config_test.go Outdated Show resolved Hide resolved
config/structs.go Outdated Show resolved Hide resolved
@Qinusty Qinusty force-pushed the qinusty/critical-packages branch 2 times, most recently from f41c06b to 90e3d12 Compare April 30, 2021 14:44
@Qinusty Qinusty force-pushed the qinusty/critical-packages branch 2 times, most recently from 31951b9 to 2a5607e Compare May 5, 2021 11:48
@Qinusty Qinusty mentioned this pull request May 5, 2021
feeds/pypi/pypi.go Outdated Show resolved Hide resolved
@Qinusty Qinusty force-pushed the qinusty/critical-packages branch 3 times, most recently from 17eb48d to 6528aa4 Compare May 7, 2021 13:56
feeds/pypi/pypi.go Outdated Show resolved Hide resolved
This allows feeds to be configured with options (starting with packages)
which acts as a list of packages to poll with the intention of avoiding
lossy firehose feeds. This is currently implemented for pypi.
@Qinusty Qinusty force-pushed the qinusty/critical-packages branch from 6528aa4 to 9754607 Compare May 7, 2021 14:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Addressing lossy feeds
2 participants