)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":5314,"name":"Brian Rosmaita","email":"rosmaita.fossdev@gmail.com","username":"brian-rosmaita"},"change_message_id":"06dc39d3f65662d48d881f31ae5053332715ab29","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"219cd35b_ab6c5865","updated":"2022-07-07 12:31:27.000000000","message":"I don\u0027t think we should do this yet, for 2 reasons (neither of which you will like).\n\n1. importlib_metadata in py 3.8 and 3.9 has backported stdlib features from 3.10 that aren\u0027t in 3.8/3.9 stdlib.  See https://review.opendev.org/c/openstack/requirements/+/843315/2#message-4f232fcc6e7f741a00638fc086f6b62be4358cf0 for more on this.  So there are legitimate reasons to continue to use importlib_metadata.\n\n2. cinderlib is relying on this being a cinder requirement, so if we don\u0027t include it cinder-side, there will be problems with cinderlib.\n\n","commit_id":"d8c6fa29a56e91f3f50ccb5cc6b78c2db140646c"},{"author":{"_account_id":10459,"name":"Luigi Toscano","email":"ltoscano@redhat.com","username":"ltoscano"},"change_message_id":"7868a2c616f977ccf8de1c4423497954a2737553","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"6ea8e379_c27c4178","updated":"2022-07-07 13:03:16.000000000","message":"So do we need officially use ","commit_id":"d8c6fa29a56e91f3f50ccb5cc6b78c2db140646c"},{"author":{"_account_id":10459,"name":"Luigi Toscano","email":"ltoscano@redhat.com","username":"ltoscano"},"change_message_id":"7868a2c616f977ccf8de1c4423497954a2737553","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"f024d1b4_25c7904c","in_reply_to":"219cd35b_ab6c5865","updated":"2022-07-07 13:03:16.000000000","message":"Ok, so all downstream consumers need to use the external importlib_metadata for now.","commit_id":"d8c6fa29a56e91f3f50ccb5cc6b78c2db140646c"},{"author":{"_account_id":16312,"name":"Alfredo Moralejo","email":"amoralej@redhat.com","username":"amoralej"},"change_message_id":"a0f9bf6e13817f966956df0de7db65ac8ff3b05d","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"407269bf_52c56567","in_reply_to":"f024d1b4_25c7904c","updated":"2022-07-07 15:24:02.000000000","message":"So, my understanding on this:\n\nfor 1. That means there may be legitimate reasons to use importlib_metadata but it\u0027s not used in cinder, is this correct?\n\nFor 2., i\u0027d say it\u0027s covered properly in cinderlib https://github.com/openstack/cinderlib/blob/master/requirements.txt","commit_id":"d8c6fa29a56e91f3f50ccb5cc6b78c2db140646c"}]}