)]}'
{"rally/plugins/openstack/scenarios/ironic/utils.py":[{"author":{"_account_id":6172,"name":"Boris Pavlovic","email":"boris@pavlovic.me","username":"boris-42"},"change_message_id":"f64f5c6d389f2b1b16ffd884be2e0bb276049db3","unresolved":false,"context_lines":[{"line_number":45,"context_line":"        :param kwargs: optional parameters to create image"},{"line_number":46,"context_line":"        :returns: node object"},{"line_number":47,"context_line":"        \"\"\""},{"line_number":48,"context_line":"        if \"name\" not in kwargs:"},{"line_number":49,"context_line":"            # NOTE(rvasilets): can\u0027t use _generate_random_name() because"},{"line_number":50,"context_line":"            # ironic have specific format for node name."},{"line_number":51,"context_line":"            # Check that the supplied hostname conforms to:"}],"source_content_type":"text/x-python","patch_set":19,"id":"3a29b11f_4eb3b4e9","side":"PARENT","line":48,"updated":"2015-10-21 04:16:16.000000000","message":"you shouldn\u0027t ignore this message imho","commit_id":"e13ab7142f438467557262bbb65a0aa9584d5bc9"},{"author":{"_account_id":11748,"name":"Chris St. Pierre","email":"cstpierr@cisco.com","username":"stpierre"},"change_message_id":"2b41dcd4f4d0bb72addcbc5a5eb420579c805370","unresolved":false,"context_lines":[{"line_number":45,"context_line":"        :param kwargs: optional parameters to create image"},{"line_number":46,"context_line":"        :returns: node object"},{"line_number":47,"context_line":"        \"\"\""},{"line_number":48,"context_line":"        if \"name\" not in kwargs:"},{"line_number":49,"context_line":"            # NOTE(rvasilets): can\u0027t use _generate_random_name() because"},{"line_number":50,"context_line":"            # ironic have specific format for node name."},{"line_number":51,"context_line":"            # Check that the supplied hostname conforms to:"}],"source_content_type":"text/x-python","patch_set":19,"id":"3a29b11f_0076d905","side":"PARENT","line":48,"in_reply_to":"3a29b11f_4eb3b4e9","updated":"2015-10-21 12:14:10.000000000","message":"I didn\u0027t. The problem is that it\u0027s not true. Details are in a comment in the new patch set.","commit_id":"e13ab7142f438467557262bbb65a0aa9584d5bc9"}],"rally/plugins/openstack/scenarios/keystone/utils.py":[{"author":{"_account_id":7369,"name":"Sergey Skripnick","email":"sskripnick@mirantis.com","username":"redixin"},"change_message_id":"d283152bb8e491b4a05aa2d4852d19db844376b4","unresolved":false,"context_lines":[{"line_number":207,"context_line":"        :param description: tenant description to be set"},{"line_number":208,"context_line":"        \"\"\""},{"line_number":209,"context_line":"        name \u003d self.generate_random_name()"},{"line_number":210,"context_line":"        description \u003d description or self.generate_random_name"},{"line_number":211,"context_line":"        self.admin_clients(\"keystone\").tenants.update(tenant.id,"},{"line_number":212,"context_line":"                                                      name, description)"},{"line_number":213,"context_line":""}],"source_content_type":"text/x-python","patch_set":13,"id":"5a2ca52d_953812a2","line":210,"updated":"2015-10-15 11:29:35.000000000","message":"Should description be callable? Or it is a typo?","commit_id":"47ab5a3eebc3a163af5dd1cd0621ca7879e5ee6f"},{"author":{"_account_id":11748,"name":"Chris St. Pierre","email":"cstpierr@cisco.com","username":"stpierre"},"change_message_id":"27106377c09e161e6c1bfe927b7539115bbee86e","unresolved":false,"context_lines":[{"line_number":207,"context_line":"        :param description: tenant description to be set"},{"line_number":208,"context_line":"        \"\"\""},{"line_number":209,"context_line":"        name \u003d self.generate_random_name()"},{"line_number":210,"context_line":"        description \u003d description or self.generate_random_name"},{"line_number":211,"context_line":"        self.admin_clients(\"keystone\").tenants.update(tenant.id,"},{"line_number":212,"context_line":"                                                      name, description)"},{"line_number":213,"context_line":""}],"source_content_type":"text/x-python","patch_set":13,"id":"5a2ca52d_e98f2bca","line":210,"in_reply_to":"5a2ca52d_953812a2","updated":"2015-10-15 12:49:00.000000000","message":"Typo. Good catch!","commit_id":"47ab5a3eebc3a163af5dd1cd0621ca7879e5ee6f"}],"rally/plugins/openstack/scenarios/manila/shares.py":[{"author":{"_account_id":10475,"name":"Alexander Maretskiy","email":"amaretskiy@mirantis.com","username":"maretskiy"},"change_message_id":"f3d273b41858e91ed3cc5bb4aa85b8aa97df1c8b","unresolved":false,"context_lines":[{"line_number":67,"context_line":"    @scenario.configure(context\u003d{\"cleanup\": [\"manila\"]})"},{"line_number":68,"context_line":"    @rutils.log_deprecated_args("},{"line_number":69,"context_line":"        \"The \u0027name\u0027 argument to create_and_delete_service will be ignored\","},{"line_number":70,"context_line":"        \"0.0.5\", [\"name\"])"},{"line_number":71,"context_line":"    def create_share_network_and_delete(self,"},{"line_number":72,"context_line":"                                        neutron_net_id\u003dNone,"},{"line_number":73,"context_line":"                                        neutron_subnet_id\u003dNone,"}],"source_content_type":"text/x-python","patch_set":18,"id":"3a29b11f_69c4985e","line":70,"updated":"2015-10-20 10:17:24.000000000","message":"wrong version \"0.0.5\" - we already have release 0.1.1","commit_id":"7070ffd363e34a5657f7c1a79d43ec377f06a28f"}],"rally/plugins/openstack/scenarios/vm/utils.py":[{"author":{"_account_id":10475,"name":"Alexander Maretskiy","email":"amaretskiy@mirantis.com","username":"maretskiy"},"change_message_id":"f3d273b41858e91ed3cc5bb4aa85b8aa97df1c8b","unresolved":false,"context_lines":[{"line_number":144,"context_line":"            if self.check_ip_address(fip[\"ip\"])(server):"},{"line_number":145,"context_line":"                self._dissociate_floating_ip(server, fip[\"ip\"],"},{"line_number":146,"context_line":"                                             atomic_action\u003dFalse)"},{"line_number":147,"context_line":"            network_wrapper.wrap(self.clients, self.task).delete_floating_ip("},{"line_number":148,"context_line":"                fip[\"id\"], wait\u003dTrue)"},{"line_number":149,"context_line":""},{"line_number":150,"context_line":"    def _delete_server_with_fip(self, server, fip, force_delete\u003dFalse):"}],"source_content_type":"text/x-python","patch_set":18,"id":"3a29b11f_244ec596","line":147,"updated":"2015-10-20 10:17:24.000000000","message":"You have changed indentation here and move delete_floating_ip() out of `if\u0027 statement - this neither related to the patch nor reflected in commit message.\n\nPlease at least mention this in commit message, as well as explain why this change is required","commit_id":"7070ffd363e34a5657f7c1a79d43ec377f06a28f"}]}