)]}'
{"docker/helm-repository/Dockerfile.j2":[{"author":{"_account_id":23928,"name":"Pete Birley","email":"petebirley@gmail.com","username":"portdirect"},"change_message_id":"2486d96d79ea35665f372bc39de48d677b4ac89b","unresolved":false,"context_lines":[{"line_number":3,"context_line":""},{"line_number":4,"context_line":"{% block helm_repository_header %}{% endblock %}"},{"line_number":5,"context_line":""},{"line_number":6,"context_line":"RUN curl -Lo /tmp/helm-v2.1.3-linux-amd64.tar.gz  http://storage.googleapis.com/kubernetes-helm/helm-v2.1.3-linux-amd64.tar.gz \\"},{"line_number":7,"context_line":"    \u0026\u0026 sudo tar --strip-components 1 -C /usr/bin linux-amd64/helm -zxvf /tmp/helm-v2.1.3-linux-amd64.tar.gz \\"},{"line_number":8,"context_line":"    \u0026\u0026 chmod 755 /usr/bin/helm"},{"line_number":9,"context_line":""}],"source_content_type":"text/x-jinja2","patch_set":1,"id":"ba5201f7_5064bab7","line":6,"range":{"start_line":6,"start_character":23,"end_line":6,"end_character":29},"updated":"2017-01-06 15:47:03.000000000","message":"Could you change this to an environment variable? as otherwise it will be harder to stay on top of helm versions.","commit_id":"d2e33620905202e8227d0272bf8f6da0ee01df68"},{"author":{"_account_id":19384,"name":"Serguei Bezverkhi","email":"sbezverk@cisco.com","username":"sbezverk"},"change_message_id":"fcbe74f63d4eaf0d02a8d3a0123c3e8d78983240","unresolved":false,"context_lines":[{"line_number":3,"context_line":""},{"line_number":4,"context_line":"{% block helm_repository_header %}{% endblock %}"},{"line_number":5,"context_line":""},{"line_number":6,"context_line":"RUN curl -Lo /tmp/helm-v2.1.3-linux-amd64.tar.gz  http://storage.googleapis.com/kubernetes-helm/helm-v2.1.3-linux-amd64.tar.gz \\"},{"line_number":7,"context_line":"    \u0026\u0026 sudo tar --strip-components 1 -C /usr/bin linux-amd64/helm -zxvf /tmp/helm-v2.1.3-linux-amd64.tar.gz \\"},{"line_number":8,"context_line":"    \u0026\u0026 chmod 755 /usr/bin/helm"},{"line_number":9,"context_line":""}],"source_content_type":"text/x-jinja2","patch_set":1,"id":"ba5201f7_f0f04ecb","line":6,"range":{"start_line":6,"start_character":23,"end_line":6,"end_character":29},"in_reply_to":"ba5201f7_5064bab7","updated":"2017-01-06 15:50:37.000000000","message":"Well, the problem here is we do not have any way to set env variable at the time when this container gets build. So it is either change version in URL, or change the version variable in the same file, I do not see much of a difference frankly.","commit_id":"d2e33620905202e8227d0272bf8f6da0ee01df68"},{"author":{"_account_id":10787,"name":"Michal Jastrzebski (inc0)","email":"michal.jastrzebski@intel.com","username":"inc0"},"change_message_id":"58e6f985b3225f62e1433f5ff2f1e607520d6099","unresolved":false,"context_lines":[{"line_number":1,"context_line":"FROM {{ namespace }}/{{ image_prefix }}base:{{ tag }}"},{"line_number":2,"context_line":"MAINTAINER {{ maintainer }}"},{"line_number":3,"context_line":""},{"line_number":4,"context_line":"ENV helm_version 2.1.3"},{"line_number":5,"context_line":""},{"line_number":6,"context_line":"{% block helm_repository_header %}{% endblock %}"},{"line_number":7,"context_line":""}],"source_content_type":"text/x-jinja2","patch_set":2,"id":"ba5201f7_8525fb5c","line":4,"updated":"2017-01-06 20:06:34.000000000","message":"{% block helm_repository_version %}\nENV helm_version 2.1.3\n{% endblock %}\n\nthis will allow you to easily override it","commit_id":"b3293a0f6a239c964399097f30afd8797b6254f0"},{"author":{"_account_id":10787,"name":"Michal Jastrzebski (inc0)","email":"michal.jastrzebski@intel.com","username":"inc0"},"change_message_id":"58e6f985b3225f62e1433f5ff2f1e607520d6099","unresolved":false,"context_lines":[{"line_number":5,"context_line":""},{"line_number":6,"context_line":"{% block helm_repository_header %}{% endblock %}"},{"line_number":7,"context_line":""},{"line_number":8,"context_line":"RUN curl -Lo /tmp/helm-v2.1.3-linux-amd64.tar.gz  http://storage.googleapis.com/kubernetes-helm/helm-v$helm_version-linux-amd64.tar.gz \\"},{"line_number":9,"context_line":"    \u0026\u0026 sudo tar --strip-components 1 -C /usr/bin linux-amd64/helm -zxvf /tmp/helm-v2.1.3-linux-amd64.tar.gz \\"},{"line_number":10,"context_line":"    \u0026\u0026 chmod 755 /usr/bin/helm \\"},{"line_number":11,"context_line":"    \u0026\u0026 sudo mkdir /helm-repo \\"}],"source_content_type":"text/x-jinja2","patch_set":2,"id":"ba5201f7_c5238354","line":8,"updated":"2017-01-06 20:06:34.000000000","message":"I assume you\u0027ll use env here right?","commit_id":"b3293a0f6a239c964399097f30afd8797b6254f0"},{"author":{"_account_id":23928,"name":"Pete Birley","email":"petebirley@gmail.com","username":"portdirect"},"change_message_id":"8326b0863108dd8ea143b5d4838af8854e1b172c","unresolved":false,"context_lines":[{"line_number":5,"context_line":""},{"line_number":6,"context_line":"{% block helm_repository_header %}{% endblock %}"},{"line_number":7,"context_line":""},{"line_number":8,"context_line":"RUN curl -Lo /tmp/helm-v2.1.3-linux-amd64.tar.gz  http://storage.googleapis.com/kubernetes-helm/helm-v$helm_version-linux-amd64.tar.gz \\"},{"line_number":9,"context_line":"    \u0026\u0026 sudo tar --strip-components 1 -C /usr/bin linux-amd64/helm -zxvf /tmp/helm-v2.1.3-linux-amd64.tar.gz \\"},{"line_number":10,"context_line":"    \u0026\u0026 chmod 755 /usr/bin/helm \\"},{"line_number":11,"context_line":"    \u0026\u0026 sudo mkdir /helm-repo \\"}],"source_content_type":"text/x-jinja2","patch_set":2,"id":"ba5201f7_e5e5e708","line":8,"range":{"start_line":8,"start_character":23,"end_line":8,"end_character":29},"updated":"2017-01-06 20:09:27.000000000","message":"would be best to either use the env-var here or not version the downloaded tar.","commit_id":"b3293a0f6a239c964399097f30afd8797b6254f0"},{"author":{"_account_id":19384,"name":"Serguei Bezverkhi","email":"sbezverk@cisco.com","username":"sbezverk"},"change_message_id":"33b012f9a944af4c27b06e0852db779118dd44fd","unresolved":false,"context_lines":[{"line_number":5,"context_line":""},{"line_number":6,"context_line":"{% block helm_repository_header %}{% endblock %}"},{"line_number":7,"context_line":""},{"line_number":8,"context_line":"RUN curl -Lo /tmp/helm-v2.1.3-linux-amd64.tar.gz  http://storage.googleapis.com/kubernetes-helm/helm-v$helm_version-linux-amd64.tar.gz \\"},{"line_number":9,"context_line":"    \u0026\u0026 sudo tar --strip-components 1 -C /usr/bin linux-amd64/helm -zxvf /tmp/helm-v2.1.3-linux-amd64.tar.gz \\"},{"line_number":10,"context_line":"    \u0026\u0026 chmod 755 /usr/bin/helm \\"},{"line_number":11,"context_line":"    \u0026\u0026 sudo mkdir /helm-repo \\"}],"source_content_type":"text/x-jinja2","patch_set":2,"id":"ba5201f7_a59bdf64","line":8,"in_reply_to":"ba5201f7_c5238354","updated":"2017-01-06 20:12:46.000000000","message":"Done","commit_id":"b3293a0f6a239c964399097f30afd8797b6254f0"},{"author":{"_account_id":23928,"name":"Pete Birley","email":"petebirley@gmail.com","username":"portdirect"},"change_message_id":"8326b0863108dd8ea143b5d4838af8854e1b172c","unresolved":false,"context_lines":[{"line_number":6,"context_line":"{% block helm_repository_header %}{% endblock %}"},{"line_number":7,"context_line":""},{"line_number":8,"context_line":"RUN curl -Lo /tmp/helm-v2.1.3-linux-amd64.tar.gz  http://storage.googleapis.com/kubernetes-helm/helm-v$helm_version-linux-amd64.tar.gz \\"},{"line_number":9,"context_line":"    \u0026\u0026 sudo tar --strip-components 1 -C /usr/bin linux-amd64/helm -zxvf /tmp/helm-v2.1.3-linux-amd64.tar.gz \\"},{"line_number":10,"context_line":"    \u0026\u0026 chmod 755 /usr/bin/helm \\"},{"line_number":11,"context_line":"    \u0026\u0026 sudo mkdir /helm-repo \\"},{"line_number":12,"context_line":"    \u0026\u0026 sudo chmod 766 /helm-repo"}],"source_content_type":"text/x-jinja2","patch_set":2,"id":"ba5201f7_45bdd3d6","line":9,"range":{"start_line":9,"start_character":82,"end_line":9,"end_character":88},"updated":"2017-01-06 20:09:27.000000000","message":"ditto","commit_id":"b3293a0f6a239c964399097f30afd8797b6254f0"},{"author":{"_account_id":19384,"name":"Serguei Bezverkhi","email":"sbezverk@cisco.com","username":"sbezverk"},"change_message_id":"33b012f9a944af4c27b06e0852db779118dd44fd","unresolved":false,"context_lines":[{"line_number":6,"context_line":"{% block helm_repository_header %}{% endblock %}"},{"line_number":7,"context_line":""},{"line_number":8,"context_line":"RUN curl -Lo /tmp/helm-v2.1.3-linux-amd64.tar.gz  http://storage.googleapis.com/kubernetes-helm/helm-v$helm_version-linux-amd64.tar.gz \\"},{"line_number":9,"context_line":"    \u0026\u0026 sudo tar --strip-components 1 -C /usr/bin linux-amd64/helm -zxvf /tmp/helm-v2.1.3-linux-amd64.tar.gz \\"},{"line_number":10,"context_line":"    \u0026\u0026 chmod 755 /usr/bin/helm \\"},{"line_number":11,"context_line":"    \u0026\u0026 sudo mkdir /helm-repo \\"},{"line_number":12,"context_line":"    \u0026\u0026 sudo chmod 766 /helm-repo"}],"source_content_type":"text/x-jinja2","patch_set":2,"id":"ba5201f7_85989b6b","line":9,"range":{"start_line":9,"start_character":82,"end_line":9,"end_character":88},"in_reply_to":"ba5201f7_45bdd3d6","updated":"2017-01-06 20:12:46.000000000","message":"Done","commit_id":"b3293a0f6a239c964399097f30afd8797b6254f0"},{"author":{"_account_id":23928,"name":"Pete Birley","email":"petebirley@gmail.com","username":"portdirect"},"change_message_id":"8326b0863108dd8ea143b5d4838af8854e1b172c","unresolved":false,"context_lines":[{"line_number":11,"context_line":"    \u0026\u0026 sudo mkdir /helm-repo \\"},{"line_number":12,"context_line":"    \u0026\u0026 sudo chmod 766 /helm-repo"},{"line_number":13,"context_line":""},{"line_number":14,"context_line":"RUN helm init --client-only"},{"line_number":15,"context_line":""},{"line_number":16,"context_line":"{% block helm_repository_footer %}{% endblock %}"},{"line_number":17,"context_line":"{% block footer %}{% endblock %}"}],"source_content_type":"text/x-jinja2","patch_set":2,"id":"ba5201f7_25c36f58","line":14,"range":{"start_line":14,"start_character":4,"end_line":14,"end_character":27},"updated":"2017-01-06 20:09:27.000000000","message":"would it not make sense to do this on the container starting if using with some PV?","commit_id":"b3293a0f6a239c964399097f30afd8797b6254f0"},{"author":{"_account_id":19384,"name":"Serguei Bezverkhi","email":"sbezverk@cisco.com","username":"sbezverk"},"change_message_id":"33b012f9a944af4c27b06e0852db779118dd44fd","unresolved":false,"context_lines":[{"line_number":11,"context_line":"    \u0026\u0026 sudo mkdir /helm-repo \\"},{"line_number":12,"context_line":"    \u0026\u0026 sudo chmod 766 /helm-repo"},{"line_number":13,"context_line":""},{"line_number":14,"context_line":"RUN helm init --client-only"},{"line_number":15,"context_line":""},{"line_number":16,"context_line":"{% block helm_repository_footer %}{% endblock %}"},{"line_number":17,"context_line":"{% block footer %}{% endblock %}"}],"source_content_type":"text/x-jinja2","patch_set":2,"id":"ba5201f7_65957731","line":14,"range":{"start_line":14,"start_character":4,"end_line":14,"end_character":27},"in_reply_to":"ba5201f7_25c36f58","updated":"2017-01-06 20:12:46.000000000","message":"nope because it just generate some files without any dependency on the running environment.","commit_id":"b3293a0f6a239c964399097f30afd8797b6254f0"},{"author":{"_account_id":23928,"name":"Pete Birley","email":"petebirley@gmail.com","username":"portdirect"},"change_message_id":"2a6ceb4df09d2d13bd6ec42c1d1de7870a0d36af","unresolved":false,"context_lines":[{"line_number":11,"context_line":"    \u0026\u0026 sudo mkdir /helm-repo \\"},{"line_number":12,"context_line":"    \u0026\u0026 sudo chmod 766 /helm-repo"},{"line_number":13,"context_line":""},{"line_number":14,"context_line":"RUN helm init --client-only"},{"line_number":15,"context_line":""},{"line_number":16,"context_line":"{% block helm_repository_footer %}{% endblock %}"},{"line_number":17,"context_line":"{% block footer %}{% endblock %}"}],"source_content_type":"text/x-jinja2","patch_set":2,"id":"ba5201f7_05e58ba6","line":14,"range":{"start_line":14,"start_character":4,"end_line":14,"end_character":27},"in_reply_to":"ba5201f7_65957731","updated":"2017-01-06 20:26:17.000000000","message":"Yup but why setup the client side of helm at all if this container is just going to be a repo server?","commit_id":"b3293a0f6a239c964399097f30afd8797b6254f0"},{"author":{"_account_id":2834,"name":"Steven Dake","email":"steven.dake@gmail.com","username":"sdake"},"change_message_id":"055f2430a15fb0e16c958ad49409ce97b08f062e","unresolved":false,"context_lines":[{"line_number":37,"context_line":"    \u0026\u0026 pip install pyyaml"},{"line_number":38,"context_line":""},{"line_number":39,"context_line":"RUN curl -Lo /tmp/helm-v$helm_version-linux-amd64.tar.gz  http://storage.googleapis.com/kubernetes-helm/helm-v$helm_version-linux-amd64.tar.gz \\"},{"line_number":40,"context_line":"    \u0026\u0026 sudo tar --strip-components 1 -C /usr/bin linux-amd64/helm -zxvf /tmp/helm-v$helm_version-linux-amd64.tar.gz \\"},{"line_number":41,"context_line":"    \u0026\u0026 chmod 755 /usr/bin/helm \\"},{"line_number":42,"context_line":"    \u0026\u0026 sudo mkdir /helm-repo \\"},{"line_number":43,"context_line":"    \u0026\u0026 sudo chmod 766 /helm-repo"}],"source_content_type":"text/x-jinja2","patch_set":9,"id":"ba5201f7_74cfd955","line":40,"updated":"2017-01-07 17:53:15.000000000","message":"I think you want $(helm_version) here to be more clear.","commit_id":"8545b876062fbfe0117b9a514b2d3b502e1e6f10"},{"author":{"_account_id":23928,"name":"Pete Birley","email":"petebirley@gmail.com","username":"portdirect"},"change_message_id":"e91a497d4ce83fa6f9b10872936f76055181a27c","unresolved":false,"context_lines":[{"line_number":37,"context_line":"    \u0026\u0026 pip install pyyaml"},{"line_number":38,"context_line":""},{"line_number":39,"context_line":"RUN curl -Lo /tmp/helm-v$helm_version-linux-amd64.tar.gz  http://storage.googleapis.com/kubernetes-helm/helm-v$helm_version-linux-amd64.tar.gz \\"},{"line_number":40,"context_line":"    \u0026\u0026 sudo tar --strip-components 1 -C /usr/bin linux-amd64/helm -zxvf /tmp/helm-v$helm_version-linux-amd64.tar.gz \\"},{"line_number":41,"context_line":"    \u0026\u0026 chmod 755 /usr/bin/helm \\"},{"line_number":42,"context_line":"    \u0026\u0026 sudo mkdir /helm-repo \\"},{"line_number":43,"context_line":"    \u0026\u0026 sudo chmod 766 /helm-repo"}],"source_content_type":"text/x-jinja2","patch_set":9,"id":"ba5201f7_8f3d0a2d","line":40,"in_reply_to":"ba5201f7_74cfd955","updated":"2017-01-07 20:41:26.000000000","message":"I agree with Steve, ${helm_version} would be better","commit_id":"8545b876062fbfe0117b9a514b2d3b502e1e6f10"},{"author":{"_account_id":2834,"name":"Steven Dake","email":"steven.dake@gmail.com","username":"sdake"},"change_message_id":"7d8e44e76bc2a588a431e2a11107b05d03d0f2e4","unresolved":false,"context_lines":[{"line_number":38,"context_line":""},{"line_number":39,"context_line":"RUN curl -Lo /tmp/helm-v$helm_version-linux-amd64.tar.gz  http://storage.googleapis.com/kubernetes-helm/helm-v$helm_version-linux-amd64.tar.gz \\"},{"line_number":40,"context_line":"    \u0026\u0026 sudo tar --strip-components 1 -C /usr/bin linux-amd64/helm -zxvf /tmp/helm-v$helm_version-linux-amd64.tar.gz \\"},{"line_number":41,"context_line":"    \u0026\u0026 chmod 755 /usr/bin/helm \\"},{"line_number":42,"context_line":"    \u0026\u0026 sudo mkdir /helm-repo \\"},{"line_number":43,"context_line":"    \u0026\u0026 sudo chmod 766 /helm-repo"},{"line_number":44,"context_line":""}],"source_content_type":"text/x-jinja2","patch_set":9,"id":"ba5201f7_94c35d37","line":41,"updated":"2017-01-07 17:51:37.000000000","message":"this should be sudo chmod","commit_id":"8545b876062fbfe0117b9a514b2d3b502e1e6f10"},{"author":{"_account_id":2834,"name":"Steven Dake","email":"steven.dake@gmail.com","username":"sdake"},"change_message_id":"7d8e44e76bc2a588a431e2a11107b05d03d0f2e4","unresolved":false,"context_lines":[{"line_number":40,"context_line":"    \u0026\u0026 sudo tar --strip-components 1 -C /usr/bin linux-amd64/helm -zxvf /tmp/helm-v$helm_version-linux-amd64.tar.gz \\"},{"line_number":41,"context_line":"    \u0026\u0026 chmod 755 /usr/bin/helm \\"},{"line_number":42,"context_line":"    \u0026\u0026 sudo mkdir /helm-repo \\"},{"line_number":43,"context_line":"    \u0026\u0026 sudo chmod 766 /helm-repo"},{"line_number":44,"context_line":""},{"line_number":45,"context_line":"RUN helm init --client-only"},{"line_number":46,"context_line":""}],"source_content_type":"text/x-jinja2","patch_set":9,"id":"ba5201f7_b4bea1ab","line":43,"updated":"2017-01-07 17:51:37.000000000","message":"This won\u0027t work without a change to the sudoers file.","commit_id":"8545b876062fbfe0117b9a514b2d3b502e1e6f10"},{"author":{"_account_id":2834,"name":"Steven Dake","email":"steven.dake@gmail.com","username":"sdake"},"change_message_id":"7d8e44e76bc2a588a431e2a11107b05d03d0f2e4","unresolved":false,"context_lines":[{"line_number":46,"context_line":""},{"line_number":47,"context_line":"{% block helm_repository_footer %}{% endblock %}"},{"line_number":48,"context_line":"{% block footer %}{% endblock %}"},{"line_number":49,"context_line":"CMD helm serve --address 0.0.0.0:8879 --repo-path /helm-repo"}],"source_content_type":"text/x-jinja2","patch_set":9,"id":"ba5201f7_14e68de7","line":49,"updated":"2017-01-07 17:51:37.000000000","message":"ideally this should be run as the helm user.  the helm user should be useradd\u0027ed.  A refactor of this has been going on elsewhere in the codebase - grep for useradd.\n\nUse USER helm","commit_id":"8545b876062fbfe0117b9a514b2d3b502e1e6f10"},{"author":{"_account_id":23928,"name":"Pete Birley","email":"petebirley@gmail.com","username":"portdirect"},"change_message_id":"e91a497d4ce83fa6f9b10872936f76055181a27c","unresolved":false,"context_lines":[{"line_number":46,"context_line":""},{"line_number":47,"context_line":"{% block helm_repository_footer %}{% endblock %}"},{"line_number":48,"context_line":"{% block footer %}{% endblock %}"},{"line_number":49,"context_line":"CMD helm serve --address 0.0.0.0:8879 --repo-path /helm-repo"}],"source_content_type":"text/x-jinja2","patch_set":9,"id":"ba5201f7_af480ecd","line":49,"in_reply_to":"ba5201f7_14e68de7","updated":"2017-01-07 20:41:26.000000000","message":"If possible this would be a really worthwhile improvement.","commit_id":"8545b876062fbfe0117b9a514b2d3b502e1e6f10"},{"author":{"_account_id":2834,"name":"Steven Dake","email":"steven.dake@gmail.com","username":"sdake"},"change_message_id":"47df47c4fa1369743778f217a33d5b5718a46031","unresolved":false,"context_lines":[{"line_number":38,"context_line":""},{"line_number":39,"context_line":"RUN curl -Lo /tmp/helm-v$helm_version-linux-amd64.tar.gz  http://storage.googleapis.com/kubernetes-helm/helm-v$helm_version-linux-amd64.tar.gz \\"},{"line_number":40,"context_line":"    \u0026\u0026 sudo tar --strip-components 1 -C /usr/bin linux-amd64/helm -zxvf /tmp/helm-v$helm_version-linux-amd64.tar.gz \\"},{"line_number":41,"context_line":"\t\u0026\u0026 sudo chmod 755 /usr/bin/helm"},{"line_number":42,"context_line":"\t"},{"line_number":43,"context_line":"RUN useradd --user-group helm \\"},{"line_number":44,"context_line":"    \u0026\u0026 sudo mkdir /helm-repo \\"}],"source_content_type":"text/x-jinja2","patch_set":11,"id":"ba5201f7_ea7be4f2","line":41,"updated":"2017-01-07 23:58:12.000000000","message":"use spaces not tabs","commit_id":"9d7eb27dbb08fff06d598c9b286e68bcccf98bd0"},{"author":{"_account_id":9237,"name":"Kevin Fox","email":"kevin@efox.cc","username":"kfox1111"},"change_message_id":"c9355d38c330f9dcd69d27129f0d6626e1052c6d","unresolved":false,"context_lines":[{"line_number":38,"context_line":""},{"line_number":39,"context_line":"RUN curl -Lo /tmp/helm-v$helm_version-linux-amd64.tar.gz  http://storage.googleapis.com/kubernetes-helm/helm-v$helm_version-linux-amd64.tar.gz \\"},{"line_number":40,"context_line":"    \u0026\u0026 sudo tar --strip-components 1 -C /usr/bin linux-amd64/helm -zxvf /tmp/helm-v$helm_version-linux-amd64.tar.gz \\"},{"line_number":41,"context_line":"\t\u0026\u0026 sudo chmod 755 /usr/bin/helm"},{"line_number":42,"context_line":"\t"},{"line_number":43,"context_line":"RUN useradd --user-group helm \\"},{"line_number":44,"context_line":"    \u0026\u0026 sudo mkdir /helm-repo \\"}],"source_content_type":"text/x-jinja2","patch_set":11,"id":"ba5201f7_6a70d40b","line":41,"updated":"2017-01-07 23:58:52.000000000","message":"ws issues","commit_id":"9d7eb27dbb08fff06d598c9b286e68bcccf98bd0"},{"author":{"_account_id":2834,"name":"Steven Dake","email":"steven.dake@gmail.com","username":"sdake"},"change_message_id":"47df47c4fa1369743778f217a33d5b5718a46031","unresolved":false,"context_lines":[{"line_number":39,"context_line":"RUN curl -Lo /tmp/helm-v$helm_version-linux-amd64.tar.gz  http://storage.googleapis.com/kubernetes-helm/helm-v$helm_version-linux-amd64.tar.gz \\"},{"line_number":40,"context_line":"    \u0026\u0026 sudo tar --strip-components 1 -C /usr/bin linux-amd64/helm -zxvf /tmp/helm-v$helm_version-linux-amd64.tar.gz \\"},{"line_number":41,"context_line":"\t\u0026\u0026 sudo chmod 755 /usr/bin/helm"},{"line_number":42,"context_line":"\t"},{"line_number":43,"context_line":"RUN useradd --user-group helm \\"},{"line_number":44,"context_line":"    \u0026\u0026 sudo mkdir /helm-repo \\"},{"line_number":45,"context_line":"    \u0026\u0026 sudo chown -R helm: /helm-repo "}],"source_content_type":"text/x-jinja2","patch_set":11,"id":"ba5201f7_0a7f48e0","line":42,"updated":"2017-01-07 23:58:12.000000000","message":"whitespace","commit_id":"9d7eb27dbb08fff06d598c9b286e68bcccf98bd0"},{"author":{"_account_id":9237,"name":"Kevin Fox","email":"kevin@efox.cc","username":"kfox1111"},"change_message_id":"c9355d38c330f9dcd69d27129f0d6626e1052c6d","unresolved":false,"context_lines":[{"line_number":40,"context_line":"    \u0026\u0026 sudo tar --strip-components 1 -C /usr/bin linux-amd64/helm -zxvf /tmp/helm-v$helm_version-linux-amd64.tar.gz \\"},{"line_number":41,"context_line":"\t\u0026\u0026 sudo chmod 755 /usr/bin/helm"},{"line_number":42,"context_line":"\t"},{"line_number":43,"context_line":"RUN useradd --user-group helm \\"},{"line_number":44,"context_line":"    \u0026\u0026 sudo mkdir /helm-repo \\"},{"line_number":45,"context_line":"    \u0026\u0026 sudo chown -R helm: /helm-repo "},{"line_number":46,"context_line":""}],"source_content_type":"text/x-jinja2","patch_set":11,"id":"ba5201f7_aa763c03","line":43,"updated":"2017-01-07 23:58:52.000000000","message":"probably should specify the uid/gid. otherwise, they can move around on the pvc.","commit_id":"9d7eb27dbb08fff06d598c9b286e68bcccf98bd0"},{"author":{"_account_id":2834,"name":"Steven Dake","email":"steven.dake@gmail.com","username":"sdake"},"change_message_id":"47df47c4fa1369743778f217a33d5b5718a46031","unresolved":false,"context_lines":[{"line_number":42,"context_line":"\t"},{"line_number":43,"context_line":"RUN useradd --user-group helm \\"},{"line_number":44,"context_line":"    \u0026\u0026 sudo mkdir /helm-repo \\"},{"line_number":45,"context_line":"    \u0026\u0026 sudo chown -R helm: /helm-repo "},{"line_number":46,"context_line":""},{"line_number":47,"context_line":"USER helm"},{"line_number":48,"context_line":""}],"source_content_type":"text/x-jinja2","patch_set":11,"id":"ba5201f7_2a824cc8","line":45,"range":{"start_line":45,"start_character":37,"end_line":45,"end_character":38},"updated":"2017-01-07 23:58:12.000000000","message":"whitespace","commit_id":"9d7eb27dbb08fff06d598c9b286e68bcccf98bd0"},{"author":{"_account_id":9237,"name":"Kevin Fox","email":"kevin@efox.cc","username":"kfox1111"},"change_message_id":"c9355d38c330f9dcd69d27129f0d6626e1052c6d","unresolved":false,"context_lines":[{"line_number":48,"context_line":""},{"line_number":49,"context_line":"RUN helm init --client-only"},{"line_number":50,"context_line":""},{"line_number":51,"context_line":"CMD helm serve --address 0.0.0.0:8879 --repo-path /helm-repo"},{"line_number":52,"context_line":""},{"line_number":53,"context_line":"{% block helm_repository_footer %}{% endblock %}"},{"line_number":54,"context_line":"{% block footer %}{% endblock %}"}],"source_content_type":"text/x-jinja2","patch_set":11,"id":"ba5201f7_4a85d0cc","line":51,"range":{"start_line":51,"start_character":33,"end_line":51,"end_character":37},"updated":"2017-01-07 23:58:52.000000000","message":"I\u0027d recommend making this an env var so it can be overridden.","commit_id":"9d7eb27dbb08fff06d598c9b286e68bcccf98bd0"},{"author":{"_account_id":7488,"name":"Jeffrey Zhang","email":"zhang.lei.fly@gmail.com","username":"Jeffrey4l"},"change_message_id":"aba4b2437e786dd6dc4a087819c6e17a6abddc70","unresolved":false,"context_lines":[{"line_number":35,"context_line":""},{"line_number":36,"context_line":"RUN pip install -U pip \\"},{"line_number":37,"context_line":"    \u0026\u0026 pip install setuptools \\"},{"line_number":38,"context_line":"    \u0026\u0026 pip install pyyaml"},{"line_number":39,"context_line":""},{"line_number":40,"context_line":"RUN curl -Lo /tmp/helm-v${helm_version}-linux-amd64.tar.gz  http://storage.googleapis.com/kubernetes-helm/helm-v${helm_version}-linux-amd64.tar.gz \\"},{"line_number":41,"context_line":"    \u0026\u0026 sudo tar --strip-components 1 -C /usr/bin linux-amd64/helm -zxvf /tmp/helm-v${helm_version}-linux-amd64.tar.gz \\"}],"source_content_type":"text/x-jinja2","patch_set":13,"id":"ba5201f7_26cc44ba","line":38,"updated":"2017-01-08 07:14:38.000000000","message":"it is a bad idea to use pip install pytho-pip. if u wanna use latest pypi, try to use\n\nRUN curl https://bootstrap.pypa.io/get-pip.py -o get-pip.py \\\n    \u0026\u0026 python get-pip.py \\\n    \u0026\u0026 rm get-pip.py","commit_id":"bb7668ebd720d65a2a0db8ad9eb5d2a240704239"},{"author":{"_account_id":19384,"name":"Serguei Bezverkhi","email":"sbezverk@cisco.com","username":"sbezverk"},"change_message_id":"cf63eb1ca1bed2db78966294bc2c169469fead3a","unresolved":false,"context_lines":[{"line_number":35,"context_line":""},{"line_number":36,"context_line":"RUN pip install -U pip \\"},{"line_number":37,"context_line":"    \u0026\u0026 pip install setuptools \\"},{"line_number":38,"context_line":"    \u0026\u0026 pip install pyyaml"},{"line_number":39,"context_line":""},{"line_number":40,"context_line":"RUN curl -Lo /tmp/helm-v${helm_version}-linux-amd64.tar.gz  http://storage.googleapis.com/kubernetes-helm/helm-v${helm_version}-linux-amd64.tar.gz \\"},{"line_number":41,"context_line":"    \u0026\u0026 sudo tar --strip-components 1 -C /usr/bin linux-amd64/helm -zxvf /tmp/helm-v${helm_version}-linux-amd64.tar.gz \\"}],"source_content_type":"text/x-jinja2","patch_set":13,"id":"ba5201f7_0c0e81e4","line":38,"in_reply_to":"ba5201f7_26cc44ba","updated":"2017-01-08 12:56:02.000000000","message":"Done","commit_id":"bb7668ebd720d65a2a0db8ad9eb5d2a240704239"},{"author":{"_account_id":7488,"name":"Jeffrey Zhang","email":"zhang.lei.fly@gmail.com","username":"Jeffrey4l"},"change_message_id":"aba4b2437e786dd6dc4a087819c6e17a6abddc70","unresolved":false,"context_lines":[{"line_number":39,"context_line":""},{"line_number":40,"context_line":"RUN curl -Lo /tmp/helm-v${helm_version}-linux-amd64.tar.gz  http://storage.googleapis.com/kubernetes-helm/helm-v${helm_version}-linux-amd64.tar.gz \\"},{"line_number":41,"context_line":"    \u0026\u0026 sudo tar --strip-components 1 -C /usr/bin linux-amd64/helm -zxvf /tmp/helm-v${helm_version}-linux-amd64.tar.gz \\"},{"line_number":42,"context_line":"    \u0026\u0026 sudo chmod 755 /usr/bin/helm"},{"line_number":43,"context_line":""},{"line_number":44,"context_line":"RUN useradd --user-group --create-home --home-dir /home/helm helm \\"},{"line_number":45,"context_line":"    \u0026\u0026 sudo mkdir /helm-repo \\"}],"source_content_type":"text/x-jinja2","patch_set":13,"id":"ba5201f7_46c78895","line":42,"updated":"2017-01-08 07:14:38.000000000","message":"rm /tmp/helm-vxxx file.","commit_id":"bb7668ebd720d65a2a0db8ad9eb5d2a240704239"},{"author":{"_account_id":19384,"name":"Serguei Bezverkhi","email":"sbezverk@cisco.com","username":"sbezverk"},"change_message_id":"cf63eb1ca1bed2db78966294bc2c169469fead3a","unresolved":false,"context_lines":[{"line_number":39,"context_line":""},{"line_number":40,"context_line":"RUN curl -Lo /tmp/helm-v${helm_version}-linux-amd64.tar.gz  http://storage.googleapis.com/kubernetes-helm/helm-v${helm_version}-linux-amd64.tar.gz \\"},{"line_number":41,"context_line":"    \u0026\u0026 sudo tar --strip-components 1 -C /usr/bin linux-amd64/helm -zxvf /tmp/helm-v${helm_version}-linux-amd64.tar.gz \\"},{"line_number":42,"context_line":"    \u0026\u0026 sudo chmod 755 /usr/bin/helm"},{"line_number":43,"context_line":""},{"line_number":44,"context_line":"RUN useradd --user-group --create-home --home-dir /home/helm helm \\"},{"line_number":45,"context_line":"    \u0026\u0026 sudo mkdir /helm-repo \\"}],"source_content_type":"text/x-jinja2","patch_set":13,"id":"ba5201f7_ec0a1dd1","line":42,"in_reply_to":"ba5201f7_46c78895","updated":"2017-01-08 12:56:02.000000000","message":"Done","commit_id":"bb7668ebd720d65a2a0db8ad9eb5d2a240704239"},{"author":{"_account_id":7488,"name":"Jeffrey Zhang","email":"zhang.lei.fly@gmail.com","username":"Jeffrey4l"},"change_message_id":"c23d31e6271c65819c7272ab3e2bd13fbe506d3b","unresolved":false,"context_lines":[{"line_number":38,"context_line":"    \u0026\u0026 rm get-pip.py"},{"line_number":39,"context_line":""},{"line_number":40,"context_line":"RUN pip install setuptools \\"},{"line_number":41,"context_line":"    \u0026\u0026 pip install pyyaml"},{"line_number":42,"context_line":""},{"line_number":43,"context_line":"RUN curl -Lo /tmp/helm-v${helm_version}-linux-amd64.tar.gz  http://storage.googleapis.com/kubernetes-helm/helm-v${helm_version}-linux-amd64.tar.gz \\"},{"line_number":44,"context_line":"    \u0026\u0026 sudo tar --strip-components 1 -C /usr/bin linux-amd64/helm -zxvf /tmp/helm-v${helm_version}-linux-amd64.tar.gz \\"}],"source_content_type":"text/x-jinja2","patch_set":14,"id":"ba5201f7_8f1c236b","line":41,"updated":"2017-01-08 14:21:15.000000000","message":"get-pip.py will install setuptools automatically.\n\npython get-pip.py\nCollecting pip\n  Downloading pip-9.0.1-py2.py3-none-any.whl (1.3MB)\n    100% |################################| 1.3MB 166kB/s \nCollecting setuptools\n  Downloading setuptools-32.3.1-py2.py3-none-any.whl (479kB)\n    100% |################################| 481kB 125kB/s \nCollecting wheel\n  Downloading wheel-0.29.0-py2.py3-none-any.whl (66kB)\n    100% |################################| 71kB 155kB/s \nInstalling collected packages: pip, setuptools, wheel\nSuccessfully installed pip-9.0.1 setuptools-32.3.1 wheel-0.29.0\n\n\n\nmerge this RUN into L #36 should be safe.","commit_id":"012051adad12912aebc4b12b7c91adfaf4c60ddc"},{"author":{"_account_id":19384,"name":"Serguei Bezverkhi","email":"sbezverk@cisco.com","username":"sbezverk"},"change_message_id":"d827a247ff9ec27a1fbc6cd73a4366d11d450a2e","unresolved":false,"context_lines":[{"line_number":38,"context_line":"    \u0026\u0026 rm get-pip.py"},{"line_number":39,"context_line":""},{"line_number":40,"context_line":"RUN pip install setuptools \\"},{"line_number":41,"context_line":"    \u0026\u0026 pip install pyyaml"},{"line_number":42,"context_line":""},{"line_number":43,"context_line":"RUN curl -Lo /tmp/helm-v${helm_version}-linux-amd64.tar.gz  http://storage.googleapis.com/kubernetes-helm/helm-v${helm_version}-linux-amd64.tar.gz \\"},{"line_number":44,"context_line":"    \u0026\u0026 sudo tar --strip-components 1 -C /usr/bin linux-amd64/helm -zxvf /tmp/helm-v${helm_version}-linux-amd64.tar.gz \\"}],"source_content_type":"text/x-jinja2","patch_set":14,"id":"ba5201f7_6ff7ffd2","line":41,"in_reply_to":"ba5201f7_8f1c236b","updated":"2017-01-08 14:55:08.000000000","message":"Done","commit_id":"012051adad12912aebc4b12b7c91adfaf4c60ddc"}]}