)]}'
{"roles/build-test-packages/tasks/dlrn-build.yml":[{"author":{"_account_id":16312,"name":"Alfredo Moralejo","email":"amoralej@redhat.com","username":"amoralej"},"change_message_id":"fc8f549453054f1ea97781c3e0e6c532b0b91d54","unresolved":false,"context_lines":[{"line_number":18,"context_line":"    regexp: \u0027{{ project_item.regexp }}\u0027"},{"line_number":19,"context_line":"    line: \u0027{{ project_item.line }}\u0027"},{"line_number":20,"context_line":"  with_items:"},{"line_number":21,"context_line":"    - { regexp: \u0027baseurl\u003d.*\u0027, line: \u0027baseurl\u003d{{ dlrn_base_url }}/centos7-{{ artg_release }}\u0027 }"},{"line_number":22,"context_line":"    - { regexp: \u0027distro\u003d.*\u0027, line: \u0027distro\u003d{{ artg_distro }}\u0027 }"},{"line_number":23,"context_line":"    - { regexp: \u0027source\u003d.*\u0027, line: \u0027source\u003d{{ artg_change.branch }}\u0027 }"},{"line_number":24,"context_line":"  loop_control:"}],"source_content_type":"text/x-yaml","patch_set":4,"id":"9f436f4f_b88df8d4","line":21,"updated":"2017-08-17 17:52:58.000000000","message":"Note this will not totally switch repos to use dlrn_base_url, but that url is only used to fetch the repo files \u003cbaseurl\u003e/delorean-deps.repo and \u003cbaseurl\u003e/current/delorean.repo. The repo server used to download packages is given by the content of those files which still will point to http://trunk.rdoproject.org.","commit_id":"f232bbb37b98d6ac85373d03229e0c1caaad126a"},{"author":{"_account_id":8652,"name":"Attila Darazs","email":"adarazs@redhat.com","username":"adarazs"},"change_message_id":"025bef28c03fb51e03e22988a15cb8155a14f920","unresolved":false,"context_lines":[{"line_number":18,"context_line":"    regexp: \u0027{{ project_item.regexp }}\u0027"},{"line_number":19,"context_line":"    line: \u0027{{ project_item.line }}\u0027"},{"line_number":20,"context_line":"  with_items:"},{"line_number":21,"context_line":"    - { regexp: \u0027baseurl\u003d.*\u0027, line: \u0027baseurl\u003d{{ dlrn_base_url }}/centos7-{{ artg_release }}\u0027 }"},{"line_number":22,"context_line":"    - { regexp: \u0027distro\u003d.*\u0027, line: \u0027distro\u003d{{ artg_distro }}\u0027 }"},{"line_number":23,"context_line":"    - { regexp: \u0027source\u003d.*\u0027, line: \u0027source\u003d{{ artg_change.branch }}\u0027 }"},{"line_number":24,"context_line":"  loop_control:"}],"source_content_type":"text/x-yaml","patch_set":4,"id":"9f436f4f_39cc5860","line":21,"in_reply_to":"9f436f4f_1e1a9a16","updated":"2017-08-18 09:38:38.000000000","message":"I was confused by your suggestion about the fake repo files originally in the bug, now I see your point.\n\nIt\u0027s somewhat hacky but it will get the job done. I will rework the change this way. Thank you for the suggestion.","commit_id":"f232bbb37b98d6ac85373d03229e0c1caaad126a"},{"author":{"_account_id":16312,"name":"Alfredo Moralejo","email":"amoralej@redhat.com","username":"amoralej"},"change_message_id":"0ad4c326958c9752b16434ad69b47f6ac43c9339","unresolved":false,"context_lines":[{"line_number":18,"context_line":"    regexp: \u0027{{ project_item.regexp }}\u0027"},{"line_number":19,"context_line":"    line: \u0027{{ project_item.line }}\u0027"},{"line_number":20,"context_line":"  with_items:"},{"line_number":21,"context_line":"    - { regexp: \u0027baseurl\u003d.*\u0027, line: \u0027baseurl\u003d{{ dlrn_base_url }}/centos7-{{ artg_release }}\u0027 }"},{"line_number":22,"context_line":"    - { regexp: \u0027distro\u003d.*\u0027, line: \u0027distro\u003d{{ artg_distro }}\u0027 }"},{"line_number":23,"context_line":"    - { regexp: \u0027source\u003d.*\u0027, line: \u0027source\u003d{{ artg_change.branch }}\u0027 }"},{"line_number":24,"context_line":"  loop_control:"}],"source_content_type":"text/x-yaml","patch_set":4,"id":"9f436f4f_1e1a9a16","line":21,"in_reply_to":"9f436f4f_711d6b72","updated":"2017-08-18 09:05:20.000000000","message":"I think we could use existing delorean repo files in the host that are already using regional mirror, see my comment in https://review.openstack.org/#/c/494262/4/roles/build-test-packages/tasks/main.yml and create empty local files, something like:\n\n/tmp/fake-repo/delorean-deps.repo\n/tmp/fake-repo/current/delorean.repo\n\nand set dlrn_base_url to /tmp/fake-repo/\n\nThat should work","commit_id":"f232bbb37b98d6ac85373d03229e0c1caaad126a"},{"author":{"_account_id":12715,"name":"John Trowbridge","email":"trown@redhat.com","username":"trown"},"change_message_id":"e1a1cf2fae200aaf303390db9d2dc7b33b6f8a50","unresolved":false,"context_lines":[{"line_number":18,"context_line":"    regexp: \u0027{{ project_item.regexp }}\u0027"},{"line_number":19,"context_line":"    line: \u0027{{ project_item.line }}\u0027"},{"line_number":20,"context_line":"  with_items:"},{"line_number":21,"context_line":"    - { regexp: \u0027baseurl\u003d.*\u0027, line: \u0027baseurl\u003d{{ dlrn_base_url }}/centos7-{{ artg_release }}\u0027 }"},{"line_number":22,"context_line":"    - { regexp: \u0027distro\u003d.*\u0027, line: \u0027distro\u003d{{ artg_distro }}\u0027 }"},{"line_number":23,"context_line":"    - { regexp: \u0027source\u003d.*\u0027, line: \u0027source\u003d{{ artg_change.branch }}\u0027 }"},{"line_number":24,"context_line":"  loop_control:"}],"source_content_type":"text/x-yaml","patch_set":4,"id":"9f436f4f_711d6b72","line":21,"in_reply_to":"9f436f4f_b88df8d4","updated":"2017-08-17 20:06:26.000000000","message":"ick... so we would need to modify the .repo files then.","commit_id":"f232bbb37b98d6ac85373d03229e0c1caaad126a"}],"roles/build-test-packages/tasks/main.yml":[{"author":{"_account_id":14985,"name":"Alex Schultz","email":"aschultz@next-development.com","username":"mwhahaha"},"change_message_id":"13d6f5089ae7bc54dc9fd2367b7747c7135ea1d6","unresolved":false,"context_lines":[{"line_number":58,"context_line":"  shell: \u003e"},{"line_number":59,"context_line":"    gawk \u0027{ print $0 }; /^# repos$/ { exit }\u0027 {{ build_repo_dir }}/DLRN/scripts/centos.cfg"},{"line_number":60,"context_line":"    \u003e {{ build_repo_dir }}/DLRN/scripts/centos-local.cfg;"},{"line_number":61,"context_line":"    cat /etc/yum.repos.d/centos-base.repo \u003e\u003e {{ build_repo_dir }}/DLRN/scripts/centos-local.cfg;"},{"line_number":62,"context_line":"    cat /etc/yum.repos.d/centos-updates.repo \u003e\u003e {{ build_repo_dir }}/DLRN/scripts/centos-local.cfg;"},{"line_number":63,"context_line":"    cat /etc/yum.repos.d/centos-extras.repo \u003e\u003e {{ build_repo_dir }}/DLRN/scripts/centos-local.cfg;"},{"line_number":64,"context_line":"    echo \u0027\"\"\"\u0027 \u003e\u003e {{ build_repo_dir }}/DLRN/scripts/centos-local.cfg;"},{"line_number":65,"context_line":"    mv {{ build_repo_dir }}/DLRN/scripts/centos-local.cfg {{ build_repo_dir }}/DLRN/scripts/centos.cfg"},{"line_number":66,"context_line":"  when: dlrn_use_local_centos_mirror|bool"}],"source_content_type":"text/x-yaml","patch_set":3,"id":"9f436f4f_e3787a2c","line":63,"range":{"start_line":61,"start_character":0,"end_line":63,"end_character":98},"updated":"2017-08-17 14:49:53.000000000","message":"This assumes these files exist.  They are also getting renamed to quickstart-centos-* in https://review.openstack.org/#/c/494056/  \n\nCan we do something else to get this repo information?","commit_id":"83cb3db3c2cb0da90683554ed6e770ac8b5509ba"},{"author":{"_account_id":16312,"name":"Alfredo Moralejo","email":"amoralej@redhat.com","username":"amoralej"},"change_message_id":"fc8f549453054f1ea97781c3e0e6c532b0b91d54","unresolved":false,"context_lines":[{"line_number":60,"context_line":"    \u003e {{ build_repo_dir }}/DLRN/scripts/centos-local.cfg;"},{"line_number":61,"context_line":"    cat /etc/yum.repos.d/quickstart-centos-base.repo \u003e\u003e {{ build_repo_dir }}/DLRN/scripts/centos-local.cfg;"},{"line_number":62,"context_line":"    cat /etc/yum.repos.d/quickstart-centos-updates.repo \u003e\u003e {{ build_repo_dir }}/DLRN/scripts/centos-local.cfg;"},{"line_number":63,"context_line":"    cat /etc/yum.repos.d/quickstart-centos-extras.repo \u003e\u003e {{ build_repo_dir }}/DLRN/scripts/centos-local.cfg;"},{"line_number":64,"context_line":"    echo \u0027\"\"\"\u0027 \u003e\u003e {{ build_repo_dir }}/DLRN/scripts/centos-local.cfg;"},{"line_number":65,"context_line":"    mv {{ build_repo_dir }}/DLRN/scripts/centos-local.cfg {{ build_repo_dir }}/DLRN/scripts/centos.cfg"},{"line_number":66,"context_line":"  when: dlrn_use_local_centos_mirror|bool"}],"source_content_type":"text/x-yaml","patch_set":4,"id":"9f436f4f_3870e8dc","line":63,"range":{"start_line":63,"start_character":18,"end_line":63,"end_character":19},"updated":"2017-08-17 17:52:58.000000000","message":"We may concatenate repo configuration for delorean here too, /etc/yum.repos.d/delorean-*.repo, /etc/yum.repos.d/quickstart-centos-qemu.repo and /etc/yum.repos.d/quickstart-centos-ceph-jewel.repo","commit_id":"f232bbb37b98d6ac85373d03229e0c1caaad126a"},{"author":{"_account_id":16312,"name":"Alfredo Moralejo","email":"amoralej@redhat.com","username":"amoralej"},"change_message_id":"e1023052e9df7c1befe966a9733af99d1f744f94","unresolved":false,"context_lines":[{"line_number":61,"context_line":"    cat /etc/yum.repos.d/quickstart-centos-*.repo \u003e\u003e {{ build_repo_dir }}/DLRN/scripts/centos-local.cfg;"},{"line_number":62,"context_line":"    cat /etc/yum.repos.d/delorean*.repo \u003e\u003e {{ build_repo_dir }}/DLRN/scripts/centos-local.cfg;"},{"line_number":63,"context_line":"    echo \u0027\"\"\"\u0027 \u003e\u003e {{ build_repo_dir }}/DLRN/scripts/centos-local.cfg;"},{"line_number":64,"context_line":"    mkdir -p /tmp/fake-dlrn-repos/current"},{"line_number":65,"context_line":"    touch /tmp/fake-dlrn-repos/delorean-deps.repo"},{"line_number":66,"context_line":"    touch /tmp/fake-dlrn-repos/current/delorean.repo"},{"line_number":67,"context_line":"  when: dlrn_use_local_mirrors|bool"}],"source_content_type":"text/x-yaml","patch_set":5,"id":"9f436f4f_716f9492","line":64,"updated":"2017-08-18 11:59:09.000000000","message":"add ; at the end of each line, it\u0027s taking all this as a single command:\n\n \"cmd\": \"gawk \u0027{ print $0 }; /^# repos$/ { exit }\u0027 /home/jenkins/DLRN/scripts/centos.cfg \u003e /home/jenkins/DLRN/scripts/centos-local.cfg; cat /etc/yum.repos.d/quickstart-centos-*.repo \u003e\u003e /home/jenkins/DLRN/scripts/centos-local.cfg; cat /etc/yum.repos.d/delorean*.repo \u003e\u003e /home/jenkins/DLRN/scripts/centos-local.cfg; echo \u0027\\\"\\\"\\\"\u0027 \u003e\u003e /home/jenkins/DLRN/scripts/centos-local.cfg; mkdir -p /tmp/fake-dlrn-repos/current touch /tmp/fake-dlrn-repos/delorean-deps.repo touch /tmp/fake-dlrn-repos/current/delorean.repo\"","commit_id":"10adfb36faf1011651edc9a424d6d1ed199cbd1d"},{"author":{"_account_id":8652,"name":"Attila Darazs","email":"adarazs@redhat.com","username":"adarazs"},"change_message_id":"1285652559068a4ed14a526b17c4833bab66223e","unresolved":false,"context_lines":[{"line_number":61,"context_line":"    cat /etc/yum.repos.d/quickstart-centos-*.repo \u003e\u003e {{ build_repo_dir }}/DLRN/scripts/centos-local.cfg;"},{"line_number":62,"context_line":"    cat /etc/yum.repos.d/delorean*.repo \u003e\u003e {{ build_repo_dir }}/DLRN/scripts/centos-local.cfg;"},{"line_number":63,"context_line":"    echo \u0027\"\"\"\u0027 \u003e\u003e {{ build_repo_dir }}/DLRN/scripts/centos-local.cfg;"},{"line_number":64,"context_line":"    mkdir -p /tmp/fake-dlrn-repos/current"},{"line_number":65,"context_line":"    touch /tmp/fake-dlrn-repos/delorean-deps.repo"},{"line_number":66,"context_line":"    touch /tmp/fake-dlrn-repos/current/delorean.repo"},{"line_number":67,"context_line":"  when: dlrn_use_local_mirrors|bool"}],"source_content_type":"text/x-yaml","patch_set":5,"id":"9f436f4f_7190b44f","line":64,"in_reply_to":"9f436f4f_716f9492","updated":"2017-08-18 12:05:55.000000000","message":"Right, thanks.","commit_id":"10adfb36faf1011651edc9a424d6d1ed199cbd1d"}]}