Skip to content

feat: enabled mirror node modularized service #1109

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

Conversation

quiet-node
Copy link
Contributor

@quiet-node quiet-node commented Jun 9, 2025

Description:

This PR enables support for the Mirror Node’s modularized services, which are required for proper gas estimation when deploying smart contracts using tools like ethers.js. Without these settings, deployments fail with INSUFFICIENT_GAS.

To activate modularized services in Mirror Node version 0.129.0, the following environment variables must be set:

HEDERA_MIRROR_WEB3_EVM_MODULARIZEDSERVICES: "true"
HEDERA_MIRROR_WEB3_EVM_MODULARIZEDTRAFFICPERCENT: "1"

These settings ensure that gas estimation functions correctly during contract deployment.

Related Issue(s):

This unblocks the following PR in the Relay: #3830

Fixes: #1110

Signed-off-by: Logan Nguyen <logan.nguyen@swirldslabs.com>
@quiet-node quiet-node requested a review from a team as a code owner June 9, 2025 21:59
@quiet-node quiet-node requested a review from natanasow June 9, 2025 21:59
@lfdt-bot
Copy link

lfdt-bot commented Jun 9, 2025

🎉 Snyk checks have passed. No issues have been found so far.

security/snyk check is complete. No issues have been found. (View Details)

license/snyk check is complete. No issues have been found. (View Details)

Copy link

codecov bot commented Jun 9, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

Impacted file tree graph

@@           Coverage Diff           @@
##             main    #1109   +/-   ##
=======================================
  Coverage        ?   60.11%           
=======================================
  Files           ?       29           
  Lines           ?     1246           
  Branches        ?      157           
=======================================
  Hits            ?      749           
  Misses          ?      471           
  Partials        ?       26           

Impacted file tree graph

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

Copy link
Contributor

@AlfredoG87 AlfredoG87 left a comment

Choose a reason for hiding this comment

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

LGTM, just one question or suggestion depending on POV.

quiet-node and others added 2 commits June 10, 2025 10:02
Signed-off-by: Logan Nguyen <logan.nguyen@swirldslabs.com>

Co-authored-by: Alfredo Gutierrez <alfredo@hashgraph.com>
Signed-off-by: Logan Nguyen <lg.131.dev@gmail.com>
Signed-off-by: Logan Nguyen <logan.nguyen@swirldslabs.com>
@georgi-l95 georgi-l95 merged commit 1b38956 into hiero-ledger:main Jun 10, 2025
13 of 14 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.

Enable modularized service for MN
4 participants