)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":10459,"name":"Luigi Toscano","email":"ltoscano@redhat.com","username":"ltoscano"},"change_message_id":"1715957932c9a3511504d89ef593e2783ee27875","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"a7c40154_1955d5a4","updated":"2022-03-18 14:48:24.000000000","message":"I was wondering about the value of this as well. There is no classifier that would make people think that python 2 is supported.\n\nThen I\u0027ve noticed this was applied to tempest as well:\n\nhttps://review.opendev.org/c/openstack/tempest/+/704096\n\nSo I\u0027m not sure. Ghanshyam, should we consistently add this at least to tempest plugins?\n","commit_id":"8e60f04ed416b0469e70b007e061c20eee2a7f84"},{"author":{"_account_id":4523,"name":"Eric Harney","email":"eharney@redhat.com","username":"eharney"},"change_message_id":"4388d76f2dab55631edc0492db725b9d1883752e","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"31c6641d_82167f46","updated":"2022-03-14 21:24:07.000000000","message":"Is there really value in adding this?\n\nPython 2 is dead, there is no Python 4... what does it actually tell anyone that isn\u0027t already stated by specifying Python 3 support?","commit_id":"8e60f04ed416b0469e70b007e061c20eee2a7f84"},{"author":{"_account_id":5314,"name":"Brian Rosmaita","email":"rosmaita.fossdev@gmail.com","username":"brian-rosmaita"},"change_message_id":"6a1fca879f417a11da3869e9eb0d64cb4a2734f0","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"7ea4f864_d41a80ea","updated":"2022-03-20 12:57:39.000000000","message":"Let\u0027s revise this patch to be consistent with tempest:\nhttps://opendev.org/openstack/tempest/src/commit/5c556626d16c2a4c6ee9c9da45d4954db6f83f27/setup.cfg#L9-L23\n\nThis would basically mean adding the python_requires as Ghanshyam suggests, and updating the \u0027classifier\u0027 section to have the same content.\n\nWhile you\u0027re working on this, please update the other metadata keys to use underscores instead of hyphens (here\u0027s the patch that did this in cinder https://review.opendev.org/c/openstack/cinder/+/787263 ).\n\nDon\u0027t forget to update your commit message to reflect this wider scope.\n\nThanks!","commit_id":"8e60f04ed416b0469e70b007e061c20eee2a7f84"},{"author":{"_account_id":30615,"name":"Tushar Trambak Gite","email":"tushargite96@gmail.com","username":"tushargite96"},"change_message_id":"262e0aec1b9a2780143fee62e87fb6750ea3266a","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"ae23d3cc_3d94f9c6","updated":"2022-03-11 12:04:53.000000000","message":"ping","commit_id":"8e60f04ed416b0469e70b007e061c20eee2a7f84"},{"author":{"_account_id":10459,"name":"Luigi Toscano","email":"ltoscano@redhat.com","username":"ltoscano"},"change_message_id":"c8d877c254dbdafabf7608cf535fedb966e00986","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"b1ec4765_3e49d99d","in_reply_to":"7ea4f864_d41a80ea","updated":"2022-03-20 13:19:57.000000000","message":"About replacing hyphens with underscores, we have just merged the change:\nhttps://review.opendev.org/c/openstack/cinder-tempest-plugin/+/789043","commit_id":"8e60f04ed416b0469e70b007e061c20eee2a7f84"},{"author":{"_account_id":8556,"name":"Ghanshyam Maan","display_name":"Ghanshyam Maan","email":"gmaan.os14@gmail.com","username":"ghanshyam"},"change_message_id":"db8efc7c10f5d3b575601b41391e874ffe1f0192","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"46316aa8_c096a94c","in_reply_to":"a7c40154_1955d5a4","updated":"2022-03-18 15:22:12.000000000","message":"I think we can add requires-python \u003d \u003e\u003d3.6 here to be more explicitly about tempest and plugins does not support \u003cpy3.6 version","commit_id":"8e60f04ed416b0469e70b007e061c20eee2a7f84"},{"author":{"_account_id":5314,"name":"Brian Rosmaita","email":"rosmaita.fossdev@gmail.com","username":"brian-rosmaita"},"change_message_id":"16c0fa30fa3e9e539a95c6a4da5cc05bb8ff8c26","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"6d8c2c3a_453991a6","updated":"2022-03-22 12:09:49.000000000","message":"Thanks, Tushar.  LGTM.","commit_id":"c70f27ca61261dc8176375275d3fd16792d8d57c"},{"author":{"_account_id":30615,"name":"Tushar Trambak Gite","email":"tushargite96@gmail.com","username":"tushargite96"},"change_message_id":"5bd4607782490135a6160b58027cedc5bf5f6f42","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"b602da10_b758decf","updated":"2022-03-23 03:38:09.000000000","message":"recheck","commit_id":"c70f27ca61261dc8176375275d3fd16792d8d57c"},{"author":{"_account_id":10459,"name":"Luigi Toscano","email":"ltoscano@redhat.com","username":"ltoscano"},"change_message_id":"dfe5ca47ae3f45ad0eba12328ef7790681756f25","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"77a5f46a_af6b0361","updated":"2022-03-22 14:06:45.000000000","message":"recheck\n\nUnrelated failure","commit_id":"c70f27ca61261dc8176375275d3fd16792d8d57c"},{"author":{"_account_id":5314,"name":"Brian Rosmaita","email":"rosmaita.fossdev@gmail.com","username":"brian-rosmaita"},"change_message_id":"aa0a51160c1a6de60811abf5874b8201e56d4749","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"ead31a1e_4d115b1a","updated":"2022-03-22 14:07:37.000000000","message":"recheck cinder-tempest-plugin-cbak-ceph - failures in a bunch of tests getting 500s from the identity service","commit_id":"c70f27ca61261dc8176375275d3fd16792d8d57c"}]}