)]}'
{"rally-jobs/rally-neutron.yaml":[{"author":{"_account_id":6172,"name":"Boris Pavlovic","email":"boris@pavlovic.me","username":"boris-42"},"change_message_id":"a4ca13a409b9046a3f67f3b9b53514efdb4df7fb","unresolved":false,"context_lines":[{"line_number":739,"context_line":"        network: {}"},{"line_number":740,"context_line":"      sla:"},{"line_number":741,"context_line":"        failure_rate:"},{"line_number":742,"context_line":"          max: 0"},{"line_number":743,"context_line":"    -"},{"line_number":744,"context_line":"      args:"},{"line_number":745,"context_line":"        command:"}],"source_content_type":"text/x-yaml","patch_set":3,"id":"7f515b1d_0ed9cff1","side":"PARENT","line":742,"updated":"2017-09-28 00:43:04.000000000","message":"why we are removing this? plus if we really need to remove it specify it in commit message","commit_id":"3eda923a3a77dd2af0599b37c863451b58f4ec34"},{"author":{"_account_id":9545,"name":"Andriy Kurilin","email":"andr.kurilin@gmail.com","username":"akurilin"},"change_message_id":"0bcebf046fd80165849f91a92c49f3c597a2e667","unresolved":false,"context_lines":[{"line_number":739,"context_line":"        network: {}"},{"line_number":740,"context_line":"      sla:"},{"line_number":741,"context_line":"        failure_rate:"},{"line_number":742,"context_line":"          max: 0"},{"line_number":743,"context_line":"    -"},{"line_number":744,"context_line":"      args:"},{"line_number":745,"context_line":"        command:"}],"source_content_type":"text/x-yaml","patch_set":3,"id":"7f515b1d_3d2391bf","side":"PARENT","line":742,"in_reply_to":"7f515b1d_0ed9cff1","updated":"2017-09-28 16:03:33.000000000","message":"Actually, I do not know what this workload for. I mean the scenario argument includes dd_test to launch at the remote path. BUT the image_command_customizer context setups the different script at the different location. I do not know how it works...\n\nBut agree that it should be done in separate patches.","commit_id":"3eda923a3a77dd2af0599b37c863451b58f4ec34"}],"rally/plugins/openstack/scenarios/vm/vmtasks.py":[{"author":{"_account_id":6172,"name":"Boris Pavlovic","email":"boris@pavlovic.me","username":"boris-42"},"change_message_id":"a4ca13a409b9046a3f67f3b9b53514efdb4df7fb","unresolved":false,"context_lines":[{"line_number":490,"context_line":"                    platform\u003d\"openstack\")"},{"line_number":491,"context_line":"class DDLoadTest(BootRuncommandDelete):"},{"line_number":492,"context_line":""},{"line_number":493,"context_line":"    @logging.log_deprecated_args("},{"line_number":494,"context_line":"        \"Use \u0027interpreter\u0027 to specify the interpreter to execute script from.\","},{"line_number":495,"context_line":"        \"0.10.0\", [\"command\"], once\u003dTrue)"},{"line_number":496,"context_line":"    def run(self, flavor, username, password\u003dNone,"}],"source_content_type":"text/x-python","patch_set":3,"id":"7f515b1d_6e6a83ed","line":493,"updated":"2017-09-28 00:43:04.000000000","message":"It\u0027s totally unclear what the heck is happening here. \n\nFirst of all DDLoadTest means that we run DD why would someone need to specify command or interpreter at all? why not just run the dd?\n\nAnd what was the command and was the interpreter and how should look like the command in interpreter argument?","commit_id":"08d3a08c79f5d34cfaec8fed32cb26e40a63d84e"},{"author":{"_account_id":9545,"name":"Andriy Kurilin","email":"andr.kurilin@gmail.com","username":"akurilin"},"change_message_id":"0bcebf046fd80165849f91a92c49f3c597a2e667","unresolved":false,"context_lines":[{"line_number":490,"context_line":"                    platform\u003d\"openstack\")"},{"line_number":491,"context_line":"class DDLoadTest(BootRuncommandDelete):"},{"line_number":492,"context_line":""},{"line_number":493,"context_line":"    @logging.log_deprecated_args("},{"line_number":494,"context_line":"        \"Use \u0027interpreter\u0027 to specify the interpreter to execute script from.\","},{"line_number":495,"context_line":"        \"0.10.0\", [\"command\"], once\u003dTrue)"},{"line_number":496,"context_line":"    def run(self, flavor, username, password\u003dNone,"}],"source_content_type":"text/x-python","patch_set":3,"id":"7f515b1d_5d788dc2","line":493,"in_reply_to":"7f515b1d_6e6a83ed","updated":"2017-09-28 16:03:33.000000000","message":"The command argument is not clear to me as well. But we should get rid of it only in the backward compatible way, that is why I deprecated it as you can see at L493-495.\n\nI understand your concern and to make the things more clear, I splitted the patch to several ones with proper commit messages.","commit_id":"08d3a08c79f5d34cfaec8fed32cb26e40a63d84e"}],"samples/tasks/scenarios/vm/dd-load-test.json":[{"author":{"_account_id":6172,"name":"Boris Pavlovic","email":"boris@pavlovic.me","username":"boris-42"},"change_message_id":"a4ca13a409b9046a3f67f3b9b53514efdb4df7fb","unresolved":false,"context_lines":[{"line_number":11,"context_line":"                },"},{"line_number":12,"context_line":"                \"floating_network\": \"public\","},{"line_number":13,"context_line":"                \"force_delete\": false,"},{"line_number":14,"context_line":"                \"interpreter\": \"/bin/sh\","},{"line_number":15,"context_line":"                \"username\": \"cirros\""},{"line_number":16,"context_line":"            },"},{"line_number":17,"context_line":"            \"runner\": {"}],"source_content_type":"application/json","patch_set":3,"id":"7f515b1d_8905b913","line":14,"updated":"2017-09-28 00:43:04.000000000","message":"add comments about changes in interface in commit message it\u0027s not only doc change it\u0027s code refactoring","commit_id":"08d3a08c79f5d34cfaec8fed32cb26e40a63d84e"}]}