)]}'
{"tools/deployment/multinode/kube-node-subnet.sh":[{"author":{"_account_id":14119,"name":"Sam Yaple","email":"sam+git@yaple.net","username":"SamYaple"},"change_message_id":"86c4be7210dda6ddd4ba22786821af3713dd6faf","unresolved":false,"context_lines":[{"line_number":21,"context_line":"# FIXME(cw); i\u0027m really not happy about this, @portdirect put the lart away"},{"line_number":22,"context_line":"if ! which ipcalc 2\u003e\u00261 \u003e /dev/null ; then"},{"line_number":23,"context_line":"    if ! which wget 2\u003e\u00261 \u003e /dev/null ; then"},{"line_number":24,"context_line":"\tyum install -y wget || apt-get install -y wget"},{"line_number":25,"context_line":"    fi"},{"line_number":26,"context_line":"    wget -cS https://github.com/cwedgwood/ipcalc/raw/0.2.2-cw0/ipcalc"},{"line_number":27,"context_line":"    chmod +x ./ipcalc"}],"source_content_type":"text/x-sh","patch_set":8,"id":"7f96bb07_06115704","line":24,"updated":"2018-01-17 22:05:40.000000000","message":"tab nitpick","commit_id":"9d2d7787b4c766c9a90c1b6a5fcf105396a40729"}],"tools/gate/files/zutils-Dockerfile":[{"author":{"_account_id":14119,"name":"Sam Yaple","email":"sam+git@yaple.net","username":"SamYaple"},"change_message_id":"95c069368f07df8b21bdac506c6172e08044703c","unresolved":false,"context_lines":[{"line_number":1,"context_line":"FROM debian"},{"line_number":2,"context_line":""},{"line_number":3,"context_line":"RUN apt-get update \u0026\u0026 apt-get install -y ipcalc --no-install-recommends"},{"line_number":4,"context_line":"RUN apt-get clean"},{"line_number":5,"context_line":"RUN find /var/lib/apt/lists/ -type f -print0 | xargs -r0 rm"},{"line_number":6,"context_line":"RUN find /var/cache/ /var/log/ -type f -print0 | xargs -r0 truncate -s0"}],"source_content_type":"application/octet-stream","patch_set":33,"id":"5f93b717_1db558cd","line":4,"updated":"2018-01-30 21:07:18.000000000","message":"a bit dirty, combine the run lines?","commit_id":"80e1efa2213d39bf55eb2a887a425636d05bdc80"},{"author":{"_account_id":8898,"name":"Chris Wedgwood","email":"cw@f00f.org","username":"anticw"},"change_message_id":"e0e6e4bcdb4a6c527e9ef8fd79fd0eeb51659e55","unresolved":false,"context_lines":[{"line_number":1,"context_line":"FROM debian"},{"line_number":2,"context_line":""},{"line_number":3,"context_line":"RUN apt-get update \u0026\u0026 apt-get install -y ipcalc --no-install-recommends"},{"line_number":4,"context_line":"RUN apt-get clean"},{"line_number":5,"context_line":"RUN find /var/lib/apt/lists/ -type f -print0 | xargs -r0 rm"},{"line_number":6,"context_line":"RUN find /var/cache/ /var/log/ -type f -print0 | xargs -r0 truncate -s0"}],"source_content_type":"application/octet-stream","patch_set":33,"id":"5f93b717_5d1be0ce","line":4,"in_reply_to":"5f93b717_1db558cd","updated":"2018-01-30 21:10:17.000000000","message":"not sure i agree it\u0027s dirty but sure","commit_id":"80e1efa2213d39bf55eb2a887a425636d05bdc80"}],"tools/gate/playbooks/multinode-deploy.yaml":[{"author":{"_account_id":14119,"name":"Sam Yaple","email":"sam+git@yaple.net","username":"SamYaple"},"change_message_id":"95c069368f07df8b21bdac506c6172e08044703c","unresolved":false,"context_lines":[{"line_number":21,"context_line":"      args:"},{"line_number":22,"context_line":"        chdir: \"{{ zuul.project.src_dir }}\""},{"line_number":23,"context_line":"    - name: make zutils container"},{"line_number":24,"context_line":"      shell: |"},{"line_number":25,"context_line":"        set -xe;"},{"line_number":26,"context_line":"        ./tools/deployment/multinode/make-zutils.sh"},{"line_number":27,"context_line":"      args:"}],"source_content_type":"text/x-yaml","patch_set":33,"id":"5f93b717_bdcd4467","line":24,"updated":"2018-01-30 21:07:18.000000000","message":"is shell really needed here? couldnt you just command: pointing to the script?","commit_id":"80e1efa2213d39bf55eb2a887a425636d05bdc80"},{"author":{"_account_id":8898,"name":"Chris Wedgwood","email":"cw@f00f.org","username":"anticw"},"change_message_id":"0a3fbac29c94a0d7cf7ee4dcc7ee5def59d29dcd","unresolved":false,"context_lines":[{"line_number":21,"context_line":"      args:"},{"line_number":22,"context_line":"        chdir: \"{{ zuul.project.src_dir }}\""},{"line_number":23,"context_line":"    - name: make zutils container"},{"line_number":24,"context_line":"      shell: |"},{"line_number":25,"context_line":"        set -xe;"},{"line_number":26,"context_line":"        ./tools/deployment/multinode/make-zutils.sh"},{"line_number":27,"context_line":"      args:"}],"source_content_type":"text/x-yaml","patch_set":33,"id":"3fa0c359_666e6230","line":24,"in_reply_to":"3fa0c359_b5f572ef","updated":"2018-02-09 01:08:15.000000000","message":"it does work, later versions of this PS do just that\n\nhttp://logs.openstack.org/23/534623/40/check/openstack-helm-multinode-fedora/fe10f96/job-output.txt.gz#_2018-02-08_18_48_06_509564\n\nfor example shows this working","commit_id":"80e1efa2213d39bf55eb2a887a425636d05bdc80"},{"author":{"_account_id":8898,"name":"Chris Wedgwood","email":"cw@f00f.org","username":"anticw"},"change_message_id":"e0e6e4bcdb4a6c527e9ef8fd79fd0eeb51659e55","unresolved":false,"context_lines":[{"line_number":21,"context_line":"      args:"},{"line_number":22,"context_line":"        chdir: \"{{ zuul.project.src_dir }}\""},{"line_number":23,"context_line":"    - name: make zutils container"},{"line_number":24,"context_line":"      shell: |"},{"line_number":25,"context_line":"        set -xe;"},{"line_number":26,"context_line":"        ./tools/deployment/multinode/make-zutils.sh"},{"line_number":27,"context_line":"      args:"}],"source_content_type":"text/x-yaml","patch_set":33,"id":"5f93b717_dd3db02b","line":24,"in_reply_to":"5f93b717_bdcd4467","updated":"2018-01-30 21:10:17.000000000","message":"i had that originally and think the \"foo \u003c path/to/bar\" maybe didn\u0027t work\n\nlet me put it back and check","commit_id":"80e1efa2213d39bf55eb2a887a425636d05bdc80"},{"author":{"_account_id":22477,"name":"Matt McEuen","email":"matt.mceuen@att.com","username":"mattmceuen"},"change_message_id":"3303e0b1a2424b4ea0d4770777d92bf1756fd9b4","unresolved":false,"context_lines":[{"line_number":21,"context_line":"      args:"},{"line_number":22,"context_line":"        chdir: \"{{ zuul.project.src_dir }}\""},{"line_number":23,"context_line":"    - name: make zutils container"},{"line_number":24,"context_line":"      shell: |"},{"line_number":25,"context_line":"        set -xe;"},{"line_number":26,"context_line":"        ./tools/deployment/multinode/make-zutils.sh"},{"line_number":27,"context_line":"      args:"}],"source_content_type":"text/x-yaml","patch_set":33,"id":"3fa0c359_b5f572ef","line":24,"in_reply_to":"5f93b717_dd3db02b","updated":"2018-02-08 22:03:47.000000000","message":"What was the result, will command: work?","commit_id":"80e1efa2213d39bf55eb2a887a425636d05bdc80"}]}