)]}'
{"nova/tests/functional/test_aggregates.py":[{"author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"change_message_id":"66d70f970c99012659146d0ce27085330c7b959b","unresolved":false,"context_lines":[{"line_number":433,"context_line":"            enabled_filters.append(\u0027AggregateMultiTenancyIsolation\u0027)"},{"line_number":434,"context_line":"            self.flags("},{"line_number":435,"context_line":"                enabled_filters\u003denabled_filters, group\u003d\u0027filter_scheduler\u0027)"},{"line_number":436,"context_line":"        # Add a custom weigher which will weigh the compute host that is"},{"line_number":437,"context_line":"        # not in the aggregate higher than the two that are in the aggregate."},{"line_number":438,"context_line":"        self.flags(weight_classes\u003d[__name__ + \u0027.HostNameWeigher\u0027],"},{"line_number":439,"context_line":"                   group\u003d\u0027filter_scheduler\u0027)"},{"line_number":440,"context_line":"        self.start_service(\u0027scheduler\u0027)"}],"source_content_type":"text/x-python","patch_set":1,"id":"5f7c97a3_47c3daed","line":437,"range":{"start_line":436,"start_character":8,"end_line":437,"end_character":77},"updated":"2018-05-30 17:46:51.000000000","message":"All of the hosts are in aggregates so this should be re-worded.","commit_id":"ceeefe038c87730d946f969e2cb3c80b9ecf13c9"},{"author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"change_message_id":"823478c5e909b567cd899d07399be67a7fa4996f","unresolved":false,"context_lines":[{"line_number":468,"context_line":"        for host in (\u0027host2\u0027, \u0027host3\u0027):"},{"line_number":469,"context_line":"            self._add_host_to_aggregate(self.admin_api, tenant_aggregate, host)"},{"line_number":470,"context_line":""},{"line_number":471,"context_line":"        # Restrict the non-admin project to the host aggregate."},{"line_number":472,"context_line":"        self._isolate_aggregate("},{"line_number":473,"context_line":"            self.admin_api, tenant_aggregate, uuids.user_project)"},{"line_number":474,"context_line":""}],"source_content_type":"text/x-python","patch_set":2,"id":"3f79a3b5_bc46a5a7","line":471,"range":{"start_line":471,"start_character":48,"end_line":471,"end_character":52},"updated":"2019-04-29 16:25:11.000000000","message":"tenant","commit_id":"954876d1dc763f6a4a2b20b4784817b84e46eac9"},{"author":{"_account_id":6167,"name":"Ken\u0027ichi Ohmichi","email":"ken1ohmichi@gmail.com","username":"oomichi"},"change_message_id":"4928d8ba7443b684d9b865b21e9a73bd310e7a9f","unresolved":false,"context_lines":[{"line_number":559,"context_line":""},{"line_number":560,"context_line":""},{"line_number":561,"context_line":"class AggregateMultiTenancyIsolationColdMigrateTest("},{"line_number":562,"context_line":"        test.TestCase, integrated_helpers.InstanceHelperMixin):"},{"line_number":563,"context_line":""},{"line_number":564,"context_line":"    @staticmethod"},{"line_number":565,"context_line":"    def _create_aggregate(admin_api, name):"}],"source_content_type":"text/x-python","patch_set":5,"id":"dfbec78f_fb77529f","line":562,"range":{"start_line":562,"start_character":4,"end_line":562,"end_character":8},"updated":"2019-05-11 02:13:52.000000000","message":"nit: why adding 4 more spaces? (surprised pep8 didn\u0027t detect)","commit_id":"f57c5aaccd70ff889821421d268d495a6a2da763"},{"author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"change_message_id":"089bba8130909518f044034ff9621606cf07dc73","unresolved":false,"context_lines":[{"line_number":559,"context_line":""},{"line_number":560,"context_line":""},{"line_number":561,"context_line":"class AggregateMultiTenancyIsolationColdMigrateTest("},{"line_number":562,"context_line":"        test.TestCase, integrated_helpers.InstanceHelperMixin):"},{"line_number":563,"context_line":""},{"line_number":564,"context_line":"    @staticmethod"},{"line_number":565,"context_line":"    def _create_aggregate(admin_api, name):"}],"source_content_type":"text/x-python","patch_set":5,"id":"9fb8cfa7_e38617dc","line":562,"range":{"start_line":562,"start_character":4,"end_line":562,"end_character":8},"in_reply_to":"9fb8cfa7_c30c3396","updated":"2019-06-19 19:44:41.000000000","message":"Actually my IDE (pycharm) complains if this line is aligned with the next logical line.","commit_id":"f57c5aaccd70ff889821421d268d495a6a2da763"},{"author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"change_message_id":"10cbcd223cbc9520d1ecaf109e0e61d816b451d6","unresolved":false,"context_lines":[{"line_number":559,"context_line":""},{"line_number":560,"context_line":""},{"line_number":561,"context_line":"class AggregateMultiTenancyIsolationColdMigrateTest("},{"line_number":562,"context_line":"        test.TestCase, integrated_helpers.InstanceHelperMixin):"},{"line_number":563,"context_line":""},{"line_number":564,"context_line":"    @staticmethod"},{"line_number":565,"context_line":"    def _create_aggregate(admin_api, name):"}],"source_content_type":"text/x-python","patch_set":5,"id":"9fb8cfa7_c30c3396","line":562,"range":{"start_line":562,"start_character":4,"end_line":562,"end_character":8},"in_reply_to":"dfbec78f_fb77529f","updated":"2019-06-19 19:43:17.000000000","message":"Force of habit when adding docstrings to a class (which I didn\u0027t do here). We don\u0027t enforce pep8 for indent on wrapped lines though so that\u0027s why pep8 doesn\u0027t fail here.","commit_id":"f57c5aaccd70ff889821421d268d495a6a2da763"},{"author":{"_account_id":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"},"change_message_id":"18500fc842028900cac1b192b6afc2a602f33b1e","unresolved":false,"context_lines":[{"line_number":389,"context_line":"            self.assertEqual(\u0027host2\u0027, self._get_instance_host(server))"},{"line_number":390,"context_line":""},{"line_number":391,"context_line":""},{"line_number":392,"context_line":"class HostNameWeigher(weights.BaseHostWeigher):"},{"line_number":393,"context_line":"    def _weigh_object(self, host_state, weight_properties):"},{"line_number":394,"context_line":"        \"\"\"Arbitrary preferring host1 over host2 over host3.\"\"\""},{"line_number":395,"context_line":"        weights \u003d {\u0027host1\u0027: 100, \u0027host2\u0027: 50, \u0027host3\u0027: 1}"}],"source_content_type":"text/x-python","patch_set":6,"id":"7faddb67_161a4f6e","line":392,"updated":"2019-08-29 16:41:30.000000000","message":"Oh, here you are","commit_id":"89e376393de31abc64d1232f9ee077e717876c77"},{"author":{"_account_id":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"},"change_message_id":"18500fc842028900cac1b192b6afc2a602f33b1e","unresolved":false,"context_lines":[{"line_number":612,"context_line":"        # Enable the AggregateMultiTenancyIsolation filter before starting the"},{"line_number":613,"context_line":"        # scheduler service."},{"line_number":614,"context_line":"        enabled_filters \u003d CONF.filter_scheduler.enabled_filters"},{"line_number":615,"context_line":"        if \u0027AggregateMultiTenancyIsolation\u0027 not in enabled_filters:"},{"line_number":616,"context_line":"            enabled_filters.append(\u0027AggregateMultiTenancyIsolation\u0027)"},{"line_number":617,"context_line":"            self.flags("},{"line_number":618,"context_line":"                enabled_filters\u003denabled_filters, group\u003d\u0027filter_scheduler\u0027)"}],"source_content_type":"text/x-python","patch_set":6,"id":"7faddb67_b609fb2b","line":615,"range":{"start_line":615,"start_character":0,"end_line":615,"end_character":67},"updated":"2019-08-29 16:41:30.000000000","message":"nit: This isn\u0027t there by default. Wonder if it would have made sense to just raise if this _was_ there (i.e. go update this test if we ever decide to enable this filter by default for some mad reason)","commit_id":"89e376393de31abc64d1232f9ee077e717876c77"},{"author":{"_account_id":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"},"change_message_id":"18500fc842028900cac1b192b6afc2a602f33b1e","unresolved":false,"context_lines":[{"line_number":619,"context_line":"        # Add a custom weigher which will weigh host1, which will be in the"},{"line_number":620,"context_line":"        # admin project aggregate, higher than the other hosts which are in"},{"line_number":621,"context_line":"        # the non-admin project aggregate."},{"line_number":622,"context_line":"        self.flags(weight_classes\u003d[__name__ + \u0027.HostNameWeigher\u0027],"},{"line_number":623,"context_line":"                   group\u003d\u0027filter_scheduler\u0027)"},{"line_number":624,"context_line":"        self.start_service(\u0027scheduler\u0027)"},{"line_number":625,"context_line":""}],"source_content_type":"text/x-python","patch_set":6,"id":"7faddb67_16332ff8","line":622,"updated":"2019-08-29 16:41:30.000000000","message":"Huh?\n\nLater: oh, I see it","commit_id":"89e376393de31abc64d1232f9ee077e717876c77"},{"author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"change_message_id":"92b454d6fc43489035a10a06c9bcfa2adcca1f30","unresolved":false,"context_lines":[{"line_number":624,"context_line":"        self.start_service(\u0027scheduler\u0027)"},{"line_number":625,"context_line":""},{"line_number":626,"context_line":"        for host in (\u0027host1\u0027, \u0027host2\u0027, \u0027host3\u0027):"},{"line_number":627,"context_line":"            fake.set_nodes([host])"},{"line_number":628,"context_line":"            self.addCleanup(fake.restore_nodes)"},{"line_number":629,"context_line":"            self.start_service(\u0027compute\u0027, host\u003dhost)"},{"line_number":630,"context_line":""},{"line_number":631,"context_line":"        # Create an admin-only aggregate for the admin project. This is needed"}],"source_content_type":"text/x-python","patch_set":6,"id":"7faddb67_766f0092","line":628,"range":{"start_line":627,"start_character":12,"end_line":628,"end_character":47},"updated":"2019-08-29 19:56:57.000000000","message":"These are gone now...need to rebase again.","commit_id":"89e376393de31abc64d1232f9ee077e717876c77"}]}