)]}'
{"placement/objects/allocation_candidate.py":[{"author":{"_account_id":14070,"name":"Eric Fried","email":"openstack@fried.cc","username":"efried"},"change_message_id":"c58ec1cf54224a72230a589131ff6bcfff59978e","unresolved":false,"context_lines":[{"line_number":672,"context_line":"    derived_alloc_to_rp \u003d sa.join("},{"line_number":673,"context_line":"        _ALLOC_TBL, _RP_TBL,"},{"line_number":674,"context_line":"        sa.and_(_ALLOC_TBL.c.resource_provider_id \u003d\u003d _RP_TBL.c.id,"},{"line_number":675,"context_line":"                _RP_TBL.c.root_provider_id.in_(root_ids))"},{"line_number":676,"context_line":"    )"},{"line_number":677,"context_line":"    usage \u003d sa.alias("},{"line_number":678,"context_line":"        sa.select(["}],"source_content_type":"text/x-python","patch_set":1,"id":"9fb8cfa7_f0af4de0","line":675,"range":{"start_line":675,"start_character":47,"end_line":675,"end_character":55},"updated":"2019-06-11 17:10:50.000000000","message":"✔","commit_id":"e1783b0087e36caebf7f22c8c1cd927bd33a0834"},{"author":{"_account_id":14070,"name":"Eric Fried","email":"openstack@fried.cc","username":"efried"},"change_message_id":"c58ec1cf54224a72230a589131ff6bcfff59978e","unresolved":false,"context_lines":[{"line_number":706,"context_line":"        inv.c.max_unit,"},{"line_number":707,"context_line":"        usage.c.used,"},{"line_number":708,"context_line":"    ]).select_from(usage_join).where("},{"line_number":709,"context_line":"        rpt.c.root_provider_id.in_(root_ids)"},{"line_number":710,"context_line":"    )"},{"line_number":711,"context_line":"    return ctx.session.execute(query).fetchall()"},{"line_number":712,"context_line":""}],"source_content_type":"text/x-python","patch_set":1,"id":"9fb8cfa7_10ad41e8","line":709,"range":{"start_line":709,"start_character":35,"end_line":709,"end_character":43},"updated":"2019-06-11 17:10:50.000000000","message":"✔","commit_id":"e1783b0087e36caebf7f22c8c1cd927bd33a0834"},{"author":{"_account_id":14070,"name":"Eric Fried","email":"openstack@fried.cc","username":"efried"},"change_message_id":"c58ec1cf54224a72230a589131ff6bcfff59978e","unresolved":false,"context_lines":[{"line_number":865,"context_line":"    # list of allocation requests."},{"line_number":866,"context_line":"    tree_uuids \u003d set()"},{"line_number":867,"context_line":"    for areq in areqs:"},{"line_number":868,"context_line":"        for arr in areq.resource_requests:"},{"line_number":869,"context_line":"            tree_uuids.add(arr.resource_provider.root_provider_uuid)"},{"line_number":870,"context_line":"    psums \u003d [psum for psum in all_psums if"},{"line_number":871,"context_line":"             psum.resource_provider.root_provider_uuid in tree_uuids]"},{"line_number":872,"context_line":""},{"line_number":873,"context_line":"    LOG.debug(\u0027Merging candidates yields %d allocation requests and %d \u0027"},{"line_number":874,"context_line":"              \u0027provider summaries\u0027, len(areqs), len(psums))"}],"source_content_type":"text/x-python","patch_set":1,"id":"9fb8cfa7_75202fad","line":871,"range":{"start_line":868,"start_character":0,"end_line":871,"end_character":69},"updated":"2019-06-11 17:10:50.000000000","message":"Cool, we were already doing the right thing here.\n\nI didn\u0027t look super close, but seems like we could DRY this filter into a helper method, which, if we had done it before, would have avoided this bug.","commit_id":"e1783b0087e36caebf7f22c8c1cd927bd33a0834"}],"placement/tests/functional/gabbits/allocation-candidates.yaml":[{"author":{"_account_id":14070,"name":"Eric Fried","email":"openstack@fried.cc","username":"efried"},"change_message_id":"c58ec1cf54224a72230a589131ff6bcfff59978e","unresolved":false,"context_lines":[{"line_number":437,"context_line":"      $.allocation_requests.`len`: 1"},{"line_number":438,"context_line":"      $.allocation_requests[0].allocations.`len`: 2"},{"line_number":439,"context_line":"      # We expect all the providers that share roots with the allocations."},{"line_number":440,"context_line":"      # In this case it the compute node, its two numa nodes and its two pfs."},{"line_number":441,"context_line":"      $.provider_summaries.`len`: 5"},{"line_number":442,"context_line":""},{"line_number":443,"context_line":"# Make sure that old microversions can return combinations where"}],"source_content_type":"text/x-yaml","patch_set":1,"id":"9fb8cfa7_3585d7cf","line":440,"range":{"start_line":440,"start_character":21,"end_line":440,"end_character":23},"updated":"2019-06-11 17:10:50.000000000","message":"it is","commit_id":"e1783b0087e36caebf7f22c8c1cd927bd33a0834"}]}