)]}'
{"rally/plugins/openstack/scenarios/cinder/volumes.py":[{"author":{"_account_id":12395,"name":"Roman Vasylets","email":"pomeo92@gmail.com","username":"rvasilets"},"change_message_id":"08e450d570637233b797ed10e5fe6a8cdd701a3f","unresolved":false,"context_lines":[{"line_number":279,"context_line":"    @validation.required_openstack(users\u003dTrue)"},{"line_number":280,"context_line":"    @scenario.configure(context\u003d{\"cleanup\": [\"cinder\", \"nova\"]})"},{"line_number":281,"context_line":"    def create_and_attach_volume(self, size, image, flavor,"},{"line_number":282,"context_line":"                                 create_volume_kwargs\u003dNone, **kwargs):"},{"line_number":283,"context_line":"        \"\"\"Create a VM and attach a volume to it."},{"line_number":284,"context_line":""},{"line_number":285,"context_line":"        Simple test to create a VM and attach a volume, then"}],"source_content_type":"text/x-python","patch_set":1,"id":"7a740942_08308d00","line":282,"updated":"2015-12-04 23:34:01.000000000","message":"Its better to rename kwargs to boot_server_kwargs.","commit_id":"acab69adcb0c88ad64d2f8814aa91ffbd3a990a7"},{"author":{"_account_id":15371,"name":"Tom Patzig","email":"tom.patzig@sap.com","username":"tpatzig"},"change_message_id":"60276b05a489a1124d9bf9a6cf98e1fb1eb1b19a","unresolved":false,"context_lines":[{"line_number":279,"context_line":"    @validation.required_openstack(users\u003dTrue)"},{"line_number":280,"context_line":"    @scenario.configure(context\u003d{\"cleanup\": [\"cinder\", \"nova\"]})"},{"line_number":281,"context_line":"    def create_and_attach_volume(self, size, image, flavor,"},{"line_number":282,"context_line":"                                 create_volume_kwargs\u003dNone, **kwargs):"},{"line_number":283,"context_line":"        \"\"\"Create a VM and attach a volume to it."},{"line_number":284,"context_line":""},{"line_number":285,"context_line":"        Simple test to create a VM and attach a volume, then"}],"source_content_type":"text/x-python","patch_set":1,"id":"7a740942_afd138cf","line":282,"in_reply_to":"7a740942_08308d00","updated":"2015-12-05 11:23:42.000000000","message":"I thought about that as well; But this would break the api and existing task definitions will no longer work.","commit_id":"acab69adcb0c88ad64d2f8814aa91ffbd3a990a7"},{"author":{"_account_id":8491,"name":"Illia Khudoshyn","email":"ikhudoshyn@mirantis.com","username":"ikhudoshyn"},"change_message_id":"c1fac7c04cace0ba7dd99b6c95ac1763985abb1f","unresolved":false,"context_lines":[{"line_number":279,"context_line":"    @validation.required_openstack(users\u003dTrue)"},{"line_number":280,"context_line":"    @scenario.configure(context\u003d{\"cleanup\": [\"cinder\", \"nova\"]})"},{"line_number":281,"context_line":"    def create_and_attach_volume(self, size, image, flavor,"},{"line_number":282,"context_line":"                                 create_volume_kwargs\u003dNone, **kwargs):"},{"line_number":283,"context_line":"        \"\"\"Create a VM and attach a volume to it."},{"line_number":284,"context_line":""},{"line_number":285,"context_line":"        Simple test to create a VM and attach a volume, then"}],"source_content_type":"text/x-python","patch_set":2,"id":"7a740942_ce7d861d","line":282,"updated":"2015-12-07 11:05:58.000000000","message":"Having part of kwargs in a dict and the rest of them as kwargs could be confusing. I believe two dedicated dicts would fit better here, like\n\ncreate_and_attach_volume(self, size, image, flavor,\t                                create_volume_params\u003dNone, create_vm_params\u003dNone)\n\nAnd using \u0027_kwargs\u0027 suffix as a part of the particular keyword is a little bit weird.","commit_id":"42952641f3ee52aab0e8642e56338cdd20e1bf13"},{"author":{"_account_id":15371,"name":"Tom Patzig","email":"tom.patzig@sap.com","username":"tpatzig"},"change_message_id":"72c0a7b718810f01a1ff0e63802f40cab369492a","unresolved":false,"context_lines":[{"line_number":279,"context_line":"    @validation.required_openstack(users\u003dTrue)"},{"line_number":280,"context_line":"    @scenario.configure(context\u003d{\"cleanup\": [\"cinder\", \"nova\"]})"},{"line_number":281,"context_line":"    def create_and_attach_volume(self, size, image, flavor,"},{"line_number":282,"context_line":"                                 create_volume_kwargs\u003dNone, **kwargs):"},{"line_number":283,"context_line":"        \"\"\"Create a VM and attach a volume to it."},{"line_number":284,"context_line":""},{"line_number":285,"context_line":"        Simple test to create a VM and attach a volume, then"}],"source_content_type":"text/x-python","patch_set":2,"id":"7a740942_f204e04a","line":282,"in_reply_to":"7a740942_07ddac6e","updated":"2015-12-07 13:16:10.000000000","message":"Ok; just let me know how you decided to go further, I can do the adjustments and re-upload a new patch set.","commit_id":"42952641f3ee52aab0e8642e56338cdd20e1bf13"},{"author":{"_account_id":8491,"name":"Illia Khudoshyn","email":"ikhudoshyn@mirantis.com","username":"ikhudoshyn"},"change_message_id":"2a18b505276f1e13e792be00af28b9be7a479f02","unresolved":false,"context_lines":[{"line_number":279,"context_line":"    @validation.required_openstack(users\u003dTrue)"},{"line_number":280,"context_line":"    @scenario.configure(context\u003d{\"cleanup\": [\"cinder\", \"nova\"]})"},{"line_number":281,"context_line":"    def create_and_attach_volume(self, size, image, flavor,"},{"line_number":282,"context_line":"                                 create_volume_kwargs\u003dNone, **kwargs):"},{"line_number":283,"context_line":"        \"\"\"Create a VM and attach a volume to it."},{"line_number":284,"context_line":""},{"line_number":285,"context_line":"        Simple test to create a VM and attach a volume, then"}],"source_content_type":"text/x-python","patch_set":2,"id":"7a740942_07ddac6e","line":282,"in_reply_to":"7a740942_1e33bd63","updated":"2015-12-07 12:48:42.000000000","message":"This seem to be a question for today\u0027s weekly meeting on IRC.\nBtw, I quickly grepped the code -- there seem to be not that much tsaks to change.","commit_id":"42952641f3ee52aab0e8642e56338cdd20e1bf13"},{"author":{"_account_id":15371,"name":"Tom Patzig","email":"tom.patzig@sap.com","username":"tpatzig"},"change_message_id":"15f3d8ccaa17cb3600f2f4327362e13973adbb3a","unresolved":false,"context_lines":[{"line_number":279,"context_line":"    @validation.required_openstack(users\u003dTrue)"},{"line_number":280,"context_line":"    @scenario.configure(context\u003d{\"cleanup\": [\"cinder\", \"nova\"]})"},{"line_number":281,"context_line":"    def create_and_attach_volume(self, size, image, flavor,"},{"line_number":282,"context_line":"                                 create_volume_kwargs\u003dNone, **kwargs):"},{"line_number":283,"context_line":"        \"\"\"Create a VM and attach a volume to it."},{"line_number":284,"context_line":""},{"line_number":285,"context_line":"        Simple test to create a VM and attach a volume, then"}],"source_content_type":"text/x-python","patch_set":2,"id":"7a740942_1e33bd63","line":282,"in_reply_to":"7a740942_ce7d861d","updated":"2015-12-07 12:01:48.000000000","message":"Totally agree. But as already commented, when we move the vm args to dict we change the api of the task and all existing task configs, will no longer work. If all agree, I can change this, just wanted to have it mentioned.\nThe kwargs suffix, I have just copied from the other occurences, to not have different variable names, for the same thing.","commit_id":"42952641f3ee52aab0e8642e56338cdd20e1bf13"},{"author":{"_account_id":8491,"name":"Illia Khudoshyn","email":"ikhudoshyn@mirantis.com","username":"ikhudoshyn"},"change_message_id":"bec0b8f263c80482c8727c9e9ca378df50c6d896","unresolved":false,"context_lines":[{"line_number":279,"context_line":"    @validation.required_openstack(users\u003dTrue)"},{"line_number":280,"context_line":"    @scenario.configure(context\u003d{\"cleanup\": [\"cinder\", \"nova\"]})"},{"line_number":281,"context_line":"    def create_and_attach_volume(self, size, image, flavor,"},{"line_number":282,"context_line":"                                 create_volume_kwargs\u003dNone, **kwargs):"},{"line_number":283,"context_line":"        \"\"\"Create a VM and attach a volume to it."},{"line_number":284,"context_line":""},{"line_number":285,"context_line":"        Simple test to create a VM and attach a volume, then"}],"source_content_type":"text/x-python","patch_set":2,"id":"7a740942_dba53c47","line":282,"in_reply_to":"7a740942_f204e04a","updated":"2015-12-07 14:48:46.000000000","message":"We agreed to the foolowing:\n1) Introduce create_volume_params and create_vm_params\n2) keep existing **kwargs for a while \n3) write a deprecation warning if any **kwargs are provided\n4) merge create_vm_params and **kwargs\n\nEventually we\u0027ll get rid of **kwargs here and in all other ocurences.\nThere going to be a spec for that.","commit_id":"42952641f3ee52aab0e8642e56338cdd20e1bf13"},{"author":{"_account_id":8491,"name":"Illia Khudoshyn","email":"ikhudoshyn@mirantis.com","username":"ikhudoshyn"},"change_message_id":"a8c625001d42dfb52d9b9c29a9d700f995445f94","unresolved":false,"context_lines":[{"line_number":301,"context_line":"        create_vm_params \u003d create_vm_params or {}"},{"line_number":302,"context_line":""},{"line_number":303,"context_line":"        if kwargs:"},{"line_number":304,"context_line":"            LOG.warning(\"(Deprecated) Use \u0027create_vm_params\u0027 \""},{"line_number":305,"context_line":"                        \"for additional instance parameters.\")"},{"line_number":306,"context_line":"            cvp \u003d kwargs.copy() or {}"},{"line_number":307,"context_line":"            cvp.update(create_vm_params)"}],"source_content_type":"text/x-python","patch_set":3,"id":"7a740942_65dc6846","line":304,"updated":"2015-12-08 13:01:50.000000000","message":"Could you please use logging.log_deprecated_args, as in https://github.com/openstack/rally/blob/master/rally/plugins/openstack/scenarios/cinder/volumes.py#L351","commit_id":"b0fc28fa16182bc290be917eb36de69e05dfcca0"},{"author":{"_account_id":8491,"name":"Illia Khudoshyn","email":"ikhudoshyn@mirantis.com","username":"ikhudoshyn"},"change_message_id":"e8947d5efb1d94380930c1d2f47e1df49848f5df","unresolved":false,"context_lines":[{"line_number":301,"context_line":"        create_vm_params \u003d create_vm_params or {}"},{"line_number":302,"context_line":""},{"line_number":303,"context_line":"        if kwargs:"},{"line_number":304,"context_line":"            LOG.warning(\"(Deprecated) Use \u0027create_vm_params\u0027 \""},{"line_number":305,"context_line":"                        \"for additional instance parameters.\")"},{"line_number":306,"context_line":"            cvp \u003d kwargs.copy() or {}"},{"line_number":307,"context_line":"            cvp.update(create_vm_params)"}],"source_content_type":"text/x-python","patch_set":3,"id":"7a740942_eb57c923","line":304,"in_reply_to":"7a740942_3071c00f","updated":"2015-12-08 13:45:34.000000000","message":"Variant that you provided looks good for me","commit_id":"b0fc28fa16182bc290be917eb36de69e05dfcca0"},{"author":{"_account_id":15371,"name":"Tom Patzig","email":"tom.patzig@sap.com","username":"tpatzig"},"change_message_id":"1264ebbaefa8a1a8e37e440e631d05b1e26702ee","unresolved":false,"context_lines":[{"line_number":301,"context_line":"        create_vm_params \u003d create_vm_params or {}"},{"line_number":302,"context_line":""},{"line_number":303,"context_line":"        if kwargs:"},{"line_number":304,"context_line":"            LOG.warning(\"(Deprecated) Use \u0027create_vm_params\u0027 \""},{"line_number":305,"context_line":"                        \"for additional instance parameters.\")"},{"line_number":306,"context_line":"            cvp \u003d kwargs.copy() or {}"},{"line_number":307,"context_line":"            cvp.update(create_vm_params)"}],"source_content_type":"text/x-python","patch_set":3,"id":"7a740942_3071c00f","line":304,"in_reply_to":"7a740942_65dc6846","updated":"2015-12-08 13:26:53.000000000","message":"Yes, can do so. I also saw that; but was not sure about certain args: Which version number? what is the deprecated argument \u0027kwargs\u0027, because that is not visible to users?\n\n@logging.log_deprecated_args(\"Use \u0027create_vm_params\u0027 for additional instance parameters\u0027\", \"0.1.2\", [\"kwargs\"], once\u003dTrue)","commit_id":"b0fc28fa16182bc290be917eb36de69e05dfcca0"},{"author":{"_account_id":9545,"name":"Andriy Kurilin","email":"andr.kurilin@gmail.com","username":"akurilin"},"change_message_id":"3076d60b5889a54958ac3a967e145d4c36b7f31b","unresolved":false,"context_lines":[{"line_number":304,"context_line":"        create_vm_params \u003d create_vm_params or {}"},{"line_number":305,"context_line":""},{"line_number":306,"context_line":"        if kwargs:"},{"line_number":307,"context_line":"            cvp \u003d kwargs.copy() or {}"},{"line_number":308,"context_line":"            cvp.update(create_vm_params)"},{"line_number":309,"context_line":"            create_vm_params \u003d cvp"},{"line_number":310,"context_line":""}],"source_content_type":"text/x-python","patch_set":4,"id":"7a740942_ac46b478","line":307,"updated":"2015-12-08 15:16:52.000000000","message":"Why not just use kwargs? I mean `create_vm_params.update(kwargs)` without copy ant additional var.","commit_id":"8ff5d90fd2246bb4014a507b3f775567b977ddf2"},{"author":{"_account_id":8491,"name":"Illia Khudoshyn","email":"ikhudoshyn@mirantis.com","username":"ikhudoshyn"},"change_message_id":"3d1cdf64d4693cd974f8efd2001bfe6a0e5f87fa","unresolved":false,"context_lines":[{"line_number":304,"context_line":"        create_vm_params \u003d create_vm_params or {}"},{"line_number":305,"context_line":""},{"line_number":306,"context_line":"        if kwargs:"},{"line_number":307,"context_line":"            cvp \u003d kwargs.copy() or {}"},{"line_number":308,"context_line":"            cvp.update(create_vm_params)"},{"line_number":309,"context_line":"            create_vm_params \u003d cvp"},{"line_number":310,"context_line":""}],"source_content_type":"text/x-python","patch_set":4,"id":"5a710552_4fd19256","line":307,"in_reply_to":"5a710552_b4a64f51","updated":"2015-12-16 15:02:43.000000000","message":"Ivan, I believe you missed the point. In your sample if the same key is provided in both create_vm_params and kwargs, the latter will be used. That is what Tom tries to avoid.","commit_id":"8ff5d90fd2246bb4014a507b3f775567b977ddf2"},{"author":{"_account_id":1736,"name":"Ivan Kolodyazhny","email":"e0ne@e0ne.info","username":"e0ne"},"change_message_id":"70066e1f880800c917f16212fa53dcfc64d66347","unresolved":false,"context_lines":[{"line_number":304,"context_line":"        create_vm_params \u003d create_vm_params or {}"},{"line_number":305,"context_line":""},{"line_number":306,"context_line":"        if kwargs:"},{"line_number":307,"context_line":"            cvp \u003d kwargs.copy() or {}"},{"line_number":308,"context_line":"            cvp.update(create_vm_params)"},{"line_number":309,"context_line":"            create_vm_params \u003d cvp"},{"line_number":310,"context_line":""}],"source_content_type":"text/x-python","patch_set":4,"id":"5a710552_b4a64f51","line":307,"in_reply_to":"7a740942_8234df49","updated":"2015-12-16 14:58:17.000000000","message":"It can be replaced with:\ncreate_vm_params \u003d create_vm_params or {}\ncreate_vm_params.update(kwargs)","commit_id":"8ff5d90fd2246bb4014a507b3f775567b977ddf2"},{"author":{"_account_id":15371,"name":"Tom Patzig","email":"tom.patzig@sap.com","username":"tpatzig"},"change_message_id":"48cffbed4290e043e9f1946307d8ab428416afa5","unresolved":false,"context_lines":[{"line_number":304,"context_line":"        create_vm_params \u003d create_vm_params or {}"},{"line_number":305,"context_line":""},{"line_number":306,"context_line":"        if kwargs:"},{"line_number":307,"context_line":"            cvp \u003d kwargs.copy() or {}"},{"line_number":308,"context_line":"            cvp.update(create_vm_params)"},{"line_number":309,"context_line":"            create_vm_params \u003d cvp"},{"line_number":310,"context_line":""}],"source_content_type":"text/x-python","patch_set":4,"id":"7a740942_8234df49","line":307,"in_reply_to":"7a740942_ac46b478","updated":"2015-12-08 15:40:05.000000000","message":"because then kwargs will take precedence over create_vm_params (when using both).","commit_id":"8ff5d90fd2246bb4014a507b3f775567b977ddf2"},{"author":{"_account_id":4428,"name":"Liyingjun","email":"yinjalee@163.com","username":"liyingjun"},"change_message_id":"fdd02beac0a8a33c9457e31efae1e3304ccee979","unresolved":false,"context_lines":[{"line_number":303,"context_line":"        create_volume_params \u003d create_volume_params or {}"},{"line_number":304,"context_line":"        create_vm_params \u003d create_vm_params or {}"},{"line_number":305,"context_line":""},{"line_number":306,"context_line":"        if kwargs:"},{"line_number":307,"context_line":"            cvp \u003d kwargs.copy() or {}"},{"line_number":308,"context_line":"            cvp.update(create_vm_params)"},{"line_number":309,"context_line":"            create_vm_params \u003d cvp"}],"source_content_type":"text/x-python","patch_set":5,"id":"fa69d971_8d06da1e","line":306,"range":{"start_line":306,"start_character":17,"end_line":306,"end_character":18},"updated":"2016-01-08 08:32:40.000000000","message":"could be simple: create_vm_params.update(kwargs)","commit_id":"8c6ceabaca0a0076c81baa2d2d5a085ed3797f95"},{"author":{"_account_id":15371,"name":"Tom Patzig","email":"tom.patzig@sap.com","username":"tpatzig"},"change_message_id":"54d6a1e0ed1d18ccce0eb217c154430b6f90f8de","unresolved":false,"context_lines":[{"line_number":303,"context_line":"        create_volume_params \u003d create_volume_params or {}"},{"line_number":304,"context_line":"        create_vm_params \u003d create_vm_params or {}"},{"line_number":305,"context_line":""},{"line_number":306,"context_line":"        if kwargs:"},{"line_number":307,"context_line":"            cvp \u003d kwargs.copy() or {}"},{"line_number":308,"context_line":"            cvp.update(create_vm_params)"},{"line_number":309,"context_line":"            create_vm_params \u003d cvp"}],"source_content_type":"text/x-python","patch_set":5,"id":"fa69d971_1e7f7267","line":306,"range":{"start_line":306,"start_character":17,"end_line":306,"end_character":18},"in_reply_to":"fa69d971_8d06da1e","updated":"2016-01-08 09:01:18.000000000","message":"As already explained:\nAsuming you have an exiting scenario, like:\n[...]\n\"args\": {\n      \"size\": 10,\n      \"image\": {\n         \"name\": \"^cirros.*uec$\"\n       },\n[...]\n\nThen if you just add the new create_vm_params, like:\n\n[...]\n\"args\": {\n      \"size\": 10,\n      \"image\": {\n         \"name\": \"^cirros.*uec$\"\n       },\n      \"create_vm_params\": {\n         \"image\": {\n             \"name\": \"ubuntu\"\n         },\n         \"flavor\": {\n             \"name\": \"m1.small\"\n         },\n      },\n[...]\n\nThen you will override image from create_vm_params, with image from args and launch a cirros vm.\n\nTo still support both, i would expect, that the new create_vm_params, will override the old args, if attributes are set in both, not vice versa.\n\nBut that\u0027s just my opinion. If you want to have it the other way, i can adjust that.","commit_id":"8c6ceabaca0a0076c81baa2d2d5a085ed3797f95"},{"author":{"_account_id":9545,"name":"Andriy Kurilin","email":"andr.kurilin@gmail.com","username":"akurilin"},"change_message_id":"ac5c169de7a0bfefd3259708157a59e5046482dd","unresolved":false,"context_lines":[{"line_number":280,"context_line":"    @scenario.configure(context\u003d{\"cleanup\": [\"cinder\", \"nova\"]})"},{"line_number":281,"context_line":"    @logging.log_deprecated_args("},{"line_number":282,"context_line":"        \"Use \u0027create_vm_params\u0027 for additional instance parameters.\","},{"line_number":283,"context_line":"        \"0.1.2\", [\"kwargs\"], once\u003dTrue)"},{"line_number":284,"context_line":"    def create_and_attach_volume(self, size, image, flavor,"},{"line_number":285,"context_line":"                                 create_volume_params\u003dNone,"},{"line_number":286,"context_line":"                                 create_vm_params\u003dNone, **kwargs):"}],"source_content_type":"text/x-python","patch_set":6,"id":"fa69d971_87546b54","line":283,"range":{"start_line":283,"start_character":8,"end_line":283,"end_character":15},"updated":"2016-01-08 11:22:41.000000000","message":"change to 0.2.0","commit_id":"569d7bb7a30ebc5e15e0f4b7d23fdad4f717b639"},{"author":{"_account_id":9545,"name":"Andriy Kurilin","email":"andr.kurilin@gmail.com","username":"akurilin"},"change_message_id":"ac5c169de7a0bfefd3259708157a59e5046482dd","unresolved":false,"context_lines":[{"line_number":302,"context_line":""},{"line_number":303,"context_line":"        create_volume_params \u003d create_volume_params or {}"},{"line_number":304,"context_line":"        create_vm_params \u003d create_vm_params or {}"},{"line_number":305,"context_line":"        create_vm_params.update(kwargs)"},{"line_number":306,"context_line":""},{"line_number":307,"context_line":"        server \u003d self._boot_server(image, flavor, **create_vm_params)"},{"line_number":308,"context_line":"        volume \u003d self._create_volume(size, **create_volume_params)"}],"source_content_type":"text/x-python","patch_set":6,"id":"fa69d971_e76f67fa","line":305,"updated":"2016-01-08 11:22:41.000000000","message":"let\u0027s add a simple check and remove update statement at all:\n\n    if kwargs and create_vm_params:\n       raise AttributeError(_(\"You can not set both `kwargs` and `create_vm_params` attributes. Please use `create_vm_params`.\")) # maybe it should be ValueError","commit_id":"569d7bb7a30ebc5e15e0f4b7d23fdad4f717b639"}],"samples/tasks/scenarios/cinder/create-and-attach-volume.json":[{"author":{"_account_id":7872,"name":"Marc Koderer","email":"marc@koderer.com","username":"mkoderer"},"change_message_id":"28233f7a1ec90db48c578dc43c44d983d9a77247","unresolved":false,"context_lines":[{"line_number":11,"context_line":"                    \"name\": \"{{flavor_name}}\""},{"line_number":12,"context_line":"                },"},{"line_number":13,"context_line":"                \"create_volume_params\": {"},{"line_number":14,"context_line":"                    \"availability_zone\": \"AZ1\""},{"line_number":15,"context_line":"                }"},{"line_number":16,"context_line":"            },"},{"line_number":17,"context_line":"            \"runner\": {"}],"source_content_type":"application/json","patch_set":5,"id":"fa69d971_7c4ebec2","line":14,"updated":"2016-01-08 07:47:23.000000000","message":"So this means the request will fail for cinder deployment without any AZ, right? I guess it would be better to have a new file then.","commit_id":"8c6ceabaca0a0076c81baa2d2d5a085ed3797f95"},{"author":{"_account_id":15371,"name":"Tom Patzig","email":"tom.patzig@sap.com","username":"tpatzig"},"change_message_id":"8d10b162bac0c9fc047c1921e31e4d055429b9a9","unresolved":false,"context_lines":[{"line_number":11,"context_line":"                    \"name\": \"{{flavor_name}}\""},{"line_number":12,"context_line":"                },"},{"line_number":13,"context_line":"                \"create_volume_params\": {"},{"line_number":14,"context_line":"                    \"availability_zone\": \"AZ1\""},{"line_number":15,"context_line":"                }"},{"line_number":16,"context_line":"            },"},{"line_number":17,"context_line":"            \"runner\": {"}],"source_content_type":"application/json","patch_set":5,"id":"fa69d971_cd95b240","line":14,"in_reply_to":"fa69d971_7c4ebec2","updated":"2016-01-08 08:25:10.000000000","message":"Yes, you are right.\nOr we provide the availibility_zone as paramater for the scenario, like:\n\"availability_zone\": \"{{availability_zone}}\"\nand default to RegionOne. What do you think. New file is also fine for me.","commit_id":"8c6ceabaca0a0076c81baa2d2d5a085ed3797f95"}]}