Skip to content

Conversation

@andrey-kozel
Copy link
Contributor

@andrey-kozel andrey-kozel commented Dec 9, 2025

What was done

Added support for disabling blob sidecars saving in the blocks service.

Changes:

  • Configuration: Added blocks.blobs.enable configuration flag (default: true) to control whether blob sidecars should be fetched and saved
  • CLI Support: Added --blocks.blob.enable flag to command-line parameters
  • ENV Variable: Added CHAIND_BLOCKS_BLOB_ENABLE variable to manage --blocks.blob.enable flag
  • Service Implementation:
    • Added blobEnable parameter to the Service structure
    • Implemented early return in updateBlobSidecarsForBlock() when blob saving is disabled, skipping unnecessary API calls
    • Added initialization logging to display blob enable status

Benefits:

  • Allows operators to disable blob saving for resource optimization when they are not needed

Testing

  • Build is working
image - Container is running image

Copy link
Collaborator

@Bez625 Bez625 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Mostly style changes. Please can you also update the CHANGELOG.md?

@andrey-kozel
Copy link
Contributor Author

@Bez625 Hello. Thank you for the review. All the suggestions were applied!

Bez625
Bez625 previously approved these changes Dec 10, 2025
@Bez625 Bez625 dismissed AntiD2ta’s stale review December 10, 2025 14:30

Changes addressed.

@Bez625 Bez625 merged commit 77bef5c into wealdtech:master Dec 10, 2025
2 checks passed
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