Update patch set 3
Patch Set 3: (1 comment) Patch-set: 3 Attention: {"person_ident":"Gerrit User 13425 \u003c13425@4a232e18-c5a9-48ee-94c0-e04e7cca6543\u003e","operation":"ADD","reason":"\u003cGERRIT_ACCOUNT_31779\u003e replied on the change"} Attention: {"person_ident":"Gerrit User 31779 \u003c31779@4a232e18-c5a9-48ee-94c0-e04e7cca6543\u003e","operation":"REMOVE","reason":"\u003cGERRIT_ACCOUNT_31779\u003e replied on the change"}
This commit is contained in:
parent
c58cb8dc2b
commit
b73c4c8cfd
|
@ -16,6 +16,24 @@
|
|||
"message": "I thought the current PowerFlex driver could only create volumes that were rounded to 8Gb anyway:https://github.com/openstack/cinder/blob/393c2e4ad90c05ebf28cc3a2c65811d7e1e0bc18/cinder/volume/drivers/dell_emc/powerflex/options.py#L109\nso there currently is no way to create a 5GB volume and extend to 7GB as the inital 5GB creation would fail. This means that there is no need for this.\nPlus, I am only aware of PowerFlex having size rounding requirements, so this seems a lot of trouble in core code for one vendor specific \u0027feature\u0027.\n-1 so you can comment.",
|
||||
"revId": "d614688d8b1c3c24235d131015cf269bd3dd833d",
|
||||
"serverId": "4a232e18-c5a9-48ee-94c0-e04e7cca6543"
|
||||
},
|
||||
{
|
||||
"unresolved": false,
|
||||
"key": {
|
||||
"uuid": "44e04a68_49c2243b",
|
||||
"filename": "/PATCHSET_LEVEL",
|
||||
"patchSetId": 3
|
||||
},
|
||||
"lineNbr": 0,
|
||||
"author": {
|
||||
"id": 31779
|
||||
},
|
||||
"writtenOn": "2023-02-09T14:32:51Z",
|
||||
"side": 1,
|
||||
"message": "Hi Simon, you are right, powerflex uses 8GB only as volume size.\nThe need here is to have both sizes (what the user wants, and what is actually allocated on the backend) displayed to the user.\nWe have discussed it in our PTG, and it was the solution that we all agreed. Let me know I have misunderstood.",
|
||||
"parentUuid": "bb57d068_4d18f90d",
|
||||
"revId": "d614688d8b1c3c24235d131015cf269bd3dd833d",
|
||||
"serverId": "4a232e18-c5a9-48ee-94c0-e04e7cca6543"
|
||||
}
|
||||
]
|
||||
}
|
Loading…
Reference in New Issue