)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":22873,"name":"Martin Kopec","email":"mkopec@redhat.com","username":"mkopec"},"change_message_id":"b646f0c31dbefa1ed7e1b0a900309224e8060066","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"d1d6acf2_ba5c504f","updated":"2023-03-16 09:30:44.000000000","message":"one detail - the documentation still contains the previous hash:\nhttps://codesearch.opendev.org/?q\u003d56d259dd78cc9ae974cc5dc24a54dbd8008770e6\u0026i\u003dnope\u0026literal\u003dnope\u0026files\u003d\u0026excludeFiles\u003d\u0026repos\u003d","commit_id":"5079cca3766b2d979d6ad65e1fc570b55197e8a7"},{"author":{"_account_id":30674,"name":"Lukas Piwowarski","email":"lpiwowar@redhat.com","username":"lukas-piwowarski"},"change_message_id":"534dda35d8a6cbd5638d4bdfa4cb560b4bf7b811","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"9e259073_359a5c0b","in_reply_to":"d1d6acf2_ba5c504f","updated":"2023-03-16 09:45:02.000000000","message":"ack, updated","commit_id":"5079cca3766b2d979d6ad65e1fc570b55197e8a7"},{"author":{"_account_id":7160,"name":"arkady kanevsky","email":"akanevsk@redhat.com","username":"arkady"},"change_message_id":"ed0885c003c67bcc17034071b9dfafc268cbbd99","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"572eb199_5f5ed165","updated":"2023-03-16 17:18:08.000000000","message":"2 questions. But patch looks good.","commit_id":"61ce470e259eb1b18fbc82cbef3dc12c7da294a9"},{"author":{"_account_id":30025,"name":"Vida Haririan","display_name":"Vida Haririan","email":"vhariria@redhat.com","username":"vhari"},"change_message_id":"2b104759f005a536690869c5615e236b553e0568","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"8c64f96b_032d3a3c","updated":"2023-03-16 17:16:54.000000000","message":"Looks good. Thanks for the change.","commit_id":"61ce470e259eb1b18fbc82cbef3dc12c7da294a9"},{"author":{"_account_id":7160,"name":"arkady kanevsky","email":"akanevsk@redhat.com","username":"arkady"},"change_message_id":"5e58d0137cd3420a1231cb9c2eb0adf43eb202e8","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"8a75574e_7155a2c7","updated":"2023-03-20 21:56:04.000000000","message":"LGTM.\nWill wait for +2 till zuul issue addressed.\n\nSlight preference toward #1 option in Martin\u0027s proposal. Simply because we will have to do it eventually anyhow.","commit_id":"9499a46716a964e8d11722d88e3097d2bea3e76d"},{"author":{"_account_id":22873,"name":"Martin Kopec","email":"mkopec@redhat.com","username":"mkopec"},"change_message_id":"c3755ef2bd928213b306f82c3a59fe5bdafe21bd","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"a7889190_46255465","updated":"2023-03-20 20:49:15.000000000","message":"hm, if i should guess, the failures are probably related to this - https://opendev.org/openstack/tempest/commit/8f4a22141ea06a0a02c41b1dd2c11e8901db90d3 \n\nthe question is, should we try to update the tempest.conf (that should be most likely enough) or should we consider using the hash from your previous patch set (the one which contains the fix we need but doesn\u0027t contain the change i linked above).","commit_id":"9499a46716a964e8d11722d88e3097d2bea3e76d"},{"author":{"_account_id":22873,"name":"Martin Kopec","email":"mkopec@redhat.com","username":"mkopec"},"change_message_id":"30db71c44d0bb8ad89466ff93563b7f463ee43f8","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":5,"id":"111fc5ae_e5b520db","updated":"2023-04-04 20:12:22.000000000","message":"recheck the fix is being merged now - https://review.opendev.org/c/openinfra/ansible-role-refstack-client/+/878198/","commit_id":"104a1475ce8db23ed7f8702c06faf6d809de811c"},{"author":{"_account_id":22873,"name":"Martin Kopec","email":"mkopec@redhat.com","username":"mkopec"},"change_message_id":"718708263695295721b85d327d3f60618c9e3945","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":5,"id":"8f8cbaf7_3db37a23","updated":"2023-04-05 07:37:09.000000000","message":"recheck the fix is merged now","commit_id":"104a1475ce8db23ed7f8702c06faf6d809de811c"},{"author":{"_account_id":30674,"name":"Lukas Piwowarski","email":"lpiwowar@redhat.com","username":"lukas-piwowarski"},"change_message_id":"f66c19902e7d86d65c556abf96f9f9a38215f2da","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":5,"id":"114b309e_90d6756a","in_reply_to":"8f8cbaf7_3db37a23","updated":"2023-04-05 07:52:33.000000000","message":"I think we need to bump up the checkout point first to include the tempest account-generator change. https://review.opendev.org/c/openinfra/refstack-client/+/878174","commit_id":"104a1475ce8db23ed7f8702c06faf6d809de811c"},{"author":{"_account_id":22873,"name":"Martin Kopec","email":"mkopec@redhat.com","username":"mkopec"},"change_message_id":"b0626340a54a3511841afe5a7a6b960eaedd8857","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":6,"id":"7ac463dc_a47f0cfd","updated":"2023-04-05 14:16:33.000000000","message":"thanks Lukas, lgtm","commit_id":"846e3d2983eb575d7bed371fb6441240f20d7ee1"}],"README.rst":[{"author":{"_account_id":7160,"name":"arkady kanevsky","email":"akanevsk@redhat.com","username":"arkady"},"change_message_id":"ed0885c003c67bcc17034071b9dfafc268cbbd99","unresolved":true,"context_lines":[{"line_number":26,"context_line":""},{"line_number":27,"context_line":"   b. -t option allows to specify tag in Tempest repository which will be installed."},{"line_number":28,"context_line":"   For example: execute ``./setup_env -t tags/3`` to install Tempest tag-3."},{"line_number":29,"context_line":"   By default, Tempest 31.1.0 will be installed from commit"},{"line_number":30,"context_line":"   d78e405364ca3725820fc7bd4b2f2af198ccc624 (March 2023)."},{"line_number":31,"context_line":""},{"line_number":32,"context_line":"   c. -p option allows to specify python version - 3.8.10 (-p 3)"}],"source_content_type":"text/x-rst","patch_set":3,"id":"4571ad0d_d612b689","line":29,"range":{"start_line":29,"start_character":14,"end_line":29,"end_character":27},"updated":"2023-03-16 17:18:08.000000000","message":"the same Tempest version as last year?","commit_id":"61ce470e259eb1b18fbc82cbef3dc12c7da294a9"},{"author":{"_account_id":30674,"name":"Lukas Piwowarski","email":"lpiwowar@redhat.com","username":"lukas-piwowarski"},"change_message_id":"9b9ceca55b73cf7059e8b1df8856b68af5d28a25","unresolved":true,"context_lines":[{"line_number":26,"context_line":""},{"line_number":27,"context_line":"   b. -t option allows to specify tag in Tempest repository which will be installed."},{"line_number":28,"context_line":"   For example: execute ``./setup_env -t tags/3`` to install Tempest tag-3."},{"line_number":29,"context_line":"   By default, Tempest 31.1.0 will be installed from commit"},{"line_number":30,"context_line":"   d78e405364ca3725820fc7bd4b2f2af198ccc624 (March 2023)."},{"line_number":31,"context_line":""},{"line_number":32,"context_line":"   c. -p option allows to specify python version - 3.8.10 (-p 3)"}],"source_content_type":"text/x-rst","patch_set":3,"id":"f96f279f_909b9fbc","line":29,"range":{"start_line":29,"start_character":14,"end_line":29,"end_character":27},"in_reply_to":"4571ad0d_d612b689","updated":"2023-03-20 17:27:39.000000000","message":"This patch is not associated with any tag. We can fix this. A new tag for tempest was released 3 days ago.","commit_id":"61ce470e259eb1b18fbc82cbef3dc12c7da294a9"}],"setup_env":[{"author":{"_account_id":7160,"name":"arkady kanevsky","email":"akanevsk@redhat.com","username":"arkady"},"change_message_id":"ed0885c003c67bcc17034071b9dfafc268cbbd99","unresolved":true,"context_lines":[{"line_number":1,"context_line":"#!/bin/bash -x"},{"line_number":2,"context_line":""},{"line_number":3,"context_line":"# Default Tempest commit:"},{"line_number":4,"context_line":"# SHA d78e405364ca3725820fc7bd4b2f2af198ccc624 (March 2023)"},{"line_number":5,"context_line":"CHECKOUT_POINT\u003dd78e405364ca3725820fc7bd4b2f2af198ccc624"},{"line_number":6,"context_line":"PY_VERSION\u003d\"3.8.10\""},{"line_number":7,"context_line":"UPPER_CONSTRAINTS_FILE\u003d\"https://opendev.org/openstack/requirements/raw/branch/master/upper-constraints.txt\""}],"source_content_type":"application/x-shellscript","patch_set":3,"id":"e0b052f9_a8504d4c","line":4,"range":{"start_line":4,"start_character":6,"end_line":4,"end_character":59},"updated":"2023-03-16 17:18:08.000000000","message":"do we want to include Tempest Tag version?","commit_id":"61ce470e259eb1b18fbc82cbef3dc12c7da294a9"},{"author":{"_account_id":30674,"name":"Lukas Piwowarski","email":"lpiwowar@redhat.com","username":"lukas-piwowarski"},"change_message_id":"9b9ceca55b73cf7059e8b1df8856b68af5d28a25","unresolved":true,"context_lines":[{"line_number":1,"context_line":"#!/bin/bash -x"},{"line_number":2,"context_line":""},{"line_number":3,"context_line":"# Default Tempest commit:"},{"line_number":4,"context_line":"# SHA d78e405364ca3725820fc7bd4b2f2af198ccc624 (March 2023)"},{"line_number":5,"context_line":"CHECKOUT_POINT\u003dd78e405364ca3725820fc7bd4b2f2af198ccc624"},{"line_number":6,"context_line":"PY_VERSION\u003d\"3.8.10\""},{"line_number":7,"context_line":"UPPER_CONSTRAINTS_FILE\u003d\"https://opendev.org/openstack/requirements/raw/branch/master/upper-constraints.txt\""}],"source_content_type":"application/x-shellscript","patch_set":3,"id":"60552888_3e65c4b7","line":4,"range":{"start_line":4,"start_character":6,"end_line":4,"end_character":59},"in_reply_to":"e0b052f9_a8504d4c","updated":"2023-03-20 17:27:39.000000000","message":"This checkpoint is not associated with any tag in tempest. The change that fixes the bug is quite new so it is not included in the tag 34.0.0. \n\nBut I noticed that since I\u0027ve made this patch a new tag 34.1.0 was created (3 days ago). I\u0027ll change the checkpoint so that we use tag 34.1.0. It\u0027ll be better to have a checkpoint associated with a tag.","commit_id":"61ce470e259eb1b18fbc82cbef3dc12c7da294a9"}]}
