openstack
/
devstack-plugin-ceph
Code Issues Proposed changes

Update patch set 12 

Patch Set 12:

(1 comment)

Patch-set: 12
Label: Code-Review=0
Attention: {"person_ident":"Gerrit User 25402 \u003c25402@4a232e18-c5a9-48ee-94c0-e04e7cca6543\u003e","operation":"REMOVE","reason":"Francesco Pantano replied on the change"}
Attention: {"person_ident":"Gerrit User 16643 \u003c16643@4a232e18-c5a9-48ee-94c0-e04e7cca6543\u003e","operation":"ADD","reason":"Francesco Pantano replied on the change"}
This commit is contained in:
Gerrit User 25402 2022-02-24 08:18:03 +00:00 committed by Gerrit Code Review
parent d7ecb5b86a
commit 515b724cc0
1 changed files with 24 additions and 0 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

24
ae18d31b8f27204340e5531e12f094ae33acaef3
View File

@ -40,6 +40,30 @@
"revId": "ae18d31b8f27204340e5531e12f094ae33acaef3",
"serverId": "4a232e18-c5a9-48ee-94c0-e04e7cca6543"
},
{
"unresolved": true,
"key": {
"uuid": "ce99858f_682d9166",
"filename": "devstack/lib/cephadm",
"patchSetId": 12
},
"lineNbr": 304,
"author": {
"id": 25402
},
"writtenOn": "2022-02-24T08:18:03Z",
"side": 1,
"message": "+1, when `ceph fs volume create ` is called, 2 mds daemons are created. We can problably remove that line and save 1 command.",
"parentUuid": "7a09a996_df65cd8d",
"range": {
"startLine": 302,
"startChar": 0,
"endLine": 304,
"endChar": 31
},
"revId": "ae18d31b8f27204340e5531e12f094ae33acaef3",
"serverId": "4a232e18-c5a9-48ee-94c0-e04e7cca6543"
},
{
"unresolved": true,
"key": {