)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":8449,"name":"Marios Andreou","email":"marios.andreou@gmail.com","username":"marios"},"change_message_id":"06bb0918dffc90c735180f1d2d6927c7310113f9","unresolved":true,"context_lines":[{"line_number":10,"context_line":"- Mirror map for ubi9, rhel9"},{"line_number":11,"context_line":"- fallback to centos(8,9) if ubi(8,9) distros are used."},{"line_number":12,"context_line":"- Look for both ubi8 and ubi9 where ubi8 is previously defined."},{"line_number":13,"context_line":""},{"line_number":14,"context_line":"Signed-off-by: Chandan Kumar (raukadah) \u003cchkumar@redhat.com\u003e"},{"line_number":15,"context_line":"Change-Id: I530b4671154e975d527cb380eda8304fec0a7f82"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"babe0166_d7500efc","line":13,"updated":"2022-06-02 07:21:12.000000000","message":"is this needed by another patch or otherwise being tested somewhere ?","commit_id":"d08e5f39ae32b3c99b7aeebeb03a4ecb3aa19acd"},{"author":{"_account_id":8449,"name":"Marios Andreou","email":"marios.andreou@gmail.com","username":"marios"},"change_message_id":"a5591fc84bd8fd4d3bcc01991b363ac09dd455b4","unresolved":false,"context_lines":[{"line_number":10,"context_line":"- Mirror map for ubi9, rhel9"},{"line_number":11,"context_line":"- fallback to centos(8,9) if ubi(8,9) distros are used."},{"line_number":12,"context_line":"- Look for both ubi8 and ubi9 where ubi8 is previously defined."},{"line_number":13,"context_line":""},{"line_number":14,"context_line":"Signed-off-by: Chandan Kumar (raukadah) \u003cchkumar@redhat.com\u003e"},{"line_number":15,"context_line":"Change-Id: I530b4671154e975d527cb380eda8304fec0a7f82"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"e0c347e2_6609d7a0","line":13,"in_reply_to":"babe0166_d7500efc","updated":"2022-06-20 12:25:10.000000000","message":"Done","commit_id":"d08e5f39ae32b3c99b7aeebeb03a4ecb3aa19acd"},{"author":{"_account_id":8449,"name":"Marios Andreou","email":"marios.andreou@gmail.com","username":"marios"},"change_message_id":"736c0cf28098dcb770e203e9d03e3f88f5e0d209","unresolved":true,"context_lines":[{"line_number":11,"context_line":"- fallback to centos(8,9) if ubi(8,9) distros are used."},{"line_number":12,"context_line":"- Look for both ubi8 and ubi9 where ubi8 is previously defined."},{"line_number":13,"context_line":""},{"line_number":14,"context_line":"Closes-Bug: #844361"},{"line_number":15,"context_line":""},{"line_number":16,"context_line":"Signed-off-by: Chandan Kumar (raukadah) \u003cchkumar@redhat.com\u003e"},{"line_number":17,"context_line":"Change-Id: I530b4671154e975d527cb380eda8304fec0a7f82"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":8,"id":"6e68e028_2f76bc78","line":14,"range":{"start_line":14,"start_character":13,"end_line":14,"end_character":19},"updated":"2022-06-20 07:07:19.000000000","message":"must be a typo... that is the gerrit number from this review\n\nit points to a cve https://bugs.launchpad.net/ubuntu/+source/linux/+bug/844361\n\n1979178","commit_id":"e7c9ac6133f0f528c9d7d11251a785f1b5ab51b7"},{"author":{"_account_id":12393,"name":"chandan kumar","display_name":"Chandan Kumar","email":"chkumar@redhat.com","username":"chkumar246"},"change_message_id":"08895228998b51c2a475afe79e21c2419f8be680","unresolved":true,"context_lines":[{"line_number":11,"context_line":"- fallback to centos(8,9) if ubi(8,9) distros are used."},{"line_number":12,"context_line":"- Look for both ubi8 and ubi9 where ubi8 is previously defined."},{"line_number":13,"context_line":""},{"line_number":14,"context_line":"Closes-Bug: #844361"},{"line_number":15,"context_line":""},{"line_number":16,"context_line":"Signed-off-by: Chandan Kumar (raukadah) \u003cchkumar@redhat.com\u003e"},{"line_number":17,"context_line":"Change-Id: I530b4671154e975d527cb380eda8304fec0a7f82"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":8,"id":"e9dd8313_dc94c184","line":14,"range":{"start_line":14,"start_character":13,"end_line":14,"end_character":19},"in_reply_to":"6e68e028_2f76bc78","updated":"2022-06-20 07:09:21.000000000","message":"just fixed it, there is no ci job where we can test the installation from source in ubi 9 environment. I have added the information in the bug itself for more details.","commit_id":"e7c9ac6133f0f528c9d7d11251a785f1b5ab51b7"},{"author":{"_account_id":8449,"name":"Marios Andreou","email":"marios.andreou@gmail.com","username":"marios"},"change_message_id":"a5591fc84bd8fd4d3bcc01991b363ac09dd455b4","unresolved":false,"context_lines":[{"line_number":11,"context_line":"- fallback to centos(8,9) if ubi(8,9) distros are used."},{"line_number":12,"context_line":"- Look for both ubi8 and ubi9 where ubi8 is previously defined."},{"line_number":13,"context_line":""},{"line_number":14,"context_line":"Closes-Bug: #844361"},{"line_number":15,"context_line":""},{"line_number":16,"context_line":"Signed-off-by: Chandan Kumar (raukadah) \u003cchkumar@redhat.com\u003e"},{"line_number":17,"context_line":"Change-Id: I530b4671154e975d527cb380eda8304fec0a7f82"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":8,"id":"b0e1c0ef_f639d140","line":14,"range":{"start_line":14,"start_character":13,"end_line":14,"end_character":19},"in_reply_to":"e9dd8313_dc94c184","updated":"2022-06-20 12:25:10.000000000","message":"Done","commit_id":"e7c9ac6133f0f528c9d7d11251a785f1b5ab51b7"}],"/PATCHSET_LEVEL":[{"author":{"_account_id":12393,"name":"chandan kumar","display_name":"Chandan Kumar","email":"chkumar@redhat.com","username":"chkumar246"},"change_message_id":"7b1ead522b0043a0fe07e23e25617d315077d239","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"9a7c0b17_6a02c88e","updated":"2022-06-02 04:50:41.000000000","message":"Fix for https://22ebcb97dd7dfe4c76ad-9609e5608bf6c8019448004f3d98b5c3.ssl.cf2.rackcdn.com/839688/53/check/tripleo-ansible-centos-stream-molecule-tripleo_cephadm/5ebb695/job-output.txt\n\n```\nTASK [test_deps : Create tripleo repos] ****************************************\n2022-06-02 03:43:08.276271 | centos-9-stream | fatal: [centos]: FAILED! \u003d\u003e changed\u003dtrue\n2022-06-02 03:43:08.276277 | centos-9-stream |   cmd:\n2022-06-02 03:43:08.276282 | centos-9-stream |   - tripleo-repos\n2022-06-02 03:43:08.276288 | centos-9-stream |   - --stream\n2022-06-02 03:43:08.276293 | centos-9-stream |   - -b\n2022-06-02 03:43:08.276298 | centos-9-stream |   - master\n2022-06-02 03:43:08.276303 | centos-9-stream |   - current-tripleo\n2022-06-02 03:43:08.276309 | centos-9-stream |   - ceph\n2022-06-02 03:43:08.276314 | centos-9-stream |   delta: \u00270:00:00.312546\u0027\n2022-06-02 03:43:08.276319 | centos-9-stream |   end: \u00272022-06-02 03:43:06.567486\u0027\n2022-06-02 03:43:08.276325 | centos-9-stream |   msg: non-zero return code\n2022-06-02 03:43:08.276330 | centos-9-stream |   rc: 1\n2022-06-02 03:43:08.276336 | centos-9-stream |   start: \u00272022-06-02 03:43:06.254940\u0027\n2022-06-02 03:43:08.276341 | centos-9-stream |   stderr: |-\n2022-06-02 03:43:08.276347 | centos-9-stream |     Traceback (most recent call last):\n2022-06-02 03:43:08.276352 | centos-9-stream |       File \"/bin/tripleo-repos\", line 10, in \u003cmodule\u003e\n2022-06-02 03:43:08.276357 | centos-9-stream |         sys.exit(main())\n2022-06-02 03:43:08.276363 | centos-9-stream |       File \"/usr/lib/python3.9/site-packages/tripleo_repos/main.py\", line 622, in main\n2022-06-02 03:43:08.276368 | centos-9-stream |         _install_repos(args, base_path)\n2022-06-02 03:43:08.276373 | centos-9-stream |       File \"/usr/lib/python3.9/site-packages/tripleo_repos/main.py\", line 488, in _install_repos\n2022-06-02 03:43:08.276379 | centos-9-stream |         content \u003d _get_repo(base_path + \u0027current-tripleo/delorean.repo\u0027,\n2022-06-02 03:43:08.276384 | centos-9-stream |       File \"/usr/lib/python3.9/site-packages/tripleo_repos/main.py\", line 257, in _get_repo\n2022-06-02 03:43:08.276390 | centos-9-stream |         r.raise_for_status()\n2022-06-02 03:43:08.276395 | centos-9-stream |       File \"/usr/lib/python3.9/site-packages/requests/models.py\", line 943, in raise_for_status\n2022-06-02 03:43:08.276400 | centos-9-stream |         raise HTTPError(http_error_msg, response\u003dself)\n2022-06-02 03:43:08.276406 | centos-9-stream |     requests.exceptions.HTTPError: 404 Client Error: Not Found for url: https://trunk.rdoproject.org/rhel9-master/current-tripleo/delorean.repo\n2022-06-02 03:43:08.276411 | centos-9-stream |   stderr_lines: \u003comitted\u003e\n2022-06-02 03:43:08.276417 | centos-9-stream |   stdout: Removed old repo \"/etc/yum.repos.d/delorean.repo\"\n2022-06-02 03:43:08.276422 | centos-9-stream |   stdout_lines: \u003comitted\u003e\n2022-06-02 03:43:08.276427 | centos-9-stream |\n\n```","commit_id":"ac9e28b290930cdbb77f314e218460344941afbe"},{"author":{"_account_id":29775,"name":"Sandeep Yadav","email":"sandyada@redhat.com","username":"sandeepyadav93"},"change_message_id":"fa7a894cac9acd25670b521f7d1122335d221778","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"a7f34b5b_8067876b","updated":"2022-06-02 07:51:25.000000000","message":"Hello Chandan,\n\nThank you for putting this patch.\n\nChanges, Looks okay to me but I am not very familiar with this code to point that if we are missing any other change somewhere else.\n\nCould you please point what is the current behavior and what this patch solves? Is there a way to confirm this patch works well. Also can you suggest any manual test or via testproject (where in logs we can confirm this is working as expected)\n\nthe content-provider job which ran in check is still using centos:stream9 container\n\nhttps://6dffb28e36a18006d846-fd1d77df4b8e8d30c5a944f1596a25a0.ssl.cf2.rackcdn.com/844361/2/check/tripleo-ci-centos-9-content-provider/ad147e7/logs/undercloud/home/zuul/tripleo_container_image_build.sh\n\n~~~\n--base quay.io/centos/centos:stream9 \n~~~\n\n","commit_id":"d08e5f39ae32b3c99b7aeebeb03a4ecb3aa19acd"},{"author":{"_account_id":30002,"name":"Douglas Viroel","email":"viroel@gmail.com","username":"dviroel"},"change_message_id":"f1ae23bdbc4f54de62bc735b83b968b37340fb60","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"8940b938_dfdbe04b","updated":"2022-06-02 17:35:28.000000000","message":"Thanks Chandan,\n\nTesting in my machine, got some bad repos:\n\n[tripleo-centos-appstream]\nname\u003dtripleo-centos-appstream\nbaseurl\u003dhttp://mirror.stream.centos.org/centos/8/AppStream/$basearch/os/\n\n^ mirror is correct, but the stream should be \u00279-stream\u0027\nSame for BaseOS, HA and PowerTools\n\n- delorean and delorean-master-testing looks correct.\n\n","commit_id":"d08e5f39ae32b3c99b7aeebeb03a4ecb3aa19acd"},{"author":{"_account_id":8449,"name":"Marios Andreou","email":"marios.andreou@gmail.com","username":"marios"},"change_message_id":"06bb0918dffc90c735180f1d2d6927c7310113f9","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"490e53c8_f3a39703","updated":"2022-06-02 07:21:12.000000000","message":"looks sane ;)","commit_id":"d08e5f39ae32b3c99b7aeebeb03a4ecb3aa19acd"},{"author":{"_account_id":12393,"name":"chandan kumar","display_name":"Chandan Kumar","email":"chkumar@redhat.com","username":"chkumar246"},"change_message_id":"bafd1c765ee39f1f656c219bcb00988f4545727a","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"dad659e2_88fe5737","in_reply_to":"a7f34b5b_8067876b","updated":"2022-06-20 07:01:47.000000000","message":"Thank you Sandeep and Marios for the review. https://bugs.launchpad.net/tripleo/+bug/1979178 contains what is the issue and how this patch addresses the issue.","commit_id":"d08e5f39ae32b3c99b7aeebeb03a4ecb3aa19acd"},{"author":{"_account_id":12393,"name":"chandan kumar","display_name":"Chandan Kumar","email":"chkumar@redhat.com","username":"chkumar246"},"change_message_id":"525a1a0c2395fbbe485bdbfcbd33cea1d99b0509","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":6,"id":"5e933639_30d7c2d9","updated":"2022-06-20 05:49:34.000000000","message":"figuring out a way to test it in CI.","commit_id":"dffa584e2650aa1a8bed7c2fe624b79f941375b0"},{"author":{"_account_id":8449,"name":"Marios Andreou","email":"marios.andreou@gmail.com","username":"marios"},"change_message_id":"736c0cf28098dcb770e203e9d03e3f88f5e0d209","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":8,"id":"8125bae6_8b9ad15d","updated":"2022-06-20 07:07:19.000000000","message":"mainly waiting to hear about testing but see inline commit message nit","commit_id":"e7c9ac6133f0f528c9d7d11251a785f1b5ab51b7"},{"author":{"_account_id":29775,"name":"Sandeep Yadav","email":"sandyada@redhat.com","username":"sandeepyadav93"},"change_message_id":"324afa5816b510609e407e0d0753b7b0b0a7bd0b","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":9,"id":"8f449747_db30ae55","updated":"2022-06-20 11:47:53.000000000","message":"Based on testing results on https://bugs.launchpad.net/tripleo/+bug/1979178/comments/1","commit_id":"8321b3fb5cacad22f8c756ce16858fbf3d420a1b"},{"author":{"_account_id":30002,"name":"Douglas Viroel","email":"viroel@gmail.com","username":"dviroel"},"change_message_id":"ef692990740eb638e63ebc08c4a423ecd7dd41f2","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":9,"id":"1d7b2776_4acf7d92","updated":"2022-06-20 12:28:58.000000000","message":"Comparing to PS2, now these repos are correct, for ubi8 and ubi9:\n\n[tripleo-centos-appstream]\nname\u003dtripleo-centos-appstream\nbaseurl\u003dhttp://mirror.stream.centos.org/9-stream/AppStream/$basearch/os/\ngpgcheck\u003d0\nenabled\u003d1\n\n---\n\n[tripleo-centos-appstream]\nname\u003dtripleo-centos-appstream\nbaseurl\u003dhttp://mirror.centos.org/centos/8-stream/AppStream/$basearch/os/\ngpgcheck\u003d0\nenabled\u003d1\n","commit_id":"8321b3fb5cacad22f8c756ce16858fbf3d420a1b"}],"plugins/module_utils/tripleo_repos/main.py":[{"author":{"_account_id":8449,"name":"Marios Andreou","email":"marios.andreou@gmail.com","username":"marios"},"change_message_id":"06bb0918dffc90c735180f1d2d6927c7310113f9","unresolved":false,"context_lines":[{"line_number":390,"context_line":"def _get_base_path(args):"},{"line_number":391,"context_line":"    if args.distro in [\u0027ubi8\u0027, \u0027ubi9\u0027]:"},{"line_number":392,"context_line":"        # there are no base paths for UBI that work well"},{"line_number":393,"context_line":"        distro \u003d args.distro.replace(\u0027ubi\u0027, \u0027centos\u0027)"},{"line_number":394,"context_line":"    else:"},{"line_number":395,"context_line":"        distro \u003d args.distro"},{"line_number":396,"context_line":""}],"source_content_type":"text/x-python","patch_set":2,"id":"ed91a138_464256b8","line":393,"range":{"start_line":393,"start_character":28,"end_line":393,"end_character":31},"updated":"2022-06-02 07:21:12.000000000","message":"ack should work well for centos8 centos9","commit_id":"d08e5f39ae32b3c99b7aeebeb03a4ecb3aa19acd"},{"author":{"_account_id":30002,"name":"Douglas Viroel","email":"viroel@gmail.com","username":"dviroel"},"change_message_id":"f1ae23bdbc4f54de62bc735b83b968b37340fb60","unresolved":true,"context_lines":[{"line_number":560,"context_line":"        content \u003d APPSTREAM_REPO_TEMPLATE % {\u0027mirror\u0027: args.mirror,"},{"line_number":561,"context_line":"                                             \u0027extra\u0027: extra,"},{"line_number":562,"context_line":"                                             \u0027legacy_url\u0027: legacy_url,"},{"line_number":563,"context_line":"                                             \u0027stream\u0027: \u00278\u0027}"},{"line_number":564,"context_line":"        _write_repo(content, distro_path)"},{"line_number":565,"context_line":"        content \u003d BASE_REPO_TEMPLATE % {\u0027mirror\u0027: args.mirror,"},{"line_number":566,"context_line":"                                        \u0027legacy_url\u0027: legacy_url,"}],"source_content_type":"text/x-python","patch_set":2,"id":"49aceedf_296386d6","line":563,"range":{"start_line":563,"start_character":56,"end_line":563,"end_character":57},"updated":"2022-06-02 17:35:28.000000000","message":"this should become \u00279-stream\u0027 if ubi9 and \u00278-stream\u0027 if ubi8. Stream \u00278\u0027 is not available anymore since CentOS 8 EOL.\n\nWe have a workaround on line 572-576 tha can be reused","commit_id":"d08e5f39ae32b3c99b7aeebeb03a4ecb3aa19acd"},{"author":{"_account_id":12393,"name":"chandan kumar","display_name":"Chandan Kumar","email":"chkumar@redhat.com","username":"chkumar246"},"change_message_id":"17fee0f2aa05fe719756aa6a28b27263290f80dc","unresolved":false,"context_lines":[{"line_number":560,"context_line":"        content \u003d APPSTREAM_REPO_TEMPLATE % {\u0027mirror\u0027: args.mirror,"},{"line_number":561,"context_line":"                                             \u0027extra\u0027: extra,"},{"line_number":562,"context_line":"                                             \u0027legacy_url\u0027: legacy_url,"},{"line_number":563,"context_line":"                                             \u0027stream\u0027: \u00278\u0027}"},{"line_number":564,"context_line":"        _write_repo(content, distro_path)"},{"line_number":565,"context_line":"        content \u003d BASE_REPO_TEMPLATE % {\u0027mirror\u0027: args.mirror,"},{"line_number":566,"context_line":"                                        \u0027legacy_url\u0027: legacy_url,"}],"source_content_type":"text/x-python","patch_set":2,"id":"bb003a0d_e1a8d4ce","line":563,"range":{"start_line":563,"start_character":56,"end_line":563,"end_character":57},"in_reply_to":"49aceedf_296386d6","updated":"2022-06-20 04:56:02.000000000","message":"Done","commit_id":"d08e5f39ae32b3c99b7aeebeb03a4ecb3aa19acd"},{"author":{"_account_id":30002,"name":"Douglas Viroel","email":"viroel@gmail.com","username":"dviroel"},"change_message_id":"f1ae23bdbc4f54de62bc735b83b968b37340fb60","unresolved":true,"context_lines":[{"line_number":564,"context_line":"        _write_repo(content, distro_path)"},{"line_number":565,"context_line":"        content \u003d BASE_REPO_TEMPLATE % {\u0027mirror\u0027: args.mirror,"},{"line_number":566,"context_line":"                                        \u0027legacy_url\u0027: legacy_url,"},{"line_number":567,"context_line":"                                        \u0027stream\u0027: \u00278\u0027}"},{"line_number":568,"context_line":"        _write_repo(content, distro_path)"},{"line_number":569,"context_line":"        distro \u003d \u0027centos8\u0027  # switch it to continue as centos8 distro"},{"line_number":570,"context_line":""}],"source_content_type":"text/x-python","patch_set":2,"id":"db48c2d3_565039af","line":567,"range":{"start_line":567,"start_character":51,"end_line":567,"end_character":52},"updated":"2022-06-02 17:35:28.000000000","message":"ditto","commit_id":"d08e5f39ae32b3c99b7aeebeb03a4ecb3aa19acd"},{"author":{"_account_id":30002,"name":"Douglas Viroel","email":"viroel@gmail.com","username":"dviroel"},"change_message_id":"f1ae23bdbc4f54de62bc735b83b968b37340fb60","unresolved":true,"context_lines":[{"line_number":566,"context_line":"                                        \u0027legacy_url\u0027: legacy_url,"},{"line_number":567,"context_line":"                                        \u0027stream\u0027: \u00278\u0027}"},{"line_number":568,"context_line":"        _write_repo(content, distro_path)"},{"line_number":569,"context_line":"        distro \u003d \u0027centos8\u0027  # switch it to continue as centos8 distro"},{"line_number":570,"context_line":""},{"line_number":571,"context_line":"    if \u0027centos\u0027 in distro:"},{"line_number":572,"context_line":"        stream \u003d str(distro[-1])"}],"source_content_type":"text/x-python","patch_set":2,"id":"40355005_55111037","line":569,"range":{"start_line":569,"start_character":8,"end_line":569,"end_character":25},"updated":"2022-06-02 17:35:28.000000000","message":"needs to consider ubi9 -\u003e centos9 ?","commit_id":"d08e5f39ae32b3c99b7aeebeb03a4ecb3aa19acd"},{"author":{"_account_id":12393,"name":"chandan kumar","display_name":"Chandan Kumar","email":"chkumar@redhat.com","username":"chkumar246"},"change_message_id":"17fee0f2aa05fe719756aa6a28b27263290f80dc","unresolved":false,"context_lines":[{"line_number":566,"context_line":"                                        \u0027legacy_url\u0027: legacy_url,"},{"line_number":567,"context_line":"                                        \u0027stream\u0027: \u00278\u0027}"},{"line_number":568,"context_line":"        _write_repo(content, distro_path)"},{"line_number":569,"context_line":"        distro \u003d \u0027centos8\u0027  # switch it to continue as centos8 distro"},{"line_number":570,"context_line":""},{"line_number":571,"context_line":"    if \u0027centos\u0027 in distro:"},{"line_number":572,"context_line":"        stream \u003d str(distro[-1])"}],"source_content_type":"text/x-python","patch_set":2,"id":"1505ad53_e7fe63d5","line":569,"range":{"start_line":569,"start_character":8,"end_line":569,"end_character":25},"in_reply_to":"40355005_55111037","updated":"2022-06-20 04:56:02.000000000","message":"Done","commit_id":"d08e5f39ae32b3c99b7aeebeb03a4ecb3aa19acd"},{"author":{"_account_id":28223,"name":"Cedric Jeanneret","display_name":"cjeanner (Tengu)","email":"cjeanner@redhat.com","username":"cjeanner"},"change_message_id":"3859647e6e7352b628af2086f63a0d0f4184e9ba","unresolved":true,"context_lines":[{"line_number":568,"context_line":"                                        \u0027stream\u0027: distro_name}"},{"line_number":569,"context_line":"        _write_repo(content, distro_path)"},{"line_number":570,"context_line":"        if distro in [\u0027centos8\u0027, \u0027centos9\u0027, \u0027ubi8\u0027, \u0027ubi9\u0027]:"},{"line_number":571,"context_line":"            distro \u003d \u0027centos\u0027 + str(distro[-1])"},{"line_number":572,"context_line":""},{"line_number":573,"context_line":"    if \u0027centos\u0027 in distro:"},{"line_number":574,"context_line":"        stream \u003d str(distro[-1])"}],"source_content_type":"text/x-python","patch_set":9,"id":"be339f5b_25e08b91","line":571,"range":{"start_line":571,"start_character":12,"end_line":571,"end_character":47},"updated":"2022-06-20 07:12:12.000000000","message":"so we\u0027re rewriting even centos8 and centos9? While it\u0027s not an issue, it\u0027s a bit ugly. Won\u0027t block on that, but... #ugly ;).","commit_id":"8321b3fb5cacad22f8c756ce16858fbf3d420a1b"},{"author":{"_account_id":30002,"name":"Douglas Viroel","email":"viroel@gmail.com","username":"dviroel"},"change_message_id":"ef692990740eb638e63ebc08c4a423ecd7dd41f2","unresolved":true,"context_lines":[{"line_number":568,"context_line":"                                        \u0027stream\u0027: distro_name}"},{"line_number":569,"context_line":"        _write_repo(content, distro_path)"},{"line_number":570,"context_line":"        if distro in [\u0027centos8\u0027, \u0027centos9\u0027, \u0027ubi8\u0027, \u0027ubi9\u0027]:"},{"line_number":571,"context_line":"            distro \u003d \u0027centos\u0027 + str(distro[-1])"},{"line_number":572,"context_line":""},{"line_number":573,"context_line":"    if \u0027centos\u0027 in distro:"},{"line_number":574,"context_line":"        stream \u003d str(distro[-1])"}],"source_content_type":"text/x-python","patch_set":9,"id":"8f349053_f5f658ec","line":571,"range":{"start_line":571,"start_character":12,"end_line":571,"end_character":47},"in_reply_to":"be339f5b_25e08b91","updated":"2022-06-20 12:28:58.000000000","message":"agree, \u0027centos8\u0027, \u0027centos9\u0027 not needed here. Chadan, please updated if you provide a new PS. Otherwise, lets merge to unblock other patches.","commit_id":"8321b3fb5cacad22f8c756ce16858fbf3d420a1b"}]}
