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.

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

Reworks the way the ledger waits for genesis start. #2502

Merged
merged 2 commits into from
Feb 5, 2024

Conversation

batconjurer
Copy link
Member

@batconjurer batconjurer commented Feb 1, 2024

Resolves #2495

Describe your changes

It now fully initializes the node and outputs logs before sleeping until genesis start time. Previously it would not start any processes until genesis times, giving no feedback to users until genesis time was reached.

Indicate on which release or other PRs this topic is based on

Checklist before merging to draft

  • I have added a changelog
  • Git history is in acceptable state

…lly initializes the node and outputs logs before sleeping until genesis start time
@batconjurer batconjurer requested review from Fraccaman and sug0 February 1, 2024 13:06
@@ -168,7 +168,7 @@ test-e2e:
NAMADA_E2E_USE_PREBUILT_BINARIES=$(NAMADA_E2E_USE_PREBUILT_BINARIES) \
NAMADA_E2E_DEBUG=$(NAMADA_E2E_DEBUG) \
RUST_BACKTRACE=$(RUST_BACKTRACE) \
$(cargo) +$(nightly) test e2e::$(TEST_FILTER) \
$(cargo) +$(nightly) test $(jobs) e2e::$(TEST_FILTER) \
Copy link
Member

Choose a reason for hiding this comment

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

e2e test must not run in parallel I think

Copy link
Member Author

Choose a reason for hiding this comment

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

That only controls how many threads are used to build the tests, not for running them

@Fraccaman Fraccaman mentioned this pull request Feb 1, 2024
@batconjurer batconjurer requested a review from Fraccaman February 1, 2024 14:13
brentstone added a commit that referenced this pull request Feb 1, 2024
* bat/feat/wait-for-genesis-logs:
  [chore]: Add changelog
  [feat]: Reworks the way the ledger waits for genesis start. It now fully initializes the node and outputs logs before sleeping until genesis start time
@tzemanovic tzemanovic merged commit c74c4f2 into main Feb 5, 2024
13 of 15 checks passed
@tzemanovic tzemanovic deleted the bat/feat/wait-for-genesis-logs branch February 5, 2024 11:10
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.

Namada node status before genesis time
5 participants