-
Notifications
You must be signed in to change notification settings - Fork 39
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
Swarm batch information updater #411
Conversation
Introduce new periodic job to retrieve Swarm batch information such as label, TTL and others. A new environment variable SWARM_BATCH_ID is used to define the batch id used to store blobs. We are querying the bee endpoint directly because bee-js doesn't support the /stamps endpoint yet.
🦋 Changeset detectedLatest commit: 490cdc2 The changes in this PR will be included in the next version bump. This PR includes changesets to release 9 packages
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 |
The latest updates on your projects. Learn more about Vercel for Git ↗︎
4 Ignored Deployments
|
📦 Next.js Bundle Analysis for @blobscan/webThis analysis was generated by the Next.js Bundle Analysis action. 🤖 This PR introduced no changes to the JavaScript bundle! 🙌 |
5eb579f
to
0d8f9b8
Compare
…onization workers
d580b78
to
3fac36c
Compare
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## next #411 +/- ##
==========================================
+ Coverage 89.70% 89.71% +0.01%
==========================================
Files 152 152
Lines 9548 9616 +68
Branches 984 984
==========================================
+ Hits 8565 8627 +62
- Misses 983 989 +6 ☔ View full report in Codecov by Sentry. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Good job @PabloCastellano !
I did some refactoring and changes as we discussed:
- Unified all syncers into a single package.
- Made some changes to the swarm stamp syncer:
- Improved error handling.
- Changed
update
forupsert
when storing the fetched swarm stamp batch data - Added missing unit tests
- Added missing prisma migration file.
Rename variable SYNCER_SWARM_STAMP_CRON_PATTERN to SWARM_STAMP_CRON_PATTERN Changed logger.warn to logger.error + improve message
Introduce new periodic job to retrieve Swarm stamp information such as label, TTL and others.
A new environment variable SWARM_BATCH_ID is used to define the batch id used to store blobs.
We are querying the bee endpoint directly
because bee-js doesn't support the /stamps endpoint yet