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.

Sign up for GitHub

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

Jump to bottom

Less logs and EC clients clean up #11

Merged
merged 34 commits into from
Sep 16, 2024
Merged

Less logs and EC clients clean up #11

merged 34 commits into from
Sep 16, 2024

Conversation

vsuharnikov
Copy link
Contributor

@vsuharnikov vsuharnikov commented Sep 5, 2024 •
edited
Loading

  • One EC API client instead of two;
  • Less logs: excluding transactions and other nonrelevant information for consensus client debugging;
  • local-network:
    • Second EC is geth instead of besu;
    • Easier setup for new EC;
    • New naming conventions for EC clients: ec-1, ec-2, ... ;
    • Updated npm dependencies.

Base automatically changed from fixes-n-tests to main September 5, 2024 15:20
@vsuharnikov
ContractBlock.toString
d24ccd6
@vsuharnikov
Additional filtration for eth_getLogs
648e906
@vsuharnikov
local-network: better start
c5d18cd
@vsuharnikov
Less logs for getLogs
631e59b
@vsuharnikov
local-network: prepare for geth
4accec6
@vsuharnikov
geth as the second EC (WIP)
f53b8bc
@vsuharnikov
besu.yml, better directory structure, removed old files
12c1ae2
@vsuharnikov
Additional keys for custom configurations
e9bf8a0
@vsuharnikov
Removed old unnecessary service
3e2c8b6
@vsuharnikov
Updated dependencies
d71bdaf
@vsuharnikov
geth: logs in file, removed redundant options
51068b2
@vsuharnikov
Merged github/main
a0f06e6
@vsuharnikov
Fixes after merge
f9703f5
@vsuharnikov
local-network: EC network settings fixes
3737189
@vsuharnikov
local-network: geth log rotation
2dafd00
@vsuharnikov
local-network: fixed geth sync issue, added ".hex" extension to secrets
1063091
@vsuharnikov
local-network: bootnode to exchange peers information (WIP)
dc50363
@vsuharnikov
local-network: bootnode address in one file
a88e47d
@vsuharnikov
local-network: bootnode logs and config in geth dir
670f312
@vsuharnikov
local-network: bootnode right IP inside script
84ea22e
@vsuharnikov
local-network: local IPs, besu knowns its IP, ECs connect each other …
c8f28a0 
…without a bootnode
@vsuharnikov
local-network: removed unnecessary config record
2a991e6
@vsuharnikov
Merge github/main
bb591a5
@vsuharnikov
Fix
950b8f8
@vsuharnikov
Update dependencies
30b47d2
@vsuharnikov vsuharnikov marked this pull request as ready for review September 12, 2024 04:33
goodmai
goodmai approved these changes Sep 12, 2024
View reviewed changes
Copy link

@goodmai goodmai left a comment

Choose a reason for hiding this comment

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

Not possible read logs from ec-2 container

@vsuharnikov vsuharnikov marked this pull request as draft September 12, 2024 09:06
@vsuharnikov
local-network: common name of log files for different EC, waiting at …
933cfbe 
…least 1 peer for waves node to solve rollback issues
@vsuharnikov
Copy link
Contributor Author

@goodmai Fixed

@vsuharnikov vsuharnikov marked this pull request as ready for review September 12, 2024 10:56
@phearnot phearnot merged commit 6d10b34 into main Sep 16, 2024
1 check passed
@phearnot phearnot deleted the less-logs branch September 16, 2024 09:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@goodmai goodmai goodmai approved these changes

@ivan-mashonskiy ivan-mashonskiy ivan-mashonskiy approved these changes

@phearnot phearnot phearnot approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@vsuharnikov @phearnot @ivan-mashonskiy @goodmai