)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":35834,"name":"Yosi Ben Shimon","display_name":"Yosi Ben Shimon","email":"ybenshim@redhat.com","username":"ybenshim"},"change_message_id":"de6eb553b7859da99a23c7d2d34a0b37e6f7c3a8","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"5661ad16_3dcdee44","updated":"2023-09-21 10:08:42.000000000","message":"On second thought, I think we should keep the setUp method.\nThe reason is that part of the tests are disruptive (like delete for example) and we don\u0027t want to change/delete the resource that is shared between the tests.\nTests like this (delete/update...) will create and delete their own resource.","commit_id":"d0d307a7749c5b9b85e808b420670f16217982fb"},{"author":{"_account_id":35834,"name":"Yosi Ben Shimon","display_name":"Yosi Ben Shimon","email":"ybenshim@redhat.com","username":"ybenshim"},"change_message_id":"d7fcc3395fed878af6923a64dfdb531d3dfcb410","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"5fde3d70_31772685","updated":"2023-09-21 08:42:52.000000000","message":"Tested the base.py changes from rbac/v3 locally and it works fine.\nLGTM","commit_id":"d0d307a7749c5b9b85e808b420670f16217982fb"},{"author":{"_account_id":8556,"name":"Ghanshyam Maan","display_name":"Ghanshyam Maan","email":"gmaan.os14@gmail.com","username":"ghanshyam"},"change_message_id":"ce239a0bf48db37690741fe59cc4481073b5e1bc","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"cf7514ee_f57be650","in_reply_to":"5661ad16_3dcdee44","updated":"2023-09-21 18:21:30.000000000","message":"ah, nice catch. I missed to think about super class setUp calling. thanks and done","commit_id":"d0d307a7749c5b9b85e808b420670f16217982fb"},{"author":{"_account_id":8556,"name":"Ghanshyam Maan","display_name":"Ghanshyam Maan","email":"gmaan.os14@gmail.com","username":"ghanshyam"},"change_message_id":"2003a154f6d34f142b6541fb1338e5fa0166cff9","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"f8d549bf_e810e857","in_reply_to":"cf7514ee_f57be650","updated":"2023-09-21 18:31:11.000000000","message":"or you mean to keep microversion setting in setUp method also? If so then we do not need it as microversion setting is per test class and not at test level. for example, you can set test required microversion at class level only and Temepst select the microversion based on test required microversion and conigured one. if any test need to call API with different microversion then it needs to be move to separate test class","commit_id":"d0d307a7749c5b9b85e808b420670f16217982fb"},{"author":{"_account_id":8556,"name":"Ghanshyam Maan","display_name":"Ghanshyam Maan","email":"gmaan.os14@gmail.com","username":"ghanshyam"},"change_message_id":"ddf7e0fac257d1930a75b3e8e3f6c4d7b899dd33","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"9449dc2a_6a8eadcf","updated":"2023-09-26 01:23:07.000000000","message":"recheck Details: {\u0027type\u0027: \u0027SubnetInUse\u0027, \u0027message\u0027: \u0027Unable to complete operation on subnet e629faec-9e2f-4bfe-aac0-a8589c47efde: One or more ports have an IP allocation from this subnet.\u0027, \u0027detail\u0027: \u0027\u0027}","commit_id":"e4f0e1105a93418dbe2140791ae48e06fecdac66"},{"author":{"_account_id":8556,"name":"Ghanshyam Maan","display_name":"Ghanshyam Maan","email":"gmaan.os14@gmail.com","username":"ghanshyam"},"change_message_id":"504c0f854993711f4430e6e9bb0e19b6ef609371","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"fc9c1bd4_1ccaf200","updated":"2023-09-22 19:32:59.000000000","message":"recheck tempest.lib.exceptions.IdentityError: Got identity error in barbican job","commit_id":"e4f0e1105a93418dbe2140791ae48e06fecdac66"},{"author":{"_account_id":8556,"name":"Ghanshyam Maan","display_name":"Ghanshyam Maan","email":"gmaan.os14@gmail.com","username":"ghanshyam"},"change_message_id":"dcd8dcd216b83f2db4354ad632c643827a08f742","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"3f8ffd6c_d7531e7c","updated":"2023-12-14 22:32:48.000000000","message":"recheck testing again to get failure log if those are related to this change","commit_id":"e4f0e1105a93418dbe2140791ae48e06fecdac66"},{"author":{"_account_id":5314,"name":"Brian Rosmaita","email":"rosmaita.fossdev@gmail.com","username":"brian-rosmaita"},"change_message_id":"4da0a2d1be4a5f2c55f66ba969f7324ca4adb434","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"4d6fcc5e_8f7ea885","updated":"2024-02-28 20:07:15.000000000","message":"This is consistent with the change made to tempest by change Ia4b884a79cc0e5005.","commit_id":"8992770bad4a9991e7e8aa5e33559a9a66599c5e"},{"author":{"_account_id":10459,"name":"Luigi Toscano","email":"ltoscano@redhat.com","username":"ltoscano"},"change_message_id":"d3b303d20e5b15027624c11951aad8ce9accd53c","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"49e7d749_b34acdc1","updated":"2024-01-17 13:48:48.000000000","message":"recheck\n\ncinder-tempest-plugin-lvm-multiattach (only failure) seems to be more green these days","commit_id":"8992770bad4a9991e7e8aa5e33559a9a66599c5e"},{"author":{"_account_id":8556,"name":"Ghanshyam Maan","display_name":"Ghanshyam Maan","email":"gmaan.os14@gmail.com","username":"ghanshyam"},"change_message_id":"7817c29afb4766518e0990487317fa7721702a83","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"9f485d9e_6c65d968","updated":"2024-01-25 06:11:26.000000000","message":"recheck another try. baribican job failing now","commit_id":"8992770bad4a9991e7e8aa5e33559a9a66599c5e"},{"author":{"_account_id":8556,"name":"Ghanshyam Maan","display_name":"Ghanshyam Maan","email":"gmaan.os14@gmail.com","username":"ghanshyam"},"change_message_id":"a1f08a5ae168391ba85edc6f8e276b8dffa3387a","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"d38ee3ac_72e4d711","updated":"2024-01-10 03:31:03.000000000","message":"recheck multiple fixes merged let\u0027s see if gate is happy","commit_id":"8992770bad4a9991e7e8aa5e33559a9a66599c5e"}]}
