)]}'
{"id":"openstack%2Fglance~743594","triplet_id":"openstack%2Fglance~master~Ieeaafcb575a53fd41d487069c07db65134528a0a","project":"openstack/glance","branch":"master","topic":"task-heartbeat","hashtags":[],"change_id":"Ieeaafcb575a53fd41d487069c07db65134528a0a","subject":"Add a test to make sure import get_flow() includes all the steps we expect","status":"ABANDONED","created":"2020-07-28 17:08:12.000000000","updated":"2020-08-04 18:58:35.000000000","total_comment_count":0,"unresolved_comment_count":0,"has_review_started":true,"meta_rev_id":"acd4f04aa0221c03c19befede9a6ba72bd4ec7fd","_number":743594,"virtual_id_number":743594,"owner":{"_account_id":4393,"name":"Dan Smith","email":"dms@danplanet.com","username":"danms"},"actions":{},"labels":{"Verified":{"recommended":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"all":[{"tag":"autogenerated:zuul:check","value":1,"date":"2020-07-31 01:40:48.000000000","permitted_voting_range":{"min":-2,"max":2},"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},{"_account_id":9303,"name":"Abhishek Kekane","email":"akekane@redhat.com","username":"abhishekkekane"},{"date":"2020-07-31 00:36:18.000000000","_account_id":9008,"name":"VMware NSX CI","username":"vmwareminesweeper","tags":["SERVICE_USER"]},{"_account_id":11904,"name":"Sean McGinnis","email":"sean.mcginnis@gmail.com","username":"SeanM"},{"_account_id":5202,"name":"Erno Kuvaja","email":"jokke@usr.fi","username":"jokke"}],"values":{"-2":"Fails","-1":"Doesn\u0027t seem to work"," 0":"No score","+1":"Works for me","+2":"Verified"},"description":"","value":1,"default_value":0,"optional":true},"Code-Review":{"approved":{"_account_id":11904,"name":"Sean McGinnis","email":"sean.mcginnis@gmail.com","username":"SeanM"},"all":[{"value":0,"permitted_voting_range":{"min":-1,"max":1},"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},{"value":2,"date":"2020-07-31 05:59:59.000000000","permitted_voting_range":{"min":-2,"max":2},"_account_id":9303,"name":"Abhishek Kekane","email":"akekane@redhat.com","username":"abhishekkekane"},{"value":0,"permitted_voting_range":{"min":-1,"max":1},"_account_id":9008,"name":"VMware NSX CI","username":"vmwareminesweeper","tags":["SERVICE_USER"]},{"value":2,"date":"2020-07-31 00:35:39.000000000","permitted_voting_range":{"min":-2,"max":2},"_account_id":11904,"name":"Sean McGinnis","email":"sean.mcginnis@gmail.com","username":"SeanM"},{"value":-1,"date":"2020-08-04 17:36:17.000000000","permitted_voting_range":{"min":-2,"max":2},"_account_id":5202,"name":"Erno Kuvaja","email":"jokke@usr.fi","username":"jokke"}],"values":{"-2":"Do not merge","-1":"This patch needs further work before it can be merged"," 0":"No score","+1":"Looks good to me, but someone else must approve","+2":"Looks good to me (core reviewer)"},"description":"","default_value":0,"optional":true},"Workflow":{"all":[{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},{"value":0,"permitted_voting_range":{"min":-1,"max":1},"_account_id":9303,"name":"Abhishek Kekane","email":"akekane@redhat.com","username":"abhishekkekane"},{"_account_id":9008,"name":"VMware NSX CI","username":"vmwareminesweeper","tags":["SERVICE_USER"]},{"value":0,"permitted_voting_range":{"min":-1,"max":1},"_account_id":11904,"name":"Sean McGinnis","email":"sean.mcginnis@gmail.com","username":"SeanM"},{"value":0,"permitted_voting_range":{"min":-1,"max":1},"_account_id":5202,"name":"Erno Kuvaja","email":"jokke@usr.fi","username":"jokke"}],"values":{"-1":"Work in progress"," 0":"Ready for reviews","+1":"Approved"},"description":"","default_value":0,"optional":true},"Review-Priority":{"recommended":{"_account_id":9303,"name":"Abhishek Kekane","email":"akekane@redhat.com","username":"abhishekkekane"},"all":[{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},{"value":1,"date":"2020-07-31 05:59:59.000000000","permitted_voting_range":{"min":-1,"max":2},"_account_id":9303,"name":"Abhishek Kekane","email":"akekane@redhat.com","username":"abhishekkekane"},{"_account_id":9008,"name":"VMware NSX CI","username":"vmwareminesweeper","tags":["SERVICE_USER"]},{"value":0,"permitted_voting_range":{"min":-1,"max":2},"_account_id":11904,"name":"Sean McGinnis","email":"sean.mcginnis@gmail.com","username":"SeanM"},{"value":0,"permitted_voting_range":{"min":-1,"max":2},"_account_id":5202,"name":"Erno Kuvaja","email":"jokke@usr.fi","username":"jokke"}],"values":{"-1":"Branch Freeze"," 0":"No Priority","+1":"Important Change","+2":"Top Priority / Holds Gate"},"description":"","value":1,"default_value":0,"optional":true}},"removable_reviewers":[],"reviewers":{"REVIEWER":[{"_account_id":5202,"name":"Erno Kuvaja","email":"jokke@usr.fi","username":"jokke"},{"_account_id":9008,"name":"VMware NSX CI","username":"vmwareminesweeper","tags":["SERVICE_USER"]},{"_account_id":9303,"name":"Abhishek Kekane","email":"akekane@redhat.com","username":"abhishekkekane"},{"_account_id":11904,"name":"Sean McGinnis","email":"sean.mcginnis@gmail.com","username":"SeanM"},{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]}]},"pending_reviewers":{},"reviewer_updates":[{"updated":"2020-07-29 20:15:38.000000000","updated_by":{"_account_id":11904,"name":"Sean McGinnis","email":"sean.mcginnis@gmail.com","username":"SeanM"},"reviewer":{"_account_id":11904,"name":"Sean McGinnis","email":"sean.mcginnis@gmail.com","username":"SeanM"},"state":"REVIEWER"},{"updated":"2020-07-31 00:36:18.000000000","updated_by":{"_account_id":9008,"name":"VMware NSX CI","username":"vmwareminesweeper","tags":["SERVICE_USER"]},"reviewer":{"_account_id":9008,"name":"VMware NSX CI","username":"vmwareminesweeper","tags":["SERVICE_USER"]},"state":"REVIEWER"},{"updated":"2020-07-31 01:40:48.000000000","updated_by":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"reviewer":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"state":"REVIEWER"},{"updated":"2020-07-31 05:59:59.000000000","updated_by":{"_account_id":9303,"name":"Abhishek Kekane","email":"akekane@redhat.com","username":"abhishekkekane"},"reviewer":{"_account_id":9303,"name":"Abhishek Kekane","email":"akekane@redhat.com","username":"abhishekkekane"},"state":"REVIEWER"},{"updated":"2020-08-04 17:36:17.000000000","updated_by":{"_account_id":5202,"name":"Erno Kuvaja","email":"jokke@usr.fi","username":"jokke"},"reviewer":{"_account_id":5202,"name":"Erno Kuvaja","email":"jokke@usr.fi","username":"jokke"},"state":"REVIEWER"}],"messages":[{"id":"193ae7eb0a27fdea529c4296a3dfd4a0f55b9e1d","author":{"_account_id":4393,"name":"Dan Smith","email":"dms@danplanet.com","username":"danms"},"date":"2020-07-28 17:08:12.000000000","message":"Uploaded patch set 1.","accounts_in_message":[],"_revision_number":1},{"id":"6cddea70d0ba35c4d69a2a255b92dd8845129d50","author":{"_account_id":9008,"name":"VMware NSX CI","username":"vmwareminesweeper","tags":["SERVICE_USER"]},"date":"2020-07-28 17:09:48.000000000","message":"Patch Set 1:\n\nBuild failed\n\n- ext-glance-zuul : NOT_REGISTERED","accounts_in_message":[],"_revision_number":1},{"id":"6f724b6af499cd43df6574de327ceff49009c745","tag":"autogenerated:zuul:check","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2020-07-28 21:08:22.000000000","message":"Patch Set 1: Verified+1\n\nBuild succeeded (check pipeline).\n\n- openstack-tox-lower-constraints https://zuul.opendev.org/t/openstack/build/0297d6cedf3d43adac58723e0c6d360c : SUCCESS in 5m 04s\n- openstack-tox-pep8 https://zuul.opendev.org/t/openstack/build/a512d34aaaf84770a26cb52db380a192 : SUCCESS in 6m 01s\n- openstack-tox-py36 https://zuul.opendev.org/t/openstack/build/9c9a79217ecf4333a78ef7c86e701910 : SUCCESS in 5m 46s\n- openstack-tox-py38 https://zuul.opendev.org/t/openstack/build/27fca1bbfa8c4fdcb101666fc14f4aae : SUCCESS in 7m 03s\n- openstack-tox-docs https://zuul.opendev.org/t/openstack/build/b48f063b12ea44289c3ec1ffb03824f9 : SUCCESS in 6m 31s\n- openstack-tox-functional-py36 https://zuul.opendev.org/t/openstack/build/9419c01af4024fb88bf7390134d13e37 : SUCCESS in 8m 30s\n- openstack-tox-functional-py38 https://zuul.opendev.org/t/openstack/build/109256f5e6a14e6aa5780a7eda86a5a1 : SUCCESS in 10m 36s\n- glance-code-constants-check https://zuul.opendev.org/t/openstack/build/8246bb45daf545ef8baf3ea55ab8b3b0 : SUCCESS in 5m 44s","accounts_in_message":[],"_revision_number":1},{"id":"c27d2bd6d638113580181315fea5f15e22a29e4f","author":{"_account_id":11904,"name":"Sean McGinnis","email":"sean.mcginnis@gmail.com","username":"SeanM"},"date":"2020-07-29 20:15:38.000000000","message":"Patch Set 1: Code-Review+2","accounts_in_message":[],"_revision_number":1},{"id":"3315eb0cebace41bbad0762b81061a29f9ff7d34","author":{"_account_id":4393,"name":"Dan Smith","email":"dms@danplanet.com","username":"danms"},"date":"2020-07-30 13:44:31.000000000","message":"Uploaded patch set 2: Patch Set 1 was rebased.","accounts_in_message":[],"_revision_number":2},{"id":"9e98383c23c21c4fd7b652e474565d3d1636160e","author":{"_account_id":9008,"name":"VMware NSX CI","username":"vmwareminesweeper","tags":["SERVICE_USER"]},"date":"2020-07-30 13:45:10.000000000","message":"Patch Set 2:\n\nBuild failed\n\n- ext-glance-zuul : NOT_REGISTERED","accounts_in_message":[],"_revision_number":2},{"id":"b007bd3803e2acc6c54be9fdcd2cab83f5810c20","tag":"autogenerated:zuul:check","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2020-07-30 18:38:38.000000000","message":"Patch Set 2: Verified-1\n\nBuild failed (check pipeline).  For information on how to proceed, see\nhttps://docs.opendev.org/opendev/infra-manual/latest/developers.html#automated-testing\n\n\n- openstack-tox-lower-constraints https://zuul.opendev.org/t/openstack/build/5986ef0a2a094d17bb7120c69d1cad3b : SUCCESS in 5m 12s\n- openstack-tox-pep8 https://zuul.opendev.org/t/openstack/build/34f6b400ceec4475834e752904a39e6e : SUCCESS in 4m 40s\n- openstack-tox-py36 https://zuul.opendev.org/t/openstack/build/e9d7e983df524011b16c7d2816e2da55 : SUCCESS in 4m 41s\n- openstack-tox-py38 https://zuul.opendev.org/t/openstack/build/7227ba64b218401da036ce1c1e8a2d23 : SUCCESS in 5m 41s\n- openstack-tox-docs https://zuul.opendev.org/t/openstack/build/214c0855a4d3446f8c47831ca50626c1 : SUCCESS in 5m 44s\n- openstack-tox-functional-py36 https://zuul.opendev.org/t/openstack/build/17bf60f371fc49a68ac7bf0fe8b23ce2 : FAILURE in 11m 15s\n- openstack-tox-functional-py38 https://zuul.opendev.org/t/openstack/build/7b28a90561cf42cda0b3bf0bf38943f9 : SUCCESS in 8m 26s\n- glance-code-constants-check https://zuul.opendev.org/t/openstack/build/44186f5007c5460690e5ee79d07cbed3 : FAILURE in 4m 06s","accounts_in_message":[],"_revision_number":2},{"id":"608033558940e1969b4a17e28409ae90ca847975","author":{"_account_id":11904,"name":"Sean McGinnis","email":"sean.mcginnis@gmail.com","username":"SeanM"},"date":"2020-07-30 18:41:53.000000000","message":"Patch Set 2:\n\nrecheck\n\nAppears to be unrelated failures:\n\nFileNotFoundError: [Errno 2] No such file or directory: \u0027/tmp/tmptwt160l2/images_3/38479b48-a197-48e7-8d62-b15aa508c8f6\u0027","accounts_in_message":[],"_revision_number":2},{"id":"d5d6eb01b3b1a1eb3bf27f7750a7a18a57fab438","author":{"_account_id":9008,"name":"VMware NSX CI","username":"vmwareminesweeper","tags":["SERVICE_USER"]},"date":"2020-07-30 18:42:19.000000000","message":"Patch Set 2:\n\nBuild failed\n\n- ext-glance-zuul : NOT_REGISTERED","accounts_in_message":[],"_revision_number":2},{"id":"6f257aae5adfbea87e9e077e1126fc2c9d1163c8","tag":"autogenerated:zuul:check","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2020-07-30 22:33:17.000000000","message":"Patch Set 2:\n\nBuild failed (check pipeline).  For information on how to proceed, see\nhttps://docs.opendev.org/opendev/infra-manual/latest/developers.html#automated-testing\n\n\n- openstack-tox-lower-constraints https://zuul.opendev.org/t/openstack/build/bf1fbf026ddd425083f57187a18dd263 : SUCCESS in 4m 51s\n- openstack-tox-pep8 https://zuul.opendev.org/t/openstack/build/da8e9cb048e447e1b473f2616993afa3 : SUCCESS in 4m 43s\n- openstack-tox-py36 https://zuul.opendev.org/t/openstack/build/a010ca0ddcf545a4bd0e750cbd681555 : SUCCESS in 5m 17s\n- openstack-tox-py38 https://zuul.opendev.org/t/openstack/build/e5dc331b3f8f4c98898e9b316b58753f : SUCCESS in 5m 29s\n- openstack-tox-docs https://zuul.opendev.org/t/openstack/build/2d9e87ec246e48ccaac03b2ef5f00587 : SUCCESS in 10m 08s\n- openstack-tox-functional-py36 https://zuul.opendev.org/t/openstack/build/738dfe800b0743cb9dbf0eccf3431bdf : SUCCESS in 7m 54s\n- openstack-tox-functional-py38 https://zuul.opendev.org/t/openstack/build/a8bcd37bab4d46d9980da40eb146c4c6 : SUCCESS in 8m 59s\n- glance-code-constants-check https://zuul.opendev.org/t/openstack/build/aa7f20c273514314a62735609a846b3a : FAILURE in 4m 21s","accounts_in_message":[],"_revision_number":2},{"id":"c2e0fdc2572b72140d7f3d2fe4f1a1e33d536744","author":{"_account_id":5202,"name":"Erno Kuvaja","email":"jokke@usr.fi","username":"jokke"},"date":"2020-07-31 00:25:58.000000000","message":"Patch Set 2:\n\nrecheck","accounts_in_message":[],"_revision_number":2},{"id":"8aa1f046a7ddc0af0677dc81150623cf2c80a7f6","author":{"_account_id":9008,"name":"VMware NSX CI","username":"vmwareminesweeper","tags":["SERVICE_USER"]},"date":"2020-07-31 00:26:28.000000000","message":"Patch Set 2:\n\nBuild failed\n\n- ext-glance-zuul : NOT_REGISTERED","accounts_in_message":[],"_revision_number":2},{"id":"abbf65eb0747bd633cfb6110403ef308dd91212f","author":{"_account_id":4393,"name":"Dan Smith","email":"dms@danplanet.com","username":"danms"},"date":"2020-07-31 00:35:39.000000000","message":"Uploaded patch set 3: Patch Set 2 was rebased.","accounts_in_message":[],"_revision_number":3},{"id":"12cc2f9fafe1ccec3761d52cb087f26394df232b","author":{"_account_id":9008,"name":"VMware NSX CI","username":"vmwareminesweeper","tags":["SERVICE_USER"]},"date":"2020-07-31 00:36:18.000000000","message":"Patch Set 3:\n\nBuild failed\n\n- ext-glance-zuul : NOT_REGISTERED","accounts_in_message":[],"_revision_number":3},{"id":"e846bb8dbfa6fe02fad597883c183577457a67ac","tag":"autogenerated:zuul:check","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2020-07-31 01:40:48.000000000","message":"Patch Set 3: Verified+1\n\nBuild succeeded (check pipeline).\n\n- openstack-tox-lower-constraints https://zuul.opendev.org/t/openstack/build/dca86720dc5d4438a1062122257fb40e : SUCCESS in 5m 28s\n- openstack-tox-pep8 https://zuul.opendev.org/t/openstack/build/1113e660e3bc46c19e80946890a2cb06 : SUCCESS in 5m 15s\n- openstack-tox-py36 https://zuul.opendev.org/t/openstack/build/a162c6a478ff41afa0c4a5985e6d04ea : SUCCESS in 5m 30s\n- openstack-tox-py38 https://zuul.opendev.org/t/openstack/build/986c533624a541dfb67bfccae1b6a441 : SUCCESS in 5m 15s\n- openstack-tox-docs https://zuul.opendev.org/t/openstack/build/11300623b69049bb807822799b102d33 : SUCCESS in 6m 08s\n- openstack-tox-functional-py36 https://zuul.opendev.org/t/openstack/build/02e682b292bd409aabad999b453811a6 : SUCCESS in 7m 27s\n- openstack-tox-functional-py38 https://zuul.opendev.org/t/openstack/build/9e275d8d77ca462ca5525c62f5295cff : SUCCESS in 9m 42s\n- glance-code-constants-check https://zuul.opendev.org/t/openstack/build/c875e63de0f84dde86d70432e5b9e446 : SUCCESS in 6m 14s","accounts_in_message":[],"_revision_number":3},{"id":"d0f872b331d6836a46385ca971593f37ec7f1718","author":{"_account_id":9303,"name":"Abhishek Kekane","email":"akekane@redhat.com","username":"abhishekkekane"},"date":"2020-07-31 05:59:59.000000000","message":"Patch Set 3: Code-Review+2 Review-Priority+1\n\nJust curious, should we tag it with related-bug?\nAs this is in chain for fixing race condition issue, we might need to backport this to stable/ussuri as well.","accounts_in_message":[],"_revision_number":3},{"id":"795a302b756843ab95731065f09cd3ba82664839","author":{"_account_id":5202,"name":"Erno Kuvaja","email":"jokke@usr.fi","username":"jokke"},"date":"2020-08-04 17:36:17.000000000","message":"Patch Set 3: Code-Review-1\n\nWhile testing is good, redundant testing is not going to do any good for us. This is already done in glance/tests/unit/async_/test_async.py. Rather not need to change it in multiple places if we refactor these some day.","accounts_in_message":[],"_revision_number":3},{"id":"438a73068781188108ffeba1d321ce5f1a313a4a","author":{"_account_id":4393,"name":"Dan Smith","email":"dms@danplanet.com","username":"danms"},"date":"2020-08-04 17:53:01.000000000","message":"Abandoned\n\nI initially broke the flow while working on this stack and those tests didn\u0027t fail, but I\u0027m not sure what I did as it seems like they should have caught that. I didn\u0027t initially find them since they\u0027re import-specific yet are in test_async.py and not test_api_image_import.py. But anyway, until and unless I can figure out why this caught what I did and the others didn\u0027t, I\u0027ll drop it out.","accounts_in_message":[],"_revision_number":3},{"id":"f6d8cbbc3d0c7f5e428a3a0f8a5d8ac20efecce9","author":{"_account_id":4393,"name":"Dan Smith","email":"dms@danplanet.com","username":"danms"},"date":"2020-08-04 17:59:57.000000000","message":"Patch Set 3:\n\n\u003e I initially broke the flow while working on this stack and those\n \u003e tests didn\u0027t fail, but I\u0027m not sure what I did as it seems like\n \u003e they should have caught that. I didn\u0027t initially find them since\n \u003e they\u0027re import-specific yet are in test_async.py and not\n \u003e test_api_image_import.py. But anyway, until and unless I can figure\n \u003e out why this caught what I did and the others didn\u0027t, I\u0027ll drop it\n \u003e out.\n\nActually, thinking back, I might\u0027ve been running only the test_api_image_import module for speed at the time, and thus saw no tests fail when I broke the flow.\n\nIs there any objection to moving those import-flow-specific tests into test_api_image_import so it\u0027s clear that they get run when testing that stuff?","accounts_in_message":[],"_revision_number":3},{"id":"5cfa016c15910b0fee471a2a65198fb18fd16e87","author":{"_account_id":11904,"name":"Sean McGinnis","email":"sean.mcginnis@gmail.com","username":"SeanM"},"date":"2020-08-04 18:43:35.000000000","message":"Patch Set 3:\n\n\u003e \u003e I initially broke the flow while working on this stack and those\n \u003e \u003e tests didn\u0027t fail, but I\u0027m not sure what I did as it seems like\n \u003e \u003e they should have caught that. I didn\u0027t initially find them since\n \u003e \u003e they\u0027re import-specific yet are in test_async.py and not\n \u003e \u003e test_api_image_import.py. But anyway, until and unless I can\n \u003e figure\n \u003e \u003e out why this caught what I did and the others didn\u0027t, I\u0027ll drop\n \u003e it\n \u003e \u003e out.\n \u003e \n \u003e Actually, thinking back, I might\u0027ve been running only the\n \u003e test_api_image_import module for speed at the time, and thus saw no\n \u003e tests fail when I broke the flow.\n \u003e \n \u003e Is there any objection to moving those import-flow-specific tests\n \u003e into test_api_image_import so it\u0027s clear that they get run when\n \u003e testing that stuff?\n\n+1 from me for anything that increases clarity and the likelihood that someone will find what they are looking for easier.","accounts_in_message":[],"_revision_number":3},{"id":"f756df5d53043669df16c97ef50f4b046d839d49","author":{"_account_id":5202,"name":"Erno Kuvaja","email":"jokke@usr.fi","username":"jokke"},"date":"2020-08-04 18:58:35.000000000","message":"Patch Set 3:\n\n\u003e \u003e I initially broke the flow while working on this stack and those\n \u003e \u003e tests didn\u0027t fail, but I\u0027m not sure what I did as it seems like\n \u003e \u003e they should have caught that. I didn\u0027t initially find them since\n \u003e \u003e they\u0027re import-specific yet are in test_async.py and not\n \u003e \u003e test_api_image_import.py. But anyway, until and unless I can\n \u003e figure\n \u003e \u003e out why this caught what I did and the others didn\u0027t, I\u0027ll drop\n \u003e it\n \u003e \u003e out.\n \u003e \n \u003e Actually, thinking back, I might\u0027ve been running only the\n \u003e test_api_image_import module for speed at the time, and thus saw no\n \u003e tests fail when I broke the flow.\n \u003e \n \u003e Is there any objection to moving those import-flow-specific tests\n \u003e into test_api_image_import so it\u0027s clear that they get run when\n \u003e testing that stuff?\n\nThe test_api_image_import.py seems to have task specific tests for that flow. All the executor tests seem to be in the test_async.py on level higher. I\u0027d rather leave it logically like that than start spreading those tests across all the flows. Just my 2cents","accounts_in_message":[],"_revision_number":3}],"current_revision_number":3,"current_revision":"9ae75d2d2aec6fc0ce0859168258382ada74ac14","revisions":{"1fe208e1d925aa3ab22d2d71fb91562246a449b3":{"kind":"REWORK","_number":1,"created":"2020-07-28 17:08:12.000000000","uploader":{"_account_id":4393,"name":"Dan Smith","email":"dms@danplanet.com","username":"danms"},"ref":"refs/changes/94/743594/1","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/glance","ref":"refs/changes/94/743594/1","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/glance refs/changes/94/743594/1 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/glance refs/changes/94/743594/1 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/glance refs/changes/94/743594/1 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/glance refs/changes/94/743594/1"}}},"commit":{"parents":[{"commit":"ee52f9a10d764c32588953add711293483631b03","subject":"Add testing for _CompleteTask in api_image_import","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/glance/commit/ee52f9a10d764c32588953add711293483631b03"}]}],"author":{"name":"Dan Smith","email":"dansmith@redhat.com","date":"2020-07-28 15:57:58.000000000","tz":-420},"committer":{"name":"Dan Smith","email":"dansmith@redhat.com","date":"2020-07-28 15:58:24.000000000","tz":-420},"subject":"Add a test to make sure import get_flow() includes all the steps we expect","message":"Add a test to make sure import get_flow() includes all the steps we expect\n\nChange-Id: Ieeaafcb575a53fd41d487069c07db65134528a0a\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/glance/commit/1fe208e1d925aa3ab22d2d71fb91562246a449b3"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/glance/commit/1fe208e1d925aa3ab22d2d71fb91562246a449b3"}]},"branch":"refs/heads/master"},"98969c363efe2cb5d74c9905b3ac29d489c028bd":{"kind":"TRIVIAL_REBASE","_number":2,"created":"2020-07-30 13:44:31.000000000","uploader":{"_account_id":4393,"name":"Dan Smith","email":"dms@danplanet.com","username":"danms"},"ref":"refs/changes/94/743594/2","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/glance","ref":"refs/changes/94/743594/2","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/glance refs/changes/94/743594/2 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/glance refs/changes/94/743594/2 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/glance refs/changes/94/743594/2 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/glance refs/changes/94/743594/2"}}},"commit":{"parents":[{"commit":"0be1639fd7990035f608813462a64bf4dc2df93d","subject":"Add testing for _CompleteTask in api_image_import","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/glance/commit/0be1639fd7990035f608813462a64bf4dc2df93d"}]}],"author":{"name":"Dan Smith","email":"dansmith@redhat.com","date":"2020-07-28 15:57:58.000000000","tz":-420},"committer":{"name":"Dan Smith","email":"dansmith@redhat.com","date":"2020-07-30 13:44:24.000000000","tz":-420},"subject":"Add a test to make sure import get_flow() includes all the steps we expect","message":"Add a test to make sure import get_flow() includes all the steps we expect\n\nChange-Id: Ieeaafcb575a53fd41d487069c07db65134528a0a\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/glance/commit/98969c363efe2cb5d74c9905b3ac29d489c028bd"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/glance/commit/98969c363efe2cb5d74c9905b3ac29d489c028bd"}]},"branch":"refs/heads/master"},"9ae75d2d2aec6fc0ce0859168258382ada74ac14":{"kind":"TRIVIAL_REBASE","_number":3,"created":"2020-07-31 00:35:39.000000000","uploader":{"_account_id":4393,"name":"Dan Smith","email":"dms@danplanet.com","username":"danms"},"ref":"refs/changes/94/743594/3","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/glance","ref":"refs/changes/94/743594/3","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/glance refs/changes/94/743594/3 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/glance refs/changes/94/743594/3 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/glance refs/changes/94/743594/3 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/glance refs/changes/94/743594/3"}}},"commit":{"parents":[{"commit":"ff54b10351ae2d2bf52e29384550b513c34307a2","subject":"Add testing for _CompleteTask in api_image_import","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/glance/commit/ff54b10351ae2d2bf52e29384550b513c34307a2"}]}],"author":{"name":"Dan Smith","email":"dansmith@redhat.com","date":"2020-07-28 15:57:58.000000000","tz":-420},"committer":{"name":"Dan Smith","email":"dansmith@redhat.com","date":"2020-07-31 00:35:33.000000000","tz":-420},"subject":"Add a test to make sure import get_flow() includes all the steps we expect","message":"Add a test to make sure import get_flow() includes all the steps we expect\n\nChange-Id: Ieeaafcb575a53fd41d487069c07db65134528a0a\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/glance/commit/9ae75d2d2aec6fc0ce0859168258382ada74ac14"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/glance/commit/9ae75d2d2aec6fc0ce0859168258382ada74ac14"}]},"branch":"refs/heads/master"}},"requirements":[],"submit_records":[],"submit_requirements":[]}