openinfra
/
python-tempestconf
Code Issues Proposed changes

Update patch set 13 

Patch Set 13:

(1 comment)

Patch-set: 13
Attention: {"person_ident":"Gerrit User 10459 \u003c10459@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-02-16 17:22:28 +00:00 committed by Gerrit Code Review
parent d8b116ccda
commit d791182731
1 changed files with 18 additions and 0 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

18
c72239914116fde2e4a90bedd1b14b2f263aae47
View File

@ -243,6 +243,24 @@
"parentUuid": "09a137f8_14b36ed9",
"revId": "c72239914116fde2e4a90bedd1b14b2f263aae47",
"serverId": "4a232e18-c5a9-48ee-94c0-e04e7cca6543"
},
{
"unresolved": true,
"key": {
"uuid": "331d5110_78eb83c2",
"filename": "config_tempest/services/image.py",
"patchSetId": 13
},
"lineNbr": 61,
"author": {
"id": 9303
},
"writtenOn": "2022-02-16T17:22:28Z",
"side": 1,
"message": "1) import image works a) if glance is configured through multistore configuration, regardless of the amount of storages (one or more) \n\nThis is correct\n\n2) import image works if glance is configured to have multiple stores, which requires the new multistore way of configuring.\n\nThis is totally wrong\n\n\nAs of now you can configure stores two ways;\n\n1. Old way\n[glance_store]\nstores \u003d file,swift,http\ndefault_store \u003d file\nfilesystem_store_datadir \u003d /path/to/file/system\n\n\n2. Multistore way\n2.1 more than one store\n[Default]\nenabled_backends \u003d fast:rbd,slow:file\n\n[glance_store]\ndefault_backend \u003d fast\n\n[fast]\n\u003crbd_specific_options\u003e\n\n[slow]\n\u003cfile_store_specific_options\u003e\n\n2.2 only one store\n[Default]\nenabled_backends \u003d fast:rbd\n\n[glance_store]\ndefault_backend \u003d fast\n\n[fast]\n\u003crbd_specific_options\u003e\n\nAFAIK, this option is required to run tests to import image in multiple stores and copy existing image in multiple stores. The first case will be possible to run even if single store is configured, but the later one i.e. copy image will not work if there is only single store configured.",
"parentUuid": "5e3f7e1d_6551b7a1",
"revId": "c72239914116fde2e4a90bedd1b14b2f263aae47",
"serverId": "4a232e18-c5a9-48ee-94c0-e04e7cca6543"
}
]
}