qa/workunits/rados: remove cache tier test#64504
Conversation
7c84515 to
10d8383
Compare
|
But do we still want to have it compiled? ceph_test_rados_api_tier_pp is a target in CMkeLists.txt |
I was not sure if we still want it, we still have the source and the cmake if we want to do full cleanup, I can add it |
|
@NitzanMordhai: let's clean the source as well. |
10d8383 to
f8b4880
Compare
qa/workunits/rados/test.sh
Outdated
| @@ -27,7 +27,6 @@ for f in \ | |||
| api_asio api_list \ | |||
| api_lock api_lock_pp \ | |||
| api_misc api_misc_pp \ | |||
There was a problem hiding this comment.
nit, s/remve/remove/ in the title of the commit message.
f8b4880 to
86c3b0a
Compare
|
jenkins test make check |
|
jenkins test make check arm64 |
|
jenkins test windows |
|
This pull request can no longer be automatically merged: a rebase is needed and changes have to be manually resolved |
86c3b0a to
60e917d
Compare
60e917d to
1649b0d
Compare
|
jenkins test api |
|
jenkins test make check arm64 |
|
@rzarzynski i don't see point to run tests against that change. Can we merge it? |
|
This pull request can no longer be automatically merged: a rebase is needed and changes have to be manually resolved |
1649b0d to
dca0961
Compare
|
jenkins test make check |
|
jenkins test make check arm64 |
|
jenkins test make check |
|
jenkins test make check arm64 |
dca0961 to
88b49a2
Compare
|
jenkins test make check |
|
This pull request can no longer be automatically merged: a rebase is needed and changes have to be manually resolved |
|
This PR was tested as part of QA Run: Unfortunately there were quite a few new failures as documented in the wiki here: New Issues raised: Please also note the api failures here which seem to be covered by other trackers: 8451486, 8451651 https://tracker.ceph.com/issues/72785 - rados/test.sh - FAILURE: Test Please review and decide whether this PR has affected these failures. Thanks |
Fixes: https://tracker.ceph.com/issues/71930 Signed-off-by: Nitzan Mordechai <nmordec@ibm.com>
88b49a2 to
2b57f43
Compare
|
@lee-j-sanders, it doesn't look like any of the above is related to the cache-tier test remove |
|
@NitzanMordhai can you update the QA batch ticket (https://tracker.ceph.com/issues/72627) with your thoughts? Then feel free to merge. |
|
merging - approved the tests |
|
jenkins test windows |
|
New trackers analysed, all unrelated. Rados approved: https://tracker.ceph.com/projects/rados/wiki/MAIN#httpstrackercephcomissues72627 |
|
jenkins test windows |
Contribution Guidelines
To sign and title your commits, please refer to Submitting Patches to Ceph.
If you are submitting a fix for a stable branch (e.g. "quincy"), please refer to Submitting Patches to Ceph - Backports for the proper workflow.
When filling out the below checklist, you may click boxes directly in the GitHub web UI. When entering or editing the entire PR message in the GitHub web UI editor, you may also select a checklist item by adding an
xbetween the brackets:[x]. Spaces and capitalization matter when checking off items this way.Checklist
Show available Jenkins commands
jenkins test classic perfJenkins Job | Jenkins Job Definitionjenkins test crimson perfJenkins Job | Jenkins Job Definitionjenkins test signedJenkins Job | Jenkins Job Definitionjenkins test make checkJenkins Job | Jenkins Job Definitionjenkins test make check arm64Jenkins Job | Jenkins Job Definitionjenkins test submodulesJenkins Job | Jenkins Job Definitionjenkins test dashboardJenkins Job | Jenkins Job Definitionjenkins test dashboard cephadmJenkins Job | Jenkins Job Definitionjenkins test apiJenkins Job | Jenkins Job Definitionjenkins test docsReadTheDocs | Github Workflow Definitionjenkins test ceph-volume allJenkins Jobs | Jenkins Jobs Definitionjenkins test windowsJenkins Job | Jenkins Job Definitionjenkins test rook e2eJenkins Job | Jenkins Job Definition