Project

General

Profile

Actions

QA Run #68711

closed

wip-bharath3-testing-2025-02-03-2127

Added by Srinivasa Bharath Kanta over 1 year ago. Updated about 1 year ago.

Status:
QA Closed
Priority:
Normal
Tags:

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()

Actions #1

Updated by Srinivasa Bharath Kanta over 1 year ago

  • Assignee set to Laura Flores
Actions #2

Updated by Laura Flores over 1 year ago

  • Assignee changed from Laura Flores to Shraddha Agrawal
Actions #3

Updated by Laura Flores about 1 year ago

  • Assignee changed from Shraddha Agrawal to Laura Flores
Actions #4

Updated by Shraddha Agrawal about 1 year ago

  • Assignee changed from Laura Flores to Shraddha Agrawal
Actions #5

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.

Actions #6

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
Actions #7

Updated by Laura Flores about 1 year ago

@Srinivasa Bharath Kanta any progress on this?

Actions #8

Updated by Srinivasa Bharath Kanta about 1 year ago

@Laura Flores - I will share the results by tomorrow(Feb 4).

Actions #10

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.

Actions #11

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
Actions #12

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.

Actions #13

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.

Actions #15

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
Actions #16

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

Actions #17

Updated by Laura Flores about 1 year ago

  • Description updated (diff)
Actions #19

Updated by Srinivasa Bharath Kanta about 1 year ago

PRs are merged hence clsoing the ticket

Actions #20

Updated by Srinivasa Bharath Kanta about 1 year ago

  • Status changed from QA Approved to QA Closed
Actions

Also available in: Atom PDF