)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":10135,"name":"Lee Yarwood","display_name":"Lee Yarwood","email":"lyarwood@redhat.com","username":"lyarwood"},"change_message_id":"9223c75ae002716246ba50f26907715d0e6000ea","unresolved":false,"context_lines":[{"line_number":25,"context_line":"u-c so that Tempest installation in venv will use stable"},{"line_number":26,"context_line":"branch constraint."},{"line_number":27,"context_line":""},{"line_number":28,"context_line":"Modify the irrelevant-files to run the nova-live-migration jobs."},{"line_number":29,"context_line":""},{"line_number":30,"context_line":"[1] https://opendev.org/openstack/tempest/src/commit/bc9fe8eca801f54915ff3eafa418e6e18ac2df63/tox.ini#L14"},{"line_number":31,"context_line":""}],"source_content_type":"text/x-gerrit-commit-message","patch_set":3,"id":"3fa7e38b_e00a3027","line":28,"range":{"start_line":28,"start_character":0,"end_line":28,"end_character":64},"updated":"2020-02-10 10:22:30.000000000","message":"This confused me at first but you\u0027re trying to ensure we run the job against changes to \tnova/tests/live_migration/hooks/utils.sh. I honestly wish we used a whitelist instead of a blacklist for these things.","commit_id":"c15afa542c88a065a50c1eb7c35a33b9ed25a0ac"},{"author":{"_account_id":8556,"name":"Ghanshyam Maan","display_name":"Ghanshyam Maan","email":"gmaan.os14@gmail.com","username":"ghanshyam"},"change_message_id":"65784e8ea2fd1de6cda1b2c42ab7131a236da4f7","unresolved":false,"context_lines":[{"line_number":25,"context_line":"u-c so that Tempest installation in venv will use stable"},{"line_number":26,"context_line":"branch constraint."},{"line_number":27,"context_line":""},{"line_number":28,"context_line":"Modify the irrelevant-files to run the nova-live-migration jobs."},{"line_number":29,"context_line":""},{"line_number":30,"context_line":"[1] https://opendev.org/openstack/tempest/src/commit/bc9fe8eca801f54915ff3eafa418e6e18ac2df63/tox.ini#L14"},{"line_number":31,"context_line":""}],"source_content_type":"text/x-gerrit-commit-message","patch_set":3,"id":"3fa7e38b_e38f9773","line":28,"range":{"start_line":28,"start_character":0,"end_line":28,"end_character":64},"in_reply_to":"3fa7e38b_702b6be9","updated":"2020-02-10 13:00:03.000000000","message":"we should add this at least to run nova-live-migration job otherwise we can break this job on run_test.py changes.","commit_id":"c15afa542c88a065a50c1eb7c35a33b9ed25a0ac"},{"author":{"_account_id":17685,"name":"Elod Illes","email":"elod.illes@est.tech","username":"elod.illes"},"change_message_id":"9f9d321e36b066431a75000bd8272b065bc59a9f","unresolved":false,"context_lines":[{"line_number":25,"context_line":"u-c so that Tempest installation in venv will use stable"},{"line_number":26,"context_line":"branch constraint."},{"line_number":27,"context_line":""},{"line_number":28,"context_line":"Modify the irrelevant-files to run the nova-live-migration jobs."},{"line_number":29,"context_line":""},{"line_number":30,"context_line":"[1] https://opendev.org/openstack/tempest/src/commit/bc9fe8eca801f54915ff3eafa418e6e18ac2df63/tox.ini#L14"},{"line_number":31,"context_line":""}],"source_content_type":"text/x-gerrit-commit-message","patch_set":3,"id":"3fa7e38b_702b6be9","line":28,"range":{"start_line":28,"start_character":0,"end_line":28,"end_character":64},"in_reply_to":"3fa7e38b_e00a3027","updated":"2020-02-10 12:33:58.000000000","message":"I thought, that this is not necessary, so agreed on chat [1] that I edit out this change, and submit a DNM/test patch [2] on top of this to see that it\u0027s still OK to merge. Hope it\u0027s not a problem. o:)\n\n[1] http://eavesdrop.openstack.org/irclogs/%23openstack-nova/%23openstack-nova.2020-02-10.log.html#t2020-02-10T12:04:23\n[2] https://review.opendev.org/#/c/706812/","commit_id":"c15afa542c88a065a50c1eb7c35a33b9ed25a0ac"},{"author":{"_account_id":17685,"name":"Elod Illes","email":"elod.illes@est.tech","username":"elod.illes"},"change_message_id":"8109ff4afd6fd9f865e22f7c51501daffed659a4","unresolved":false,"context_lines":[{"line_number":25,"context_line":"u-c so that Tempest installation in venv will use stable"},{"line_number":26,"context_line":"branch constraint."},{"line_number":27,"context_line":""},{"line_number":28,"context_line":"Modify the irrelevant-files to run the nova-live-migration jobs."},{"line_number":29,"context_line":""},{"line_number":30,"context_line":"[1] https://opendev.org/openstack/tempest/src/commit/bc9fe8eca801f54915ff3eafa418e6e18ac2df63/tox.ini#L14"},{"line_number":31,"context_line":""}],"source_content_type":"text/x-gerrit-commit-message","patch_set":3,"id":"3fa7e38b_23312f59","line":28,"range":{"start_line":28,"start_character":0,"end_line":28,"end_character":64},"in_reply_to":"3fa7e38b_e38f9773","updated":"2020-02-10 13:15:57.000000000","message":"Ok, sorry, let\u0027s fix stable/rocky. We discussed in chat that maybe it would have been cleaner if it was fixed in master first. This patch could have been passed without this. But I also understand your concern.","commit_id":"c15afa542c88a065a50c1eb7c35a33b9ed25a0ac"}]}