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

Bump citrea e2e #1792

Merged
merged 3 commits into from
Jan 31, 2025
Merged

Bump citrea e2e #1792

merged 3 commits into from
Jan 31, 2025

Conversation

jfldde
Copy link
Contributor

@jfldde jfldde commented Jan 31, 2025

  • Accept restarting citrea test node with extra_args
  • Bump tail log lines default values to 100 (easier to debug CI)
  • Propagate initialization's wait_for_ready error
  • Remove config trait
  • Fix odd port conflict when running bitcoind out of docker

@jfldde jfldde requested a review from eyusufatik as a code owner January 31, 2025 11:53
@auto-assign auto-assign bot requested a review from kpp January 31, 2025 11:53
@codecov Codecov
Copy link

codecov bot commented Jan 31, 2025 •
edited
Loading

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 77.4%. Comparing base (347d641) to head (daecac4).
Report is 1 commits behind head on nightly.

Additional details and impacted files

see 6 files with indirect coverage changes

@eyusufatik eyusufatik merged commit 84e51db into nightly Jan 31, 2025
14 checks passed
@eyusufatik eyusufatik deleted the bump-citrea-e2e branch January 31, 2025 19:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@eyusufatik eyusufatik eyusufatik approved these changes

@kpp kpp Awaiting requested review from kpp

Assignees

@jfldde jfldde

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@jfldde @eyusufatik