)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":16495,"name":"Jean-Emile DARTOIS","email":"jean-emile.dartois@b-com.com","username":"JeD"},"change_message_id":"a36a07e7ee8f5f07334f3b70d4d0c14f6f941aae","unresolved":false,"context_lines":[{"line_number":10,"context_line":"and McCabe\u0027s cyclomatic complexity 22."},{"line_number":11,"context_line":"(watcher/decision_engine/strategy/strategies/basic_consolidation.py)"},{"line_number":12,"context_line":"I extracted some of the functionalities into helper functions"},{"line_number":13,"context_line":"to reduce the length and complexity of execute()."},{"line_number":14,"context_line":"http://openqa.sed.hu/dashboard/index/544838?did\u003d1"},{"line_number":15,"context_line":"Additionally it became more readable as well, without"},{"line_number":16,"context_line":"changing its functionality."}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"da6ed579_c23036a1","line":13,"updated":"2016-01-18 08:34:06.000000000","message":"For the next time, can please create a bug ?\n\nYou can take a look to the openstack documentation:\nhttp://docs.openstack.org/infra/manual/developers.html#working-on-bugs","commit_id":"28a53f88b3c5b3128a226518dcbe3e31b620efe7"}],"watcher/decision_engine/strategy/strategies/basic_consolidation.py":[{"author":{"_account_id":16495,"name":"Jean-Emile DARTOIS","email":"jean-emile.dartois@b-com.com","username":"JeD"},"change_message_id":"f06d5a8d346663758d8811cd653684d17490658b","unresolved":false,"context_lines":[{"line_number":360,"context_line":"                score.append((hypervisor_id, result))"},{"line_number":361,"context_line":"        return score"},{"line_number":362,"context_line":""},{"line_number":363,"context_line":"    def node_and_vmScore(self, s, score, current_model):"},{"line_number":364,"context_line":"        \"\"\"Get List of VMs from Node\"\"\""},{"line_number":365,"context_line":"        node_to_release \u003d s[len(score) - 1][0]"},{"line_number":366,"context_line":"        vms_to_mig \u003d current_model.get_mapping().get_node_vms_from_id("}],"source_content_type":"text/x-python","patch_set":2,"id":"da6ed579_07b6603d","line":363,"updated":"2016-01-18 08:11:19.000000000","message":"nit: the function should be lowercase.","commit_id":"28a53f88b3c5b3128a226518dcbe3e31b620efe7"},{"author":{"_account_id":16495,"name":"Jean-Emile DARTOIS","email":"jean-emile.dartois@b-com.com","username":"JeD"},"change_message_id":"f06d5a8d346663758d8811cd653684d17490658b","unresolved":false,"context_lines":[{"line_number":415,"context_line":""},{"line_number":416,"context_line":"    def unsuccessful_migration_actualization(self, m, unsuccessful_migration):"},{"line_number":417,"context_line":"        if m \u003e 0:"},{"line_number":418,"context_line":"            self.number_of_migrations \u003d self.number_of_migrations + m"},{"line_number":419,"context_line":"            return 0"},{"line_number":420,"context_line":"        else:"},{"line_number":421,"context_line":"            return unsuccessful_migration + 1"}],"source_content_type":"text/x-python","patch_set":2,"id":"da6ed579_c7d4c848","line":418,"updated":"2016-01-18 08:11:19.000000000","message":"Can you please replace this line by:\nself.number_of_migrations +\u003d m","commit_id":"28a53f88b3c5b3128a226518dcbe3e31b620efe7"},{"author":{"_account_id":16495,"name":"Jean-Emile DARTOIS","email":"jean-emile.dartois@b-com.com","username":"JeD"},"change_message_id":"f06d5a8d346663758d8811cd653684d17490658b","unresolved":false,"context_lines":[{"line_number":457,"context_line":"            first \u003d False"},{"line_number":458,"context_line":"            score \u003d []"},{"line_number":459,"context_line":""},{"line_number":460,"context_line":"            score \u003d self.calculate_score_of_nodes(current_model, score)"},{"line_number":461,"context_line":""},{"line_number":462,"context_line":"            \u0027\u0027\u0027 sort compute nodes by Score decreasing \u0027\u0027\u0027\u0027\u0027"},{"line_number":463,"context_line":"            s \u003d sorted(score, reverse\u003dTrue, key\u003dlambda x: (x[1]))"}],"source_content_type":"text/x-python","patch_set":2,"id":"da6ed579_27204449","line":460,"range":{"start_line":460,"start_character":15,"end_line":460,"end_character":16},"updated":"2016-01-18 08:11:19.000000000","message":"nit: can you please rename this variable to help us to understand.\n\nWhat do you think about score_of_nodes ?","commit_id":"28a53f88b3c5b3128a226518dcbe3e31b620efe7"},{"author":{"_account_id":16495,"name":"Jean-Emile DARTOIS","email":"jean-emile.dartois@b-com.com","username":"JeD"},"change_message_id":"f06d5a8d346663758d8811cd653684d17490658b","unresolved":false,"context_lines":[{"line_number":470,"context_line":"                    \" of the cluster is zero\"))"},{"line_number":471,"context_line":"                break"},{"line_number":472,"context_line":""},{"line_number":473,"context_line":"            node_to_release, vm_score \u003d self.node_and_vmScore("},{"line_number":474,"context_line":"                s, score, current_model)"},{"line_number":475,"context_line":""},{"line_number":476,"context_line":"            \u0027\u0027\u0027 sort VMs by Score \u0027\u0027\u0027"}],"source_content_type":"text/x-python","patch_set":2,"id":"da6ed579_07ee60ec","line":473,"updated":"2016-01-18 08:11:19.000000000","message":"nit: can you please rename this variable to help us to understand.","commit_id":"28a53f88b3c5b3128a226518dcbe3e31b620efe7"}]}