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

bug: some Zebra and Lightwalletd logs have changed, causing some tests to fail #9012

Closed
1 of 4 tasks
gustavovalverde opened this issue Nov 11, 2024 · 1 comment · Fixed by #9052
Closed
1 of 4 tasks
Assignees
Labels
A-devops Area: Pipelines, CI/CD and Dockerfiles A-rust Area: Updates to Rust code C-bug Category: This is a bug C-testing Category: These are tests I-integration-fail Continuous integration fails, including build and test failures P-Critical 🚑 S-needs-triage Status: A bug report needs triage

Comments

@gustavovalverde
Copy link
Member

What happened?

I expected to see this happen:
Tests passing

Instead, this happened:
Cached states are not being created as we're failing the disk creation if the expected logs are not found:

We have multiple regexes in Zebra for disk versioning, that are not being found:

And the height of the disk, which varies by test:

What were you doing when the issue happened?

CI was running

Zebra logs

This is an example: https://github.com/ZcashFoundation/zebra/actions/runs/11725652272/job/32664982469#step:12:170

Zebra Version

No response

Which operating systems does the issue happen on?

  • Linux
  • macOS
  • Windows
  • Other OS

OS details

No response

Additional information

No response

@gustavovalverde gustavovalverde added A-devops Area: Pipelines, CI/CD and Dockerfiles A-rust Area: Updates to Rust code C-bug Category: This is a bug C-testing Category: These are tests I-integration-fail Continuous integration fails, including build and test failures P-Critical 🚑 S-needs-triage Status: A bug report needs triage labels Nov 11, 2024
@github-project-automation github-project-automation bot moved this to New in Zebra Nov 11, 2024
@conradoplg
Copy link
Collaborator

The database version and height issues seems to be independent.

The database version issue cause was pinpointed by @arya2, needs updating the test

@mpguerra mpguerra moved this from New to Sprint Backlog in Zebra Dec 2, 2024
@mpguerra mpguerra moved this from Sprint Backlog to In progress in Zebra Dec 2, 2024
@mpguerra mpguerra linked a pull request Dec 2, 2024 that will close this issue
7 tasks
@mpguerra mpguerra moved this from In progress to Review/QA in Zebra Dec 11, 2024
@mergify mergify bot closed this as completed in #9052 Dec 11, 2024
@github-project-automation github-project-automation bot moved this from Review/QA to Done in Zebra Dec 11, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A-devops Area: Pipelines, CI/CD and Dockerfiles A-rust Area: Updates to Rust code C-bug Category: This is a bug C-testing Category: These are tests I-integration-fail Continuous integration fails, including build and test failures P-Critical 🚑 S-needs-triage Status: A bug report needs triage
Projects
Status: Done
Development

Successfully merging a pull request may close this issue.

3 participants