Project

General

Profile

Actions
Copy link

Bug #64042

open

mgr/snap_schedule: Adding retention which already exists gives improper error message

Added by Milind Changire about 2 years ago. Updated 2 months ago.

Status:
Pending Backport
Priority:
Normal
Assignee:
Mahesh Mohan
Category:
Administration/Usability
Target version:
Ceph - v21.0.0
% Done:

0%

Source:
Development
Backport:
reef,squid,tentacle
Regression:
No
Severity:
3 - minor
Reviewed:
Affected Versions:
ceph-qa-suite:
Component(FS):
mgr/snap_schedule
Labels (FS):
Pull request ID:
59161
Tags (freeform):
temp-assign backport_processed
Merge Commit:
1275f1a3ce0fb9cf2fa9fabc64adcda0f520837a
Fixed In:
v20.3.0-532-g1275f1a3ce
Released In:
Upkeep Timestamp:
2025-07-14T17:41:30+00:00

Description

This is a message formatting error along with an incorrect errno error.

Snap Schedule Retention add for value which already exists generates error as expected but with improper message.

Error message "Error ENOENT: Retention for M is already present with value10. Please remove first" in below example mentions retention value 10 for M interval already exists, but represent existing value as 'value10'. There is space missing in 'value10', it had to be 'value 10' instead.

Example cmd:
[root@ceph-sumar-regression-5ohzjt-node6 ~]# ceph fs snap-schedule retention add /volumes/svg_1/sv1 M 10 --fs cephfs
Retention added to path /volumes/svg_1/sv1
[root@ceph-sumar-regression-5ohzjt-node6 ~]# ceph fs snap-schedule retention add /volumes/svg_1/sv1 M 10 --fs cephfs
Error ENOENT: Retention for M is already present with value10. Please remove first

Related issues 3 (2 open — 1 closed)

Copied to CephFS - Backport #71436: reef: mgr/snap_schedule: Adding retention which already exists gives improper error messageResolvedJos CollinActions
Copied to CephFS - Backport #71437: tentacle: mgr/snap_schedule: Adding retention which already exists gives improper error messageQA TestingMahesh MohanActions
Copied to CephFS - Backport #71438: squid: mgr/snap_schedule: Adding retention which already exists gives improper error messageQA TestingJos CollinActions
Actions
Copy link
 #1

Updated by Milind Changire about 2 years ago

  • Pull request ID set to 55193
Actions
Copy link
 #2

Updated by Venky Shankar about 2 years ago

  • Category set to Administration/Usability
  • Status changed from New to Triaged
  • Assignee set to Milind Changire
  • Target version set to v19.0.0
  • Backport set to quincy, reef
Actions
Copy link
 #3

Updated by Venky Shankar about 2 years ago

  • Status changed from Triaged to Fix Under Review
  • Backport changed from quincy, reef to quincy,reef,squid
Actions
Copy link
 #4

Updated by Milind Changire over 1 year ago

  • Pull request ID changed from 55193 to 59161
  • PR#59161 continued from PR#55193
  • PR#55193 got locked up after pushing changes to a closed PR
Actions
Copy link
 #5

Updated by Konstantin Shalygin about 1 year ago

  • Backport changed from quincy,reef,squid to reef,squid
Actions
Copy link
 #6

Updated by Venky Shankar 10 months ago

  • Status changed from Fix Under Review to Pending Backport
  • Target version changed from v19.0.0 to v21.0.0
  • Source set to Development
  • Backport changed from reef,squid to reef,squid,tentacle
  • Component(FS) mgr/snap_schedule added
Actions
Copy link
 #7

Updated by Upkeep Bot 10 months ago

  • Copied to Backport #71436: reef: mgr/snap_schedule: Adding retention which already exists gives improper error message added
Actions
Copy link
 #8

Updated by Upkeep Bot 10 months ago

  • Copied to Backport #71437: tentacle: mgr/snap_schedule: Adding retention which already exists gives improper error message added
Actions
Copy link
 #9

Updated by Upkeep Bot 10 months ago

  • Copied to Backport #71438: squid: mgr/snap_schedule: Adding retention which already exists gives improper error message added
Actions
Copy link
 #10

Updated by Upkeep Bot 10 months ago

  • Tags (freeform) set to backport_processed
Actions
Copy link
 #11

Updated by Upkeep Bot 9 months ago

  • Merge Commit set to 1275f1a3ce0fb9cf2fa9fabc64adcda0f520837a
  • Fixed In set to v20.3.0-532-g1275f1a3ce0
  • Upkeep Timestamp set to 2025-07-09T14:05:40+00:00
Actions
Copy link
 #12

Updated by Upkeep Bot 8 months ago

  • Fixed In changed from v20.3.0-532-g1275f1a3ce0 to v20.3.0-532-g1275f1a3ce
  • Upkeep Timestamp changed from 2025-07-09T14:05:40+00:00 to 2025-07-14T17:41:30+00:00
Actions
Copy link
 #13

Updated by Venky Shankar 2 months ago

  • Assignee changed from Milind Changire to Mahesh Mohan
  • Tags (freeform) changed from backport_processed to temp-assign

@Milind Changire is moving away from CephFS development. Assigning this to @Mahesh Mohan in the interim.

Actions
Copy link
 #14

Updated by Upkeep Bot 2 months ago

  • Tags (freeform) changed from temp-assign to temp-assign backport_processed
Actions
Copy link

Also available in: Atom PDF