)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":8556,"name":"Ghanshyam Maan","display_name":"Ghanshyam Maan","email":"gmaan.os14@gmail.com","username":"ghanshyam"},"change_message_id":"33dd5dc89c72c759a04aa8c6d88a16e1be519212","unresolved":true,"context_lines":[{"line_number":4,"context_line":"Commit:     sairam vengala \u003csairamvengala007@gmail.com\u003e"},{"line_number":5,"context_line":"CommitDate: 2023-09-29 07:27:45 +0000"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"update elastic-recheck.rst file"},{"line_number":8,"context_line":"What to do if a test job fails"},{"line_number":9,"context_line":""},{"line_number":10,"context_line":"Change-Id: I825a4b25c44a0be8fe4ca98db09609181de9fa89"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"b75dc16b_78ad2489","line":8,"range":{"start_line":7,"start_character":0,"end_line":8,"end_character":30},"updated":"2023-09-29 15:56:23.000000000","message":"you need to follow commit message guielines. Add something like below\n\nAdd how to debug test failure in contributors guide\n\nOpenStack community encourage to debug the gate test failure instead of just do recheck to re-trigger the test. Add those in contributor guidelines.","commit_id":"391d2f4865447d2b118d214ef2002c7dea109b58"},{"author":{"_account_id":35978,"name":"sairam vengala","email":"sairamvengala007@gmail.com","username":"Sairam"},"change_message_id":"0ebd0a4adc156c6cd9c7d84733ae605ff20795a2","unresolved":false,"context_lines":[{"line_number":4,"context_line":"Commit:     sairam vengala \u003csairamvengala007@gmail.com\u003e"},{"line_number":5,"context_line":"CommitDate: 2023-09-29 07:27:45 +0000"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"update elastic-recheck.rst file"},{"line_number":8,"context_line":"What to do if a test job fails"},{"line_number":9,"context_line":""},{"line_number":10,"context_line":"Change-Id: I825a4b25c44a0be8fe4ca98db09609181de9fa89"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"413e58a8_41548463","line":8,"range":{"start_line":7,"start_character":0,"end_line":8,"end_character":30},"in_reply_to":"b75dc16b_78ad2489","updated":"2023-09-29 16:32:13.000000000","message":"Thanks for the suggestion, updated in patch set 4","commit_id":"391d2f4865447d2b118d214ef2002c7dea109b58"},{"author":{"_account_id":14482,"name":"Ian Y. Choi","email":"ianyrchoi@gmail.com","username":"ianychoi"},"change_message_id":"29d5056a2e6c3f354a1b4bfe659a45acb989151e","unresolved":true,"context_lines":[{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Add how to debug test failure in contributors guide"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"OpenStack community encourage to debug the gate test failure"},{"line_number":10,"context_line":"instead of just do recheck to re-trigger the test. Add those in contributor guidelines."},{"line_number":11,"context_line":""},{"line_number":12,"context_line":"Change-Id: I825a4b25c44a0be8fe4ca98db09609181de9fa89"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":4,"id":"fc64ec4c_1de1038d","line":9,"range":{"start_line":9,"start_character":20,"end_line":9,"end_character":30},"updated":"2023-10-02 04:03:44.000000000","message":"-\u003e encourages","commit_id":"a659aed1bd6eee5882273382bcde25ba2699a2e1"},{"author":{"_account_id":35978,"name":"sairam vengala","email":"sairamvengala007@gmail.com","username":"Sairam"},"change_message_id":"222aead30cdf0d2d943b148c1516e030b5c8e893","unresolved":false,"context_lines":[{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Add how to debug test failure in contributors guide"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"OpenStack community encourage to debug the gate test failure"},{"line_number":10,"context_line":"instead of just do recheck to re-trigger the test. Add those in contributor guidelines."},{"line_number":11,"context_line":""},{"line_number":12,"context_line":"Change-Id: I825a4b25c44a0be8fe4ca98db09609181de9fa89"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":4,"id":"c7d1d94b_9819d9c9","line":9,"range":{"start_line":9,"start_character":20,"end_line":9,"end_character":30},"in_reply_to":"fc64ec4c_1de1038d","updated":"2023-10-03 03:51:45.000000000","message":"Thanks for the suggestion, updated in Patchset 5","commit_id":"a659aed1bd6eee5882273382bcde25ba2699a2e1"}],"/PATCHSET_LEVEL":[{"author":{"_account_id":14482,"name":"Ian Y. Choi","email":"ianyrchoi@gmail.com","username":"ianychoi"},"change_message_id":"29d5056a2e6c3f354a1b4bfe659a45acb989151e","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"6823ad81_5163706c","updated":"2023-10-02 04:03:44.000000000","message":"Although there is a +2 and I agree with the purpose of this patch, let me suggest some more for better documentation.","commit_id":"a659aed1bd6eee5882273382bcde25ba2699a2e1"},{"author":{"_account_id":8556,"name":"Ghanshyam Maan","display_name":"Ghanshyam Maan","email":"gmaan.os14@gmail.com","username":"ghanshyam"},"change_message_id":"80be34a457c2b6a0adf567b3543c8222b8b8d2b1","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"0b823436_f16f08ca","updated":"2023-09-29 16:47:17.000000000","message":"thanks lgtm","commit_id":"a659aed1bd6eee5882273382bcde25ba2699a2e1"},{"author":{"_account_id":14482,"name":"Ian Y. Choi","email":"ianyrchoi@gmail.com","username":"ianychoi"},"change_message_id":"2b1282d7eeef1068dc4cbd939d06f126eb9cd4e4","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":11,"id":"84ede14e_37ac3ebc","updated":"2023-10-03 11:13:41.000000000","message":"Thank you for the update!","commit_id":"21eca93b30bd839eab53d5c1c6e2b2c2cc6d8953"},{"author":{"_account_id":8556,"name":"Ghanshyam Maan","display_name":"Ghanshyam Maan","email":"gmaan.os14@gmail.com","username":"ghanshyam"},"change_message_id":"664aeb5ef09179f09148784fb63289ae73d913f5","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":11,"id":"9a29a16b_d03dbe3f","updated":"2023-10-03 17:59:35.000000000","message":"lgtm","commit_id":"21eca93b30bd839eab53d5c1c6e2b2c2cc6d8953"}],"doc/source/code-and-documentation/elastic-recheck.rst":[{"author":{"_account_id":8556,"name":"Ghanshyam Maan","display_name":"Ghanshyam Maan","email":"gmaan.os14@gmail.com","username":"ghanshyam"},"change_message_id":"33dd5dc89c72c759a04aa8c6d88a16e1be519212","unresolved":true,"context_lines":[{"line_number":23,"context_line":"Once you see the failure, you will be tempted to just \u0027recheck\u0027 the patch to"},{"line_number":24,"context_line":"see if it fails again, but please DO NOT DO THAT. CI test resources are avery"},{"line_number":25,"context_line":"scarce resource (and becoming more so all the time), so please be extremely"},{"line_number":26,"context_line":"sparing when asking the system to re-run tests. Read this document to know how"},{"line_number":27,"context_line":"to handle test failures \u003chttps://docs.openstack.org/project-team-guide/testing.html#how-to-handle-test-failures\u003e"},{"line_number":28,"context_line":""},{"line_number":29,"context_line":"OpenSearch"},{"line_number":30,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"}],"source_content_type":"text/x-rst","patch_set":2,"id":"3a922889_414bd61b","line":27,"range":{"start_line":26,"start_character":75,"end_line":27,"end_character":112},"updated":"2023-09-29 15:56:23.000000000","message":"to add link do it like below:\n\n`how\nto handle test failures \u003chttps://docs.openstack.org/project-team-guide/testing.html#how-to-handle-test-failures\u003e`_","commit_id":"391d2f4865447d2b118d214ef2002c7dea109b58"},{"author":{"_account_id":35978,"name":"sairam vengala","email":"sairamvengala007@gmail.com","username":"Sairam"},"change_message_id":"0ebd0a4adc156c6cd9c7d84733ae605ff20795a2","unresolved":false,"context_lines":[{"line_number":23,"context_line":"Once you see the failure, you will be tempted to just \u0027recheck\u0027 the patch to"},{"line_number":24,"context_line":"see if it fails again, but please DO NOT DO THAT. CI test resources are avery"},{"line_number":25,"context_line":"scarce resource (and becoming more so all the time), so please be extremely"},{"line_number":26,"context_line":"sparing when asking the system to re-run tests. Read this document to know how"},{"line_number":27,"context_line":"to handle test failures \u003chttps://docs.openstack.org/project-team-guide/testing.html#how-to-handle-test-failures\u003e"},{"line_number":28,"context_line":""},{"line_number":29,"context_line":"OpenSearch"},{"line_number":30,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"}],"source_content_type":"text/x-rst","patch_set":2,"id":"6ce4f16a_a9a783bc","line":27,"range":{"start_line":26,"start_character":75,"end_line":27,"end_character":112},"in_reply_to":"3a922889_414bd61b","updated":"2023-09-29 16:32:13.000000000","message":"Thanks for the suggestion, updated in patch set 4","commit_id":"391d2f4865447d2b118d214ef2002c7dea109b58"},{"author":{"_account_id":14482,"name":"Ian Y. Choi","email":"ianyrchoi@gmail.com","username":"ianychoi"},"change_message_id":"29d5056a2e6c3f354a1b4bfe659a45acb989151e","unresolved":true,"context_lines":[{"line_number":19,"context_line":"catching it. Sometimes the test run might have tripped over an underlying"},{"line_number":20,"context_line":"pre-existing bug in OpenStack. Additionally, some times the infrastructure for"},{"line_number":21,"context_line":"running the tests might have had a failure. To figure this out you\u0027ll always"},{"line_number":22,"context_line":"need to look at the logs from the failed job to understand what\u0027s happening."},{"line_number":23,"context_line":"Once you see the failure, you will be tempted to just \u0027recheck\u0027 the patch to"},{"line_number":24,"context_line":"see if it fails again, but please DO NOT DO THAT. CI test resources are avery"},{"line_number":25,"context_line":"scarce resource (and becoming more so all the time), so please be extremely"}],"source_content_type":"text/x-rst","patch_set":4,"id":"5ee8b8eb_0ca35efe","line":22,"range":{"start_line":22,"start_character":75,"end_line":22,"end_character":76},"updated":"2023-10-02 04:03:44.000000000","message":"Let\u0027s add a blank line between L22 and L23.","commit_id":"a659aed1bd6eee5882273382bcde25ba2699a2e1"},{"author":{"_account_id":35978,"name":"sairam vengala","email":"sairamvengala007@gmail.com","username":"Sairam"},"change_message_id":"222aead30cdf0d2d943b148c1516e030b5c8e893","unresolved":false,"context_lines":[{"line_number":19,"context_line":"catching it. Sometimes the test run might have tripped over an underlying"},{"line_number":20,"context_line":"pre-existing bug in OpenStack. Additionally, some times the infrastructure for"},{"line_number":21,"context_line":"running the tests might have had a failure. To figure this out you\u0027ll always"},{"line_number":22,"context_line":"need to look at the logs from the failed job to understand what\u0027s happening."},{"line_number":23,"context_line":"Once you see the failure, you will be tempted to just \u0027recheck\u0027 the patch to"},{"line_number":24,"context_line":"see if it fails again, but please DO NOT DO THAT. CI test resources are avery"},{"line_number":25,"context_line":"scarce resource (and becoming more so all the time), so please be extremely"}],"source_content_type":"text/x-rst","patch_set":4,"id":"b0fea043_bd9b5b8b","line":22,"range":{"start_line":22,"start_character":75,"end_line":22,"end_character":76},"in_reply_to":"5ee8b8eb_0ca35efe","updated":"2023-10-03 03:51:45.000000000","message":"Thanks for the suggestion, updated in Patchset 6","commit_id":"a659aed1bd6eee5882273382bcde25ba2699a2e1"},{"author":{"_account_id":14482,"name":"Ian Y. Choi","email":"ianyrchoi@gmail.com","username":"ianychoi"},"change_message_id":"29d5056a2e6c3f354a1b4bfe659a45acb989151e","unresolved":true,"context_lines":[{"line_number":20,"context_line":"pre-existing bug in OpenStack. Additionally, some times the infrastructure for"},{"line_number":21,"context_line":"running the tests might have had a failure. To figure this out you\u0027ll always"},{"line_number":22,"context_line":"need to look at the logs from the failed job to understand what\u0027s happening."},{"line_number":23,"context_line":"Once you see the failure, you will be tempted to just \u0027recheck\u0027 the patch to"},{"line_number":24,"context_line":"see if it fails again, but please DO NOT DO THAT. CI test resources are avery"},{"line_number":25,"context_line":"scarce resource (and becoming more so all the time), so please be extremely"},{"line_number":26,"context_line":"sparing when asking the system to re-run tests. Read this document to know `how"},{"line_number":27,"context_line":"to handle test failures \u003chttps://docs.openstack.org/project-team-guide/testing.html#how-to-handle-test-failures\u003e`_"},{"line_number":28,"context_line":""},{"line_number":29,"context_line":"OpenSearch"},{"line_number":30,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"}],"source_content_type":"text/x-rst","patch_set":4,"id":"b78316ae_454ce3aa","line":27,"range":{"start_line":23,"start_character":0,"end_line":27,"end_character":114},"updated":"2023-10-02 04:03:44.000000000","message":"I don\u0027t like just copying \u0026 pasting the same sentence from the target document but agree with the purpose of this patch. So, let me a little paraphrase to be more suitable to contributor-guide readers - adding a context: `recheck` needs to be written as a comment, not pressing a button on gerrit:\n\n```\nA comment `recheck` on the patch would trigger the failed job to execute again. **DO NOT** just recheck the patch only to see if it fails again. CI test resources are very scarce. Read this document to know `how\nto handle test failures \u003chttps://docs.openstack.org/project-team-guide/testing.html#how-to-handle-test-failures\u003e`_\n```","commit_id":"a659aed1bd6eee5882273382bcde25ba2699a2e1"},{"author":{"_account_id":35978,"name":"sairam vengala","email":"sairamvengala007@gmail.com","username":"Sairam"},"change_message_id":"222aead30cdf0d2d943b148c1516e030b5c8e893","unresolved":false,"context_lines":[{"line_number":20,"context_line":"pre-existing bug in OpenStack. Additionally, some times the infrastructure for"},{"line_number":21,"context_line":"running the tests might have had a failure. To figure this out you\u0027ll always"},{"line_number":22,"context_line":"need to look at the logs from the failed job to understand what\u0027s happening."},{"line_number":23,"context_line":"Once you see the failure, you will be tempted to just \u0027recheck\u0027 the patch to"},{"line_number":24,"context_line":"see if it fails again, but please DO NOT DO THAT. CI test resources are avery"},{"line_number":25,"context_line":"scarce resource (and becoming more so all the time), so please be extremely"},{"line_number":26,"context_line":"sparing when asking the system to re-run tests. Read this document to know `how"},{"line_number":27,"context_line":"to handle test failures \u003chttps://docs.openstack.org/project-team-guide/testing.html#how-to-handle-test-failures\u003e`_"},{"line_number":28,"context_line":""},{"line_number":29,"context_line":"OpenSearch"},{"line_number":30,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"}],"source_content_type":"text/x-rst","patch_set":4,"id":"9bb1c514_f614d655","line":27,"range":{"start_line":23,"start_character":0,"end_line":27,"end_character":114},"in_reply_to":"b78316ae_454ce3aa","updated":"2023-10-03 03:51:45.000000000","message":"Thanks for the suggestion, updated in Patchset 6","commit_id":"a659aed1bd6eee5882273382bcde25ba2699a2e1"}]}