)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":5314,"name":"Brian Rosmaita","email":"rosmaita.fossdev@gmail.com","username":"brian-rosmaita"},"change_message_id":"3f309dfe2147e862be1913adbb2f10f56bd8428a","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"2eb49f14_7317ac13","updated":"2022-05-19 21:46:53.000000000","message":"Asked whether this backport is acceptable on the ML: http://lists.openstack.org/pipermail/openstack-discuss/2022-May/028608.html","commit_id":"9043efc5fff4daf83fdbcba1e200ab2e113e0464"},{"author":{"_account_id":5202,"name":"Erno Kuvaja","email":"jokke@usr.fi","username":"jokke"},"change_message_id":"2b02b898355b518a6fe475b13ff3fa399f40663b","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"9e287c78_0ad7be51","updated":"2022-06-02 11:54:57.000000000","message":"Backport is clean, e don\u0027t have objections from the ML and https://review.opendev.org/c/openstack/glance/+/843208 addresses my concerns as follow-up. Good to go.\n\nThanks Brian and Abhishek!","commit_id":"9043efc5fff4daf83fdbcba1e200ab2e113e0464"},{"author":{"_account_id":5314,"name":"Brian Rosmaita","email":"rosmaita.fossdev@gmail.com","username":"brian-rosmaita"},"change_message_id":"5764107064e04651e345002b9deeda7d1e90cb11","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"a4c45675_dc8d1c2e","updated":"2022-05-25 16:14:20.000000000","message":"Előd Illés (Release Management PTL) is OK with this backport: http://lists.openstack.org/pipermail/openstack-discuss/2022-May/028617.html","commit_id":"9043efc5fff4daf83fdbcba1e200ab2e113e0464"},{"author":{"_account_id":5314,"name":"Brian Rosmaita","email":"rosmaita.fossdev@gmail.com","username":"brian-rosmaita"},"change_message_id":"6123a88345a3f56f39e417e76031ac0fe4e08ad6","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"e0ab8096_d4d9906c","updated":"2022-05-25 15:57:06.000000000","message":"Follow-up patch to increment the API version is https://review.opendev.org/c/openstack/glance/+/843208","commit_id":"9043efc5fff4daf83fdbcba1e200ab2e113e0464"},{"author":{"_account_id":5314,"name":"Brian Rosmaita","email":"rosmaita.fossdev@gmail.com","username":"brian-rosmaita"},"change_message_id":"67e3c27df0bd7db4d290f44314fbe0ac1e05e68d","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"4f073a20_2f1e7378","updated":"2022-05-13 18:33:26.000000000","message":"I completely disagree with this.  I think this is a legitimate exception.\n\n(1) The documentation was that this should be a 202: https://opendev.org/openstack/glance-specs/blame/commit/2638ada23d92f714f54b71db00330e4a6c921beb/specs/xena/approved/glance/cache-api.rst#L153\n\n(2) This is a purely administrative function, so the user impact is not widespread.\n\n(3) Yoga was released a month and a half ago, there hasn\u0027t been much time for admins to begin consuming this feature from change I69162e19bf095ef.\n\nThe policy Erno quoted says that \"Proposed backports breaking any of the above guidelines can be discussed as exception requests on the openstack-discuss list (prefix with [stable]) where the stable maintenance core team will have the final say\".  I\u0027ll put this on the agenda for the next glance meeting; if I\u0027m the only person in favor of this change, I\u0027ll withdraw my objection.  Otherwise we can take it to openstack-discuss.\n\nThe alternative that we have two versions of the api, one that returns 200 for this call and one that returns 202 is too stupid to contemplate.  If we don\u0027t allow this backport, we should just revert the change from the zed development branch, just as we would any other patch merged into the development branch that caused a regression.","commit_id":"9043efc5fff4daf83fdbcba1e200ab2e113e0464"},{"author":{"_account_id":5202,"name":"Erno Kuvaja","email":"jokke@usr.fi","username":"jokke"},"change_message_id":"46a1863134a777e27d101be81eaac0c9fe288cfb","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"56844cde_056c3cd8","updated":"2022-05-23 10:07:51.000000000","message":"No objections from ML so far.","commit_id":"9043efc5fff4daf83fdbcba1e200ab2e113e0464"},{"author":{"_account_id":5314,"name":"Brian Rosmaita","email":"rosmaita.fossdev@gmail.com","username":"brian-rosmaita"},"change_message_id":"dd1f397893ceaed860a12e8fcf74630fce05cc0d","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"2536ecba_5f0752f6","updated":"2022-05-26 12:16:19.000000000","message":"Release management has granted us an exception to backport this change, and I believe that the child patch addresses Erno\u0027s concerns about versioning.  So I think this is ready to go.","commit_id":"9043efc5fff4daf83fdbcba1e200ab2e113e0464"},{"author":{"_account_id":27615,"name":"Rajat Dhasmana","email":"rajatdhasmana@gmail.com","username":"whoami-rajat"},"change_message_id":"456096e48b28e5b866deb8bf53e9258ee08219a0","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"bba4c247_48562657","updated":"2022-05-11 11:06:47.000000000","message":"The main problem is regarding the timing, [1] shows when devstack checks for active images and [2] shows when the image transitions to active state.\n\n07:50:33 -- devstack requests images\n07:50:37 -- image transitions to active state\n\nNote that the devstack requests early hence is unable to find active images. This seems to be a problem with creating the image with import plugin (which is async) hence devstack proceeds with other steps and the image creation doesn\u0027t happen in time.\n\nYou can also confirm that image create was successful in the glance validation steps[3]\n\nI checked cinder side things and they look good.\nvolume creation (c-vol)[4],  glance store returning success (g-api)[5]\n\n[1] 2022-05-11 07:50:33.918925 | controller | ++ lib/tempest:configure_tempest:181        :   echo \u0027Found no valid images to use!\u0027\n\n[2] May 11 07:50:37.298347 ubuntu-focal-rax-dfw-0029612136 devstack@g-api.service[100807]: DEBUG glance.async_.flows.api_image_import [-] Image 5095fa8f-e22e-4b1b-8b62-8428d204e027 status changing from importing to active {{(pid\u003d100807) __exit__ /opt/stack/glance/glance/async_/flows/api_image_import.py:134}}\n\n[3] https://paste.opendev.org/show/b9AornouTEm4tsMJQYKS/\n\n[4] The image-volume was created successfully (c-vol)\nMay 11 07:50:09.238661 ubuntu-focal-rax-dfw-0029612136 cinder-volume[99871]: INFO cinder.volume.flows.manager.create_volume [None req-02d382ad-122f-4e1f-818f-d74aa161b0a4 service None] Volume volume-f3f3ff97-f45e-48c2-ac32-ac0a69620db1 (f3f3ff97-f45e-48c2-ac32-ac0a69620db1): created successfully\n\n[5] Glance cinder store returned success\nMay 11 07:50:37.264436 ubuntu-focal-rax-dfw-0029612136 devstack@g-api.service[100807]: DEBUG glance_store._drivers.cinder [-] Wrote 117440512 bytes to volume f3f3ff97-f45e-48c2-ac32-ac0a69620db1 with checksum 318f4eb47a165b55db0be707e0ecbf63. {{(pid\u003d100807) add /usr/local/lib/python3.8/dist-packages/glance_store/_drivers/cinder.py:1048}}\n\n","commit_id":"9043efc5fff4daf83fdbcba1e200ab2e113e0464"},{"author":{"_account_id":5202,"name":"Erno Kuvaja","email":"jokke@usr.fi","username":"jokke"},"change_message_id":"2501ccc0cc9fd66dbd833c3bb1102536a7feba94","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"350b5c34_b329294e","updated":"2022-05-11 11:23:57.000000000","message":"This is external API breaking change. Not allowed for stable backport [0].\n\nIn general we have been avoiding these kind of changes once the API has been released as there might be already consumers relying on the specific response codes implemented and documented. Such break would be totally unacceptable on stable minor/patch version upgrade.\n\nAs the patch was already merged in master we need to have follow up patch there to bump the api minor version to highlight the change and put warning of the change into RENOs.\n\n[0]https://docs.openstack.org/project-team-guide/stable-branches.html#review-guidelines","commit_id":"9043efc5fff4daf83fdbcba1e200ab2e113e0464"},{"author":{"_account_id":5314,"name":"Brian Rosmaita","email":"rosmaita.fossdev@gmail.com","username":"brian-rosmaita"},"change_message_id":"756d5cc5288cf5892f093335e503976e06e03961","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"f075c7b5_cab412f1","updated":"2022-05-19 21:37:28.000000000","message":"We discussed this at today\u0027s glance meeting [0], and Erno indicated that he will lift his -2 if we get a green light from the stable-maint team for this change.\n\n[0] https://meetings.opendev.org/meetings/glance/2022/glance.2022-05-19-14.00.log.html#l-34","commit_id":"9043efc5fff4daf83fdbcba1e200ab2e113e0464"},{"author":{"_account_id":5314,"name":"Brian Rosmaita","email":"rosmaita.fossdev@gmail.com","username":"brian-rosmaita"},"change_message_id":"7b1346b92e90113df06cd24bab466f062a3d5fa3","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"0d26973d_ad0057f5","updated":"2022-06-02 18:49:28.000000000","message":"openstack-tox-functional-py39 job failed when trying to upload logs:\n\nkex_exchange_identification: Connection closed by remote host\nConnection closed by 198.72.124.52 port 22\nrsync: connection unexpectedly closed (0 bytes received so far) [Receiver]\nrsync error: unexplained error (code 255) at io.c(228) [Receiver\u003d3.2.3]\n\nAll the tests actually passed.","commit_id":"9043efc5fff4daf83fdbcba1e200ab2e113e0464"},{"author":{"_account_id":5314,"name":"Brian Rosmaita","email":"rosmaita.fossdev@gmail.com","username":"brian-rosmaita"},"change_message_id":"cc4a7ab79ff7008ce0351b7283aa506600993822","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"0ade97d7_70ffb70a","updated":"2022-05-19 21:49:12.000000000","message":"recheck - glance-multistore-cinder-import failure should be fixed by https://review.opendev.org/c/openstack/devstack/+/842390 which has merged in stable/yoga","commit_id":"9043efc5fff4daf83fdbcba1e200ab2e113e0464"},{"author":{"_account_id":9303,"name":"Abhishek Kekane","email":"akekane@redhat.com","username":"abhishekkekane"},"change_message_id":"a84dff7a9912daf03f5be939fa8396ffd22f49a9","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"9e3127b8_b45174cb","updated":"2022-05-11 07:14:05.000000000","message":"recheck cinder failure","commit_id":"9043efc5fff4daf83fdbcba1e200ab2e113e0464"},{"author":{"_account_id":9303,"name":"Abhishek Kekane","email":"akekane@redhat.com","username":"abhishekkekane"},"change_message_id":"106cc2fcfb90083be9e6aa60ed059216738bbf0e","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"b25d749f_4ab5a81a","updated":"2022-06-02 13:59:59.000000000","message":"recheck functional failure","commit_id":"9043efc5fff4daf83fdbcba1e200ab2e113e0464"},{"author":{"_account_id":5314,"name":"Brian Rosmaita","email":"rosmaita.fossdev@gmail.com","username":"brian-rosmaita"},"change_message_id":"2ab2b6bc9506d3a82aa2ac61b6b927382412c85f","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"cafb3f0b_085d51a8","updated":"2022-06-02 17:26:51.000000000","message":"recheck glance-multistore-cinder-import : failures during setup and tests in tempest.api.volume.test_volumes_snapshots.VolumesSnapshotTestJSON - oom_reaper killed mysqld, leading to a bunch of db connection errors","commit_id":"9043efc5fff4daf83fdbcba1e200ab2e113e0464"},{"author":{"_account_id":5314,"name":"Brian Rosmaita","email":"rosmaita.fossdev@gmail.com","username":"brian-rosmaita"},"change_message_id":"5697538a3867e9bb69def0db71aa0dee065066b3","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"79964174_304bde18","updated":"2022-06-02 18:58:38.000000000","message":"recheck openstack-tox-functional-py39 : POST_FAILURE (but all tests passed)","commit_id":"9043efc5fff4daf83fdbcba1e200ab2e113e0464"},{"author":{"_account_id":5314,"name":"Brian Rosmaita","email":"rosmaita.fossdev@gmail.com","username":"brian-rosmaita"},"change_message_id":"085a263a3d3cd82fed9e9f1619003c0a2f880cbe","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"5304f803_225639b8","updated":"2022-06-03 03:23:52.000000000","message":"recheck openstack-tox-functional-py{36,39} : all failures are:\n      \n  File \"/home/zuul/src/opendev.org/openstack/glance/glance/tests/utils.py\", line 339, in wait_for_fork\n  raise RuntimeError(\u0027Gave up waiting for %i to exit!\u0027 % pid)","commit_id":"9043efc5fff4daf83fdbcba1e200ab2e113e0464"},{"author":{"_account_id":5314,"name":"Brian Rosmaita","email":"rosmaita.fossdev@gmail.com","username":"brian-rosmaita"},"change_message_id":"ad89393cc7d4e9379b79dc24d81d55e603967447","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"44aee3a3_7ee952bb","updated":"2022-06-02 20:31:42.000000000","message":"recheck tempest-ipv6-only : failed during devstack setup","commit_id":"9043efc5fff4daf83fdbcba1e200ab2e113e0464"},{"author":{"_account_id":5314,"name":"Brian Rosmaita","email":"rosmaita.fossdev@gmail.com","username":"brian-rosmaita"},"change_message_id":"7adc4efff73d25e3d4bec680aae4d3c203154caf","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"c55b87e0_c27eee52","updated":"2022-06-02 19:32:35.000000000","message":"tempest-ipv6-only failed during devstack setup:\n\n[ERROR] /opt/stack/devstack/lib/neutron_plugins/ovn_agent:178 Socket /var/run/openvswitch/ovnnb_db.sock not found","commit_id":"9043efc5fff4daf83fdbcba1e200ab2e113e0464"},{"author":{"_account_id":4393,"name":"Dan Smith","email":"dms@danplanet.com","username":"danms"},"change_message_id":"0ac84ccbfc7142d43595d1f1e84377b57ca747c6","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"e734a981_a620cd98","in_reply_to":"4f073a20_2f1e7378","updated":"2022-05-18 15:25:07.000000000","message":"Yeah, I definitely appreciate Erno\u0027s sensitivity to breaking API changes. But I agree, very (very) few people have deployed yoga at this point, and the scope of people that have started to use this is a fraction of that fraction. I would support quickly fixing and backporting this as an exception on the basis that far more people will notice that it changed from yoga to zed if we *don\u0027t* backport, and backport ASAP.","commit_id":"9043efc5fff4daf83fdbcba1e200ab2e113e0464"}]}