Project

General

Profile

Bug #55242

cephfs-shell: put command should accept both path mandatorily and validate local_path

Added by Dhairya Parmar 8 months ago. Updated 4 months ago.

Status:
Resolved
Priority:
Normal
Category:
Correctness/Safety
Target version:
% Done:

0%

Source:
Tags:
Backport:
quincy, pacific
Regression:
No
Severity:
3 - minor
Reviewed:
Affected Versions:
ceph-qa-suite:
Component(FS):
cephfs-shell
Labels (FS):
Pull request ID:
Crash signature (v1):
Crash signature (v2):

Description

Currently, there are no checks to make sure, local_path is valid. For instance for a file "helloworld" at /home/dparmar/foo,

[dparmar@fedora foo]$ pwd
/home/dparmar/foo
[dparmar@fedora foo]$ ls
helloworld

When tried using "put" command for helloworld1 instead of helloworld,
CephFS:~/>>> put /home/dparmar/foo/helloworld1 /helloworld
CephFS:~/>>> ls
CephFS:~/>>> 

It shows no warning or any other error message, but it should.


Related issues

Copied to CephFS - Backport #55625: pacific: cephfs-shell: put command should accept both path mandatorily and validate local_path Resolved
Copied to CephFS - Backport #55626: quincy: cephfs-shell: put command should accept both path mandatorily and validate local_path Resolved

History

#1 Updated by Venky Shankar 8 months ago

  • Assignee set to Dhairya Parmar
  • Target version set to v18.0.0
  • Backport set to quincy, pacific

#2 Updated by Dhairya Parmar 8 months ago

  • Status changed from New to Fix Under Review
  • Pull request ID set to 45829

#3 Updated by Venky Shankar 7 months ago

  • Status changed from Fix Under Review to Pending Backport

#4 Updated by Backport Bot 7 months ago

  • Copied to Backport #55625: pacific: cephfs-shell: put command should accept both path mandatorily and validate local_path added

#5 Updated by Backport Bot 7 months ago

  • Copied to Backport #55626: quincy: cephfs-shell: put command should accept both path mandatorily and validate local_path added

#6 Updated by Dhairya Parmar 4 months ago

  • Status changed from Pending Backport to Resolved

Also available in: Atom PDF