)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":9656,"name":"Ihar Hrachyshka","email":"ihrachys@redhat.com","username":"ihrachys","status":"Red Hat Networking Systems Engineer"},"change_message_id":"108079971eb950e43cb1f506c57801bab4125aa3","unresolved":false,"context_lines":[{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Relocate Flat Allocation DB model"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"As there would be issue of cyclic imports while"},{"line_number":10,"context_line":"implementation of objects for flat allocation  which has"},{"line_number":11,"context_line":"db models definition and mixins in same file, this patch will"},{"line_number":12,"context_line":"refactor flat allocation models."},{"line_number":13,"context_line":""},{"line_number":14,"context_line":"Change-Id: I7876d28d35c5c4b9167520733e1a1b08dabc6951"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":4,"id":"1ac06dbe_3e1406f4","line":11,"range":{"start_line":9,"start_character":0,"end_line":11,"end_character":44},"updated":"2016-08-19 15:32:24.000000000","message":"doesn\u0027t sound like correct English, some words are probably missing. Try:\n\n\u0027Since there would be a circular import issue once we try to implement an object relying on the model, and adopt it in tree for database accessing code, this patch prepares the ground for this work by moving the model out of database accessing modules into neutron/db/models.\u0027","commit_id":"e5eca2ee50974b8e3d4a18657519b7de76f7018d"},{"author":{"_account_id":18332,"name":"Ankur","email":"ankur.gupta@intel.com","username":"ankur-gupta-f"},"change_message_id":"92ff3d2cf93e65e1cfc9654600095111cc37414f","unresolved":false,"context_lines":[{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Relocate Flat Allocation DB model"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"As there would be issue of cyclic imports while"},{"line_number":10,"context_line":"implementation of objects for flat allocation  which has"},{"line_number":11,"context_line":"db models definition and mixins in same file, this patch will"},{"line_number":12,"context_line":"refactor flat allocation models."},{"line_number":13,"context_line":""},{"line_number":14,"context_line":"Change-Id: I7876d28d35c5c4b9167520733e1a1b08dabc6951"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":4,"id":"1ac06dbe_0c2d80a7","line":11,"range":{"start_line":9,"start_character":0,"end_line":11,"end_character":44},"in_reply_to":"1ac06dbe_3e1406f4","updated":"2016-08-20 19:14:46.000000000","message":"Done","commit_id":"e5eca2ee50974b8e3d4a18657519b7de76f7018d"},{"author":{"_account_id":9656,"name":"Ihar Hrachyshka","email":"ihrachys@redhat.com","username":"ihrachys","status":"Red Hat Networking Systems Engineer"},"change_message_id":"d41d49c5b264c4728ca1bdf16096dc9dae2eb425","unresolved":false,"context_lines":[{"line_number":11,"context_line":"for database accessing code, this patch prepares the ground for"},{"line_number":12,"context_line":"this work by moving the model out of database accessing"},{"line_number":13,"context_line":"modules into neutron/db/models. This patch will"},{"line_number":14,"context_line":"refactor flat allocation models."},{"line_number":15,"context_line":""},{"line_number":16,"context_line":"Change-Id: I7876d28d35c5c4b9167520733e1a1b08dabc6951"},{"line_number":17,"context_line":"Partial-Bug: #1597913"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":7,"id":"fa7ab95a_703f43d6","line":14,"updated":"2016-08-29 10:01:10.000000000","message":"no refactoring involved here...","commit_id":"21c2a7a05fe79a06b2127e647db6ef689c1661bb"}],"neutron/db/models/flatallocation.py":[{"author":{"_account_id":7715,"name":"Hirofumi Ichihara","email":"ichihara.hirofumi@gmail.com","username":"Hirofumi-Ichihara"},"change_message_id":"21c7105df08d31aa37e05f9cd4158b56de5b24ac","unresolved":false,"context_lines":[{"line_number":1,"context_line":"# Copyright (c) 2013 OpenStack Foundation"},{"line_number":2,"context_line":"# All Rights Reserved."},{"line_number":3,"context_line":"#"},{"line_number":4,"context_line":"#    Licensed under the Apache License, Version 2.0 (the \"License\"); you may"},{"line_number":5,"context_line":"#    not use this file except in compliance with the License. You may obtain"}],"source_content_type":"text/x-python","patch_set":2,"id":"3ac371cc_b348e688","line":2,"range":{"start_line":1,"start_character":0,"end_line":2,"end_character":22},"updated":"2016-08-16 13:23:20.000000000","message":"Please remove useless lines.","commit_id":"f171c7bc995ed3b6f4465bfe0877d2961b03573c"},{"author":{"_account_id":22218,"name":"Sindhu Devale","email":"sindhu.devale@intel.com","username":"sindhudevale"},"change_message_id":"9bbbf65bfd7b9aea471235526f4b68da6b797263","unresolved":false,"context_lines":[{"line_number":1,"context_line":"# Copyright (c) 2013 OpenStack Foundation"},{"line_number":2,"context_line":"# All Rights Reserved."},{"line_number":3,"context_line":"#"},{"line_number":4,"context_line":"#    Licensed under the Apache License, Version 2.0 (the \"License\"); you may"},{"line_number":5,"context_line":"#    not use this file except in compliance with the License. You may obtain"}],"source_content_type":"text/x-python","patch_set":2,"id":"3ac371cc_0c151b48","line":2,"range":{"start_line":1,"start_character":0,"end_line":2,"end_character":22},"in_reply_to":"3ac371cc_b348e688","updated":"2016-08-17 14:04:50.000000000","message":"Done","commit_id":"f171c7bc995ed3b6f4465bfe0877d2961b03573c"}],"neutron/db/models/plugins/ml2/flatallocation.py":[{"author":{"_account_id":7715,"name":"Hirofumi Ichihara","email":"ichihara.hirofumi@gmail.com","username":"Hirofumi-Ichihara"},"change_message_id":"02e6419da298dc4dc03fdbb4973255c68f8a96aa","unresolved":false,"context_lines":[{"line_number":15,"context_line":"from neutron.db import model_base"},{"line_number":16,"context_line":""},{"line_number":17,"context_line":""},{"line_number":18,"context_line":"class FlatAllocation(model_base.BASEV2, model_base.HasId):"},{"line_number":19,"context_line":"    \"\"\"Represent persistent allocation state of a physical network."},{"line_number":20,"context_line":""},{"line_number":21,"context_line":"    If a record exists for a physical network, then that physical"}],"source_content_type":"text/x-python","patch_set":6,"id":"1ac06dbe_c25bd5f4","line":18,"range":{"start_line":18,"start_character":40,"end_line":18,"end_character":56},"updated":"2016-08-23 06:13:46.000000000","message":"Why did you add the class here?","commit_id":"cb0ffd6b2597ca359a4e7ad8b81681487b7be327"},{"author":{"_account_id":22218,"name":"Sindhu Devale","email":"sindhu.devale@intel.com","username":"sindhudevale"},"change_message_id":"04bd2ece77e34a37b5e7577a13fee54b24134a59","unresolved":false,"context_lines":[{"line_number":15,"context_line":"from neutron.db import model_base"},{"line_number":16,"context_line":""},{"line_number":17,"context_line":""},{"line_number":18,"context_line":"class FlatAllocation(model_base.BASEV2, model_base.HasId):"},{"line_number":19,"context_line":"    \"\"\"Represent persistent allocation state of a physical network."},{"line_number":20,"context_line":""},{"line_number":21,"context_line":"    If a record exists for a physical network, then that physical"}],"source_content_type":"text/x-python","patch_set":6,"id":"1ac06dbe_72e578da","line":18,"range":{"start_line":18,"start_character":40,"end_line":18,"end_character":56},"in_reply_to":"1ac06dbe_c25bd5f4","updated":"2016-08-23 12:41:43.000000000","message":"While creating object from this model, I was getting \n\"AttributeError: \u0027FlatAllocation\u0027 object has no attribute \u0027id\u0027 \". This was because the model itself didn\u0027t have the id field. So added this.","commit_id":"cb0ffd6b2597ca359a4e7ad8b81681487b7be327"}],"neutron/plugins/ml2/drivers/type_flat.py":[{"author":{"_account_id":7715,"name":"Hirofumi Ichihara","email":"ichihara.hirofumi@gmail.com","username":"Hirofumi-Ichihara"},"change_message_id":"21c7105df08d31aa37e05f9cd4158b56de5b24ac","unresolved":false,"context_lines":[{"line_number":13,"context_line":"#    License for the specific language governing permissions and limitations"},{"line_number":14,"context_line":"#    under the License."},{"line_number":15,"context_line":""},{"line_number":16,"context_line":"import six"},{"line_number":17,"context_line":"import sys"},{"line_number":18,"context_line":""},{"line_number":19,"context_line":"from neutron_lib import exceptions as exc"}],"source_content_type":"text/x-python","patch_set":2,"id":"3ac371cc_363184ed","line":16,"range":{"start_line":16,"start_character":0,"end_line":16,"end_character":10},"updated":"2016-08-16 13:23:20.000000000","message":"The six is third-party lib.\n\nhttp://docs.openstack.org/developer/hacking/#import-order-template","commit_id":"f171c7bc995ed3b6f4465bfe0877d2961b03573c"},{"author":{"_account_id":22218,"name":"Sindhu Devale","email":"sindhu.devale@intel.com","username":"sindhudevale"},"change_message_id":"9bbbf65bfd7b9aea471235526f4b68da6b797263","unresolved":false,"context_lines":[{"line_number":13,"context_line":"#    License for the specific language governing permissions and limitations"},{"line_number":14,"context_line":"#    under the License."},{"line_number":15,"context_line":""},{"line_number":16,"context_line":"import six"},{"line_number":17,"context_line":"import sys"},{"line_number":18,"context_line":""},{"line_number":19,"context_line":"from neutron_lib import exceptions as exc"}],"source_content_type":"text/x-python","patch_set":2,"id":"3ac371cc_676ae8b4","line":16,"range":{"start_line":16,"start_character":0,"end_line":16,"end_character":10},"in_reply_to":"3ac371cc_363184ed","updated":"2016-08-17 14:04:50.000000000","message":"Done","commit_id":"f171c7bc995ed3b6f4465bfe0877d2961b03573c"}]}