)]}'
{"docker/devenv/Dockerfile.j2":[{"author":{"_account_id":10787,"name":"Michal Jastrzebski (inc0)","email":"michal.jastrzebski@intel.com","username":"inc0"},"change_message_id":"b6a113bd37f5fb45528063ac5982355af2c8e6d6","unresolved":false,"context_lines":[{"line_number":1,"context_line":"FROM {{ namespace }}/{{ image_prefix }}openstack-base:{{ tag }}"},{"line_number":2,"context_line":"LABEL maintainer\u003d\"{{ maintainer }}\" name\u003d\"{{ image_name }}\" build-date\u003d\"{{ build_date }}\""},{"line_number":3,"context_line":""},{"line_number":4,"context_line":"{% block devenv_base_header %}{% endblock %}"}],"source_content_type":"text/x-jinja2","patch_set":40,"id":"7f515b1d_0a10494d","line":1,"updated":"2017-10-03 00:46:47.000000000","message":"ok that is my fail, I mixed two patchsets I was working on :/","commit_id":"901a6b35e4ced7f61dee1904e7e060c2c3f528c7"}],"tests/playbooks/pre.yml":[{"author":{"_account_id":13039,"name":"Martin André","email":"m.andre@redhat.com","username":"mandre"},"change_message_id":"4bfb9399af7e96d719991cb6d6f4464a9817de50","unresolved":false,"context_lines":[{"line_number":1,"context_line":"- hosts: all"},{"line_number":2,"context_line":"  tasks:"},{"line_number":3,"context_line":"    - name: Install dbus for debian system"},{"line_number":4,"context_line":"      apt: name\u003ddbus"},{"line_number":5,"context_line":"      become: true"},{"line_number":6,"context_line":"      when: ansible_os_family \u003d\u003d \u0027Debian\u0027"}],"source_content_type":"text/x-yaml","patch_set":42,"id":"7f515b1d_b0fbe3bb","line":3,"updated":"2017-10-03 08:30:16.000000000","message":"Is it really necessary? I would expect one of the pre-run playbook to install everything that is in bindep.txt. If it\u0027s not the case then we may need to move everything that is in bindep.txt here.","commit_id":"911b0d34addcb2f084936e3b315034809f316a1a"},{"author":{"_account_id":7488,"name":"Jeffrey Zhang","email":"zhang.lei.fly@gmail.com","username":"Jeffrey4l"},"change_message_id":"c12480a5883546bf3952e18ff930107f4d2a6368","unresolved":false,"context_lines":[{"line_number":8,"context_line":""},{"line_number":9,"context_line":"    - name: Install ansible"},{"line_number":10,"context_line":"      pip:"},{"line_number":11,"context_line":"        name: ansible"}],"source_content_type":"text/x-yaml","patch_set":45,"id":"7f515b1d_c6c6a24a","line":11,"updated":"2017-10-03 13:01:05.000000000","message":"we can use the ansible installed zuul.","commit_id":"f99052e5ed1b2e23669d3abf70c93e2917940f9c"}],"tests/playbooks/run.yml":[{"author":{"_account_id":9927,"name":"Britt Houser","email":"bhouser@cisco.com","username":"britthouser"},"change_message_id":"f3be7927081fd754e2985ef385b6bd3c73993a94","unresolved":false,"context_lines":[{"line_number":17,"context_line":"          mkdir -p /tmp/logs/{ansible,build,kolla,kolla_configs,system_logs}"},{"line_number":18,"context_line":"        executable: /bin/bash"},{"line_number":19,"context_line":"        chdir: \"{{ zuul.project.src_dir }}\""},{"line_number":20,"context_line":"            "},{"line_number":21,"context_line":"    - name: copy setup script"},{"line_number":22,"context_line":"      copy:"},{"line_number":23,"context_line":"        src: \"{{ zuul.project.src_dir }}/tools/setup_{{ ansible_os_family }}.sh\""}],"source_content_type":"text/x-yaml","patch_set":38,"id":"7f515b1d_c26223ae","line":20,"updated":"2017-10-02 13:55:35.000000000","message":"don\u0027t forget to delete this whitespace.","commit_id":"a9c8f4d238c6445abd89176abf746a2bf15308ff"},{"author":{"_account_id":13039,"name":"Martin André","email":"m.andre@redhat.com","username":"mandre"},"change_message_id":"a0f49d6f4c471d87e22b355ed8f4c40406401894","unresolved":false,"context_lines":[{"line_number":20,"context_line":"            "},{"line_number":21,"context_line":"    - name: copy setup script"},{"line_number":22,"context_line":"      copy:"},{"line_number":23,"context_line":"        src: \"{{ zuul.project.src_dir }}/tools/setup_{{ ansible_os_family }}.sh\""},{"line_number":24,"context_line":"        dest: /tmp/setup.sh"},{"line_number":25,"context_line":"        mode: 0755"},{"line_number":26,"context_line":""}],"source_content_type":"text/x-yaml","patch_set":38,"id":"7f515b1d_5843c642","line":23,"updated":"2017-10-02 15:15:37.000000000","message":"I\u0027m thinking we should move the tools/setup_XXX.sh scripts to the playbook and make them jinja templates.","commit_id":"a9c8f4d238c6445abd89176abf746a2bf15308ff"},{"author":{"_account_id":13039,"name":"Martin André","email":"m.andre@redhat.com","username":"mandre"},"change_message_id":"4bfb9399af7e96d719991cb6d6f4464a9817de50","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":42,"id":"7f515b1d_b079a3c5","updated":"2017-10-03 08:30:16.000000000","message":"Maybe all the node setup should be moved to the pre playbook?","commit_id":"911b0d34addcb2f084936e3b315034809f316a1a"},{"author":{"_account_id":13039,"name":"Martin André","email":"m.andre@redhat.com","username":"mandre"},"change_message_id":"4bfb9399af7e96d719991cb6d6f4464a9817de50","unresolved":false,"context_lines":[{"line_number":34,"context_line":"        mode: 666"},{"line_number":35,"context_line":"      become: true"},{"line_number":36,"context_line":""},{"line_number":37,"context_line":"    - name: Install ansible"},{"line_number":38,"context_line":"      pip:"},{"line_number":39,"context_line":"        name: ansible"},{"line_number":40,"context_line":""}],"source_content_type":"text/x-yaml","patch_set":42,"id":"7f515b1d_2b58a42e","line":37,"updated":"2017-10-03 08:30:16.000000000","message":"Current failure due to missing \u0027become: true\u0027.","commit_id":"911b0d34addcb2f084936e3b315034809f316a1a"}],"tools/deploy_aio.sh":[{"author":{"_account_id":13039,"name":"Martin André","email":"m.andre@redhat.com","username":"mandre"},"change_message_id":"4bfb9399af7e96d719991cb6d6f4464a9817de50","unresolved":false,"context_lines":[{"line_number":9,"context_line":"export KOLLA_TYPE\u003d$2"},{"line_number":10,"context_line":"export KOLLA_ANSIBLE_DIR\u003d$(mktemp -d)"},{"line_number":11,"context_line":""},{"line_number":12,"context_line":"cat \u003e /tmp/clonemap \u003c\u003cEOF"},{"line_number":13,"context_line":"clonemap:"},{"line_number":14,"context_line":" - name: openstack/kolla-ansible"},{"line_number":15,"context_line":"   dest: ${KOLLA_ANSIBLE_DIR}"}],"source_content_type":"text/x-sh","patch_set":42,"id":"7f515b1d_d04d3fc4","line":12,"updated":"2017-10-03 08:30:16.000000000","message":"IIUC we\u0027re now cloning kolla-ansible via https://review.openstack.org/#/c/508661/42/.zuul.yaml@44, we should stop using zuul-cloner.","commit_id":"911b0d34addcb2f084936e3b315034809f316a1a"}],"tools/gate_run.sh":[{"author":{"_account_id":9927,"name":"Britt Houser","email":"bhouser@cisco.com","username":"britthouser"},"change_message_id":"f3be7927081fd754e2985ef385b6bd3c73993a94","unresolved":false,"context_lines":[{"line_number":3,"context_line":"set -o errexit"},{"line_number":4,"context_line":"set -o xtrace"},{"line_number":5,"context_line":""},{"line_number":6,"context_line":"# exist when the jobs is not project jobs."},{"line_number":7,"context_line":"if [[ -z $IN_PROJECT_JOBS ]]; then"},{"line_number":8,"context_line":"    exit 0"},{"line_number":9,"context_line":"fi"}],"source_content_type":"text/x-sh","patch_set":38,"id":"7f515b1d_a20a87ce","line":6,"updated":"2017-10-02 13:55:35.000000000","message":"exit","commit_id":"a9c8f4d238c6445abd89176abf746a2bf15308ff"}],"tools/setup_Debian.sh":[{"author":{"_account_id":9927,"name":"Britt Houser","email":"bhouser@cisco.com","username":"britthouser"},"change_message_id":"f3be7927081fd754e2985ef385b6bd3c73993a94","unresolved":false,"context_lines":[{"line_number":68,"context_line":"if [[ ${DISTRIB_CODENAME} \u003d\u003d \"trusty\" ]]; then"},{"line_number":69,"context_line":"    sudo apt-get -y install --no-install-recommends btrfs-tools"},{"line_number":70,"context_line":"    setup_disk"},{"line_number":71,"context_line":"    echo \"DOCKER_OPTS\u003d\\\"-s btrfs --insecure-registry 0.0.0.0/0\\\"\" | sudo tee /etc/default/docker"},{"line_number":72,"context_line":"    sudo mount --make-shared /run"},{"line_number":73,"context_line":"    sudo service docker start"},{"line_number":74,"context_line":"else"}],"source_content_type":"text/x-sh","patch_set":38,"id":"7f515b1d_a233a7aa","line":71,"updated":"2017-10-02 13:55:35.000000000","message":"Specifying 0.0.0.0/0 for insecure registry, allows all ports as well?","commit_id":"a9c8f4d238c6445abd89176abf746a2bf15308ff"},{"author":{"_account_id":13039,"name":"Martin André","email":"m.andre@redhat.com","username":"mandre"},"change_message_id":"a0f49d6f4c471d87e22b355ed8f4c40406401894","unresolved":false,"context_lines":[{"line_number":68,"context_line":"if [[ ${DISTRIB_CODENAME} \u003d\u003d \"trusty\" ]]; then"},{"line_number":69,"context_line":"    sudo apt-get -y install --no-install-recommends btrfs-tools"},{"line_number":70,"context_line":"    setup_disk"},{"line_number":71,"context_line":"    echo \"DOCKER_OPTS\u003d\\\"-s btrfs --insecure-registry 0.0.0.0/0\\\"\" | sudo tee /etc/default/docker"},{"line_number":72,"context_line":"    sudo mount --make-shared /run"},{"line_number":73,"context_line":"    sudo service docker start"},{"line_number":74,"context_line":"else"}],"source_content_type":"text/x-sh","patch_set":38,"id":"7f515b1d_38100a4e","line":71,"in_reply_to":"7f515b1d_a233a7aa","updated":"2017-10-02 15:15:37.000000000","message":"If we render this file from a jinja template then we can replace $(cat /etc/nodepool/primary_node_private) with the content of nodepool.private_ipv4 variable.","commit_id":"a9c8f4d238c6445abd89176abf746a2bf15308ff"}]}