)]}'
{"test/probe/test_object_expirer.py":[{"author":{"_account_id":15343,"name":"Tim Burke","email":"tburke@nvidia.com","username":"tburke"},"change_message_id":"feb99728c378371bceec539650ab9a4af4c28115","unresolved":false,"context_lines":[{"line_number":186,"context_line":"                         \u0027X-Timestamp\u0027: Timestamp(now).normal},"},{"line_number":187,"context_line":"                contents\u003d\u0027dontcare\u0027)"},{"line_number":188,"context_line":"            onodes.extend(policy.object_ring.get_nodes("},{"line_number":189,"context_line":"                self.account, container_name, object_name)[1])"},{"line_number":190,"context_line":""},{"line_number":191,"context_line":"        time.sleep(2.0)"},{"line_number":192,"context_line":"        # make sure auto-created expirer-queue containers get in the account"}],"source_content_type":"text/x-python","patch_set":2,"id":"7f96bb07_9a6ce5ce","line":189,"updated":"2018-01-17 17:30:16.000000000","message":"This doesn\u0027t necessarily hit all nodes... and with the work-distribution improvements from https://review.openstack.org/#/c/524548/ I\u0027m not even sure that we *do* expect node index 1 to get hit. Maybe better to hit all nodes regardless?\n\n onodes \u003d set()\n for policy in ENABLED_POLICIES:\n     onodes.update(policy.object_ring.devs)","commit_id":"a6a93ade8505de41df8fb06a84bc52ab5927bfc6"},{"author":{"_account_id":7847,"name":"Alistair Coles","email":"alistairncoles@gmail.com","username":"acoles"},"change_message_id":"69e135022103f0aa0befe06fd52beb39dd5307d4","unresolved":false,"context_lines":[{"line_number":186,"context_line":"                         \u0027X-Timestamp\u0027: Timestamp(now).normal},"},{"line_number":187,"context_line":"                contents\u003d\u0027dontcare\u0027)"},{"line_number":188,"context_line":"            onodes.extend(policy.object_ring.get_nodes("},{"line_number":189,"context_line":"                self.account, container_name, object_name)[1])"},{"line_number":190,"context_line":""},{"line_number":191,"context_line":"        time.sleep(2.0)"},{"line_number":192,"context_line":"        # make sure auto-created expirer-queue containers get in the account"}],"source_content_type":"text/x-python","patch_set":2,"id":"7f96bb07_b501b23a","line":189,"in_reply_to":"7f96bb07_9a6ce5ce","updated":"2018-01-17 18:29:00.000000000","message":"the [1] is indexing the tuple returned from get_nodes to get the list of nodes\n\nbut your suggestion is slicker","commit_id":"a6a93ade8505de41df8fb06a84bc52ab5927bfc6"}]}