)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":25023,"name":"Jonathan Rosser","email":"jonathan.rosser@rd.bbc.co.uk","username":"jrosser"},"change_message_id":"b4d26c45e212c38f14ab70d9b9da2652ef6a95e1","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"4565e369_74bf6d05","updated":"2025-10-14 07:44:50.000000000","message":"Doesn\u0027t this mean that any attempt to install barbican-tempest-plugin with pip requiring a particular version, a git hash, a branch, or even `master` will fail due to incompatible constraints?","commit_id":"5f006c64d1bfd4a0d11f2f116986ce40f7c75b84"},{"author":{"_account_id":8556,"name":"Ghanshyam Maan","display_name":"Ghanshyam Maan","email":"gmaan.os14@gmail.com","username":"ghanshyam"},"change_message_id":"1fea6ebfaadea3041c73052f6f5becb282b5befc","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"cfdad27f_92ae83a9","updated":"2025-10-14 16:13:04.000000000","message":"I do not think we need to add tempest plugins in the g-r. We had it tempest in g-r previously and due to its branchless nature we have to remove it and add it in denylist file. If we add plugin it will create the issue as CI will enforce the max version of this plugin more details can be fgound in the notes here https://github.com/openstack/requirements/blob/73fb1fe9fc5f158ec49e33a79be73ba3641800f3/denylist.txt#L45\n\nIf really needed then dependent plugins can be added to the requirements file on plugin side so that they can maintain the min required version. This is how plugin adds tempest also in their requirement file - https://github.com/openstack/cinder-tempest-plugin/blob/058e172729cde892e61ee65c931d5defc0d3d81c/requirements.txt#L8","commit_id":"5f006c64d1bfd4a0d11f2f116986ce40f7c75b84"},{"author":{"_account_id":22629,"name":"Michal Nasiadka","email":"mnasiadka@gmail.com","username":"mnasiadka"},"change_message_id":"f832362963d722a33f518765663c9b3c9c67c69b","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":3,"id":"b136d98d_3d203195","updated":"2025-10-14 10:59:00.000000000","message":"Shouldn\u0027t it be added to https://github.com/openstack/requirements/blob/73fb1fe9fc5f158ec49e33a79be73ba3641800f3/denylist.txt#L45 ?","commit_id":"5f006c64d1bfd4a0d11f2f116986ce40f7c75b84"},{"author":{"_account_id":12898,"name":"Tony Breeds","email":"tony@bakeyournoodle.com","username":"tonyb"},"change_message_id":"7eb11d05d128767c6150ed3431637bbf69aa57b1","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"0b6fbf66_c566fa34","updated":"2025-10-14 02:16:26.000000000","message":"While this is typically the right thing to do because this is a tempest plugin I\u0027m a little hesitant.   I\u0027m checking with QA but I think it\u0027d be better to:\n\na) disable the requirements-check job in manila-tempest-plugin ;\nb) add barbican-tempest-plugin to the required-projects ;\nc) #profit\n\nno vote until the picture is a little clearer","commit_id":"5f006c64d1bfd4a0d11f2f116986ce40f7c75b84"},{"author":{"_account_id":12898,"name":"Tony Breeds","email":"tony@bakeyournoodle.com","username":"tonyb"},"change_message_id":"4431a6024903795f8e6f4dc20ab95c50c16b5118","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"ac531252_a02224cb","in_reply_to":"b136d98d_3d203195","updated":"2025-10-19 23:24:13.000000000","message":"Done","commit_id":"5f006c64d1bfd4a0d11f2f116986ce40f7c75b84"},{"author":{"_account_id":32919,"name":"kiran pawar","display_name":"Kiran Pawar","email":"kinpaa@gmail.com","username":"kpdev"},"change_message_id":"7cc70609d29f4f5a0cb4ad659cdda85b561e2cf7","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"6ef3f744_061f738b","updated":"2025-10-24 06:44:36.000000000","message":"Hi core,\nplease merge this change","commit_id":"bb869cb895c61e91793153b62e830c7514a1a508"},{"author":{"_account_id":8556,"name":"Ghanshyam Maan","display_name":"Ghanshyam Maan","email":"gmaan.os14@gmail.com","username":"ghanshyam"},"change_message_id":"6237d283030b7633350041b72215d976f4e16aae","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"3daa42ca_77c04270","updated":"2025-10-15 01:06:40.000000000","message":"I was more thinking of adding based on need but adding those in one shot also not bad idea.","commit_id":"bb869cb895c61e91793153b62e830c7514a1a508"}]}