)]}'
{"config/release/tripleo-ci/RedHat-8/master.yml":[{"author":{"_account_id":8449,"name":"Marios Andreou","email":"marios.andreou@gmail.com","username":"marios"},"change_message_id":"43c5241dfc9b9ccf1e9a95d90b6d85cc64378ce3","unresolved":false,"context_lines":[{"line_number":96,"context_line":"  sudo dnf install -y chrony;"},{"line_number":97,"context_line":"  sudo dnf install -y mock;"},{"line_number":98,"context_line":"  sudo touch /usr/share/rhn/RHN-ORG-TRUSTED-SSL-CERT;"},{"line_number":99,"context_line":"  sudo dnf repolist;"},{"line_number":100,"context_line":"  sudo dnf clean all;"},{"line_number":101,"context_line":"  {% if repo_setup_run_update|default(true)|bool %}"},{"line_number":102,"context_line":"  sudo dnf update -y;"}],"source_content_type":"text/x-yaml","patch_set":5,"id":"7faddb67_18d4a71f","line":99,"range":{"start_line":99,"start_character":7,"end_line":99,"end_character":19},"updated":"2019-07-31 06:46:41.000000000","message":"why do we need the repolist (debug?)\n(hmm i see we have it on the other file already)","commit_id":"6532ade7fcdd716b46e1e065a97e345887799a8b"}],"config/release/tripleo-ci/RedHat-8/promotion-testing-hash-master.yml":[{"author":{"_account_id":13294,"name":"Javier Peña","email":"jpena@redhat.com","username":"jpena"},"change_message_id":"effcf3df687fe70923b6dc17b558d3511a733a58","unresolved":false,"context_lines":[{"line_number":60,"context_line":"    baseurl: ${NODEPOOL_RDO_PROXY}/{{ distro_ver }}-{{ release }}/{{ distro_deps_repo_name }}/latest/"},{"line_number":61,"context_line":""},{"line_number":62,"context_line":"repo_cmd_after: |"},{"line_number":63,"context_line":"  sudo dnf install -y chrony;"},{"line_number":64,"context_line":"  sudo dnf install -y mock;"},{"line_number":65,"context_line":"  sudo dnf repolist;"},{"line_number":66,"context_line":"  sudo dnf clean all;"}],"source_content_type":"text/x-yaml","patch_set":2,"id":"7faddb67_25e1489c","line":63,"updated":"2019-07-30 14:18:55.000000000","message":"I think we should re-enable the rhui-* repos here, based on the log output.","commit_id":"0c65c0ce9f30317b82477647effd44c923944a30"},{"author":{"_account_id":12393,"name":"chandan kumar","display_name":"Chandan Kumar","email":"chkumar@redhat.com","username":"chkumar246"},"change_message_id":"720f1bc3655964f0de5063a84783c4277f7065ef","unresolved":false,"context_lines":[{"line_number":60,"context_line":"    baseurl: ${NODEPOOL_RDO_PROXY}/{{ distro_ver }}-{{ release }}/{{ distro_deps_repo_name }}/latest/"},{"line_number":61,"context_line":""},{"line_number":62,"context_line":"repo_cmd_after: |"},{"line_number":63,"context_line":"  sudo dnf install -y chrony;"},{"line_number":64,"context_line":"  sudo dnf install -y mock;"},{"line_number":65,"context_line":"  sudo dnf repolist;"},{"line_number":66,"context_line":"  sudo dnf clean all;"}],"source_content_type":"text/x-yaml","patch_set":2,"id":"7faddb67_2556282c","line":63,"in_reply_to":"7faddb67_25e1489c","updated":"2019-07-30 14:28:24.000000000","message":"Done","commit_id":"0c65c0ce9f30317b82477647effd44c923944a30"},{"author":{"_account_id":8449,"name":"Marios Andreou","email":"marios.andreou@gmail.com","username":"marios"},"change_message_id":"43c5241dfc9b9ccf1e9a95d90b6d85cc64378ce3","unresolved":false,"context_lines":[{"line_number":65,"context_line":"  sudo dnf config-manager --enable rhui-*;"},{"line_number":66,"context_line":"  sudo dnf install -y chrony;"},{"line_number":67,"context_line":"  sudo dnf install -y mock;"},{"line_number":68,"context_line":"  sudo touch /usr/share/rhn/RHN-ORG-TRUSTED-SSL-CERT;"},{"line_number":69,"context_line":"  sudo dnf repolist;"},{"line_number":70,"context_line":"  sudo dnf clean all;"},{"line_number":71,"context_line":"  {% if repo_setup_run_update|default(true)|bool %}"}],"source_content_type":"text/x-yaml","patch_set":5,"id":"7faddb67_38d96356","line":68,"updated":"2019-07-31 06:46:41.000000000","message":"needs a comment on this one?","commit_id":"6532ade7fcdd716b46e1e065a97e345887799a8b"},{"author":{"_account_id":8449,"name":"Marios Andreou","email":"marios.andreou@gmail.com","username":"marios"},"change_message_id":"e8c3815ee3cb1481bf32b0fe65790e85971c0b06","unresolved":false,"context_lines":[{"line_number":65,"context_line":"  sudo dnf config-manager --enable rhui-*;"},{"line_number":66,"context_line":"  sudo dnf install -y chrony;"},{"line_number":67,"context_line":"  sudo dnf install -y mock;"},{"line_number":68,"context_line":"  sudo touch /usr/share/rhn/RHN-ORG-TRUSTED-SSL-CERT;"},{"line_number":69,"context_line":"  sudo dnf repolist;"},{"line_number":70,"context_line":"  sudo dnf clean all;"},{"line_number":71,"context_line":"  {% if repo_setup_run_update|default(true)|bool %}"}],"source_content_type":"text/x-yaml","patch_set":5,"id":"7faddb67_be71bb78","line":68,"in_reply_to":"7faddb67_38d96356","updated":"2019-07-31 09:02:43.000000000","message":"really confusing to have so many reviews touchign the same file \n\nhttps://review.opendev.org/#/c/673738/2/config/release/tripleo-ci/RedHat-8/promotion-testing-hash-master.yml\n\nhttps://review.opendev.org/#/c/673517/5/config/release/tripleo-ci/RedHat-8/promotion-testing-hash-master.yml\n\nhttps://review.opendev.org/#/c/673709/3/config/release/tripleo-ci/RedHat-8/promotion-testing-hash-master.yml\n\n\nthe last two especially look like they have the same stuff? maybe comment there and combine?","commit_id":"6532ade7fcdd716b46e1e065a97e345887799a8b"}]}
