Skip to content

feat: migrate logging to ipfs/go-log/v2 #2416

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

Merged
tac0turtle merged 19 commits into from
Jul 1, 2025
Merged

feat: migrate logging to ipfs/go-log/v2 #2416

Show file tree
Hide file tree
Changes from 1 commit
Commits
Show all changes
19 commits
Select commit Hold shift + click to select a range
286c65e
feat: migrate logging to ipfs/go-log/v2
google-labs-jules[bot] Jun 30, 2025
4a33609
lint tidy and removal of comments
Jun 30, 2025
fb6847b
fix tests
Jun 30, 2025
1e9b6c1
fix lint
Jun 30, 2025
9b4de61
remove extra comments
Jun 30, 2025
0536a16
remove extra tests
Jul 1, 2025
92c02c9
skip tests to work backwards
Jul 1, 2025
98c7279
Merge branch 'main' into feat/migrate-logging-ipfs-go-log
tac0turtle Jul 1, 2025
6df1356
remove op-geth
Jul 1, 2025
d395f2f
remove op-geth
Jul 1, 2025
5783cb6
fix some logs and add script to spin up two nodes
Jul 1, 2025
f4d1a07
Merge branch 'main' into feat/migrate-logging-ipfs-go-log
Jul 1, 2025
9833124
remove extra comments
Jul 1, 2025
901ecd2
remove other skips
Jul 1, 2025
b425103
linting
Jul 1, 2025
961267e
Merge branch 'main' into feat/migrate-logging-ipfs-go-log
Jul 1, 2025
f86e282
some cleanup
Jul 1, 2025
469e1fb
make p2pid extract more robust
Jul 1, 2025
7d1db82
remove some logs
Jul 1, 2025
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
remove other skips
  • Loading branch information
tac0turtle
tac0turtle committed Jul 1, 2025
commit 901ecd209735a5730d074b5a3a49814733f0c958
3 changes: 0 additions & 3 deletions test/e2e/evm_full_node_e2e_test.go
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -424,7 +424,6 @@ func TestEvmSequencerWithFullNodeE2E(t *testing.T) {
// ensuring that the network can scale to multiple full nodes while maintaining
// data consistency and integrity across all participants.
func TestEvmFullNodeBlockPropagationE2E(t *testing.T) {
t.Skip("Skipping EVM full node block propagation test for now, needs to be re-enabled after refactoring")
flag.Parse()
workDir := t.TempDir()
sequencerHome := filepath.Join(workDir, "evm-sequencer")
Expand Down Expand Up @@ -661,7 +660,6 @@ func setupSequencerWithFullNodeLazy(t *testing.T, sut *SystemUnderTest, sequence
// producing blocks when necessary (when transactions are available), while
// maintaining proper P2P sync functionality.
func TestEvmLazyModeSequencerE2E(t *testing.T) {
t.Skip("Skipping EVM lazy mode sequencer test for now, needs to be re-enabled after refactoring")
flag.Parse()
workDir := t.TempDir()
sequencerHome := filepath.Join(workDir, "evm-lazy-sequencer")
Expand Down Expand Up @@ -937,7 +935,6 @@ func restartSequencerAndFullNode(t *testing.T, sut *SystemUnderTest, sequencerHo
// consistency and continues to function correctly even when all nodes
// are restarted simultaneously, including mode changes.
func TestEvmSequencerFullNodeRestartE2E(t *testing.T) {
t.Skip("Skipping EVM sequencer full node restart test for now, needs to be re-enabled after refactoring")
flag.Parse()

t.Run("StandardRestart", func(t *testing.T) {
Expand Down
2 changes: 1 addition & 1 deletion test/e2e/sut_helper.go
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -63,7 +63,7 @@ func (s *SystemUnderTest) RunCmd(cmd string, args ...string) (string, error) {
args...,
)
// Use CombinedOutput to capture both stdout and stderr
combinedOutput, err := c.CombinedOutput() // Changed from c.Output()
combinedOutput, err := c.CombinedOutput()

return string(combinedOutput), err
}
Expand Down