)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"change_message_id":"8f8c7d7be6333712ff2f81b22a8183fcfcf8d483","unresolved":false,"context_lines":[{"line_number":4,"context_line":"Commit:     Yikun Jiang \u003cyikunkero@gmail.com\u003e"},{"line_number":5,"context_line":"CommitDate: 2018-07-11 15:57:08 +0800"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Address nit in afc7650e64753ab7687ae2c4f2714d4bb78a4e5a"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Change-Id: I126b51892056e3241640bbf05acf16d7f72b4329"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"5f7c97a3_4ce876de","line":7,"updated":"2018-07-12 12:16:40.000000000","message":"Might want to just call this \"Address nits in server group policy series\" since it\u0027s going to have fixes from more than one change.","commit_id":"5a50ed361767f20a1de5be01ef5fa2cce7397842"}],"nova/tests/unit/scheduler/filters/test_affinity_filters.py":[{"author":{"_account_id":8556,"name":"Ghanshyam Maan","display_name":"Ghanshyam Maan","email":"gmaan.os14@gmail.com","username":"ghanshyam"},"change_message_id":"728c1c747f97fe30e4ecf89672c87d5bab06ca58","unresolved":false,"context_lines":[{"line_number":149,"context_line":"            policy\u003d\u0027affinity\u0027))"},{"line_number":150,"context_line":"        self.assertTrue(filt_cls.host_passes(host, spec_obj))"},{"line_number":151,"context_line":"        spec_obj \u003d objects.RequestSpec(instance_group\u003dobjects.InstanceGroup("},{"line_number":152,"context_line":"            policy\u003dpolicy, members\u003d[], rules\u003d{}), instance_uuid\u003duuids.fake)"},{"line_number":153,"context_line":"        spec_obj.instance_group.hosts \u003d []"},{"line_number":154,"context_line":"        self.assertTrue(filt_cls.host_passes(host, spec_obj))"},{"line_number":155,"context_line":"        spec_obj.instance_group.hosts \u003d [\u0027host2\u0027]"}],"source_content_type":"text/x-python","patch_set":1,"id":"5f7c97a3_65ca2d70","side":"PARENT","line":152,"range":{"start_line":152,"start_character":37,"end_line":152,"end_character":47},"updated":"2018-07-12 08:56:44.000000000","message":"why you are removing this ?","commit_id":"f890401e3cb20356255bb992c68dfb61a976add4"},{"author":{"_account_id":20722,"name":"Yikun Jiang","email":"yikunkero@gmail.com","username":"yikunkero"},"change_message_id":"2c38be3432947507dcf91b39c274c95fe41e5c25","unresolved":false,"context_lines":[{"line_number":149,"context_line":"            policy\u003d\u0027affinity\u0027))"},{"line_number":150,"context_line":"        self.assertTrue(filt_cls.host_passes(host, spec_obj))"},{"line_number":151,"context_line":"        spec_obj \u003d objects.RequestSpec(instance_group\u003dobjects.InstanceGroup("},{"line_number":152,"context_line":"            policy\u003dpolicy, members\u003d[], rules\u003d{}), instance_uuid\u003duuids.fake)"},{"line_number":153,"context_line":"        spec_obj.instance_group.hosts \u003d []"},{"line_number":154,"context_line":"        self.assertTrue(filt_cls.host_passes(host, spec_obj))"},{"line_number":155,"context_line":"        spec_obj.instance_group.hosts \u003d [\u0027host2\u0027]"}],"source_content_type":"text/x-python","patch_set":1,"id":"5f7c97a3_38d21cff","side":"PARENT","line":152,"range":{"start_line":152,"start_character":37,"end_line":152,"end_character":47},"in_reply_to":"5f7c97a3_65ca2d70","updated":"2018-07-12 09:36:44.000000000","message":"we would set default rules to {}, if we don;t have rules args. So, this rules is used.\n\nand matt pointed out it, so I address this.\nhttps://review.openstack.org/#/c/571166/24/nova/tests/unit/scheduler/filters/test_affinity_filters.py@152","commit_id":"f890401e3cb20356255bb992c68dfb61a976add4"}]}
