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

Adjust and/or increase scrape times? #31

Open
colinmahns opened this issue Aug 30, 2023 · 2 comments
Open

Adjust and/or increase scrape times? #31

colinmahns opened this issue Aug 30, 2023 · 2 comments
Labels
enhancement New feature or request question Further information is requested

Comments

@colinmahns
Copy link

Currently Planet updates for three scenarios: push/merge actions, pull requests (to test the generation of the website), and a cron set to 12:30 UTC / 09:30 ET / 06:30 PT every day.

From .github/workflows/jekyll.yml:

on:
  push:
    branches: [ master ]
  pull_request:
    branches: [ master ]
  schedule:
    # * is a special character in YAML so you have to quote this string
    # We will pull once a day, UTC to start to not blow up our Github actions
    - cron: '30 12 * * *'

Our upstream feeds are currently only sparingly updated, some once a week, some less. I don't think it's worth periodically pulling updates as if we use too many actions we will accrue some costs and quite frankly if there's no updates to the feeds it's a wasted run. But as more chapters start updating their feeds (I've already anecdotally seen a few chapters are now posting updates for the first time in months), I think we can increase this to a couple updates a day. Maybe one in the morning and one in the evening, or even an afternoon pull if we see updates happen from these feeds.

Any suggestions for time? 09:30 ET was selected purely arbitrarily. Happy to adjust this to earlier for catching East Coast commutes and mornings, along with adding a mid or late day scrape.

@colinmahns colinmahns added enhancement New feature or request question Further information is requested labels Aug 30, 2023
@victoria-riley-barnett
Copy link

Do we have analytics hooked in? Recommend setting that up. Willing to bet theres a clear spike or two and as long as we pull close to but before those, once a day is probably fine.

@colinmahns
Copy link
Author

colinmahns commented Aug 31, 2023 via email

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request question Further information is requested
Projects
None yet
Development

No branches or pull requests

2 participants