)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":15197,"name":"Pierre Riteau","email":"pierre@stackhpc.com","username":"priteau","status":"StackHPC"},"change_message_id":"2cada5b301ae0b13650ba0d7225d64b3f58b614d","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":8,"id":"9ac7f39d_119ecb06","updated":"2024-05-28 08:18:36.000000000","message":"Please address comments and rebase on top of master.","commit_id":"6b42f3b96d664d744f626bba8fe7a8b9250e8420"}],"blazar/tests/db/migration/__init__.py":[{"author":{"_account_id":15197,"name":"Pierre Riteau","email":"pierre@stackhpc.com","username":"priteau","status":"StackHPC"},"change_message_id":"2939639ebe0be17ccc717b125d35c247079dda2c","unresolved":true,"context_lines":[{"line_number":435,"context_line":""},{"line_number":436,"context_line":"        connection.close()"},{"line_number":437,"context_line":""},{"line_number":438,"context_line":"        del (self.engines[database])"},{"line_number":439,"context_line":"        del (self.test_databases[database])"},{"line_number":440,"context_line":""},{"line_number":441,"context_line":"    def _test_postgresql_opportunistically(self):"},{"line_number":442,"context_line":"        # Test postgresql database migration walk"}],"source_content_type":"text/x-python","patch_set":8,"id":"95754552_7d15dcd1","line":439,"range":{"start_line":438,"start_character":0,"end_line":439,"end_character":43},"updated":"2024-05-28 08:18:20.000000000","message":"I think you can remove the parentheses.","commit_id":"6b42f3b96d664d744f626bba8fe7a8b9250e8420"},{"author":{"_account_id":36765,"name":"Seunghun Lee","display_name":"Seunghun Lee","email":"seunghun@stackhpc.com","username":"seunghun1ee","status":"Cloud Engineer at StackHPC"},"change_message_id":"8485d5a772c1a76b3079f23b3a37eaf56195c72a","unresolved":false,"context_lines":[{"line_number":435,"context_line":""},{"line_number":436,"context_line":"        connection.close()"},{"line_number":437,"context_line":""},{"line_number":438,"context_line":"        del (self.engines[database])"},{"line_number":439,"context_line":"        del (self.test_databases[database])"},{"line_number":440,"context_line":""},{"line_number":441,"context_line":"    def _test_postgresql_opportunistically(self):"},{"line_number":442,"context_line":"        # Test postgresql database migration walk"}],"source_content_type":"text/x-python","patch_set":8,"id":"f6e8ea21_6bb98ec5","line":439,"range":{"start_line":438,"start_character":0,"end_line":439,"end_character":43},"in_reply_to":"95754552_7d15dcd1","updated":"2024-05-30 15:32:00.000000000","message":"Done","commit_id":"6b42f3b96d664d744f626bba8fe7a8b9250e8420"},{"author":{"_account_id":15197,"name":"Pierre Riteau","email":"pierre@stackhpc.com","username":"priteau","status":"StackHPC"},"change_message_id":"2939639ebe0be17ccc717b125d35c247079dda2c","unresolved":true,"context_lines":[{"line_number":455,"context_line":"        # build a fully populated postgresql database with all the tables"},{"line_number":456,"context_line":"        self._reset_database(database)"},{"line_number":457,"context_line":"        self._walk_versions(engine, self.snake_walk, self.downgrade)"},{"line_number":458,"context_line":"        del (self.engines[database])"},{"line_number":459,"context_line":"        del (self.test_databases[database])"},{"line_number":460,"context_line":""},{"line_number":461,"context_line":"    def _alembic_command(self, alembic_command, engine, *args, **kwargs):"},{"line_number":462,"context_line":"        \"\"\"Alembic command redefine reasons"}],"source_content_type":"text/x-python","patch_set":8,"id":"bee1c00f_d0f82add","line":459,"range":{"start_line":458,"start_character":0,"end_line":459,"end_character":43},"updated":"2024-05-28 08:18:20.000000000","message":"I think you can remove the parentheses.","commit_id":"6b42f3b96d664d744f626bba8fe7a8b9250e8420"},{"author":{"_account_id":36765,"name":"Seunghun Lee","display_name":"Seunghun Lee","email":"seunghun@stackhpc.com","username":"seunghun1ee","status":"Cloud Engineer at StackHPC"},"change_message_id":"8485d5a772c1a76b3079f23b3a37eaf56195c72a","unresolved":false,"context_lines":[{"line_number":455,"context_line":"        # build a fully populated postgresql database with all the tables"},{"line_number":456,"context_line":"        self._reset_database(database)"},{"line_number":457,"context_line":"        self._walk_versions(engine, self.snake_walk, self.downgrade)"},{"line_number":458,"context_line":"        del (self.engines[database])"},{"line_number":459,"context_line":"        del (self.test_databases[database])"},{"line_number":460,"context_line":""},{"line_number":461,"context_line":"    def _alembic_command(self, alembic_command, engine, *args, **kwargs):"},{"line_number":462,"context_line":"        \"\"\"Alembic command redefine reasons"}],"source_content_type":"text/x-python","patch_set":8,"id":"b5079c34_17cb0f1e","line":459,"range":{"start_line":458,"start_character":0,"end_line":459,"end_character":43},"in_reply_to":"bee1c00f_d0f82add","updated":"2024-05-30 15:32:00.000000000","message":"Done","commit_id":"6b42f3b96d664d744f626bba8fe7a8b9250e8420"}],"blazar/tests/db/sqlalchemy/test_sqlalchemy_api.py":[{"author":{"_account_id":35263,"name":"Matt Crees","email":"mattc@stackhpc.com","username":"mattcrees"},"change_message_id":"db0d19cc9cc40717b151936e425a541f7f1a04cf","unresolved":true,"context_lines":[{"line_number":209,"context_line":"    def setUp(self):"},{"line_number":210,"context_line":"        super(SQLAlchemyDBApiTestCase, self).setUp()"},{"line_number":211,"context_line":""},{"line_number":212,"context_line":"    def test_model_query(self):"},{"line_number":213,"context_line":"        lease \u003d db_api.lease_create(_get_fake_phys_lease_values())"},{"line_number":214,"context_line":"        query \u003d db_api.model_query(models.Lease)"},{"line_number":215,"context_line":"        self.assertEqual([lease.to_dict()],"}],"source_content_type":"text/x-python","patch_set":11,"id":"dabdc310_33c77427","line":212,"updated":"2024-06-28 14:10:30.000000000","message":"model_query was dropped in this patch https://review.opendev.org/c/openstack/blazar/+/918121, so we don\u0027t need this here anymore.","commit_id":"1b23461acbff30fc6ba34129e40716bb5680f4ce"},{"author":{"_account_id":36765,"name":"Seunghun Lee","display_name":"Seunghun Lee","email":"seunghun@stackhpc.com","username":"seunghun1ee","status":"Cloud Engineer at StackHPC"},"change_message_id":"d4191b88f752da831d15890ed550ddc4f2507c2d","unresolved":false,"context_lines":[{"line_number":209,"context_line":"    def setUp(self):"},{"line_number":210,"context_line":"        super(SQLAlchemyDBApiTestCase, self).setUp()"},{"line_number":211,"context_line":""},{"line_number":212,"context_line":"    def test_model_query(self):"},{"line_number":213,"context_line":"        lease \u003d db_api.lease_create(_get_fake_phys_lease_values())"},{"line_number":214,"context_line":"        query \u003d db_api.model_query(models.Lease)"},{"line_number":215,"context_line":"        self.assertEqual([lease.to_dict()],"}],"source_content_type":"text/x-python","patch_set":11,"id":"197c42e4_0be98206","line":212,"in_reply_to":"dabdc310_33c77427","updated":"2024-08-27 08:17:00.000000000","message":"Acknowledged","commit_id":"1b23461acbff30fc6ba34129e40716bb5680f4ce"}],"test-requirements.txt":[{"author":{"_account_id":15197,"name":"Pierre Riteau","email":"pierre@stackhpc.com","username":"priteau","status":"StackHPC"},"change_message_id":"75c2aa58be73d3cfa20c11e1aaab4c65d804d56e","unresolved":true,"context_lines":[{"line_number":1,"context_line":"# The order of packages is significant, because pip processes them in the order"},{"line_number":2,"context_line":"# of appearance. Changing the order has an impact on the overall integration"},{"line_number":3,"context_line":"# process, which may cause wedges in the gate later."},{"line_number":4,"context_line":"hacking\u003e\u003d6.1.0,\u003c6.2.0 # Apache-2.0"},{"line_number":5,"context_line":""},{"line_number":6,"context_line":"ddt\u003e\u003d1.0.1 # MIT"},{"line_number":7,"context_line":"fixtures\u003e\u003d3.0.0 # Apache-2.0/BSD"}],"source_content_type":"text/plain","patch_set":13,"id":"e88c0c58_5e9c24d7","line":4,"updated":"2024-08-27 08:30:21.000000000","message":"Can you try bumping to hacking 7.0.0?\n\nhacking\u003e\u003d7.0.0,\u003c7.1.0 # Apache-2.0","commit_id":"fdaa9c3dcaa6f7f79778a4f62ed9b7f680a75095"},{"author":{"_account_id":36765,"name":"Seunghun Lee","display_name":"Seunghun Lee","email":"seunghun@stackhpc.com","username":"seunghun1ee","status":"Cloud Engineer at StackHPC"},"change_message_id":"9dc2830300af516081e1954d08af65bbf75af540","unresolved":false,"context_lines":[{"line_number":1,"context_line":"# The order of packages is significant, because pip processes them in the order"},{"line_number":2,"context_line":"# of appearance. Changing the order has an impact on the overall integration"},{"line_number":3,"context_line":"# process, which may cause wedges in the gate later."},{"line_number":4,"context_line":"hacking\u003e\u003d6.1.0,\u003c6.2.0 # Apache-2.0"},{"line_number":5,"context_line":""},{"line_number":6,"context_line":"ddt\u003e\u003d1.0.1 # MIT"},{"line_number":7,"context_line":"fixtures\u003e\u003d3.0.0 # Apache-2.0/BSD"}],"source_content_type":"text/plain","patch_set":13,"id":"99f9672b_b369ff74","line":4,"in_reply_to":"e88c0c58_5e9c24d7","updated":"2024-08-27 09:11:17.000000000","message":"I tested pep8 and py310 module of tox with 7.0.0 hacking and they all succeeded.","commit_id":"fdaa9c3dcaa6f7f79778a4f62ed9b7f680a75095"}]}