Project

General

Profile

Feature #36560

mgr/dashboard: Allow renaming an existing Pool

Added by Lenz Grimmer about 2 months ago. Updated 20 days ago.

Status:
Resolved
Priority:
Normal
Assignee:
Category:
dashboard/pools
Target version:
Start date:
10/23/2018
Due date:
% Done:

0%

Source:
Tags:
Backport:
Reviewed:
Affected Versions:
Pull request ID:

Description

According to the documentation it is possible to rename an existing Ceph Pool. The Pool Management in the Dashboard should support this functionality as well.

History

#1 Updated by Dan Guo about 1 month ago

Hi, I did a little implementation of this feature, just like fixing a bug, adding some code without changing the structure of original code, I don't known if I did is or not could meets the requirements, but I can post it here if someone is interested to see.
I will be very happy if my work could make things go well.

#2 Updated by Lenz Grimmer about 1 month ago

Dan Guo wrote:

Hi, I did a little implementation of this feature, just like fixing a bug, adding some code without changing the structure of original code, I don't known if I did is or not could meets the requirements, but I can post it here if someone is interested to see.
I will be very happy if my work could make things go well.

Sounds great - would you mind submitting a "RFC" pull request about this? We'd be grateful for your help here.

#3 Updated by Dan Guo about 1 month ago

Lenz Grimmer wrote:

Sounds great - would you mind submitting a "RFC" pull request about this? We'd be grateful for your help here.

I have submmited a PR just did it like normal. Very glad hear from everyone.

#4 Updated by Lenz Grimmer about 1 month ago

  • Status changed from New to Need Review
  • Assignee set to Dan Guo
  • Pull request ID set to 25107

Thank you! Assigning this issue to you then.

#6 Updated by Lenz Grimmer 20 days ago

  • Status changed from Need Review to Resolved

#7 Updated by Lenz Grimmer 20 days ago

Merged! Thanks a lot for your contribution.

#8 Updated by Dan Guo 20 days ago

Lenz Grimmer wrote:

Merged! Thanks a lot for your contribution.

It's my pleasure.

Also available in: Atom PDF