QA Run #68711
closedwip-bharath3-testing-2025-02-03-2127
Description
The following PRs were included:
https://github.com/ceph/ceph/pull/59529 - blk/kerneldevice: notify_all only required when discard_drain wait for condition
https://github.com/ceph/ceph/pull/59633 - tools/ceph-bluestore-tool: optimize offline trim report info
https://github.com/ceph/ceph/pull/60205 - os/bluestore: make retry_max and initial_delay configurable for aio s
https://github.com/ceph/ceph/pull/60363 - os/bluestore: Create additional bdev labels when expanding block device.
Excluded:
https://github.com/ceph/ceph/pull/59979 - os: simplify os::Transaction -- get rid of the Transaction::decode_bp()
Updated by Laura Flores over 1 year ago
- Assignee changed from Laura Flores to Shraddha Agrawal
Updated by Laura Flores about 1 year ago
- Assignee changed from Shraddha Agrawal to Laura Flores
Updated by Shraddha Agrawal about 1 year ago
- Assignee changed from Laura Flores to Shraddha Agrawal
Updated by Shraddha Agrawal about 1 year ago
Hey @Srinivasa Bharath Kanta, can we rerun the above batch excluding these two PRs:
1. https://github.com/ceph/ceph/pull/60363
2. https://github.com/ceph/ceph/pull/59979
Both of them likely have related errors seen in the run.
More details for both can be found here(https://github.com/ceph/ceph/pull/59979#issuecomment-2618601678) and here(https://github.com/ceph/ceph/pull/60363#issuecomment-2618629867) respectively.
Updated by Shraddha Agrawal about 1 year ago
- Status changed from QA Testing to QA Needs Rerun/Rebuilt
- Assignee changed from Shraddha Agrawal to Srinivasa Bharath Kanta
Updated by Laura Flores about 1 year ago
@Srinivasa Bharath Kanta any progress on this?
Updated by Srinivasa Bharath Kanta about 1 year ago
@Laura Flores - I will share the results by tomorrow(Feb 4).
Updated by Srinivasa Bharath Kanta about 1 year ago
The tests are in progress.
https://pulpito.ceph.com/?branch=wip-bharath3-testing-2025-02-03-2127
Updated by Srinivasa Bharath Kanta about 1 year ago
- Subject changed from wip-bharath3-testing-2024-10-23-1509 to wip-bharath3-testing-2025-02-03-2127
- Shaman Build changed from wip-bharath3-testing-2024-10-23-1509 to wip-bharath3-testing-2025-02-03-2127
- QA Runs changed from wip-bharath3-testing-2024-10-23-1509 to wip-bharath3-testing-2025-02-03-2127
- Git Branch changed from SrinivasaBharath/ceph/commits/wip-bharath3-testing-2024-10-23-1509 to SrinivasaBharath/ceph/commits/wip-bharath3-testing-2025-02-03-2127
The PR#60363 and PR#59979 are excluded from the PR list.
The new PR List are -
https://github.com/ceph/ceph/pull/59529 - blk/kerneldevice: notify_all only required when discard_drain wait for condition
https://github.com/ceph/ceph/pull/59633 - tools/ceph-bluestore-tool: optimize offline trim report info
https://github.com/ceph/ceph/pull/60205 - os/bluestore: make retry_max and initial_delay configurable for aio s
Srinivasa Bharath Kanta wrote:
The following PRs were included:
https://github.com/ceph/ceph/pull/59529 - blk/kerneldevice: notify_all only required when discard_drain wait for condition
https://github.com/ceph/ceph/pull/59633 - tools/ceph-bluestore-tool: optimize offline trim report info
https://github.com/ceph/ceph/pull/59979 - os: simplify os::Transaction -- get rid of the Transaction::decode_bp()
https://github.com/ceph/ceph/pull/60205 - os/bluestore: make retry_max and initial_delay configurable for aio s
https://github.com/ceph/ceph/pull/60363 - os/bluestore: Create additional bdev labels when expanding block device.
Updated by Srinivasa Bharath Kanta about 1 year ago
- Status changed from QA Needs Rerun/Rebuilt to QA Needs Approval
- Assignee changed from Srinivasa Bharath Kanta to Laura Flores
Updated by Laura Flores about 1 year ago
- Assignee changed from Laura Flores to Shraddha Agrawal
Hi @Shraddha Agrawal, the tests have been rescheduled. They are still running at this point, but will need a review when they've completed.
Updated by Laura Flores about 1 year ago
@Shraddha Agrawal this one is ready to go. On the initial run, there is one job that is stuck in "waiting" state, but that is a teuthology problem, so I think we can proceed.
Updated by Shraddha Agrawal about 1 year ago ยท Edited
Rados approved - https://tracker.ceph.com/projects/rados/wiki/MAIN#httpstrackercephcomissues68711
Excluding these two PRs:
1. https://github.com/ceph/ceph/pull/60363
2. https://github.com/ceph/ceph/pull/59979
Updated by Shraddha Agrawal about 1 year ago
- Status changed from QA Needs Approval to QA Approved
- Assignee changed from Shraddha Agrawal to Srinivasa Bharath Kanta
Updated by Laura Flores about 1 year ago
Awesome! @Srinivasa Bharath Kanta the following PRs are excluded:
1. https://github.com/ceph/ceph/pull/60363 -- already merged
2. https://github.com/ceph/ceph/pull/59979 -- DNM
Updated by Laura Flores about 1 year ago
@Srinivasa Bharath Kanta it looks like https://github.com/ceph/ceph/pull/60205 can be merged.
Updated by Srinivasa Bharath Kanta about 1 year ago
PRs are merged hence clsoing the ticket
Updated by Srinivasa Bharath Kanta about 1 year ago
- Status changed from QA Approved to QA Closed