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

hostd integrity check sector data and general dataset refresh intervals #488

Merged
merged 1 commit into from
Feb 16, 2024

Conversation

alexfreska
Copy link
Member

@alexfreska alexfreska commented Feb 14, 2024

  • Integrity check alert sector data will now update when there is an ongoing integrity check.
  • All data will now refresh more frequently.

Copy link

changeset-bot bot commented Feb 14, 2024

🦋 Changeset detected

Latest commit: 4506593

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 3 packages
Name Type
hostd Minor
renterd Minor
walletd Minor

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

@n8maninger
Copy link
Member

Can everything refresh more frequently? Maybe every 15 seconds?

ChrisSchinnerl
ChrisSchinnerl previously approved these changes Feb 15, 2024
@alexfreska
Copy link
Member Author

alexfreska commented Feb 15, 2024

@n8maninger currently the change is to 20s when viewing alerts, 60s in background. Are you suggesting 15s when viewing? In general, most datasets are set to refresh every 60 seconds + when tab is refocused + explicitly refreshed when relevant data mutations happen - I could lower this, was aiming minimize requests while still keeping data fresh.

@n8maninger
Copy link
Member

Yeah. Im suggesting every data set should refresh at a 10-15s interval.

@alexfreska alexfreska changed the title hostd integrity check sector data and alert revalidation hostd integrity check sector data and general dataset refresh intervals Feb 15, 2024
@alexfreska alexfreska force-pushed the alerts branch 2 times, most recently from 7c8e52b to 5f56038 Compare February 15, 2024 21:54
@alexfreska
Copy link
Member Author

@n8maninger alright, updated all apps to use a default of 15s for all primary data.

@n8maninger n8maninger merged commit 924a653 into main Feb 16, 2024
1 check passed
@n8maninger n8maninger deleted the alerts branch February 16, 2024 14:43
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.

3 participants