)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":13861,"name":"yatin","email":"ykarel@redhat.com","username":"yatinkarel"},"change_message_id":"323ee0ca18d0ff3ecede87ba2aa5ad4b43f216a8","unresolved":true,"context_lines":[{"line_number":9,"context_line":"DLRN deps repo contains centos mirrors url which is not in our"},{"line_number":10,"context_line":"control got timeout in the jobs."},{"line_number":11,"context_line":""},{"line_number":12,"context_line":"TripleO repos does not update those which nodepool CentOS mirrors,"},{"line_number":13,"context_line":"It adds the tasks to do the same and also remove fedora  related"},{"line_number":14,"context_line":"repo renaming, since fedora is longer used in CI."},{"line_number":15,"context_line":""}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"130df273_0c7ba9e1","line":12,"range":{"start_line":12,"start_character":0,"end_line":12,"end_character":66},"updated":"2021-03-31 13:25:01.000000000","message":"tripleo-repos -b master --mirror http://mirror.centos.org --rdo-mirror https://trunk.rdoproject.org tripleo-ci-testing ceph opstools --stream\n\nTripleO repos supports overriding mirror overrides so let\u0027s use that instead","commit_id":"afa5b27cdcda99bf7b27e72fcdb1ea35481273e5"},{"author":{"_account_id":12393,"name":"chandan kumar","display_name":"Chandan Kumar","email":"chkumar@redhat.com","username":"chkumar246"},"change_message_id":"beda276b4bb78f199011ffbb65c7bf99c1fde083","unresolved":true,"context_lines":[{"line_number":9,"context_line":"DLRN deps repo contains centos mirrors url which is not in our"},{"line_number":10,"context_line":"control got timeout in the jobs."},{"line_number":11,"context_line":""},{"line_number":12,"context_line":"TripleO repos does not update those which nodepool CentOS mirrors,"},{"line_number":13,"context_line":"It adds the tasks to do the same and also remove fedora  related"},{"line_number":14,"context_line":"repo renaming, since fedora is longer used in CI."},{"line_number":15,"context_line":""}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"6113309d_4d0807a9","line":12,"range":{"start_line":12,"start_character":0,"end_line":12,"end_character":66},"in_reply_to":"130df273_0c7ba9e1","updated":"2021-03-31 13:41:43.000000000","message":"It is already there https://opendev.org/openstack/tripleo-ci/src/branch/master/roles/tripleo-repos/tasks/main.yml#L102 let me update the commit message","commit_id":"afa5b27cdcda99bf7b27e72fcdb1ea35481273e5"},{"author":{"_account_id":8449,"name":"Marios Andreou","email":"marios.andreou@gmail.com","username":"marios"},"change_message_id":"13c4937d70274b4a9b22e0098b61a59cd4f55f65","unresolved":true,"context_lines":[{"line_number":12,"context_line":"TripleO repos does not update those dlrn deps repo centos mirror"},{"line_number":13,"context_line":"url, It adds the tasks to do the same and also remove fedora"},{"line_number":14,"context_line":"related repo renaming, since fedora is longer used in CI."},{"line_number":15,"context_line":""},{"line_number":16,"context_line":"Closes-Bug: #1922032"},{"line_number":17,"context_line":""},{"line_number":18,"context_line":"Signed-off-by: Chandan Kumar (raukadah) \u003cchkumar@redhat.com\u003e"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":4,"id":"d3dc3e2b_cdd4ec19","line":15,"updated":"2021-04-02 07:05:22.000000000","message":"\"also removes stale fedora repo\"","commit_id":"5ca3c96306c6f0b45e1c3973a02b8560efd3718e"},{"author":{"_account_id":12393,"name":"chandan kumar","display_name":"Chandan Kumar","email":"chkumar@redhat.com","username":"chkumar246"},"change_message_id":"f9b0599cdd828061633e4dd4624bb66f413fd4e9","unresolved":false,"context_lines":[{"line_number":12,"context_line":"TripleO repos does not update those dlrn deps repo centos mirror"},{"line_number":13,"context_line":"url, It adds the tasks to do the same and also remove fedora"},{"line_number":14,"context_line":"related repo renaming, since fedora is longer used in CI."},{"line_number":15,"context_line":""},{"line_number":16,"context_line":"Closes-Bug: #1922032"},{"line_number":17,"context_line":""},{"line_number":18,"context_line":"Signed-off-by: Chandan Kumar (raukadah) \u003cchkumar@redhat.com\u003e"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":4,"id":"a7d907d2_7fbba67e","line":15,"in_reply_to":"d3dc3e2b_cdd4ec19","updated":"2021-04-05 10:19:49.000000000","message":"Done","commit_id":"5ca3c96306c6f0b45e1c3973a02b8560efd3718e"}],"roles/tripleo-repos/tasks/main.yml":[{"author":{"_account_id":24162,"name":"Sorin Sbârnea","display_name":"zbr","email":"ssbarnea@redhat.com","username":"ssbarnea","status":"do not feed the troll 🥕"},"change_message_id":"6d9c05802564d04f29817f8af927b0eea73bdddd","unresolved":true,"context_lines":[{"line_number":124,"context_line":"  shell: |"},{"line_number":125,"context_line":"    set -ex"},{"line_number":126,"context_line":"    sed -i -e \"s|http://mirror.centos.org/centos|$NODEPOOL_CENTOS_MIRROR|g\" \\"},{"line_number":127,"context_line":"      /etc/yum.repos.d/delorean-{{ ci_branch | default(\u0027master\u0027) }}-testing.repo"},{"line_number":128,"context_line":""},{"line_number":129,"context_line":"- name: Update all installed packages after new repos are setup  # noqa package-latest"},{"line_number":130,"context_line":"  become: true"}],"source_content_type":"text/x-yaml","patch_set":1,"id":"e9c9357b_01fb1019","line":127,"range":{"start_line":127,"start_character":23,"end_line":127,"end_character":80},"updated":"2021-03-31 09:57:36.000000000","message":"Wouldn\u0027t it be more flexible to do the sed on all repos and replace the original URL with the mirror one?","commit_id":"534f06656e848f11b8a665827e203c53cc80e21f"},{"author":{"_account_id":12393,"name":"chandan kumar","display_name":"Chandan Kumar","email":"chkumar@redhat.com","username":"chkumar246"},"change_message_id":"d7d2132d9e7f6f53a28822b3aae9130c499311d2","unresolved":false,"context_lines":[{"line_number":124,"context_line":"  shell: |"},{"line_number":125,"context_line":"    set -ex"},{"line_number":126,"context_line":"    sed -i -e \"s|http://mirror.centos.org/centos|$NODEPOOL_CENTOS_MIRROR|g\" \\"},{"line_number":127,"context_line":"      /etc/yum.repos.d/delorean-{{ ci_branch | default(\u0027master\u0027) }}-testing.repo"},{"line_number":128,"context_line":""},{"line_number":129,"context_line":"- name: Update all installed packages after new repos are setup  # noqa package-latest"},{"line_number":130,"context_line":"  become: true"}],"source_content_type":"text/x-yaml","patch_set":1,"id":"6da139ca_d34c3b9a","line":127,"range":{"start_line":127,"start_character":23,"end_line":127,"end_character":80},"in_reply_to":"e9c9357b_01fb1019","updated":"2021-03-31 10:33:02.000000000","message":"Done","commit_id":"534f06656e848f11b8a665827e203c53cc80e21f"},{"author":{"_account_id":13861,"name":"yatin","email":"ykarel@redhat.com","username":"yatinkarel"},"change_message_id":"323ee0ca18d0ff3ecede87ba2aa5ad4b43f216a8","unresolved":true,"context_lines":[{"line_number":123,"context_line":"  become: true"},{"line_number":124,"context_line":"  shell: |"},{"line_number":125,"context_line":"    set -ex"},{"line_number":126,"context_line":"    sed -i -e \"s|http://mirror.centos.org/centos|$NODEPOOL_CENTOS_MIRROR|g\" \\"},{"line_number":127,"context_line":"      /etc/yum.repos.d/*.repo"},{"line_number":128,"context_line":""},{"line_number":129,"context_line":"- name: Update all installed packages after new repos are setup  # noqa package-latest"}],"source_content_type":"text/x-yaml","patch_set":2,"id":"881d5053_a669d920","line":126,"range":{"start_line":126,"start_character":50,"end_line":126,"end_character":72},"updated":"2021-03-31 13:25:01.000000000","message":"do replace only when it\u0027s defined else it will cause issues","commit_id":"afa5b27cdcda99bf7b27e72fcdb1ea35481273e5"},{"author":{"_account_id":13861,"name":"yatin","email":"ykarel@redhat.com","username":"yatinkarel"},"change_message_id":"1de203528123e7933e0e93c4adc1df8caceaea6b","unresolved":true,"context_lines":[{"line_number":115,"context_line":"          --rdo-mirror $RDO_MIRROR_HOST $REPO ceph opstools"},{"line_number":116,"context_line":"    fi"},{"line_number":117,"context_line":""},{"line_number":118,"context_line":"    # Workaround for setting nodepool centos mirror for delorean-deps"},{"line_number":119,"context_line":"    if [ ! -z \"$DISTRO_MIRROR_HOST\" ]; then"},{"line_number":120,"context_line":"        sed -i -e \"s|http://mirror.centos.org/centos|$DISTRO_MIRROR_HOST|g\" \\"},{"line_number":121,"context_line":"         /etc/yum.repos.d/delorean-{{ ci_branch | default(\u0027master\u0027) }}-testing.repo"}],"source_content_type":"text/x-yaml","patch_set":3,"id":"1c65da27_18504ab7","line":118,"updated":"2021-03-31 13:51:09.000000000","message":"this would also need DISTRO_MIRROR_HOST to be defined which is not defined in the failing jobs, else it would have already set to correct values by tripleo-repos.","commit_id":"1f7c2342d86e996b305badde9806a35d4aa85287"},{"author":{"_account_id":12393,"name":"chandan kumar","display_name":"Chandan Kumar","email":"chkumar@redhat.com","username":"chkumar246"},"change_message_id":"df26638ddb02a0907f62865a668cb35cc47b38c9","unresolved":false,"context_lines":[{"line_number":115,"context_line":"          --rdo-mirror $RDO_MIRROR_HOST $REPO ceph opstools"},{"line_number":116,"context_line":"    fi"},{"line_number":117,"context_line":""},{"line_number":118,"context_line":"    # Workaround for setting nodepool centos mirror for delorean-deps"},{"line_number":119,"context_line":"    if [ ! -z \"$DISTRO_MIRROR_HOST\" ]; then"},{"line_number":120,"context_line":"        sed -i -e \"s|http://mirror.centos.org/centos|$DISTRO_MIRROR_HOST|g\" \\"},{"line_number":121,"context_line":"         /etc/yum.repos.d/delorean-{{ ci_branch | default(\u0027master\u0027) }}-testing.repo"}],"source_content_type":"text/x-yaml","patch_set":3,"id":"423161b2_24c532dd","line":118,"in_reply_to":"1c65da27_18504ab7","updated":"2021-04-01 08:00:46.000000000","message":"Done","commit_id":"1f7c2342d86e996b305badde9806a35d4aa85287"},{"author":{"_account_id":8449,"name":"Marios Andreou","email":"marios.andreou@gmail.com","username":"marios"},"change_message_id":"13c4937d70274b4a9b22e0098b61a59cd4f55f65","unresolved":true,"context_lines":[{"line_number":100,"context_line":""},{"line_number":101,"context_line":"    if [ ! -z \"$DISTRO_MIRROR_HOST\" ]; then"},{"line_number":102,"context_line":"        MIRROR\u003d\"--mirror $DISTRO_MIRROR_HOST\""},{"line_number":103,"context_line":"    fi"},{"line_number":104,"context_line":""},{"line_number":105,"context_line":"    if [ \"{{ is_stream_node|string }}\" \u003d\u003d \"True\" ]; then"},{"line_number":106,"context_line":"        tripleo-repos \\"}],"source_content_type":"text/x-yaml","patch_set":4,"id":"e426b921_1a54c812","line":103,"range":{"start_line":103,"start_character":4,"end_line":103,"end_character":6},"updated":"2021-04-02 07:05:22.000000000","message":"maybe you can just add it here as else?\n\nhmmm actually you probably want to move this task _after_ your one? So it sets the MIRROR variable to the rdo mirror?","commit_id":"5ca3c96306c6f0b45e1c3973a02b8560efd3718e"},{"author":{"_account_id":12393,"name":"chandan kumar","display_name":"Chandan Kumar","email":"chkumar@redhat.com","username":"chkumar246"},"change_message_id":"f9b0599cdd828061633e4dd4624bb66f413fd4e9","unresolved":false,"context_lines":[{"line_number":100,"context_line":""},{"line_number":101,"context_line":"    if [ ! -z \"$DISTRO_MIRROR_HOST\" ]; then"},{"line_number":102,"context_line":"        MIRROR\u003d\"--mirror $DISTRO_MIRROR_HOST\""},{"line_number":103,"context_line":"    fi"},{"line_number":104,"context_line":""},{"line_number":105,"context_line":"    if [ \"{{ is_stream_node|string }}\" \u003d\u003d \"True\" ]; then"},{"line_number":106,"context_line":"        tripleo-repos \\"}],"source_content_type":"text/x-yaml","patch_set":4,"id":"bb2158c1_9237ee84","line":103,"range":{"start_line":103,"start_character":4,"end_line":103,"end_character":6},"in_reply_to":"e426b921_1a54c812","updated":"2021-04-05 10:19:49.000000000","message":"Done","commit_id":"5ca3c96306c6f0b45e1c3973a02b8560efd3718e"},{"author":{"_account_id":8449,"name":"Marios Andreou","email":"marios.andreou@gmail.com","username":"marios"},"change_message_id":"13c4937d70274b4a9b22e0098b61a59cd4f55f65","unresolved":true,"context_lines":[{"line_number":115,"context_line":"          --rdo-mirror $RDO_MIRROR_HOST $REPO ceph opstools"},{"line_number":116,"context_line":"    fi"},{"line_number":117,"context_line":""},{"line_number":118,"context_line":"    # Workaround for setting nodepool centos mirror for delorean-deps"},{"line_number":119,"context_line":"    if [ -z \"$DISTRO_MIRROR_HOST\" ]; then"},{"line_number":120,"context_line":"        export $DISTRO_MIRROR_HOST\u003d\"http://mirror.regionone.vexxhost.rdoproject.org/centos\""},{"line_number":121,"context_line":"        sed -i -e \"s|http://mirror.centos.org/centos|$DISTRO_MIRROR_HOST|g\" \\"}],"source_content_type":"text/x-yaml","patch_set":4,"id":"c2da7674_4e960530","line":118,"range":{"start_line":118,"start_character":6,"end_line":118,"end_character":16},"updated":"2021-04-02 07:05:22.000000000","message":"since you went to the trouble of adding the comment (and thank you for doing that) may as well add reference to the launchpad e.g. \"launchpad tripleo/+bug/1922032\"","commit_id":"5ca3c96306c6f0b45e1c3973a02b8560efd3718e"},{"author":{"_account_id":12393,"name":"chandan kumar","display_name":"Chandan Kumar","email":"chkumar@redhat.com","username":"chkumar246"},"change_message_id":"f9b0599cdd828061633e4dd4624bb66f413fd4e9","unresolved":false,"context_lines":[{"line_number":115,"context_line":"          --rdo-mirror $RDO_MIRROR_HOST $REPO ceph opstools"},{"line_number":116,"context_line":"    fi"},{"line_number":117,"context_line":""},{"line_number":118,"context_line":"    # Workaround for setting nodepool centos mirror for delorean-deps"},{"line_number":119,"context_line":"    if [ -z \"$DISTRO_MIRROR_HOST\" ]; then"},{"line_number":120,"context_line":"        export $DISTRO_MIRROR_HOST\u003d\"http://mirror.regionone.vexxhost.rdoproject.org/centos\""},{"line_number":121,"context_line":"        sed -i -e \"s|http://mirror.centos.org/centos|$DISTRO_MIRROR_HOST|g\" \\"}],"source_content_type":"text/x-yaml","patch_set":4,"id":"5ae8c4d0_79b44cc4","line":118,"range":{"start_line":118,"start_character":6,"end_line":118,"end_character":16},"in_reply_to":"c2da7674_4e960530","updated":"2021-04-05 10:19:49.000000000","message":"Done","commit_id":"5ca3c96306c6f0b45e1c3973a02b8560efd3718e"},{"author":{"_account_id":13861,"name":"yatin","email":"ykarel@redhat.com","username":"yatinkarel"},"change_message_id":"e50058fd71e642b11e0345a8e0523be6aed8d2f8","unresolved":true,"context_lines":[{"line_number":104,"context_line":"    # Workaround for setting nodepool centos mirror for delorean-deps"},{"line_number":105,"context_line":"    # check launchpad tripleo/+bug/1922032"},{"line_number":106,"context_line":"    else"},{"line_number":107,"context_line":"        MIRROR\u003d\"--mirror http://mirror.regionone.vexxhost.rdoproject.org/centos\""},{"line_number":108,"context_line":"    fi"},{"line_number":109,"context_line":""},{"line_number":110,"context_line":"    if [ \"{{ is_stream_node|string }}\" \u003d\u003d \"True\" ]; then"}],"source_content_type":"text/x-yaml","patch_set":5,"id":"4dd4fe45_6191a0cd","line":107,"range":{"start_line":107,"start_character":32,"end_line":107,"end_character":79},"updated":"2021-04-06 11:26:37.000000000","message":"the issue on centos side was solved, hardcoding here is not good, if still needed can add a var which can be used to override mirror in jobs, like in third party CI where the issue got caught as DISTRO_MIRROR_HOST not set.","commit_id":"c27ef835a0021a12f1825fb7174d880c77502646"},{"author":{"_account_id":12393,"name":"chandan kumar","display_name":"Chandan Kumar","email":"chkumar@redhat.com","username":"chkumar246"},"change_message_id":"18f12a70f19e03f11e322d5b8eb615b1b15835ba","unresolved":true,"context_lines":[{"line_number":104,"context_line":"    # Workaround for setting nodepool centos mirror for delorean-deps"},{"line_number":105,"context_line":"    # check launchpad tripleo/+bug/1922032"},{"line_number":106,"context_line":"    else"},{"line_number":107,"context_line":"        MIRROR\u003d\"--mirror http://mirror.regionone.vexxhost.rdoproject.org/centos\""},{"line_number":108,"context_line":"    fi"},{"line_number":109,"context_line":""},{"line_number":110,"context_line":"    if [ \"{{ is_stream_node|string }}\" \u003d\u003d \"True\" ]; then"}],"source_content_type":"text/x-yaml","patch_set":5,"id":"4634a6ed_2d435a6c","line":107,"range":{"start_line":107,"start_character":32,"end_line":107,"end_character":79},"in_reply_to":"4dd4fe45_6191a0cd","updated":"2021-04-07 03:52:06.000000000","message":"Seems like a good idea, let me propose a follow up patch","commit_id":"c27ef835a0021a12f1825fb7174d880c77502646"}]}