)]}'
{"nova/db/sqlalchemy/migrate_repo/versions/216_havana.py":[{"author":{"_account_id":2750,"name":"Sean Dague","email":"sean@dague.net","username":"sdague"},"change_message_id":"60736764ee0df1136a3e17ef2878acd2f002bbb7","unresolved":false,"context_lines":[{"line_number":15,"context_line":"from migrate.changeset import UniqueConstraint"},{"line_number":16,"context_line":"from migrate import ForeignKeyConstraint"},{"line_number":17,"context_line":"from oslo_log import log as logging"},{"line_number":18,"context_line":"import six"},{"line_number":19,"context_line":"from sqlalchemy import Boolean, BigInteger, Column, DateTime, Enum, Float"},{"line_number":20,"context_line":"from sqlalchemy import dialects"},{"line_number":21,"context_line":"from sqlalchemy import ForeignKey, Index, Integer, MetaData, String, Table"}],"source_content_type":"text/x-python","patch_set":1,"id":"fa32b979_44c27a86","line":18,"updated":"2015-06-24 16:52:29.000000000","message":"this is probably a pep8 issue now","commit_id":"4f9fec89494acbc3c1edc506743fdbf62c581a49"},{"author":{"_account_id":9107,"name":"Victor Stinner","email":"vstinner@redhat.com","username":"haypo"},"change_message_id":"1317b896e8fdf64865be91e9061edcba8983b7e4","unresolved":false,"context_lines":[{"line_number":15,"context_line":"from migrate.changeset import UniqueConstraint"},{"line_number":16,"context_line":"from migrate import ForeignKeyConstraint"},{"line_number":17,"context_line":"from oslo_log import log as logging"},{"line_number":18,"context_line":"import six"},{"line_number":19,"context_line":"from sqlalchemy import Boolean, BigInteger, Column, DateTime, Enum, Float"},{"line_number":20,"context_line":"from sqlalchemy import dialects"},{"line_number":21,"context_line":"from sqlalchemy import ForeignKey, Index, Integer, MetaData, String, Table"}],"source_content_type":"text/x-python","patch_set":1,"id":"fa32b979_c0fbb6bf","line":18,"in_reply_to":"fa32b979_44c27a86","updated":"2015-06-25 10:21:16.000000000","message":"Done, thanks.","commit_id":"4f9fec89494acbc3c1edc506743fdbf62c581a49"}],"nova/tests/unit/db/test_db_api.py":[{"author":{"_account_id":9107,"name":"Victor Stinner","email":"vstinner@redhat.com","username":"haypo"},"change_message_id":"d10e8f167f3d498b91a50cb39261a5e4b11c8fae","unresolved":false,"context_lines":[{"line_number":1368,"context_line":"    def _dict_from_object(self, obj, ignored_keys):"},{"line_number":1369,"context_line":"        if ignored_keys is None:"},{"line_number":1370,"context_line":"            ignored_keys \u003d []"},{"line_number":1371,"context_line":"        if isinstance(obj, dict):"},{"line_number":1372,"context_line":"            obj_items \u003d obj.items()"},{"line_number":1373,"context_line":"        else:"},{"line_number":1374,"context_line":"            obj_items \u003d obj.iteritems()"}],"source_content_type":"text/x-python","patch_set":3,"id":"fa32b979_cd7f6e15","line":1371,"updated":"2015-06-25 10:20:57.000000000","message":"This ugly check will be removed with the next oslo.db release including my change to add an items() method to oslo.db objects: Ibd4f454183c2bd3465539c3e572fa23208975bdb\n\nWhen oslo.db will be released, I will propose a patch to remove this code (always use obj.items()), and also replace remaining obj.iteritems() calls with obj.items().","commit_id":"532162d561001105f1b22d44e110018fc0cd89a8"},{"author":{"_account_id":4393,"name":"Dan Smith","email":"dms@danplanet.com","username":"danms"},"change_message_id":"82f6cdf280a80cd1d2a7c4052811095872f744f8","unresolved":false,"context_lines":[{"line_number":2793,"context_line":"        # loaded or not. The attribute is always present. This test should be"},{"line_number":2794,"context_line":"        # updated when oslo.db 1.13 will be released with the fix"},{"line_number":2795,"context_line":"        # If9c3ccc03f1dc9746936b0b83ea132508491e577."},{"line_number":2796,"context_line":"        # self.assertNotIn(\u0027pci_requests\u0027, extra)"},{"line_number":2797,"context_line":"        self.assertIn(\u0027numa_topology\u0027, extra)"},{"line_number":2798,"context_line":"        self.assertIn(\u0027vcpu_model\u0027, extra)"},{"line_number":2799,"context_line":""}],"source_content_type":"text/x-python","patch_set":3,"id":"ba3cc151_88ada28a","line":2796,"updated":"2015-06-29 15:37:27.000000000","message":"I normally would hate this sort of skip action. However, if we skip this whole test, we lose the checking on the next two lines. So, I think we ask Victor to queue up a change to remove this behind this patch, which fixes it as soon as we land the other thing.","commit_id":"532162d561001105f1b22d44e110018fc0cd89a8"}]}