Skip to content

squid: pybind/ceph_argparse: Fix error message for ceph tell command#59005

Merged
kotreshhr merged 2 commits intoceph:squidfrom
neesingh-rh:wip-67300-squid
Jun 17, 2025
Merged

squid: pybind/ceph_argparse: Fix error message for ceph tell command#59005
kotreshhr merged 2 commits intoceph:squidfrom
neesingh-rh:wip-67300-squid

Conversation

@neesingh-rh
Copy link
Copy Markdown
Contributor

@neesingh-rh neesingh-rh commented Aug 2, 2024

backport tracker: https://tracker.ceph.com/issues/67300

backport of #51332
parent tracker: https://tracker.ceph.com/issues/59624

this backport was staged using ceph-backport.sh version 16.0.0.6848
find the latest version at https://github.com/ceph/ceph/blob/main/src/script/ceph-backport.sh

@neesingh-rh neesingh-rh added this to the squid milestone Aug 2, 2024
@github-actions github-actions Bot added cephfs Ceph File System tests labels Aug 2, 2024
@github-actions
Copy link
Copy Markdown

github-actions Bot commented Aug 5, 2024

This pull request can no longer be automatically merged: a rebase is needed and changes have to be manually resolved

neeraj pratap singh added 2 commits August 5, 2024 17:24
pybind/ceph_argparse: fixing error message for ceph tell command
7748a3b 
Fixes: https://tracker.ceph.com/issues/59624
Signed-off-by: Neeraj Pratap Singh <neesingh@redhat.com>
(cherry picked from commit 2811bb0)
qa: add test for ceph tell with unknown cephtype
6d28415 
Fixes: https://tracker.ceph.com/issues/59624
Signed-off-by: Neeraj Pratap Singh <neesingh@redhat.com>
(cherry picked from commit decf32e)

Conflicts:
	qa/tasks/cephfs/test_misc.py
@rishabh-d-dave
Copy link
Copy Markdown
Contributor

rishabh-d-dave commented Sep 8, 2024

jenkins test make check

1 similar comment
@neesingh-rh
Copy link
Copy Markdown
Contributor Author

neesingh-rh commented Oct 17, 2024

jenkins test make check

@github-actions
Copy link
Copy Markdown

github-actions Bot commented Dec 16, 2024

This pull request has been automatically marked as stale because it has not had any activity for 60 days. It will be closed if no further activity occurs for another 30 days.
If you are a maintainer or core committer, please follow-up on this pull request to identify what steps should be taken by the author to move this proposed change forward.
If you are the author of this pull request, thank you for your proposed contribution. If you believe this change is still appropriate, please ensure that any feedback has been addressed and ask for a code review.

@github-actions github-actions Bot added the stale label Dec 16, 2024
@github-actions github-actions Bot removed the stale label Jan 13, 2025
@hkadam134
Copy link
Copy Markdown
Contributor

hkadam134 commented Mar 3, 2025

PR is under test - https://tracker.ceph.com/issues/70262

@hkadam134
Copy link
Copy Markdown
Contributor

hkadam134 commented Mar 11, 2025

No New Issues found,
QA Run Analysis - https://tracker.ceph.com/issues/70262#note-4
wiki page for existing Issues - https://tracker.ceph.com/issues/70262#note-5

PR is Tested and needs approval to merge.

@ljflores
Copy link
Copy Markdown
Member

ljflores commented Apr 23, 2025

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

tchaikov
tchaikov approved these changes Apr 24, 2025
View reviewed changes
Copy link
Copy Markdown
Contributor

@tchaikov tchaikov left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm

@kotreshhr
Copy link
Copy Markdown
Contributor

kotreshhr commented Jun 17, 2025

Approved based on - https://tracker.ceph.com/projects/cephfs/wiki/Squid#wip-hemanth-testing-2025-03-03-1505-squid

@kotreshhr kotreshhr merged commit 2d28e14 into ceph:squid Jun 17, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@tchaikov tchaikov tchaikov approved these changes

@vshankar vshankar Awaiting requested review from vshankar

@rishabh-d-dave rishabh-d-dave Awaiting requested review from rishabh-d-dave

Assignees

No one assigned

Labels

cephfs Ceph File System needs-qa pybind tests wip-hemanth-testing

Projects

None yet

Milestone

squid

Development

Successfully merging this pull request may close these issues.

6 participants

@neesingh-rh @rishabh-d-dave @hkadam134 @ljflores @kotreshhr @tchaikov