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

Tests: TestHarness wait... methods should be able to wait for a node to start #1562

Open
jgiszczak opened this issue Aug 25, 2023 · 0 comments
Open

Tests: TestHarness wait... methods should be able to wait for a node to start #1562

jgiszczak opened this issue Aug 25, 2023 · 0 comments
Labels
👍 lgtm test-refinement
Milestone
Leap v6.0.0 Cusp

Comments

@jgiszczak
Copy link
Contributor

Currently waitForBlock() and the other wait... methods use get info with exitOnFailure hardcoded to True. If the node hasn't finished enough of its startup to respond to get info the endpoint will be 404 and the whole test shuts down.

If the Node's popenProc is not None and killed is False, the wait... methods should tolerate 404 and other errors and retry, as long as popenProc tests good.
@jgiszczak jgiszczak mentioned this issue Aug 25, 2023
Closed
@bhazzard bhazzard changed the title TestHarness wait... methods should be able to wait for a node to start Tests: TestHarness wait... methods should be able to wait for a node to start Sep 28, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
👍 lgtm test-refinement
Projects
Team Backlog
Archived in project
Milestone
Leap v6.0.0 Cusp
Development

No branches or pull requests
3 participants
@bhazzard @jgiszczak @enf-ci-bot