)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":11105,"name":"Swapnil Kulkarni","email":"coolsvap@gmail.com","username":"coolsvap"},"change_message_id":"1ff0dd95cf1002c98e54ff4ad2d9bebeba2d7a88","unresolved":false,"context_lines":[{"line_number":4,"context_line":"Commit:     Christian Berendt \u003cberendt@betacloud-solutions.de\u003e"},{"line_number":5,"context_line":"CommitDate: 2016-06-25 16:59:01 +0200"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Cleanup RUN comamnds in base Dockerfile"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"This will merge several RUN statements into one RUN statement to reduce"},{"line_number":10,"context_line":"the number of the layers of the base Docker image."}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"3aaa91ec_324a8828","line":7,"updated":"2016-06-27 12:36:01.000000000","message":"commands","commit_id":"e513b01757f01f4b13b05c7f31605a1a395a59de"}],"docker/base/Dockerfile.j2":[{"author":{"_account_id":13039,"name":"Martin André","email":"m.andre@redhat.com","username":"mandre"},"change_message_id":"3b6f172c8fa0c02f255c0252d688bca48c49ff81","unresolved":false,"context_lines":[{"line_number":26,"context_line":"{% if base_distro in [\u0027fedora\u0027, \u0027centos\u0027, \u0027oraclelinux\u0027, \u0027rhel\u0027] %}"},{"line_number":27,"context_line":""},{"line_number":28,"context_line":"RUN if [ $(awk \u0027{match($0, /[0-9]+/,version)}END{print version[0]}\u0027 /etc/system-release) !\u003d \"{{ supported_distro_release }}\" ]; then \\"},{"line_number":29,"context_line":"        echo \"Only supported {{ supported_distro_release }} release on {{ base_distro }}\"; false; fi \\"},{"line_number":30,"context_line":"    \u0026\u0026 cat /tmp/kolla_bashrc \u003e\u003e /etc/bashrc"},{"line_number":31,"context_line":""},{"line_number":32,"context_line":"#### BEGIN REPO ENABLEMENT"}],"source_content_type":"text/x-jinja2","patch_set":1,"id":"3aaa91ec_9b3a7797","line":29,"updated":"2016-06-27 13:14:56.000000000","message":"This is really confusing now. I suggest moving the \u0027fi\u0027 to the line below to clearly show that we\u0027re out of the if block.","commit_id":"e513b01757f01f4b13b05c7f31605a1a395a59de"},{"author":{"_account_id":11105,"name":"Swapnil Kulkarni","email":"coolsvap@gmail.com","username":"coolsvap"},"change_message_id":"1ff0dd95cf1002c98e54ff4ad2d9bebeba2d7a88","unresolved":false,"context_lines":[{"line_number":39,"context_line":"# Turns on MariaDB repos throughout the RPM build"},{"line_number":40,"context_line":"COPY mariadb.yum.repo /etc/yum.repos.d/MariaDB.repo"},{"line_number":41,"context_line":""},{"line_number":42,"context_line":"RUN yum -y install http://repo.percona.com/release/7/RPMS/x86_64/percona-release-0.1-3.noarch.rpm \\"},{"line_number":43,"context_line":"    \u0026\u0026 rpm --import http://yum.mariadb.org/RPM-GPG-KEY-MariaDB \\"},{"line_number":44,"context_line":"    \u0026\u0026 rpm --import /etc/pki/rpm-gpg/RPM-GPG-KEY-Percona \\"},{"line_number":45,"context_line":"    \u0026\u0026 rpm --import https://packages.elastic.co/GPG-KEY-elasticsearch"}],"source_content_type":"text/x-jinja2","patch_set":1,"id":"3aaa91ec_b2b4b8d4","line":42,"updated":"2016-06-27 12:36:01.000000000","message":"put the package on the next line below install.","commit_id":"e513b01757f01f4b13b05c7f31605a1a395a59de"},{"author":{"_account_id":11105,"name":"Swapnil Kulkarni","email":"coolsvap@gmail.com","username":"coolsvap"},"change_message_id":"1ff0dd95cf1002c98e54ff4ad2d9bebeba2d7a88","unresolved":false,"context_lines":[{"line_number":81,"context_line":""},{"line_number":82,"context_line":"# Enable couple required repositories for all RHEL builds"},{"line_number":83,"context_line":"# Turn on EPEL throughout the build"},{"line_number":84,"context_line":"RUN yum -y install https://mirror.centos.org/centos-7/7/extras/x86_64/Packages/epel-release-7-5.noarch.rpm \\"},{"line_number":85,"context_line":"    \u0026\u0026 rpm --import /etc/pki/rpm-gpg/RPM-GPG-KEY-EPEL-7 \\"},{"line_number":86,"context_line":"    \u0026\u0026 yum -y install yum-plugin-priorities \\"},{"line_number":87,"context_line":"    \u0026\u0026 yum clean all \\"}],"source_content_type":"text/x-jinja2","patch_set":1,"id":"3aaa91ec_32a2889b","line":84,"updated":"2016-06-27 12:36:01.000000000","message":"put the package on the next line below install.","commit_id":"e513b01757f01f4b13b05c7f31605a1a395a59de"},{"author":{"_account_id":11105,"name":"Swapnil Kulkarni","email":"coolsvap@gmail.com","username":"coolsvap"},"change_message_id":"1ff0dd95cf1002c98e54ff4ad2d9bebeba2d7a88","unresolved":false,"context_lines":[{"line_number":83,"context_line":"# Turn on EPEL throughout the build"},{"line_number":84,"context_line":"RUN yum -y install https://mirror.centos.org/centos-7/7/extras/x86_64/Packages/epel-release-7-5.noarch.rpm \\"},{"line_number":85,"context_line":"    \u0026\u0026 rpm --import /etc/pki/rpm-gpg/RPM-GPG-KEY-EPEL-7 \\"},{"line_number":86,"context_line":"    \u0026\u0026 yum -y install yum-plugin-priorities \\"},{"line_number":87,"context_line":"    \u0026\u0026 yum clean all \\"},{"line_number":88,"context_line":"    \u0026\u0026 yum-config-manager --enable rhel-7-server-optional-rpms \\"},{"line_number":89,"context_line":"    \u0026\u0026 yum-config-manager --enable rhel-7-server-extras-rpms"}],"source_content_type":"text/x-jinja2","patch_set":1,"id":"3aaa91ec_f2976038","line":86,"updated":"2016-06-27 12:36:01.000000000","message":"put the package on the next line below install.","commit_id":"e513b01757f01f4b13b05c7f31605a1a395a59de"},{"author":{"_account_id":11105,"name":"Swapnil Kulkarni","email":"coolsvap@gmail.com","username":"coolsvap"},"change_message_id":"1ff0dd95cf1002c98e54ff4ad2d9bebeba2d7a88","unresolved":false,"context_lines":[{"line_number":109,"context_line":"    \u0026\u0026 rpm --import /etc/pki/rpm-gpg/RPM-GPG-KEY-CentOS-SIG-Storage \\"},{"line_number":110,"context_line":"    \u0026\u0026 rpm --import /etc/pki/rpm-gpg/RPM-GPG-KEY-CentOS-SIG-Virtualization \\"},{"line_number":111,"context_line":"    \u0026\u0026 yum-config-manager --enable ol7_optional_latest ol7_addons \\"},{"line_number":112,"context_line":"    \u0026\u0026 yum -y install yum-plugin-priorities \\"},{"line_number":113,"context_line":"    \u0026\u0026 yum clean all"},{"line_number":114,"context_line":""},{"line_number":115,"context_line":"    {% endif %}"}],"source_content_type":"text/x-jinja2","patch_set":1,"id":"3aaa91ec_12914c53","line":112,"updated":"2016-06-27 12:36:01.000000000","message":"put the package on the next line below install. Also the order is changed.","commit_id":"e513b01757f01f4b13b05c7f31605a1a395a59de"},{"author":{"_account_id":19316,"name":"Eduardo Gonzalez","email":"dabarren@gmail.com","username":"egonzalez90"},"change_message_id":"ddec86d2d6e60037076b4ccd960929646e618d85","unresolved":false,"context_lines":[{"line_number":136,"context_line":"RUN yum -y update \\"},{"line_number":137,"context_line":"    # Pin package versions"},{"line_number":138,"context_line":"    \u0026\u0026 yum -y install \\"},{"line_number":139,"context_line":"    \u0026\u0026 yum-plugin-versionlock \\"},{"line_number":140,"context_line":"    \u0026\u0026 yum clean all"},{"line_number":141,"context_line":"COPY versionlock.list /etc/yum/pluginconf.d/"},{"line_number":142,"context_line":""}],"source_content_type":"text/x-jinja2","patch_set":1,"id":"3aaa91ec_0ad3e9d5","line":139,"updated":"2016-06-27 06:29:11.000000000","message":"This is returning error because yum is expecting one package to install, remove doubles \u0026\u0026","commit_id":"e513b01757f01f4b13b05c7f31605a1a395a59de"},{"author":{"_account_id":11105,"name":"Swapnil Kulkarni","email":"coolsvap@gmail.com","username":"coolsvap"},"change_message_id":"1ff0dd95cf1002c98e54ff4ad2d9bebeba2d7a88","unresolved":false,"context_lines":[{"line_number":136,"context_line":"RUN yum -y update \\"},{"line_number":137,"context_line":"    # Pin package versions"},{"line_number":138,"context_line":"    \u0026\u0026 yum -y install \\"},{"line_number":139,"context_line":"    \u0026\u0026 yum-plugin-versionlock \\"},{"line_number":140,"context_line":"    \u0026\u0026 yum clean all"},{"line_number":141,"context_line":"COPY versionlock.list /etc/yum/pluginconf.d/"},{"line_number":142,"context_line":""}],"source_content_type":"text/x-jinja2","patch_set":1,"id":"3aaa91ec_52c4745e","line":139,"in_reply_to":"3aaa91ec_0ad3e9d5","updated":"2016-06-27 12:36:01.000000000","message":"Yes remove \u0026\u0026 and maintain the ordering","commit_id":"e513b01757f01f4b13b05c7f31605a1a395a59de"},{"author":{"_account_id":11105,"name":"Swapnil Kulkarni","email":"coolsvap@gmail.com","username":"coolsvap"},"change_message_id":"80c5fe7a8fd09ffb17e46ba25980c35262ae2e8d","unresolved":false,"context_lines":[{"line_number":41,"context_line":"COPY mariadb.yum.repo /etc/yum.repos.d/MariaDB.repo"},{"line_number":42,"context_line":""},{"line_number":43,"context_line":"RUN yum -y install \\"},{"line_number":44,"context_line":"           http://repo.percona.com/release/7/RPMS/x86_64/percona-release-0.1-3.noarch.rpm \\"},{"line_number":45,"context_line":"    \u0026\u0026 rpm --import http://yum.mariadb.org/RPM-GPG-KEY-MariaDB \\"},{"line_number":46,"context_line":"    \u0026\u0026 rpm --import /etc/pki/rpm-gpg/RPM-GPG-KEY-Percona \\"},{"line_number":47,"context_line":"    \u0026\u0026 rpm --import https://packages.elastic.co/GPG-KEY-elasticsearch"}],"source_content_type":"text/x-jinja2","patch_set":2,"id":"3aaa91ec_55298536","line":44,"updated":"2016-06-29 11:03:45.000000000","message":"spacing is not correct\n\nit should be like \n\nRUN yum -y install \\\n        http://repo....","commit_id":"2e77b49ac01ecf2ae7c4680e449df04b8eee33a0"},{"author":{"_account_id":11105,"name":"Swapnil Kulkarni","email":"coolsvap@gmail.com","username":"coolsvap"},"change_message_id":"80c5fe7a8fd09ffb17e46ba25980c35262ae2e8d","unresolved":false,"context_lines":[{"line_number":67,"context_line":""},{"line_number":68,"context_line":"RUN rpm --import /etc/pki/rpm-gpg/RPM-GPG-KEY-CentOS-7 \\"},{"line_number":69,"context_line":"    \u0026\u0026 yum -y install \\"},{"line_number":70,"context_line":"              epel-release \\"},{"line_number":71,"context_line":"              yum-plugin-priorities \\"},{"line_number":72,"context_line":"              centos-release-ceph-hammer \\"},{"line_number":73,"context_line":"              centos-release-qemu-ev \\"}],"source_content_type":"text/x-jinja2","patch_set":2,"id":"3aaa91ec_951bed1b","line":70,"updated":"2016-06-29 11:03:45.000000000","message":"ditto","commit_id":"2e77b49ac01ecf2ae7c4680e449df04b8eee33a0"},{"author":{"_account_id":11105,"name":"Swapnil Kulkarni","email":"coolsvap@gmail.com","username":"coolsvap"},"change_message_id":"80c5fe7a8fd09ffb17e46ba25980c35262ae2e8d","unresolved":false,"context_lines":[{"line_number":84,"context_line":"# Enable couple required repositories for all RHEL builds"},{"line_number":85,"context_line":"# Turn on EPEL throughout the build"},{"line_number":86,"context_line":"RUN yum -y install \\"},{"line_number":87,"context_line":"           https://mirror.centos.org/centos-7/7/extras/x86_64/Packages/epel-release-7-5.noarch.rpm \\"},{"line_number":88,"context_line":"    \u0026\u0026 rpm --import /etc/pki/rpm-gpg/RPM-GPG-KEY-EPEL-7 \\"},{"line_number":89,"context_line":"    \u0026\u0026 yum -y install"},{"line_number":90,"context_line":"              yum-plugin-priorities \\"}],"source_content_type":"text/x-jinja2","patch_set":2,"id":"3aaa91ec_f51a3116","line":87,"updated":"2016-06-29 11:03:45.000000000","message":"ditto","commit_id":"2e77b49ac01ecf2ae7c4680e449df04b8eee33a0"},{"author":{"_account_id":11105,"name":"Swapnil Kulkarni","email":"coolsvap@gmail.com","username":"coolsvap"},"change_message_id":"80c5fe7a8fd09ffb17e46ba25980c35262ae2e8d","unresolved":false,"context_lines":[{"line_number":87,"context_line":"           https://mirror.centos.org/centos-7/7/extras/x86_64/Packages/epel-release-7-5.noarch.rpm \\"},{"line_number":88,"context_line":"    \u0026\u0026 rpm --import /etc/pki/rpm-gpg/RPM-GPG-KEY-EPEL-7 \\"},{"line_number":89,"context_line":"    \u0026\u0026 yum -y install"},{"line_number":90,"context_line":"              yum-plugin-priorities \\"},{"line_number":91,"context_line":"    \u0026\u0026 yum clean all \\"},{"line_number":92,"context_line":"    \u0026\u0026 yum-config-manager --enable rhel-7-server-optional-rpms \\"},{"line_number":93,"context_line":"    \u0026\u0026 yum-config-manager --enable rhel-7-server-extras-rpms"}],"source_content_type":"text/x-jinja2","patch_set":2,"id":"3aaa91ec_15d6dd26","line":90,"updated":"2016-06-29 11:03:45.000000000","message":"ditto","commit_id":"2e77b49ac01ecf2ae7c4680e449df04b8eee33a0"},{"author":{"_account_id":11105,"name":"Swapnil Kulkarni","email":"coolsvap@gmail.com","username":"coolsvap"},"change_message_id":"80c5fe7a8fd09ffb17e46ba25980c35262ae2e8d","unresolved":false,"context_lines":[{"line_number":114,"context_line":"    \u0026\u0026 rpm --import /etc/pki/rpm-gpg/RPM-GPG-KEY-CentOS-SIG-Virtualization \\"},{"line_number":115,"context_line":"    \u0026\u0026 yum-config-manager --enable ol7_optional_latest ol7_addons \\"},{"line_number":116,"context_line":"    \u0026\u0026 yum -y install"},{"line_number":117,"context_line":"              yum-plugin-priorities \\"},{"line_number":118,"context_line":"    \u0026\u0026 yum clean all"},{"line_number":119,"context_line":""},{"line_number":120,"context_line":"    {% endif %}"}],"source_content_type":"text/x-jinja2","patch_set":2,"id":"3aaa91ec_f5f191ce","line":117,"updated":"2016-06-29 11:03:45.000000000","message":"ditto","commit_id":"2e77b49ac01ecf2ae7c4680e449df04b8eee33a0"},{"author":{"_account_id":11105,"name":"Swapnil Kulkarni","email":"coolsvap@gmail.com","username":"coolsvap"},"change_message_id":"80c5fe7a8fd09ffb17e46ba25980c35262ae2e8d","unresolved":false,"context_lines":[{"line_number":124,"context_line":""},{"line_number":125,"context_line":"# Set up repositories"},{"line_number":126,"context_line":"RUN yum -y install \\"},{"line_number":127,"context_line":"           https://rdo.fedorapeople.org/rdo-release.rpm \\"},{"line_number":128,"context_line":"           dnf \\"},{"line_number":129,"context_line":"           dnf-plugins-core \\"},{"line_number":130,"context_line":"           yum-plugin-priorities \\"}],"source_content_type":"text/x-jinja2","patch_set":2,"id":"3aaa91ec_15bd3dd7","line":127,"updated":"2016-06-29 11:03:45.000000000","message":"ditto","commit_id":"2e77b49ac01ecf2ae7c4680e449df04b8eee33a0"},{"author":{"_account_id":11105,"name":"Swapnil Kulkarni","email":"coolsvap@gmail.com","username":"coolsvap"},"change_message_id":"80c5fe7a8fd09ffb17e46ba25980c35262ae2e8d","unresolved":false,"context_lines":[{"line_number":141,"context_line":"RUN yum -y update \\"},{"line_number":142,"context_line":"    # Pin package versions"},{"line_number":143,"context_line":"    \u0026\u0026 yum -y install \\"},{"line_number":144,"context_line":"              yum-plugin-versionlock \\"},{"line_number":145,"context_line":"    \u0026\u0026 yum clean all"},{"line_number":146,"context_line":"COPY versionlock.list /etc/yum/pluginconf.d/"},{"line_number":147,"context_line":""}],"source_content_type":"text/x-jinja2","patch_set":2,"id":"3aaa91ec_55b7c5f4","line":144,"updated":"2016-06-29 11:03:45.000000000","message":"ditto","commit_id":"2e77b49ac01ecf2ae7c4680e449df04b8eee33a0"},{"author":{"_account_id":11105,"name":"Swapnil Kulkarni","email":"coolsvap@gmail.com","username":"coolsvap"},"change_message_id":"80c5fe7a8fd09ffb17e46ba25980c35262ae2e8d","unresolved":false,"context_lines":[{"line_number":149,"context_line":""},{"line_number":150,"context_line":"# Install base packages"},{"line_number":151,"context_line":"RUN yum -y install \\"},{"line_number":152,"context_line":"           sudo \\"},{"line_number":153,"context_line":"           which \\"},{"line_number":154,"context_line":"           python \\"},{"line_number":155,"context_line":"           lvm2 \\"}],"source_content_type":"text/x-jinja2","patch_set":2,"id":"3aaa91ec_75abe189","line":152,"updated":"2016-06-29 11:03:45.000000000","message":"ditto","commit_id":"2e77b49ac01ecf2ae7c4680e449df04b8eee33a0"},{"author":{"_account_id":11105,"name":"Swapnil Kulkarni","email":"coolsvap@gmail.com","username":"coolsvap"},"change_message_id":"80c5fe7a8fd09ffb17e46ba25980c35262ae2e8d","unresolved":false,"context_lines":[{"line_number":164,"context_line":""},{"line_number":165,"context_line":"# Update packages"},{"line_number":166,"context_line":"RUN yum -y install \\"},{"line_number":167,"context_line":"           curl \\"},{"line_number":168,"context_line":"           sudo \\"},{"line_number":169,"context_line":"           tar \\"},{"line_number":170,"context_line":"           which \\"}],"source_content_type":"text/x-jinja2","patch_set":2,"id":"3aaa91ec_f59fd15c","line":167,"updated":"2016-06-29 11:03:45.000000000","message":"ditto","commit_id":"2e77b49ac01ecf2ae7c4680e449df04b8eee33a0"}]}
