You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
[2022-11-04T12:32:37Z ERROR ya_market::testing::backtrace_util] No backtrace support. Generating default name from UUIDv4. uuid4=b6b0d3e5-2371-4da7-b2d3-8f1309e8f4b1,
The unhappy path taken when backtrace doesn't parse properly. The issue was that all market tests with deduced names (MarketsNetwork::new(None), 116 occurances as of now) will have auto-gen'd ones of the form unknown-test-{:#32x}, because the bactrace does not conform to the assumptions made when the name deduction was first written.
A proper fix would parse the new backtrace format or, even better, not use backtraces for this at all, as symbol names do not have a guaranteed structure.
Followed up in Rethink MarketsNetwork::new name deduction #2282.
A bug in cargo made the valid flag split-debuginfo="unpacked" cause windows builds to fail, so I removed it from Cargo.toml with an appropriate remark. This should be reverted when my fix in cargo is merged and published Fix split-debuginfo support detection rust-lang/cargo#11347.
Removed -D warnings from clippy. This largely defeats the purpose of using clippy, but we just can't have that with automatically updated toolchain, as new / broadened lints will pose a risk to CI on every update. Continued in Enforcing clippy may break CI on each update #2281.
kamirr
changed the title
Broken market test suite
CI fails on master
Nov 7, 2022
Likely cause:
Full logs in https://github.com/golemfactory/yagna/actions/runs/3393331703/jobs/5641357957
The text was updated successfully, but these errors were encountered: