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

new script-based update strategy #44

Merged
merged 13 commits into from
Jan 28, 2025
Merged

new script-based update strategy #44

merged 13 commits into from
Jan 28, 2025

Conversation

chuckadams
Copy link
Contributor

Pull Request

What changed?

AS is no longer using the internal svn listing/tracking mechanisms to determine which plugins to fetch, but instead some shell scripts in svn/bin that return the list of changed slugs since a given past date (default 2 days), which are then fed to the fetch command to grab unconditionally. To avoid re-fetching on every run, slugs that have been checked recently are skipped, though all metadata still in the check interval will eventually be re-checked every 4 hours (also configurable).

Why did it change?

This greatly simplifies the revision state tracking logic in AspireSync by not having any at all. Since the update runs every 10 minutes and the push every 30 minutes, slugs that failed to fetch in the last run due to request timeouts will usually get picked up in the next one. The re-fetch every 4 hours will also take care of any plugins that have been closed, which the old tracking logic would not do.

Did you fix any specific issues?

none

CERTIFICATION

By opening this pull request, I do agree to abide by the Code of Conduct and be bound by the terms of the Contribution Guidelines in effect on the date and time of my contribution as proven by the revision information in GitHub.

@chuckadams chuckadams merged commit f9b671a into main Jan 28, 2025
1 check passed
@chuckadams chuckadams deleted the chuck/dev/svn branch January 28, 2025 15:27
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.

None yet

1 participant