)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":22873,"name":"Martin Kopec","email":"mkopec@redhat.com","username":"mkopec"},"change_message_id":"61c6bb502c390962f5b4bc04ffe92abf3c23df47","unresolved":true,"context_lines":[{"line_number":4,"context_line":"Commit:     Milana Levy \u003cmillevy@redhat.com\u003e"},{"line_number":5,"context_line":"CommitDate: 2022-06-14 15:25:30 +0300"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":" Changed pip install tempest\\ to sudo pip3 install tempest\\"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Change-Id: I9ba22ab48cb1737a6917479089cddf961c30efe4"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"b4ef55ae_14f68458","line":7,"range":{"start_line":7,"start_character":0,"end_line":7,"end_character":1},"updated":"2022-06-14 18:52:02.000000000","message":"a leading space is not wanted.","commit_id":"1c90e79efce53258d52f7365dc80d2a738d5a0c8"},{"author":{"_account_id":22873,"name":"Martin Kopec","email":"mkopec@redhat.com","username":"mkopec"},"change_message_id":"61c6bb502c390962f5b4bc04ffe92abf3c23df47","unresolved":true,"context_lines":[{"line_number":5,"context_line":"CommitDate: 2022-06-14 15:25:30 +0300"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":" Changed pip install tempest\\ to sudo pip3 install tempest\\"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Change-Id: I9ba22ab48cb1737a6917479089cddf961c30efe4"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"5e2cfdff_62ab22be","line":8,"updated":"2022-06-14 18:52:02.000000000","message":"the general structure of a commit message is:\n\u003ctitle of the change no longer than 50 characters\u003e\nempty line\n\u003cbody of the commit msg explaining what\u0027s changing, why, what it fixes etc, the key is to provide as much context as possible Every line should be shorter than ~70 characters\u003e\n\nhttps://docs.openstack.org/contributors/common/git.html#commit-messages","commit_id":"1c90e79efce53258d52f7365dc80d2a738d5a0c8"}],"/PATCHSET_LEVEL":[{"author":{"_account_id":30742,"name":"Soniya Murlidhar Vyas","email":"svyas@redhat.com","username":"svyas"},"change_message_id":"c7e40e42f2bf135488355289191159aa52808650","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"0d311bac_708771cf","updated":"2022-12-13 06:36:43.000000000","message":"Thanks for updating documentation. Please address the comments asked by Martin and we can merge this change.","commit_id":"1c90e79efce53258d52f7365dc80d2a738d5a0c8"},{"author":{"_account_id":22873,"name":"Martin Kopec","email":"mkopec@redhat.com","username":"mkopec"},"change_message_id":"61c6bb502c390962f5b4bc04ffe92abf3c23df47","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"c96b99af_9d641c77","updated":"2022-06-14 18:52:02.000000000","message":"this is a good start, thank you for going through the tempest\u0027s documentation. Any chance you found other inconsistencies in the documentation such as invalid links, typos, pip/python instead of pip3/python3 occurrences etc?","commit_id":"1c90e79efce53258d52f7365dc80d2a738d5a0c8"}],"doc/source/overview.rst":[{"author":{"_account_id":22873,"name":"Martin Kopec","email":"mkopec@redhat.com","username":"mkopec"},"change_message_id":"61c6bb502c390962f5b4bc04ffe92abf3c23df47","unresolved":true,"context_lines":[{"line_number":61,"context_line":"   the Tempest repo::"},{"line_number":62,"context_line":""},{"line_number":63,"context_line":"    $ git clone https://opendev.org/openstack/tempest"},{"line_number":64,"context_line":"    $ sudo pip3 install tempest/"},{"line_number":65,"context_line":""},{"line_number":66,"context_line":"   This can be done within a venv, but the assumption for this guide is that"},{"line_number":67,"context_line":"   the Tempest CLI entry point will be in your shell\u0027s PATH."}],"source_content_type":"text/x-rst","patch_set":1,"id":"611d9c36_6cb4fa02","line":64,"range":{"start_line":64,"start_character":6,"end_line":64,"end_character":10},"updated":"2022-06-14 18:52:02.000000000","message":"having sudo here doesn\u0027t feel right because the note below says this can be done within a venv ... if it\u0027s done within a venv, sudo won\u0027t be needed, it\u0027s needed only when you want to install tempest globally ... so it\u0027s up to the user, if installing globally then sudo might be needed, if installing in a venv, sudo is not needed.","commit_id":"1c90e79efce53258d52f7365dc80d2a738d5a0c8"},{"author":{"_account_id":22873,"name":"Martin Kopec","email":"mkopec@redhat.com","username":"mkopec"},"change_message_id":"61c6bb502c390962f5b4bc04ffe92abf3c23df47","unresolved":true,"context_lines":[{"line_number":61,"context_line":"   the Tempest repo::"},{"line_number":62,"context_line":""},{"line_number":63,"context_line":"    $ git clone https://opendev.org/openstack/tempest"},{"line_number":64,"context_line":"    $ sudo pip3 install tempest/"},{"line_number":65,"context_line":""},{"line_number":66,"context_line":"   This can be done within a venv, but the assumption for this guide is that"},{"line_number":67,"context_line":"   the Tempest CLI entry point will be in your shell\u0027s PATH."}],"source_content_type":"text/x-rst","patch_set":1,"id":"046fb0f2_b7ab9ca5","line":64,"range":{"start_line":64,"start_character":11,"end_line":64,"end_character":15},"updated":"2022-06-14 18:52:02.000000000","message":"this looks ok as we support only python3","commit_id":"1c90e79efce53258d52f7365dc80d2a738d5a0c8"},{"author":{"_account_id":8556,"name":"Ghanshyam Maan","display_name":"Ghanshyam Maan","email":"gmaan.os14@gmail.com","username":"ghanshyam"},"change_message_id":"4d0224ef33719a179a81d9ad66b47f6fce647abf","unresolved":true,"context_lines":[{"line_number":61,"context_line":"   the Tempest repo::"},{"line_number":62,"context_line":""},{"line_number":63,"context_line":"    $ git clone https://opendev.org/openstack/tempest"},{"line_number":64,"context_line":"    $ sudo pip3 install tempest/"},{"line_number":65,"context_line":""},{"line_number":66,"context_line":"   This can be done within a venv, but the assumption for this guide is that"},{"line_number":67,"context_line":"   the Tempest CLI entry point will be in your shell\u0027s PATH."}],"source_content_type":"text/x-rst","patch_set":1,"id":"27006bfe_7bee6009","line":64,"range":{"start_line":64,"start_character":6,"end_line":64,"end_character":10},"in_reply_to":"611d9c36_6cb4fa02","updated":"2022-12-24 05:27:18.000000000","message":"agree","commit_id":"1c90e79efce53258d52f7365dc80d2a738d5a0c8"}]}