)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":10135,"name":"Lee Yarwood","display_name":"Lee Yarwood","email":"lyarwood@redhat.com","username":"lyarwood"},"change_message_id":"f2efa6e82679c197b0525b07741aea820a5c4acf","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"72d78fe7_9cb21add","updated":"2021-12-03 18:31:27.000000000","message":"\u003e Patch Set 1: Code-Review+2 Workflow+1\n\u003e \n\u003e do we include tempest only libs in GR?  also for any new lib we need https://docs.openstack.org/project-team-guide/dependency-management.html#for-new-requirements answered in the commit message\n\nGah Apologies *again*, I\u0027ll go over these on Monday but I can already see a there\u0027s a lack of testing in the codebase that might force us to use something different. ","commit_id":"04862fb4e130081b84c3b6290c1b9c6d7e0fcd82"},{"author":{"_account_id":1955,"name":"Alan Pevec","email":"alan.pevec@redhat.com","username":"apevec"},"change_message_id":"9588ea53f42a343465617e45856ed41dc60bd15e","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"cbcf7cb9_557ec190","updated":"2022-03-01 11:37:17.000000000","message":"let\u0027s abandon this since Tempest is not constrained","commit_id":"04862fb4e130081b84c3b6290c1b9c6d7e0fcd82"},{"author":{"_account_id":22873,"name":"Martin Kopec","email":"mkopec@redhat.com","username":"mkopec"},"change_message_id":"7efc21f76145465221a7b195909c3b014143901c","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"79cf65b3_4be660f3","updated":"2022-03-01 14:53:59.000000000","message":"let\u0027s avoid this lib, it\u0027s too much risk (stability concerns mentioned in the comments) plus additional work for packaging .. if we need only a simple ping from the lib, we can easily implement our own method for that, we don\u0027t need to import a special library in this case\n\nI proposed a new patch set without using this lib: https://review.opendev.org/c/openstack/tempest/+/817635","commit_id":"04862fb4e130081b84c3b6290c1b9c6d7e0fcd82"},{"author":{"_account_id":8556,"name":"Ghanshyam Maan","display_name":"Ghanshyam Maan","email":"gmaan.os14@gmail.com","username":"ghanshyam"},"change_message_id":"27d324c63e673e662579c90fd5d39b9add82a791","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"be260200_b1a899e5","updated":"2021-12-03 18:19:32.000000000","message":"thanks","commit_id":"04862fb4e130081b84c3b6290c1b9c6d7e0fcd82"},{"author":{"_account_id":8556,"name":"Ghanshyam Maan","display_name":"Ghanshyam Maan","email":"gmaan.os14@gmail.com","username":"ghanshyam"},"change_message_id":"7fe5bc95d11c0b70cb0bc8c4491ac972cf2498f4","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"b3038c2d_df9fff9a","in_reply_to":"72d78fe7_9cb21add","updated":"2021-12-03 18:41:22.000000000","message":"tempest is in blacklist and not constrained (since https://review.opendev.org/c/openstack/requirements/+/787621). I am not sure on lib needed by tempest needs to be in blacklist too, I think those can be used in another place also?","commit_id":"04862fb4e130081b84c3b6290c1b9c6d7e0fcd82"}]}
