)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":8449,"name":"Marios Andreou","email":"marios.andreou@gmail.com","username":"marios"},"change_message_id":"e0a0151ce11c43a71e3e03e7464dab7e99234d77","unresolved":true,"context_lines":[{"line_number":15,"context_line":"rdo registry. This patch ensure that if there is a zuul variable"},{"line_number":16,"context_line":"passed by content provider, it will use that hash, instead of"},{"line_number":17,"context_line":"read it from rdo registry."},{"line_number":18,"context_line":""},{"line_number":19,"context_line":"Change-Id: I273243ced8b83aecec9da775e7f7a3c3865b4fc3"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":9,"id":"9cc6c3ab_fa8627f8","line":18,"updated":"2021-10-29 11:38:05.000000000","message":"please add the related bug !","commit_id":"a04e64b3f214bbb039e9f44544e1822f35b6340e"}],"/PATCHSET_LEVEL":[{"author":{"_account_id":8367,"name":"Arx Cruz","email":"arxcruz@redhat.com","username":"arxcruz"},"change_message_id":"1ec0a31bfabb8b8965ac892bff5b001232921cbe","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":7,"id":"6e8652d2_2995f018","updated":"2021-10-25 14:06:24.000000000","message":"test project: https://review.rdoproject.org/r/c/testproject/+/36136","commit_id":"e52296a0de2002e233c532adcf5758ebec750edc"},{"author":{"_account_id":12393,"name":"chandan kumar","display_name":"Chandan Kumar","email":"chkumar@redhat.com","username":"chkumar246"},"change_message_id":"163d3270500b40b27306d2d2560bd1e66cafedb5","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":9,"id":"c624bd7d_3bad4983","updated":"2021-10-28 11:04:16.000000000","message":"Can we fire a DNM patch to run a update job ? As these issues were seen on those jobs.","commit_id":"a04e64b3f214bbb039e9f44544e1822f35b6340e"},{"author":{"_account_id":12393,"name":"chandan kumar","display_name":"Chandan Kumar","email":"chkumar@redhat.com","username":"chkumar246"},"change_message_id":"f763861fccb4c5cdcb2ca2829628a22b0e9d523e","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":9,"id":"6f45f05a_7b20fb6c","updated":"2021-10-29 11:33:25.000000000","message":"Let;s merge it, We can see if it break, we can follow up on that","commit_id":"a04e64b3f214bbb039e9f44544e1822f35b6340e"},{"author":{"_account_id":30002,"name":"Douglas Viroel","email":"viroel@gmail.com","username":"dviroel"},"change_message_id":"39f672b52667e7c885113872971add40f1b2f2d5","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":9,"id":"bca3e3d2_c26b2fd9","updated":"2021-10-29 11:35:38.000000000","message":"Thanks Arx","commit_id":"a04e64b3f214bbb039e9f44544e1822f35b6340e"},{"author":{"_account_id":8449,"name":"Marios Andreou","email":"marios.andreou@gmail.com","username":"marios"},"change_message_id":"bbefa54f8c87214bc198ca55f934b1754d47a32d","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":9,"id":"daa460d1_37e143fe","updated":"2021-10-27 11:41:20.000000000","message":"so we don\u0027t need to block on that... if the bug has only ever been seen on the upgrade to version this will fix it\n\nwe can revisit with upgrade from if needed","commit_id":"a04e64b3f214bbb039e9f44544e1822f35b6340e"},{"author":{"_account_id":12393,"name":"chandan kumar","display_name":"Chandan Kumar","email":"chkumar@redhat.com","username":"chkumar246"},"change_message_id":"cad1e55e13f065b4bd775791dc3287fe06325ba6","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":10,"id":"7399b841_b2c9a436","updated":"2021-11-01 10:51:50.000000000","message":"Thanks for updating commit msg","commit_id":"0b0aa53a9bc3298ca593ac43afe873e8b3c5dfe9"},{"author":{"_account_id":9976,"name":"Ronelle Landy","email":"rlandy@redhat.com","username":"rlandy"},"change_message_id":"36260ac86deebe7783a813a0d1d90cdeb86d3760","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":10,"id":"d10c2f45_9dc43c9d","updated":"2021-11-01 11:14:10.000000000","message":"let\u0027s try this","commit_id":"0b0aa53a9bc3298ca593ac43afe873e8b3c5dfe9"},{"author":{"_account_id":8449,"name":"Marios Andreou","email":"marios.andreou@gmail.com","username":"marios"},"change_message_id":"c4a6e3a802a8ad043075a53338ec700e2169c1e9","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":10,"id":"e4f76314_b845b9d3","updated":"2021-10-29 14:36:07.000000000","message":"thank you","commit_id":"0b0aa53a9bc3298ca593ac43afe873e8b3c5dfe9"}],"scripts/emit_releases_file/emit_releases_file.py":[{"author":{"_account_id":8449,"name":"Marios Andreou","email":"marios.andreou@gmail.com","username":"marios"},"change_message_id":"589d20d61601d2ef82710fa72a8ef38b22b716fe","unresolved":true,"context_lines":[{"line_number":475,"context_line":"    )"},{"line_number":476,"context_line":""},{"line_number":477,"context_line":"    parser.add_argument("},{"line_number":478,"context_line":"        \u0027--current-hash\u0027,"},{"line_number":479,"context_line":"        help\u003d\u0027Force an specific hash instead of current-tripleo\u0027,"},{"line_number":480,"context_line":"    )"},{"line_number":481,"context_line":""}],"source_content_type":"text/x-python","patch_set":7,"id":"8a79d989_e44a0f98","line":478,"range":{"start_line":478,"start_character":11,"end_line":478,"end_character":23},"updated":"2021-10-26 06:57:07.000000000","message":"--hash-override \n\n*(just suggestion... i think it should be something more descriptive... the word \u0027current\u0027 is quite overloaded in rdo/tripleo)","commit_id":"e52296a0de2002e233c532adcf5758ebec750edc"},{"author":{"_account_id":8449,"name":"Marios Andreou","email":"marios.andreou@gmail.com","username":"marios"},"change_message_id":"589d20d61601d2ef82710fa72a8ef38b22b716fe","unresolved":true,"context_lines":[{"line_number":478,"context_line":"        \u0027--current-hash\u0027,"},{"line_number":479,"context_line":"        help\u003d\u0027Force an specific hash instead of current-tripleo\u0027,"},{"line_number":480,"context_line":"    )"},{"line_number":481,"context_line":""},{"line_number":482,"context_line":"    args \u003d parser.parse_args()"},{"line_number":483,"context_line":""},{"line_number":484,"context_line":"    setup_logging(args.log_file)"}],"source_content_type":"text/x-python","patch_set":7,"id":"3eba01cb_5cd9a056","line":481,"updated":"2021-10-26 06:57:07.000000000","message":"don\u0027t we need two though? like --install-hash-override  and --target-hash-override.\n\nSo the \u0027from\u0027 and \u0027to\u0027 hash in an upgrade ... i.e. the version we install during the deployment then the version we are upgrading to.\n\nyour patch is addressing the \u0027upgrading to\u0027 version only","commit_id":"e52296a0de2002e233c532adcf5758ebec750edc"},{"author":{"_account_id":8449,"name":"Marios Andreou","email":"marios.andreou@gmail.com","username":"marios"},"change_message_id":"f407eb6b32f6c749be016c21e7fea1812c71d66e","unresolved":true,"context_lines":[{"line_number":478,"context_line":"        \u0027--current-hash\u0027,"},{"line_number":479,"context_line":"        help\u003d\u0027Force an specific hash instead of current-tripleo\u0027,"},{"line_number":480,"context_line":"    )"},{"line_number":481,"context_line":""},{"line_number":482,"context_line":"    args \u003d parser.parse_args()"},{"line_number":483,"context_line":""},{"line_number":484,"context_line":"    setup_logging(args.log_file)"}],"source_content_type":"text/x-python","patch_set":7,"id":"6437ddb6_c05cf1bc","line":481,"in_reply_to":"3eba01cb_5cd9a056","updated":"2021-10-27 11:37:51.000000000","message":"i think you missed this comment though Arx wdyt? We can get this with a promotion in the version we are installing as well as the version we are upgrading to no?","commit_id":"e52296a0de2002e233c532adcf5758ebec750edc"},{"author":{"_account_id":8449,"name":"Marios Andreou","email":"marios.andreou@gmail.com","username":"marios"},"change_message_id":"e0a0151ce11c43a71e3e03e7464dab7e99234d77","unresolved":true,"context_lines":[{"line_number":475,"context_line":"    )"},{"line_number":476,"context_line":""},{"line_number":477,"context_line":"    parser.add_argument("},{"line_number":478,"context_line":"        \u0027--hash-override\u0027,"},{"line_number":479,"context_line":"        help\u003d\u0027Force an specific hash instead of current-tripleo\u0027,"},{"line_number":480,"context_line":"    )"},{"line_number":481,"context_line":""}],"source_content_type":"text/x-python","patch_set":9,"id":"2431cb98_68b138df","line":478,"range":{"start_line":478,"start_character":11,"end_line":478,"end_character":15},"updated":"2021-10-29 11:38:05.000000000","message":"so as commented earlier here and as discussed in yesterday\u0027s scrum, we will also need to address the \u0027--install-hash-override\u0027 as well at some point ie. the version that is installed. \n\nThis change is addressing the \u0027--upgrade-hash-override\u0027 i.e. the target/upgrade-to version.\n\nFTR I found an example of the issue for the upgrade-from version this morning\n\n        * https://e4cf1ab71b6ca5f5dc46-fe390436bababd65005a5c1c9412b532.ssl.cf5.rackcdn.com/815775/1/gate/tripleo-ci-centos-8-undercloud-upgrade/de8cbed/logs/undercloud/home/zuul/undercloud_install.log\n        * 2021-10-28 20:43:10.182778 | bc764e10-04ed-6e6e-aa45-000000000da8 |      FATAL | Container image prepare | undercloud | error\u003d{\"changed\": false, \"error\": \"Not found image: http://104.130.219.215:5001/v2/tripleowallaby/openstack-rsyslog/manifests/8019012886459075100400fbcc95c36a\", \"msg\": \"Error running container image prepare: Not found image: http://104.130.219.215:5001/v2/tripleowallaby/openstack-rsyslog/manifests/8019012886459075100400fbcc95c36a\", \"params\": {}, \"success\": false}\n        09:55 \u003c marios\u003e arxcruz: just seen a case of missing containers for the \u0027install\u0027 version (not the \u0027upgrade to\u0027 \n                version) there \n\n\n\n\nThat being said, I think we should go ahead with this especially if we are seeing it more frequently hitting the target/upgrade-to version, but bearing in mind we aren\u0027t quite done until we can override the install version too.","commit_id":"a04e64b3f214bbb039e9f44544e1822f35b6340e"}]}
