)]}'
{"lib/tempest":[{"author":{"_account_id":7350,"name":"Jordan Pittier","email":"jordan.pittier@gmail.com","username":"JordanP"},"change_message_id":"1a2e0dff6ea95b8e8034a1c6831be64e181cf55e","unresolved":false,"context_lines":[{"line_number":442,"context_line":"    # Using ``CINDER_ENABLED_BACKENDS``"},{"line_number":443,"context_line":"    if [[ -n \"$CINDER_ENABLED_BACKENDS\" ]] \u0026\u0026 [[ $CINDER_ENABLED_BACKENDS \u003d~ .*,.* ]]; then"},{"line_number":444,"context_line":"        iniset $TEMPEST_CONFIG volume-feature-enabled multi_backend \"True\""},{"line_number":445,"context_line":"        add_comma_seperator\u003d0"},{"line_number":446,"context_line":"        backends_list\u003d\u0027\u0027"},{"line_number":447,"context_line":"        # create a backend list, backend_names \u003d BACKEND_1,BACKEND_2"},{"line_number":448,"context_line":"        for be in ${CINDER_ENABLED_BACKENDS//,/ }; do"}],"source_content_type":"application/x-shellscript","patch_set":1,"id":"7aa08908_ddcb2271","line":445,"updated":"2016-06-06 15:01:01.000000000","message":"What about replacing all this with a single:\n\n  iniset $TEMPEST_CONFIG volume \"backend_names\" $CINDER_ENABLED_BACKENDS","commit_id":"59b78aa114be7e2e0e6ce4928f25cf9a7d90affb"},{"author":{"_account_id":11075,"name":"Benny Kopilov","email":"bkopilov@redhat.com","username":"bkopilov"},"change_message_id":"e0e1afc03c666765a533a9b8e5e6c7266506b814","unresolved":false,"context_lines":[{"line_number":442,"context_line":"    # Using ``CINDER_ENABLED_BACKENDS``"},{"line_number":443,"context_line":"    if [[ -n \"$CINDER_ENABLED_BACKENDS\" ]] \u0026\u0026 [[ $CINDER_ENABLED_BACKENDS \u003d~ .*,.* ]]; then"},{"line_number":444,"context_line":"        iniset $TEMPEST_CONFIG volume-feature-enabled multi_backend \"True\""},{"line_number":445,"context_line":"        add_comma_seperator\u003d0"},{"line_number":446,"context_line":"        backends_list\u003d\u0027\u0027"},{"line_number":447,"context_line":"        # create a backend list, backend_names \u003d BACKEND_1,BACKEND_2"},{"line_number":448,"context_line":"        for be in ${CINDER_ENABLED_BACKENDS//,/ }; do"}],"source_content_type":"application/x-shellscript","patch_set":1,"id":"7aa08908_f6c43b93","line":445,"in_reply_to":"7aa08908_ddcb2271","updated":"2016-06-06 15:39:09.000000000","message":"Hi , \nI think they extract from backend name the value , the format is :  CINDER_ENABLED_BACKENDS+\u003d,lvm:lvm1  (type:name)\n\nI tried to keep the same style\n\nHere is what i found under /lib/cinder:\n\n# Default backends\n# The backend format is type:name where type is one of the supported backend\n# types (lvm, nfs, etc) and name is the identifier used in the Cinder\n# configuration and for the volume type name.  Multiple backends are\n# comma-separated.\n# The old ``CINDER_MULTI_LVM_BACKEND\u003dTrue`` setting had a default of:\n# CINDER_ENABLED_BACKENDS\u003d${CINDER_ENABLED_BACKENDS:-lvm:lvmdriver-1,lvm:lvmdriver-2}","commit_id":"59b78aa114be7e2e0e6ce4928f25cf9a7d90affb"},{"author":{"_account_id":7173,"name":"Scott DAngelo","email":"scott.dangelo@gmail.com","username":"scottda"},"change_message_id":"700537756aa74afd679ccd10ae879d411771bfdf","unresolved":false,"context_lines":[{"line_number":442,"context_line":"    # Using ``CINDER_ENABLED_BACKENDS``"},{"line_number":443,"context_line":"    if [[ -n \"$CINDER_ENABLED_BACKENDS\" ]] \u0026\u0026 [[ $CINDER_ENABLED_BACKENDS \u003d~ .*,.* ]]; then"},{"line_number":444,"context_line":"        iniset $TEMPEST_CONFIG volume-feature-enabled multi_backend \"True\""},{"line_number":445,"context_line":"        add_comma_seperator\u003d0"},{"line_number":446,"context_line":"        backends_list\u003d\u0027\u0027"},{"line_number":447,"context_line":"        # create a backend list, backend_names \u003d BACKEND_1,BACKEND_2"},{"line_number":448,"context_line":"        for be in ${CINDER_ENABLED_BACKENDS//,/ }; do"}],"source_content_type":"application/x-shellscript","patch_set":1,"id":"7aa08908_706a611b","line":445,"in_reply_to":"7aa08908_f6c43b93","updated":"2016-06-14 19:57:04.000000000","message":"Yes, this is correct. We must strip out the key.","commit_id":"59b78aa114be7e2e0e6ce4928f25cf9a7d90affb"},{"author":{"_account_id":5803,"name":"Attila Fazekas","email":"afazekas@redhat.com","username":"afazekas"},"change_message_id":"816c2bb2f015642119e92d4d6ac5b45b66d15e06","unresolved":false,"context_lines":[{"line_number":442,"context_line":"    # Using ``CINDER_ENABLED_BACKENDS``"},{"line_number":443,"context_line":"    if [[ -n \"$CINDER_ENABLED_BACKENDS\" ]] \u0026\u0026 [[ $CINDER_ENABLED_BACKENDS \u003d~ .*,.* ]]; then"},{"line_number":444,"context_line":"        iniset $TEMPEST_CONFIG volume-feature-enabled multi_backend \"True\""},{"line_number":445,"context_line":"        add_comma_seperator\u003d0"},{"line_number":446,"context_line":"        backends_list\u003d\u0027\u0027"},{"line_number":447,"context_line":"        # create a backend list, backend_names \u003d BACKEND_1,BACKEND_2"},{"line_number":448,"context_line":"        for be in ${CINDER_ENABLED_BACKENDS//,/ }; do"}],"source_content_type":"application/x-shellscript","patch_set":1,"id":"7aa08908_d669d8fa","line":445,"in_reply_to":"7aa08908_f6c43b93","updated":"2016-06-08 05:04:55.000000000","message":"please declare it as local variable.","commit_id":"59b78aa114be7e2e0e6ce4928f25cf9a7d90affb"},{"author":{"_account_id":5803,"name":"Attila Fazekas","email":"afazekas@redhat.com","username":"afazekas"},"change_message_id":"816c2bb2f015642119e92d4d6ac5b45b66d15e06","unresolved":false,"context_lines":[{"line_number":443,"context_line":"    if [[ -n \"$CINDER_ENABLED_BACKENDS\" ]] \u0026\u0026 [[ $CINDER_ENABLED_BACKENDS \u003d~ .*,.* ]]; then"},{"line_number":444,"context_line":"        iniset $TEMPEST_CONFIG volume-feature-enabled multi_backend \"True\""},{"line_number":445,"context_line":"        add_comma_seperator\u003d0"},{"line_number":446,"context_line":"        backends_list\u003d\u0027\u0027"},{"line_number":447,"context_line":"        # create a backend list, backend_names \u003d BACKEND_1,BACKEND_2"},{"line_number":448,"context_line":"        for be in ${CINDER_ENABLED_BACKENDS//,/ }; do"},{"line_number":449,"context_line":"            if [ \"$add_comma_seperator\" -eq \"1\" ]; then"}],"source_content_type":"application/x-shellscript","patch_set":1,"id":"7aa08908_d6c278e8","line":446,"updated":"2016-06-08 05:04:55.000000000","message":"please declare it as local variable.","commit_id":"59b78aa114be7e2e0e6ce4928f25cf9a7d90affb"},{"author":{"_account_id":5803,"name":"Attila Fazekas","email":"afazekas@redhat.com","username":"afazekas"},"change_message_id":"816c2bb2f015642119e92d4d6ac5b45b66d15e06","unresolved":false,"context_lines":[{"line_number":447,"context_line":"        # create a backend list, backend_names \u003d BACKEND_1,BACKEND_2"},{"line_number":448,"context_line":"        for be in ${CINDER_ENABLED_BACKENDS//,/ }; do"},{"line_number":449,"context_line":"            if [ \"$add_comma_seperator\" -eq \"1\" ]; then"},{"line_number":450,"context_line":"                   backends_list+\u003d,${be##*:}"},{"line_number":451,"context_line":"            else"},{"line_number":452,"context_line":"            # first element in the list"},{"line_number":453,"context_line":"                backends_list+\u003d${be##*:}"}],"source_content_type":"application/x-shellscript","patch_set":1,"id":"7aa08908_b6fa4c41","line":450,"updated":"2016-06-08 05:04:55.000000000","message":"bash8 would like to see less spaces here.","commit_id":"59b78aa114be7e2e0e6ce4928f25cf9a7d90affb"},{"author":{"_account_id":5803,"name":"Attila Fazekas","email":"afazekas@redhat.com","username":"afazekas"},"change_message_id":"4ba28588abb68d0e5b1b13d3ff512e236d20a7cd","unresolved":false,"context_lines":[{"line_number":443,"context_line":"    if [[ -n \"$CINDER_ENABLED_BACKENDS\" ]] \u0026\u0026 [[ $CINDER_ENABLED_BACKENDS \u003d~ .*,.* ]]; then"},{"line_number":444,"context_line":"        iniset $TEMPEST_CONFIG volume-feature-enabled multi_backend \"True\""},{"line_number":445,"context_line":"        local i\u003d1"},{"line_number":446,"context_line":"        local be"},{"line_number":447,"context_line":"        for be in ${CINDER_ENABLED_BACKENDS//,/ }; do"},{"line_number":448,"context_line":"            local be_name\u003d${be##*:}"},{"line_number":449,"context_line":"            iniset $TEMPEST_CONFIG volume \"backend${i}_name\" \"$be_name\""}],"source_content_type":"application/x-shellscript","patch_set":2,"id":"7aa08908_1fbc05db","side":"PARENT","line":446,"updated":"2016-06-08 08:56:22.000000000","message":"be is still local.","commit_id":"18ce57e55c736542a0f328c6cdf0f521faf5e48a"},{"author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"change_message_id":"25135f5076297a8fff5796f759570fecfa4e2858","unresolved":false,"context_lines":[{"line_number":446,"context_line":"        local be"},{"line_number":447,"context_line":"        for be in ${CINDER_ENABLED_BACKENDS//,/ }; do"},{"line_number":448,"context_line":"            local be_name\u003d${be##*:}"},{"line_number":449,"context_line":"            iniset $TEMPEST_CONFIG volume \"backend${i}_name\" \"$be_name\""},{"line_number":450,"context_line":"            i\u003d$(( i + 1 ))"},{"line_number":451,"context_line":"        done"},{"line_number":452,"context_line":"    fi"}],"source_content_type":"application/x-shellscript","patch_set":3,"id":"5a9d85d2_ffbedefd","side":"PARENT","line":449,"updated":"2016-06-20 17:26:00.000000000","message":"Wouldn\u0027t this break any existing CI jobs that are relying on this being set this way?","commit_id":"18ce57e55c736542a0f328c6cdf0f521faf5e48a"},{"author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"change_message_id":"cd77579932062414fdeb4a0d55a67fcf5ad6241c","unresolved":false,"context_lines":[{"line_number":446,"context_line":"        local be"},{"line_number":447,"context_line":"        for be in ${CINDER_ENABLED_BACKENDS//,/ }; do"},{"line_number":448,"context_line":"            local be_name\u003d${be##*:}"},{"line_number":449,"context_line":"            iniset $TEMPEST_CONFIG volume \"backend${i}_name\" \"$be_name\""},{"line_number":450,"context_line":"            i\u003d$(( i + 1 ))"},{"line_number":451,"context_line":"        done"},{"line_number":452,"context_line":"    fi"}],"source_content_type":"application/x-shellscript","patch_set":3,"id":"5a9d85d2_05e55808","side":"PARENT","line":449,"in_reply_to":"5a9d85d2_829ade12","updated":"2016-06-20 18:22:00.000000000","message":"There is a drbd job in project-config that sets CINDER_ENABLED_BACKENDS and wasn\u0027t running the multibackend tests, but I\u0027m sure it\u0027s a non-voting job on whatever it\u0027s on anyway, so no one would have noticed.","commit_id":"18ce57e55c736542a0f328c6cdf0f521faf5e48a"},{"author":{"_account_id":5196,"name":"Matthew Treinish","email":"mtreinish@kortar.org","username":"treinish"},"change_message_id":"2911d3b494ed73b629ac745aafaeb3d4f34e2794","unresolved":false,"context_lines":[{"line_number":446,"context_line":"        local be"},{"line_number":447,"context_line":"        for be in ${CINDER_ENABLED_BACKENDS//,/ }; do"},{"line_number":448,"context_line":"            local be_name\u003d${be##*:}"},{"line_number":449,"context_line":"            iniset $TEMPEST_CONFIG volume \"backend${i}_name\" \"$be_name\""},{"line_number":450,"context_line":"            i\u003d$(( i + 1 ))"},{"line_number":451,"context_line":"        done"},{"line_number":452,"context_line":"    fi"}],"source_content_type":"application/x-shellscript","patch_set":3,"id":"5a9d85d2_829ade12","side":"PARENT","line":449,"in_reply_to":"5a9d85d2_ffbedefd","updated":"2016-06-20 18:12:22.000000000","message":"Tempest uses a list opt for this particular option. IIRC, oslo only accepts a comma separated list for ListOpt which probably just means devstack has been misconfiguring this for a while. If a CI depends on a bug misconfiguring tempest that probably is a bigger problem...","commit_id":"18ce57e55c736542a0f328c6cdf0f521faf5e48a"},{"author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"change_message_id":"9b495e6b06dc6273ba2255cace1119e4c81dfda7","unresolved":false,"context_lines":[{"line_number":455,"context_line":"                add_comma_seperator\u003d1"},{"line_number":456,"context_line":"            fi"},{"line_number":457,"context_line":"        done"},{"line_number":458,"context_line":"        iniset $TEMPEST_CONFIG volume \"backend_names\" \"$backends_list\""},{"line_number":459,"context_line":"    fi"},{"line_number":460,"context_line":""},{"line_number":461,"context_line":"    if [ $TEMPEST_VOLUME_DRIVER !\u003d \"default\" -o \\"}],"source_content_type":"application/x-shellscript","patch_set":3,"id":"5a9d85d2_e49013ef","line":458,"updated":"2016-06-20 17:17:50.000000000","message":"Does this have to be configurable in Tempest? Is this not something we can figure out from the Cinder API?","commit_id":"a38e7d539a066b3e0a62ceca1719e3926bbcb3ce"},{"author":{"_account_id":5196,"name":"Matthew Treinish","email":"mtreinish@kortar.org","username":"treinish"},"change_message_id":"2911d3b494ed73b629ac745aafaeb3d4f34e2794","unresolved":false,"context_lines":[{"line_number":455,"context_line":"                add_comma_seperator\u003d1"},{"line_number":456,"context_line":"            fi"},{"line_number":457,"context_line":"        done"},{"line_number":458,"context_line":"        iniset $TEMPEST_CONFIG volume \"backend_names\" \"$backends_list\""},{"line_number":459,"context_line":"    fi"},{"line_number":460,"context_line":""},{"line_number":461,"context_line":"    if [ $TEMPEST_VOLUME_DRIVER !\u003d \"default\" -o \\"}],"source_content_type":"application/x-shellscript","patch_set":3,"id":"5a9d85d2_02242e0b","line":458,"in_reply_to":"5a9d85d2_e49013ef","updated":"2016-06-20 18:12:22.000000000","message":"It needs to be configurable in tempest because we need to tell tempest which backends to use for operations that require that information. If a cloud has 4 cinder backends available how does tempest know which ones to use?","commit_id":"a38e7d539a066b3e0a62ceca1719e3926bbcb3ce"},{"author":{"_account_id":7118,"name":"Ian Wienand","email":"iwienand@redhat.com","username":"iwienand"},"change_message_id":"0a24b60e17aaae90395fc8afb3849976a274a636","unresolved":false,"context_lines":[{"line_number":454,"context_line":"                backends_list+\u003d${be##*:}"},{"line_number":455,"context_line":"                add_comma_seperator\u003d1"},{"line_number":456,"context_line":"            fi"},{"line_number":457,"context_line":"        done"},{"line_number":458,"context_line":"        iniset $TEMPEST_CONFIG volume \"backend_names\" \"$backends_list\""},{"line_number":459,"context_line":"    fi"},{"line_number":460,"context_line":""}],"source_content_type":"application/x-shellscript","patch_set":5,"id":"3aaa91ec_8ec77b11","line":457,"updated":"2016-06-28 21:53:55.000000000","message":"this seems rather complicated when you\u0027ve already got a comma separated list.  doesn\u0027t the list look like \"be:volume\" .. so it seems a simple sed could clear this out?\n\n $ echo \"glusterfs:fooo,netapp_isci:blah,vmdk:moo\" | sed \u0027s/:[^,]*//g\u0027","commit_id":"4e01122222865594dd3c23ba080b2b7bfc3f0d1d"},{"author":{"_account_id":11075,"name":"Benny Kopilov","email":"bkopilov@redhat.com","username":"bkopilov"},"change_message_id":"5ff8f9994c41d75029c298cfb8c890c813f552f5","unresolved":false,"context_lines":[{"line_number":454,"context_line":"                backends_list+\u003d${be##*:}"},{"line_number":455,"context_line":"                add_comma_seperator\u003d1"},{"line_number":456,"context_line":"            fi"},{"line_number":457,"context_line":"        done"},{"line_number":458,"context_line":"        iniset $TEMPEST_CONFIG volume \"backend_names\" \"$backends_list\""},{"line_number":459,"context_line":"    fi"},{"line_number":460,"context_line":""}],"source_content_type":"application/x-shellscript","patch_set":5,"id":"3aaa91ec_1e472e4c","line":457,"in_reply_to":"3aaa91ec_8ec77b11","updated":"2016-06-29 05:15:52.000000000","message":"I liked your suggestion , the only problem that we may hit when we have the same backend type , example:\necho \"glusterfs:fooo,glusterfs:blah,vmdk:moo\" | sed \u0027s/:[^,]*//g\u0027\n\nI will replace it with :\nCINDER_ENABLED_BACKENDS\u003dlvm:lvm1,lvm:lvm2\n\necho \"glusterfs:fooo,glusterfs:blah,vmdk:moo\" | sed \u0027s/:/_/g\u0027\nBenny","commit_id":"4e01122222865594dd3c23ba080b2b7bfc3f0d1d"},{"author":{"_account_id":7118,"name":"Ian Wienand","email":"iwienand@redhat.com","username":"iwienand"},"change_message_id":"2567f4584422ead423796cec174034d56f1438da","unresolved":false,"context_lines":[{"line_number":442,"context_line":"    # Using ``CINDER_ENABLED_BACKENDS``"},{"line_number":443,"context_line":"    if [[ -n \"$CINDER_ENABLED_BACKENDS\" ]] \u0026\u0026 [[ $CINDER_ENABLED_BACKENDS \u003d~ .*,.* ]]; then"},{"line_number":444,"context_line":"        iniset $TEMPEST_CONFIG volume-feature-enabled multi_backend \"True\""},{"line_number":445,"context_line":"        iniset $TEMPEST_CONFIG volume backend_names \"$CINDER_ENABLED_BACKENDS\""},{"line_number":446,"context_line":"    fi"},{"line_number":447,"context_line":"    if [ $TEMPEST_VOLUME_DRIVER !\u003d \"default\" -o \\"},{"line_number":448,"context_line":"        \"$TEMPEST_VOLUME_VENDOR\" !\u003d \"$TEMPEST_DEFAULT_VOLUME_VENDOR\" ]; then"}],"source_content_type":"application/x-shellscript","patch_set":6,"id":"3aaa91ec_8f29f59d","line":445,"updated":"2016-06-30 01:16:59.000000000","message":"so tempest handles it as is?  that seems even better, but wasn\u0027t quite the same as your previous change...","commit_id":"51556200391fa3c9595d798f297d3e63e787a88e"},{"author":{"_account_id":11075,"name":"Benny Kopilov","email":"bkopilov@redhat.com","username":"bkopilov"},"change_message_id":"db2a9297b81e3edc6e91ba4f6ef976d08e4e1408","unresolved":false,"context_lines":[{"line_number":442,"context_line":"    # Using ``CINDER_ENABLED_BACKENDS``"},{"line_number":443,"context_line":"    if [[ -n \"$CINDER_ENABLED_BACKENDS\" ]] \u0026\u0026 [[ $CINDER_ENABLED_BACKENDS \u003d~ .*,.* ]]; then"},{"line_number":444,"context_line":"        iniset $TEMPEST_CONFIG volume-feature-enabled multi_backend \"True\""},{"line_number":445,"context_line":"        iniset $TEMPEST_CONFIG volume backend_names \"$CINDER_ENABLED_BACKENDS\""},{"line_number":446,"context_line":"    fi"},{"line_number":447,"context_line":"    if [ $TEMPEST_VOLUME_DRIVER !\u003d \"default\" -o \\"},{"line_number":448,"context_line":"        \"$TEMPEST_VOLUME_VENDOR\" !\u003d \"$TEMPEST_DEFAULT_VOLUME_VENDOR\" ]; then"}],"source_content_type":"application/x-shellscript","patch_set":6,"id":"3aaa91ec_1154fffb","line":445,"in_reply_to":"3aaa91ec_8746f9bd","updated":"2016-06-30 07:22:59.000000000","message":"Hi Ian ,","commit_id":"51556200391fa3c9595d798f297d3e63e787a88e"},{"author":{"_account_id":11075,"name":"Benny Kopilov","email":"bkopilov@redhat.com","username":"bkopilov"},"change_message_id":"db2a9297b81e3edc6e91ba4f6ef976d08e4e1408","unresolved":false,"context_lines":[{"line_number":442,"context_line":"    # Using ``CINDER_ENABLED_BACKENDS``"},{"line_number":443,"context_line":"    if [[ -n \"$CINDER_ENABLED_BACKENDS\" ]] \u0026\u0026 [[ $CINDER_ENABLED_BACKENDS \u003d~ .*,.* ]]; then"},{"line_number":444,"context_line":"        iniset $TEMPEST_CONFIG volume-feature-enabled multi_backend \"True\""},{"line_number":445,"context_line":"        iniset $TEMPEST_CONFIG volume backend_names \"$CINDER_ENABLED_BACKENDS\""},{"line_number":446,"context_line":"    fi"},{"line_number":447,"context_line":"    if [ $TEMPEST_VOLUME_DRIVER !\u003d \"default\" -o \\"},{"line_number":448,"context_line":"        \"$TEMPEST_VOLUME_VENDOR\" !\u003d \"$TEMPEST_DEFAULT_VOLUME_VENDOR\" ]; then"}],"source_content_type":"application/x-shellscript","patch_set":6,"id":"3aaa91ec_56279d11","line":445,"in_reply_to":"3aaa91ec_8746f9bd","updated":"2016-06-30 07:22:59.000000000","message":"The part that bothers me , that the configuration is based on :\n# Default backends\n# The backend format is type:name where type is one of the supported backend\n# types (lvm, nfs, etc) and name is the identifier used in the Cinder\n# configuration and for the volume type name.  Multiple backends are\n\n\nSo the string format is : type:name  example: lvm:lvmTest\n\nWe will need to set into tempest.conf the name only .","commit_id":"51556200391fa3c9595d798f297d3e63e787a88e"},{"author":{"_account_id":11075,"name":"Benny Kopilov","email":"bkopilov@redhat.com","username":"bkopilov"},"change_message_id":"d3472a6f52b960b4cf8cb2cf404874c44684bdfe","unresolved":false,"context_lines":[{"line_number":442,"context_line":"    # Using ``CINDER_ENABLED_BACKENDS``"},{"line_number":443,"context_line":"    if [[ -n \"$CINDER_ENABLED_BACKENDS\" ]] \u0026\u0026 [[ $CINDER_ENABLED_BACKENDS \u003d~ .*,.* ]]; then"},{"line_number":444,"context_line":"        iniset $TEMPEST_CONFIG volume-feature-enabled multi_backend \"True\""},{"line_number":445,"context_line":"        iniset $TEMPEST_CONFIG volume backend_names \"$CINDER_ENABLED_BACKENDS\""},{"line_number":446,"context_line":"    fi"},{"line_number":447,"context_line":"    if [ $TEMPEST_VOLUME_DRIVER !\u003d \"default\" -o \\"},{"line_number":448,"context_line":"        \"$TEMPEST_VOLUME_VENDOR\" !\u003d \"$TEMPEST_DEFAULT_VOLUME_VENDOR\" ]; then"}],"source_content_type":"application/x-shellscript","patch_set":6,"id":"3aaa91ec_8746f9bd","line":445,"in_reply_to":"3aaa91ec_8f29f59d","updated":"2016-06-30 03:08:21.000000000","message":"Yes , it will handle it , the CINDER_ENABLED_BACKENDS is a list of strings seperated by comma , same as  backend_names in tempest.\n\nWe need to enable multi-backend support in tempest upstream, there are tests but they are not running. multi-backend is a very basic function and is used by customers !\nBenny","commit_id":"51556200391fa3c9595d798f297d3e63e787a88e"},{"author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"change_message_id":"5f927b0a18b45f954f9a47dc00e24af0e51e83c7","unresolved":false,"context_lines":[{"line_number":462,"context_line":"    fi"},{"line_number":463,"context_line":""},{"line_number":464,"context_line":"    # Using ``CINDER_ENABLED_BACKENDS``"},{"line_number":465,"context_line":"    if [[ -n \"$CINDER_ENABLED_BACKENDS\" ]] \u0026\u0026 [[ $CINDER_ENABLED_BACKENDS \u003d~ .*,.* ]]; then"},{"line_number":466,"context_line":"        iniset $TEMPEST_CONFIG volume-feature-enabled multi_backend \"True\""},{"line_number":467,"context_line":"        local add_comma_seperator\u003d0"},{"line_number":468,"context_line":"        local backends_list\u003d\u0027\u0027"}],"source_content_type":"application/x-shellscript","patch_set":11,"id":"bacf61ea_69901a44","line":465,"range":{"start_line":465,"start_character":15,"end_line":465,"end_character":38},"updated":"2016-08-04 16:32:34.000000000","message":"Why isn\u0027t this just a passthrough to tempest.conf? Why do we need to parse CINDER_ENABLED_BACKENDS to split on the colon? e.g.:\n\n# CINDER_ENABLED_BACKENDS\u003d${CINDER_ENABLED_BACKENDS:-lvm:lvmdriver-1,lvm:lvmdriver-2}\n\nSo backend_names in tempest would be \u0027lvmdriver-1,lvmdriver-2\u0027 right? But is that what is defined in cinder.conf?\n\nI see the other code in devstack parsing that variable is splitting it between backend type and name, I\u0027m just wondering how Tempest is supposed to be handling it (only name and not type)?","commit_id":"7688122689dcdc5915725392177f4234abdfed4d"},{"author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"change_message_id":"43cd251636d64742164949b8fb3ccf5a7c1369ae","unresolved":false,"context_lines":[{"line_number":462,"context_line":"    fi"},{"line_number":463,"context_line":""},{"line_number":464,"context_line":"    # Using ``CINDER_ENABLED_BACKENDS``"},{"line_number":465,"context_line":"    if [[ -n \"$CINDER_ENABLED_BACKENDS\" ]] \u0026\u0026 [[ $CINDER_ENABLED_BACKENDS \u003d~ .*,.* ]]; then"},{"line_number":466,"context_line":"        iniset $TEMPEST_CONFIG volume-feature-enabled multi_backend \"True\""},{"line_number":467,"context_line":"        local add_comma_seperator\u003d0"},{"line_number":468,"context_line":"        local backends_list\u003d\u0027\u0027"}],"source_content_type":"application/x-shellscript","patch_set":11,"id":"bacf61ea_e9036a40","line":465,"range":{"start_line":465,"start_character":15,"end_line":465,"end_character":38},"in_reply_to":"bacf61ea_69901a44","updated":"2016-08-04 16:37:30.000000000","message":"I guess tempest creates volume types using the backend name:\n\nhttps://github.com/openstack/tempest/blob/master/tempest/api/volume/admin/test_multi_backend.py#L56\n\nAnd I guess that\u0027s what\u0027s used for the capabilities for the backend:\n\nhttp://developer.openstack.org/api-ref-blockstorage-v2.html#showBackendCapabilities\n\nSo OK, I guess this is just backend name and it\u0027s necessary to split the type from the name.","commit_id":"7688122689dcdc5915725392177f4234abdfed4d"}]}