Project

General

Profile

Actions

Bug #55242

closed

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

Added by Dhairya Parmar about 2 years ago. Updated over 1 year 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 2 (0 open2 closed)

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

Updated by Venky Shankar about 2 years ago

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

Updated by Dhairya Parmar about 2 years ago

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

Updated by Venky Shankar almost 2 years ago

  • Status changed from Fix Under Review to Pending Backport
Actions #4

Updated by Backport Bot almost 2 years ago

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

Updated by Backport Bot almost 2 years ago

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

Updated by Dhairya Parmar over 1 year ago

  • Status changed from Pending Backport to Resolved
Actions

Also available in: Atom PDF