test: Fix a blocklist API test#46010
Conversation
Otherwise, our grep fails! Fixes: https://tracker.ceph.com/issues/55419 Signed-off-by: Greg Farnum <gfarnum@redhat.com>
|
This is entirely untested as a fix because I can't figure out how to run the cephtool test locally just yet, but from log inspection this is the issue. |
|
@ljflores you reported this; don't suppose you know how to run individual tests like this locally? Otherwise I guess I'll spin up a small suite run that invokes it. |
|
@gregsfortytwo yeah, there's a way we can rerun the exact test that it originally failed. I can look into doing that on Monday. |
|
jenkins test make check |
ljflores
left a comment
There was a problem hiding this comment.
Tested this on one of Yuri's qa runs, where the cephtool test failed. Re-running that branch with the suite repo set to this branch passed all the cephtool tests:
|
The make check failure is a recent regression. See more details here: #43413 (comment) |
|
jenkins test make check |
|
@neha-ojha this test passed when run against Yuri's master branch (wip-yuri2-testing-2022-04-22-0500). Would you say that's thorough enough, or should this go through the whole rados suite? |
This is enough. |
This test case was added in #44151, but I screwed up and did less testing on that PR than I thought I had before merging. :/
Fixes: https://tracker.ceph.com/issues/55419
Signed-off-by: Greg Farnum gfarnum@redhat.com
Checklist
Show available Jenkins commands
jenkins retest this pleasejenkins test classic perfjenkins test crimson perfjenkins test signedjenkins test make checkjenkins test make check arm64jenkins test submodulesjenkins test dashboardjenkins test dashboard cephadmjenkins test apijenkins test docsjenkins render docsjenkins test ceph-volume alljenkins test ceph-volume toxjenkins test windows