Project

General

Profile

Actions

QA Run #70262

closed

wip-hemanth-testing-2025-03-03-1505-squid

Added by Kumar Hemanth about 1 year ago. Updated 11 months ago.

Status:
QA Closed
Priority:
Normal
Assignee:

Description

--- done. these PRs were included:
https://github.com/ceph/ceph/pull/59005 - squid: pybind/ceph_argparse: Fix error message for ceph tell command
https://github.com/ceph/ceph/pull/60396 - squid: mgr/vol : shortening the name of helper method
https://github.com/ceph/ceph/pull/61287 - squid: log: concatenate thread names and print once per thread


Related issues 1 (0 open1 closed)

Related to Ceph QA - QA Run #69323: wip-bharath6-testing-2024-12-19-0956-squidQA ClosedSrinivasa Bharath KantaActions
Actions #2

Updated by Venky Shankar about 1 year ago

  • Related to QA Run #69323: wip-bharath6-testing-2024-12-19-0956-squid added
Actions #5

Updated by Kumar Hemanth about 1 year ago

No New Failures were found.

Existing Issues were updated in the wiki -
https://tracker.ceph.com/projects/cephfs/wiki/Squid#wip-hemanth-testing-2025-03-03-1505-squid

Actions #6

Updated by Kumar Hemanth about 1 year ago

@Venky Shankar - QA runs have been analyzed, and the wiki has been updated.
The PRs need approval before I can proceed with merging.

Assigning to you for further action.

Actions #7

Updated by Venky Shankar about 1 year ago

  • Assignee changed from Kumar Hemanth to Venky Shankar

Kumar Hemanth wrote in #note-6:

@Venky Shankar - QA runs have been analyzed, and the wiki has been updated.
The PRs need approval before I can proceed with merging.

Assigning to you for further action.

I will have a look and do the needful.

Actions #9

Updated by Venky Shankar about 1 year ago

Reviewing this now...

Actions #10

Updated by Venky Shankar about 1 year ago

Venky Shankar wrote in #note-9:

Reviewing this now...

Got pulled into something else. Finishing this up.

Actions #11

Updated by Srinivasa Bharath Kanta about 1 year ago

  • Tags changed from cephfs to cephfs, core
Actions #12

Updated by Kumar Hemanth about 1 year ago

  • Tags changed from cephfs, core to cephfs, core, rgw
Actions #13

Updated by Kumar Hemanth about 1 year ago

  • Assignee changed from Venky Shankar to Laura Flores

Hi @Laura Flores :

Can you review https://tracker.ceph.com/issues/70262#note-8 and provide approval?

Actions #14

Updated by Laura Flores about 1 year ago

  • Status changed from QA Testing to QA Needs Approval
Actions #15

Updated by Kumar Hemanth about 1 year ago

@Laura Flores - Any update on the approval?

Actions #16

Updated by Laura Flores about 1 year ago

  • Assignee changed from Laura Flores to Kumar Hemanth

@Kumar Hemanth yes, apologies for the delay.

RADOS approved: https://tracker.ceph.com/projects/rados/wiki/SQUID#httpstrackercephcomissues70262

Not sure if any other approvals are needed, so reassigning to you w/o moving status to "QA Approved".

Actions #17

Updated by Kumar Hemanth about 1 year ago

  • Assignee changed from Kumar Hemanth to J. Eric Ivancich

Hi @J. Eric Ivancich -

Can you review https://tracker.ceph.com/issues/70262#note-8 and provide approval from RGW?

Actions #18

Updated by J. Eric Ivancich about 1 year ago

I found two issues and I need to get input from @Yuval Lifshitz and @Gal Salomon whether these need to be fixed.

Yuval: https://tracker.ceph.com/issues/71114
Gal: https://tracker.ceph.com/issues/71111

Actions #19

Updated by J. Eric Ivancich about 1 year ago

Apparently the s3select error is fixed by https://github.com/ceph/ceph/pull/62959 so that should be part of the next test run.

Still waiting on @Yuval Lifshitz for the other issue.

@Kumar Hemanth : I'll keep this tracker assigned to me until we get the other issue figured out.

Actions #20

Updated by Kumar Hemanth 11 months ago

Hi @J. Eric Ivancich , any update on this PR?

Actions #21

Updated by Kotresh Hiremath Ravishankar 11 months ago · Edited

Hi @Kumar Hemanth,

I have approved and merged the following two cephfs PRs based on the results.

https://github.com/ceph/ceph/pull/59005 - squid: pybind/ceph_argparse: Fix error message for ceph tell command
https://github.com/ceph/ceph/pull/60396 - squid: mgr/vol : shortening the name of helper method

The PR https://github.com/ceph/ceph/pull/61287 touching the common code was already merged.

I see the RGW trackers - both of them looks like existing?

I think we should close this tracker.

Actions #22

Updated by J. Eric Ivancich 11 months ago

Looking....

Actions #23

Updated by J. Eric Ivancich 11 months ago

  • Assignee changed from J. Eric Ivancich to Kumar Hemanth

RGW approved. Am I the last one?

Actions #24

Updated by Kumar Hemanth 11 months ago

  • Status changed from QA Needs Approval to QA Approved
Actions #25

Updated by Kumar Hemanth 11 months ago

  • Status changed from QA Approved to QA Closed
Actions

Also available in: Atom PDF