)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":10239,"name":"Dmitry Tantsur","email":"dtantsur@protonmail.com","username":"dtantsur"},"change_message_id":"a79c7138993d8c5a6fdad5ad8493f1fbf6783b7b","unresolved":false,"context_lines":[{"line_number":4,"context_line":"Commit:     Sam Betts \u003csam@code-smash.net\u003e"},{"line_number":5,"context_line":"CommitDate: 2016-05-06 20:34:12 +0100"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"TinyIPA: Shave off some file size from tinyipa ramdisk"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Change-Id: I6b8d648800284444d67a6988511707e57de1e6c9"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"dab17558_a85ec60f","line":7,"updated":"2016-05-08 15:24:41.000000000","message":"if happen to create one more patch set, please explain what you actually did","commit_id":"04202fa3203ed1ed379da922433a60aac59aff04"},{"author":{"_account_id":18893,"name":"Miles Gould","email":"mgould@redhat.com","username":"pozorvlak"},"change_message_id":"a405f84a1fc3eac5d73b6f2b802c13b374e075a2","unresolved":false,"context_lines":[{"line_number":7,"context_line":"TinyIPA: Shave off some file size from tinyipa ramdisk"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"This patch aims to reduce the TinyIPA ramdisk file size by removing"},{"line_number":10,"context_line":"some files that aren\u0027t required to run IPA once built and also striping"},{"line_number":11,"context_line":"the binaries we build for qemu-utils and tgt to make them as small as"},{"line_number":12,"context_line":"possible."},{"line_number":13,"context_line":""}],"source_content_type":"text/x-gerrit-commit-message","patch_set":3,"id":"dab17558_808b8467","line":10,"updated":"2016-05-10 16:09:26.000000000","message":"Typo: \"stripping\"","commit_id":"672360166393367b1ad3823444df82ec0dc4b83e"},{"author":{"_account_id":18893,"name":"Miles Gould","email":"mgould@redhat.com","username":"pozorvlak"},"change_message_id":"46102e2771ba4b1946d904aa758bf7e38b78f3d8","unresolved":false,"context_lines":[{"line_number":11,"context_line":"the binaries we build for qemu-utils and tgt to make them as small as"},{"line_number":12,"context_line":"possible. This patch also starts to build gdisk as part of the build"},{"line_number":13,"context_line":"process so that we can control our sub-dependencies better, removing"},{"line_number":14,"context_line":"libraries such ICU that bloat the ramdisk."},{"line_number":15,"context_line":""},{"line_number":16,"context_line":"Change-Id: I6b8d648800284444d67a6988511707e57de1e6c9"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":5,"id":"dab17558_07ae5de5","line":14,"updated":"2016-05-11 10:59:00.000000000","message":"Nit: \"such as\"","commit_id":"ad7905d533aab87f0ae3047ebc8d4666dd60f075"},{"author":{"_account_id":10342,"name":"Jay Faulkner","display_name":"JayF","email":"jay@jvf.cc","username":"JayF","status":"youtube.com/@oss-gr / podcast.gr-oss.io"},"change_message_id":"f27d1126e0ec4fa62ea3429b0811862778ced7f9","unresolved":false,"context_lines":[{"line_number":9,"context_line":"This patch aims to reduce the TinyIPA ramdisk file size by removing"},{"line_number":10,"context_line":"some files that aren\u0027t required to run IPA once built and also striping"},{"line_number":11,"context_line":"the binaries we build for qemu-utils and tgt to make them as small as"},{"line_number":12,"context_line":"possible. This patch also starts to build gdisk as part of the build"},{"line_number":13,"context_line":"process so that we can control our sub-dependencies better, removing"},{"line_number":14,"context_line":"libraries such ICU that bloat the ramdisk."},{"line_number":15,"context_line":""}],"source_content_type":"text/x-gerrit-commit-message","patch_set":6,"id":"dab17558_74753d62","line":12,"updated":"2016-05-11 18:09:02.000000000","message":"I don\u0027t think this patch builds gdisk anymore.","commit_id":"04a358d4695d3867afb71d9b53d1f10fc0072ae7"}],"imagebuild/tinyipa/build-tinyipa.sh":[{"author":{"_account_id":14525,"name":"Vasyl Saienko","email":"vsaienko@mirantis.com","username":"vsaienko"},"change_message_id":"7f136d1aee19797931a9eb0eeb2df1928c658ead","unresolved":false,"context_lines":[{"line_number":85,"context_line":"# Build tgt"},{"line_number":86,"context_line":"rm -rf $WORKDIR/build_files/tgt.tcz"},{"line_number":87,"context_line":"$CHROOT_CMD /bin/sh -c \"cd /tmp/tgt \u0026\u0026 make \u0026\u0026 make install-programs install-conf install-scripts DESTDIR\u003d/tmp/tgt-installed\""},{"line_number":88,"context_line":"find $BUILDDIR/tmp/tgt-installed/ -type f -executable | xargs file | grep ELF | cut -d \u0027:\u0027 -f 1 | sudo xargs strip"},{"line_number":89,"context_line":"cd $WORKDIR/build_files \u0026\u0026 mksquashfs $BUILDDIR/tmp/tgt-installed tgt.tcz \u0026\u0026 md5sum tgt.tcz \u003e tgt.tcz.md5.txt"},{"line_number":90,"context_line":""},{"line_number":91,"context_line":"# Build qemu-utils"}],"source_content_type":"text/x-sh","patch_set":2,"id":"dab17558_d2f1f402","line":88,"range":{"start_line":88,"start_character":69,"end_line":88,"end_character":97},"updated":"2016-05-10 10:00:38.000000000","message":"\u0027grep ELF | cut -d \u0027:\u0027 -f 1 \u0027 may be replaced with \nawk -F \u0027:\u0027 \u0027/ELF/ {print $1}\u0027","commit_id":"f5ea573d6a6e5e283bafe257110202faa4cc0770"},{"author":{"_account_id":6637,"name":"Sam Betts","email":"sam@code-smash.net","username":"sambetts"},"change_message_id":"dee858bcd40aaa73860d785d645896579276f2ec","unresolved":false,"context_lines":[{"line_number":85,"context_line":"# Build tgt"},{"line_number":86,"context_line":"rm -rf $WORKDIR/build_files/tgt.tcz"},{"line_number":87,"context_line":"$CHROOT_CMD /bin/sh -c \"cd /tmp/tgt \u0026\u0026 make \u0026\u0026 make install-programs install-conf install-scripts DESTDIR\u003d/tmp/tgt-installed\""},{"line_number":88,"context_line":"find $BUILDDIR/tmp/tgt-installed/ -type f -executable | xargs file | grep ELF | cut -d \u0027:\u0027 -f 1 | sudo xargs strip"},{"line_number":89,"context_line":"cd $WORKDIR/build_files \u0026\u0026 mksquashfs $BUILDDIR/tmp/tgt-installed tgt.tcz \u0026\u0026 md5sum tgt.tcz \u003e tgt.tcz.md5.txt"},{"line_number":90,"context_line":""},{"line_number":91,"context_line":"# Build qemu-utils"}],"source_content_type":"text/x-sh","patch_set":2,"id":"dab17558_313dd5c4","line":88,"range":{"start_line":88,"start_character":69,"end_line":88,"end_character":97},"in_reply_to":"dab17558_d2f1f402","updated":"2016-05-10 13:08:49.000000000","message":"Done","commit_id":"f5ea573d6a6e5e283bafe257110202faa4cc0770"},{"author":{"_account_id":14525,"name":"Vasyl Saienko","email":"vsaienko@mirantis.com","username":"vsaienko"},"change_message_id":"7f136d1aee19797931a9eb0eeb2df1928c658ead","unresolved":false,"context_lines":[{"line_number":91,"context_line":"# Build qemu-utils"},{"line_number":92,"context_line":"rm -rf $WORKDIR/build_files/qemu-utils.tcz"},{"line_number":93,"context_line":"$CHROOT_CMD /bin/sh -c \"cd /tmp/qemu \u0026\u0026 ./configure --disable-system --disable-user --disable-linux-user --disable-bsd-user --disable-guest-agent --disable-blobs \u0026\u0026 make \u0026\u0026 make install DESTDIR\u003d/tmp/qemu-utils\""},{"line_number":94,"context_line":"find $BUILDDIR/tmp/qemu-utils/ -type f -executable | xargs file | grep ELF | cut -d \u0027:\u0027 -f 1 | sudo xargs strip"},{"line_number":95,"context_line":"cd $WORKDIR/build_files \u0026\u0026 mksquashfs $BUILDDIR/tmp/qemu-utils qemu-utils.tcz \u0026\u0026 md5sum qemu-utils.tcz \u003e qemu-utils.tcz.md5.txt"},{"line_number":96,"context_line":""},{"line_number":97,"context_line":"# Create qemu-utils.tcz.dep"}],"source_content_type":"text/x-sh","patch_set":2,"id":"dab17558_b2f6f0f6","line":94,"updated":"2016-05-10 10:00:38.000000000","message":"ditto","commit_id":"f5ea573d6a6e5e283bafe257110202faa4cc0770"},{"author":{"_account_id":6637,"name":"Sam Betts","email":"sam@code-smash.net","username":"sambetts"},"change_message_id":"dee858bcd40aaa73860d785d645896579276f2ec","unresolved":false,"context_lines":[{"line_number":91,"context_line":"# Build qemu-utils"},{"line_number":92,"context_line":"rm -rf $WORKDIR/build_files/qemu-utils.tcz"},{"line_number":93,"context_line":"$CHROOT_CMD /bin/sh -c \"cd /tmp/qemu \u0026\u0026 ./configure --disable-system --disable-user --disable-linux-user --disable-bsd-user --disable-guest-agent --disable-blobs \u0026\u0026 make \u0026\u0026 make install DESTDIR\u003d/tmp/qemu-utils\""},{"line_number":94,"context_line":"find $BUILDDIR/tmp/qemu-utils/ -type f -executable | xargs file | grep ELF | cut -d \u0027:\u0027 -f 1 | sudo xargs strip"},{"line_number":95,"context_line":"cd $WORKDIR/build_files \u0026\u0026 mksquashfs $BUILDDIR/tmp/qemu-utils qemu-utils.tcz \u0026\u0026 md5sum qemu-utils.tcz \u003e qemu-utils.tcz.md5.txt"},{"line_number":96,"context_line":""},{"line_number":97,"context_line":"# Create qemu-utils.tcz.dep"}],"source_content_type":"text/x-sh","patch_set":2,"id":"dab17558_7133ddf6","line":94,"in_reply_to":"dab17558_b2f6f0f6","updated":"2016-05-10 13:08:49.000000000","message":"Done","commit_id":"f5ea573d6a6e5e283bafe257110202faa4cc0770"},{"author":{"_account_id":18893,"name":"Miles Gould","email":"mgould@redhat.com","username":"pozorvlak"},"change_message_id":"a405f84a1fc3eac5d73b6f2b802c13b374e075a2","unresolved":false,"context_lines":[{"line_number":91,"context_line":"# Build qemu-utils"},{"line_number":92,"context_line":"rm -rf $WORKDIR/build_files/qemu-utils.tcz"},{"line_number":93,"context_line":"$CHROOT_CMD /bin/sh -c \"cd /tmp/qemu \u0026\u0026 ./configure --disable-system --disable-user --disable-linux-user --disable-bsd-user --disable-guest-agent --disable-blobs \u0026\u0026 make \u0026\u0026 make install DESTDIR\u003d/tmp/qemu-utils\""},{"line_number":94,"context_line":"find $BUILDDIR/tmp/qemu-utils/ -type f -executable | xargs file | awk -F \u0027:\u0027 \u0027/ELF/ {print $1}\u0027 | sudo xargs strip"},{"line_number":95,"context_line":"cd $WORKDIR/build_files \u0026\u0026 mksquashfs $BUILDDIR/tmp/qemu-utils qemu-utils.tcz \u0026\u0026 md5sum qemu-utils.tcz \u003e qemu-utils.tcz.md5.txt"},{"line_number":96,"context_line":""},{"line_number":97,"context_line":"# Create qemu-utils.tcz.dep"}],"source_content_type":"text/x-sh","patch_set":3,"id":"dab17558_125b66f7","line":94,"updated":"2016-05-10 16:09:26.000000000","message":"Is the qemu-utils directory flat? If so, you could simplify this to eg\n\nfor i in $(file /tmp/qemu-utils/* | grep ELF); do sudo strip $i; done","commit_id":"672360166393367b1ad3823444df82ec0dc4b83e"},{"author":{"_account_id":10239,"name":"Dmitry Tantsur","email":"dtantsur@protonmail.com","username":"dtantsur"},"change_message_id":"ab11872af39733b1f53b6946b45d96ae9b20911f","unresolved":false,"context_lines":[{"line_number":49,"context_line":"git clone git://git.qemu-project.org/qemu.git $BUILDDIR/tmp/qemu --depth\u003d1 --branch v2.5.0"},{"line_number":50,"context_line":""},{"line_number":51,"context_line":"# Download gdisk source"},{"line_number":52,"context_line":"( cd \"$BUILDDIR/tmp\" \u0026\u0026 wget -N http://www.rodsbooks.com/gdisk/gptfdisk-1.0.1.tar.gz )"},{"line_number":53,"context_line":""},{"line_number":54,"context_line":"# Create directory for python local mirror"},{"line_number":55,"context_line":"mkdir -p \"$BUILDDIR/tmp/localpip\""}],"source_content_type":"text/x-sh","patch_set":4,"id":"dab17558_b3e9a0b9","line":52,"updated":"2016-05-11 07:51:31.000000000","message":"thinking out load: before we can switch to tinyipa, we should make sure all these downloads are available from infra mirrors...","commit_id":"7dcc368f7f1c2ca3e197df2ed3876ef9beaa65c6"},{"author":{"_account_id":6637,"name":"Sam Betts","email":"sam@code-smash.net","username":"sambetts"},"change_message_id":"3826ffd103f8bdd5bc20dd545618049491d7ab5e","unresolved":false,"context_lines":[{"line_number":49,"context_line":"git clone git://git.qemu-project.org/qemu.git $BUILDDIR/tmp/qemu --depth\u003d1 --branch v2.5.0"},{"line_number":50,"context_line":""},{"line_number":51,"context_line":"# Download gdisk source"},{"line_number":52,"context_line":"( cd \"$BUILDDIR/tmp\" \u0026\u0026 wget -N http://www.rodsbooks.com/gdisk/gptfdisk-1.0.1.tar.gz )"},{"line_number":53,"context_line":""},{"line_number":54,"context_line":"# Create directory for python local mirror"},{"line_number":55,"context_line":"mkdir -p \"$BUILDDIR/tmp/localpip\""}],"source_content_type":"text/x-sh","patch_set":4,"id":"dab17558_783d508e","line":52,"in_reply_to":"dab17558_b3e9a0b9","updated":"2016-05-11 09:18:41.000000000","message":"Do you know how we go about doing this?","commit_id":"7dcc368f7f1c2ca3e197df2ed3876ef9beaa65c6"},{"author":{"_account_id":18893,"name":"Miles Gould","email":"mgould@redhat.com","username":"pozorvlak"},"change_message_id":"500fb2a5b78ff6974247960185dba3f4f960af3f","unresolved":false,"context_lines":[{"line_number":100,"context_line":"# Create qemu-utils.tcz.dep"},{"line_number":101,"context_line":"echo \"glib2.tcz\" \u003e qemu-utils.tcz.dep"},{"line_number":102,"context_line":""},{"line_number":103,"context_line":"# Build gdisk"},{"line_number":104,"context_line":"rm -rf $WORKDIR/build_files/gdisk.tcz"},{"line_number":105,"context_line":"cd $BUILDDIR/tmp"},{"line_number":106,"context_line":"tar -zxf gptfdisk-1.0.1.tar.gz"}],"source_content_type":"text/x-sh","patch_set":5,"id":"dab17558_d111b705","line":103,"updated":"2016-05-11 10:30:28.000000000","message":"Probably worth documenting *why* we\u0027re building gdisk from source.","commit_id":"ad7905d533aab87f0ae3047ebc8d4666dd60f075"},{"author":{"_account_id":18893,"name":"Miles Gould","email":"mgould@redhat.com","username":"pozorvlak"},"change_message_id":"46102e2771ba4b1946d904aa758bf7e38b78f3d8","unresolved":false,"context_lines":[{"line_number":100,"context_line":"# Create qemu-utils.tcz.dep"},{"line_number":101,"context_line":"echo \"glib2.tcz\" \u003e qemu-utils.tcz.dep"},{"line_number":102,"context_line":""},{"line_number":103,"context_line":"# Build gdisk"},{"line_number":104,"context_line":"rm -rf $WORKDIR/build_files/gdisk.tcz"},{"line_number":105,"context_line":"cd $BUILDDIR/tmp"},{"line_number":106,"context_line":"tar -zxf gptfdisk-1.0.1.tar.gz"}],"source_content_type":"text/x-sh","patch_set":5,"id":"dab17558_c75a95f1","line":103,"in_reply_to":"dab17558_d111b705","updated":"2016-05-11 10:59:00.000000000","message":"The bogus dependency on ICU has apparently now been removed (http://forum.tinycorelinux.net/index.php/topic,19951.0.html - thanks mat128!), but the change doesn\u0027t seem to have been backported to tinycorelinux 6.x.","commit_id":"ad7905d533aab87f0ae3047ebc8d4666dd60f075"},{"author":{"_account_id":18893,"name":"Miles Gould","email":"mgould@redhat.com","username":"pozorvlak"},"change_message_id":"500fb2a5b78ff6974247960185dba3f4f960af3f","unresolved":false,"context_lines":[{"line_number":107,"context_line":"$CHROOT_CMD /bin/sh -c \"cd /tmp/gptfdisk-1.0.1 \u0026\u0026 make\""},{"line_number":108,"context_line":"mkdir -p $BUILDDIR/tmp/gdisk/usr/local/sbin"},{"line_number":109,"context_line":"cd $BUILDDIR/tmp/gptfdisk-1.0.1/"},{"line_number":110,"context_line":"for file in $(find -type f -executable -not -path \"*.sh\"); do"},{"line_number":111,"context_line":"  strip $file"},{"line_number":112,"context_line":"  cp $file $BUILDDIR/tmp/gdisk/usr/local/sbin"},{"line_number":113,"context_line":"done"}],"source_content_type":"text/x-sh","patch_set":5,"id":"dab17558_f1dcbb47","line":110,"updated":"2016-05-11 10:30:28.000000000","message":"Do we need all the executables, or just gdisk?","commit_id":"ad7905d533aab87f0ae3047ebc8d4666dd60f075"}],"imagebuild/tinyipa/finalise-tinyipa.sh":[{"author":{"_account_id":14525,"name":"Vasyl Saienko","email":"vsaienko@mirantis.com","username":"vsaienko"},"change_message_id":"7f136d1aee19797931a9eb0eeb2df1928c658ead","unresolved":false,"context_lines":[{"line_number":97,"context_line":"find $FINALDIR/usr/local/lib/python2.7 -name \"*.py\" -not -path \"*ironic_python_agent/api/config.py\" | sudo xargs rm"},{"line_number":98,"context_line":"find $FINALDIR/usr/local/lib/python2.7 -name \"*.pyc\" | sudo xargs rm"},{"line_number":99,"context_line":""},{"line_number":100,"context_line":"find $FINALDIR -path \"*babel/locale-data/*.dat\" -not -path \"*en_US*\" | sudo xargs rm -rf"},{"line_number":101,"context_line":""},{"line_number":102,"context_line":"# Allow an extension to be added to the generated files by specifying"},{"line_number":103,"context_line":"# $BRANCH_PATH e.g. export BRANCH_PATH\u003dmaster results in tinyipa-master.gz etc"}],"source_content_type":"text/x-sh","patch_set":2,"id":"dab17558_b25b90f5","line":100,"updated":"2016-05-10 10:00:38.000000000","message":"-r option is not needed.\n\n\u0027| sudo xargs rm -rf\u0027 may be replaced with\n-exec rm -f {} \\;","commit_id":"f5ea573d6a6e5e283bafe257110202faa4cc0770"},{"author":{"_account_id":6637,"name":"Sam Betts","email":"sam@code-smash.net","username":"sambetts"},"change_message_id":"dee858bcd40aaa73860d785d645896579276f2ec","unresolved":false,"context_lines":[{"line_number":97,"context_line":"find $FINALDIR/usr/local/lib/python2.7 -name \"*.py\" -not -path \"*ironic_python_agent/api/config.py\" | sudo xargs rm"},{"line_number":98,"context_line":"find $FINALDIR/usr/local/lib/python2.7 -name \"*.pyc\" | sudo xargs rm"},{"line_number":99,"context_line":""},{"line_number":100,"context_line":"find $FINALDIR -path \"*babel/locale-data/*.dat\" -not -path \"*en_US*\" | sudo xargs rm -rf"},{"line_number":101,"context_line":""},{"line_number":102,"context_line":"# Allow an extension to be added to the generated files by specifying"},{"line_number":103,"context_line":"# $BRANCH_PATH e.g. export BRANCH_PATH\u003dmaster results in tinyipa-master.gz etc"}],"source_content_type":"text/x-sh","patch_set":2,"id":"dab17558_513059e8","line":100,"in_reply_to":"dab17558_b25b90f5","updated":"2016-05-10 13:08:49.000000000","message":"Done","commit_id":"f5ea573d6a6e5e283bafe257110202faa4cc0770"},{"author":{"_account_id":10239,"name":"Dmitry Tantsur","email":"dtantsur@protonmail.com","username":"dtantsur"},"change_message_id":"ab11872af39733b1f53b6946b45d96ae9b20911f","unresolved":false,"context_lines":[{"line_number":110,"context_line":"fi"},{"line_number":111,"context_line":""},{"line_number":112,"context_line":"# Rebuild build directory into gz file"},{"line_number":113,"context_line":"( cd \"$FINALDIR\" \u0026\u0026 sudo find | sudo cpio -o -H newc | gzip -9 \u003e \"$WORKDIR/tinyipa${branch_ext}.gz\" )"},{"line_number":114,"context_line":""},{"line_number":115,"context_line":"# Copy vmlinuz to new name"},{"line_number":116,"context_line":"cp \"$WORKDIR/build_files/vmlinuz64\" \"$WORKDIR/tinyipa${branch_ext}.vmlinuz\""}],"source_content_type":"text/x-sh","patch_set":4,"id":"dab17558_13d50ce7","line":113,"updated":"2016-05-11 07:51:31.000000000","message":"lets print here the final size, so that we can see it in gate","commit_id":"7dcc368f7f1c2ca3e197df2ed3876ef9beaa65c6"},{"author":{"_account_id":6637,"name":"Sam Betts","email":"sam@code-smash.net","username":"sambetts"},"change_message_id":"3826ffd103f8bdd5bc20dd545618049491d7ab5e","unresolved":false,"context_lines":[{"line_number":110,"context_line":"fi"},{"line_number":111,"context_line":""},{"line_number":112,"context_line":"# Rebuild build directory into gz file"},{"line_number":113,"context_line":"( cd \"$FINALDIR\" \u0026\u0026 sudo find | sudo cpio -o -H newc | gzip -9 \u003e \"$WORKDIR/tinyipa${branch_ext}.gz\" )"},{"line_number":114,"context_line":""},{"line_number":115,"context_line":"# Copy vmlinuz to new name"},{"line_number":116,"context_line":"cp \"$WORKDIR/build_files/vmlinuz64\" \"$WORKDIR/tinyipa${branch_ext}.vmlinuz\""}],"source_content_type":"text/x-sh","patch_set":4,"id":"dab17558_252a81db","line":113,"in_reply_to":"dab17558_13d50ce7","updated":"2016-05-11 09:18:41.000000000","message":"Done","commit_id":"7dcc368f7f1c2ca3e197df2ed3876ef9beaa65c6"}]}