)]}'
{"nova/tests/functional/api/openstack/placement/db/test_allocation_candidates.py":[{"author":{"_account_id":14070,"name":"Eric Fried","email":"openstack@fried.cc","username":"efried"},"change_message_id":"ee47ebf1b501ac797844b6c3de15191937274b9c","unresolved":false,"context_lines":[{"line_number":1909,"context_line":"        self._validate_provider_summary_resources({}, alloc_cands)"},{"line_number":1910,"context_line":"        self._validate_provider_summary_traits({}, alloc_cands)"},{"line_number":1911,"context_line":""},{"line_number":1912,"context_line":"    def _get_root_ids_matching_names(self, names):"},{"line_number":1913,"context_line":"        \"\"\"Utility function to look up root provider IDs from a set of supplied"},{"line_number":1914,"context_line":"        provider names directly from the API DB."},{"line_number":1915,"context_line":"        \"\"\""},{"line_number":1916,"context_line":"        names \u003d map(six.text_type, names)"},{"line_number":1917,"context_line":"        sel \u003d sa.select([rp_obj._RP_TBL.c.root_provider_id])"},{"line_number":1918,"context_line":"        sel \u003d sel.where(rp_obj._RP_TBL.c.name.in_(names))"},{"line_number":1919,"context_line":"        with self.api_db.get_engine().connect() as conn:"},{"line_number":1920,"context_line":"            cn_root_ids \u003d set([r[0] for r in conn.execute(sel)])"},{"line_number":1921,"context_line":"        return cn_root_ids"},{"line_number":1922,"context_line":""},{"line_number":1923,"context_line":"    def _get_rp_ids_matching_names(self, names):"},{"line_number":1924,"context_line":"        \"\"\"Utility function to look up resource provider IDs from a set of"}],"source_content_type":"text/x-python","patch_set":3,"id":"5f7c97a3_3be8cd14","line":1921,"range":{"start_line":1912,"start_character":0,"end_line":1921,"end_character":26},"updated":"2018-05-18 20:13:25.000000000","message":"We don\u0027t need this anymore.  Everywhere it\u0027s used we could just use _get_rp_ids_matching_names instead.","commit_id":"7668efb8259061cd4a121df282fb05bd0cfb2fd2"},{"author":{"_account_id":14070,"name":"Eric Fried","email":"openstack@fried.cc","username":"efried"},"change_message_id":"ee47ebf1b501ac797844b6c3de15191937274b9c","unresolved":false,"context_lines":[{"line_number":1934,"context_line":"    def test_trees_matching_all(self):"},{"line_number":1935,"context_line":"        \"\"\"Creates a few provider trees having different inventories and"},{"line_number":1936,"context_line":"        allocations and tests the _get_trees_matching_all_resources() utility"},{"line_number":1937,"context_line":"        function to ensure that only the root provider IDs of matching provider"},{"line_number":1938,"context_line":"        trees are returned."},{"line_number":1939,"context_line":"        \"\"\""},{"line_number":1940,"context_line":"        # NOTE(jaypipes): _get_trees_matching_all() expects a dict of resource"}],"source_content_type":"text/x-python","patch_set":3,"id":"5f7c97a3_3b752d99","line":1937,"range":{"start_line":1937,"start_character":32,"end_line":1937,"end_character":58},"updated":"2018-05-18 20:13:25.000000000","message":"I think this comment is wrong (and maybe has been for a while).","commit_id":"7668efb8259061cd4a121df282fb05bd0cfb2fd2"}]}