openinfra
/
python-tempestconf
Code Issues Proposed changes

Update patch set 3 

Patch Set 3: Code-Review-1

(1 comment)

Patch-set: 3
Reviewer: Gerrit User 9303 <9303@4a232e18-c5a9-48ee-94c0-e04e7cca6543>
Label: Code-Review=-1
Attention: {"person_ident":"Gerrit User 30742 \u003c30742@4a232e18-c5a9-48ee-94c0-e04e7cca6543\u003e","operation":"ADD","reason":"Abhishek Kekane replied on the change"}
Attention: {"person_ident":"Gerrit User 9303 \u003c9303@4a232e18-c5a9-48ee-94c0-e04e7cca6543\u003e","operation":"REMOVE","reason":"Abhishek Kekane replied on the change"}
This commit is contained in:
Gerrit User 9303 2022-01-10 14:12:49 +00:00 committed by Gerrit Code Review
parent 0676dee387
commit 6ef0f6737d
1 changed files with 23 additions and 0 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

23
b284f4f7b7dfdce06054bdb546956d9c11832e32
View File

@ -52,6 +52,29 @@
"revId": "b284f4f7b7dfdce06054bdb546956d9c11832e32",
"serverId": "4a232e18-c5a9-48ee-94c0-e04e7cca6543"
},
{
"unresolved": true,
"key": {
"uuid": "4855f0fd_fdc336af",
"filename": "config_tempest/services/image.py",
"patchSetId": 3
},
"lineNbr": 62,
"author": {
"id": 9303
},
"writtenOn": "2022-01-10T14:12:49Z",
"side": 1,
"message": "There is a possibility that I can configure multiplestores with actual single store in it;\nLike;\n\nenabled_backends \u003d efficient:rbd\n\nAlso going forward we are going to get rid of old way of store configuration, so devstack will define default store as \n\nenabled_backends \u003d default:file\n\nI think this code should be generic and should work with num_stores \u003d\u003d 1",
"range": {
"startLine": 60,
"startChar": 8,
"endLine": 62,
"endChar": 69
},
"revId": "b284f4f7b7dfdce06054bdb546956d9c11832e32",
"serverId": "4a232e18-c5a9-48ee-94c0-e04e7cca6543"
},
{
"unresolved": true,
"key": {