-
Notifications
You must be signed in to change notification settings - Fork 115
Bump gomod deps #2943
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
Merged
Merged
Bump gomod deps #2943
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Simplify avalanche-go binary path configuration by setting it on the default runtime config instead of iterating through all nodes. Also handle case where Process config might be nil.
- Add db cache directory creation and management - Implement caching for avalanchego database downloads with 7-day TTL - Refactor file copy operations to use streaming for memory efficiency - Extract and expose archive extraction utilities for reuse - Add proper error handling and logging for cache operations
Add .tar.gz extension to cached database file path to ensure proper file type identification and handling.
- Update avalanchego to v1.13.3-0.20250911181736-31dcd1ea2540 - Remove local avalanchego replace directive - Simplify GetTmpNetNetworkID to use network.GetNetworkID() directly - Remove unused GetTmpNetNodeNetworkID function
Implements migration logic to set NetworkID in network config files when missing, using old tmpnet network ID retrieval logic for backward compatibility.
Moved ERC20 implementation from pkg/ictt/erc20.go to new pkg/erc20/ package structure and updated all references across commands and tests. Removed obsolete SOV public subnet test suite and updated E2E test configurations accordingly.
- Add migrateNodeBlockchainConfigs function to migration.go for converting file-based chain configs to ChainConfigContentKey - Update tmpnet.go to use ChainConfigContentKey instead of ChainConfigDirKey for blockchain configurations - Remove file-based blockchain config directory setup and operations - Add cleanup of binary files in package manager test suite
Removes the manual copying of chain config files when adding a new node to the local cluster, along with unused imports for filepath and dircopy. The chain configuration is now handled through content-based storage. Signed-off-by: Felipe Madero <felipe.madero@gmail.com>
- Add dynamic port allocation for ICM relayer metrics to avoid conflicts - Implement FindAvailablePort utility function for port discovery - Fix import ordering in whitelist.go to follow Go conventions Signed-off-by: Felipe Madero <felipe.madero@gmail.com>
Update ContainSubstring calls to use proper string formatting for acp-support values in log file assertions.
d9cfe3b to
7e75ae2
Compare
sukantoraymond
approved these changes
Sep 17, 2025
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Why this should be merged
How this works
How this was tested
How is this documented