)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":8449,"name":"Marios Andreou","email":"marios.andreou@gmail.com","username":"marios"},"change_message_id":"7f1107bb907587eba8c0b413b22a4b91163901cf","unresolved":true,"context_lines":[{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Don\u0027t always validate distro name"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"When the output path is not /etc/yum.repos.d, the repo files won\u0027t be"},{"line_number":10,"context_line":"used to install packages on this host, so there is no need to be"},{"line_number":11,"context_line":"strict about host OS name validation."},{"line_number":12,"context_line":""}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"78eefc6b_27338378","line":9,"range":{"start_line":9,"start_character":7,"end_line":9,"end_character":37},"updated":"2022-01-25 07:46:37.000000000","message":"can you add some more context please where/why is this necessary (is it for centos9? rhel9? if we have a bug please add reference here)","commit_id":"253992a1e246e553c27ce8bc4afb5bc51a2d6ff4"},{"author":{"_account_id":4571,"name":"Steve Baker","email":"sbaker@redhat.com","username":"steve-stevebaker"},"change_message_id":"6ab99796b4c485fce84e37f88295cfc78c04506c","unresolved":true,"context_lines":[{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Don\u0027t always validate distro name"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"When the output path is not /etc/yum.repos.d, the repo files won\u0027t be"},{"line_number":10,"context_line":"used to install packages on this host, so there is no need to be"},{"line_number":11,"context_line":"strict about host OS name validation."},{"line_number":12,"context_line":""}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"2d87b7a7_d4350ce8","line":9,"range":{"start_line":9,"start_character":7,"end_line":9,"end_character":37},"in_reply_to":"55f0338e_95753976","updated":"2022-01-25 19:37:00.000000000","message":"Yes, what Sagi said, building centos images on a fedora host with a custom output path, so my host /etc/yum.repos.d doesn\u0027t get messed up","commit_id":"253992a1e246e553c27ce8bc4afb5bc51a2d6ff4"},{"author":{"_account_id":10969,"name":"Shnaidman Sagi (Sergey)","display_name":"Shnaidman Sagi","email":"sshnaidm@redhat.com","username":"sergsh"},"change_message_id":"d06a2f94b9f76c3c5fd33e9c6f8f090e1d248c29","unresolved":true,"context_lines":[{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Don\u0027t always validate distro name"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"When the output path is not /etc/yum.repos.d, the repo files won\u0027t be"},{"line_number":10,"context_line":"used to install packages on this host, so there is no need to be"},{"line_number":11,"context_line":"strict about host OS name validation."},{"line_number":12,"context_line":""}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"55f0338e_95753976","line":9,"range":{"start_line":9,"start_character":7,"end_line":9,"end_character":37},"in_reply_to":"78eefc6b_27338378","updated":"2022-01-25 11:04:37.000000000","message":"I think it\u0027s necessary for development use case when you generate repositories for CentOS 9 on your laptop that runs Fedora for example, and later copy them to a CentOS virtual machine.","commit_id":"253992a1e246e553c27ce8bc4afb5bc51a2d6ff4"}],"/PATCHSET_LEVEL":[{"author":{"_account_id":10969,"name":"Shnaidman Sagi (Sergey)","display_name":"Shnaidman Sagi","email":"sshnaidm@redhat.com","username":"sergsh"},"change_message_id":"68e9a77ef14b75deb402317696dbfdf7ff4a25b0","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"ef233d69_c5109a0c","updated":"2022-01-24 21:21:09.000000000","message":"Need to fix unittests: test_invalid_no_stream, test_invalid_stream\nhttps://storage.bhs.cloud.ovh.net/v1/AUTH_dcaab5e32b234d56b626f72581e3644c/zuul_opendev_logs_ea8/789679/2/check/openstack-tox-cover/ea8fc8f/testr_results.html","commit_id":"253992a1e246e553c27ce8bc4afb5bc51a2d6ff4"},{"author":{"_account_id":8449,"name":"Marios Andreou","email":"marios.andreou@gmail.com","username":"marios"},"change_message_id":"7f1107bb907587eba8c0b413b22a4b91163901cf","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"7c9fa438_48c96aaa","updated":"2022-01-25 07:46:37.000000000","message":"as sagi noted tests need some attention but generally i need to understand more about why we need it ... otherwise it seems like a hack and might be hard to maintain/understand later","commit_id":"253992a1e246e553c27ce8bc4afb5bc51a2d6ff4"}],"plugins/module_utils/tripleo_repos/main.py":[{"author":{"_account_id":14985,"name":"Alex Schultz","email":"aschultz@next-development.com","username":"mwhahaha"},"change_message_id":"b501f00e43dcbdc63b83f4d7c573b2be23775ef7","unresolved":true,"context_lines":[{"line_number":347,"context_line":"    _validate_current_tripleo(args.repos)"},{"line_number":348,"context_line":"    _validate_distro_repos(args)"},{"line_number":349,"context_line":"    _validate_tripleo_ci_testing(args.repos)"},{"line_number":350,"context_line":"    _validate_distro_stream(args, distro_name, distro_major_version_id)"},{"line_number":351,"context_line":""},{"line_number":352,"context_line":""},{"line_number":353,"context_line":"def _remove_existing(args):"}],"source_content_type":"text/x-python","patch_set":3,"id":"de7e385c_469b0b91","line":350,"updated":"2022-01-25 20:54:40.000000000","message":"an alternative would be a --skip-distro-validation and don\u0027t do this call.","commit_id":"c5df4a6aca1d5701c1b4407ab3b52a244d37dd88"}]}