)]}'
{"mini-mirror/Chart.yaml":[{"author":{"_account_id":22477,"name":"Matt McEuen","email":"matt.mceuen@att.com","username":"mattmceuen"},"change_message_id":"2149734202bd7a698470700065c1f745cbcb6c1b","unresolved":false,"context_lines":[{"line_number":1,"context_line":"# Copyright 2017 The Openstack-Helm Authors."},{"line_number":2,"context_line":"#"},{"line_number":3,"context_line":"# Licensed under the Apache License, Version 2.0 (the \"License\");"},{"line_number":4,"context_line":"# you may not use this file except in compliance with the License."}],"source_content_type":"text/x-yaml","patch_set":11,"id":"dfd5e7cf_15836255","line":1,"updated":"2019-01-08 22:30:43.000000000","message":"2019","commit_id":"c7647bf536616eb600463f0c05b9159fed094708"},{"author":{"_account_id":28618,"name":"Drew Walters","email":"drewwalters@microsoft.com","username":"drewwalters96"},"change_message_id":"13714666503921b9a98a51fef87c17f33e423822","unresolved":false,"context_lines":[{"line_number":1,"context_line":"# Copyright 2017 The Openstack-Helm Authors."},{"line_number":2,"context_line":"#"},{"line_number":3,"context_line":"# Licensed under the Apache License, Version 2.0 (the \"License\");"},{"line_number":4,"context_line":"# you may not use this file except in compliance with the License."}],"source_content_type":"text/x-yaml","patch_set":11,"id":"dfd5e7cf_211df1bc","line":1,"in_reply_to":"dfd5e7cf_15836255","updated":"2019-01-09 18:39:06.000000000","message":"\u003e 2019\n\nHeaders updated per our offline discussion.","commit_id":"c7647bf536616eb600463f0c05b9159fed094708"}],"mini-mirror/templates/configmap-etc.yaml":[{"author":{"_account_id":22477,"name":"Matt McEuen","email":"matt.mceuen@att.com","username":"mattmceuen"},"change_message_id":"2149734202bd7a698470700065c1f745cbcb6c1b","unresolved":false,"context_lines":[{"line_number":23,"context_line":"  name: mini-mirror-etc"},{"line_number":24,"context_line":"data:"},{"line_number":25,"context_line":"  nginx.conf: |"},{"line_number":26,"context_line":"    server {"},{"line_number":27,"context_line":"      root /srv;"},{"line_number":28,"context_line":"      server_name nginx;"},{"line_number":29,"context_line":""}],"source_content_type":"text/x-yaml","patch_set":11,"id":"dfd5e7cf_9019a07a","line":26,"updated":"2019-01-08 22:30:43.000000000","message":"Can we put this in values.yaml, so that it can be overridden when needed at runtime, e.g.:\nhttps://github.com/openstack/openstack-helm-infra/blob/master/kibana/values.yaml#L118","commit_id":"c7647bf536616eb600463f0c05b9159fed094708"},{"author":{"_account_id":28618,"name":"Drew Walters","email":"drewwalters@microsoft.com","username":"drewwalters96"},"change_message_id":"13714666503921b9a98a51fef87c17f33e423822","unresolved":false,"context_lines":[{"line_number":23,"context_line":"  name: mini-mirror-etc"},{"line_number":24,"context_line":"data:"},{"line_number":25,"context_line":"  nginx.conf: |"},{"line_number":26,"context_line":"    server {"},{"line_number":27,"context_line":"      root /srv;"},{"line_number":28,"context_line":"      server_name nginx;"},{"line_number":29,"context_line":""}],"source_content_type":"text/x-yaml","patch_set":11,"id":"dfd5e7cf_c17985fc","line":26,"in_reply_to":"dfd5e7cf_9019a07a","updated":"2019-01-09 18:39:06.000000000","message":"\u003e Can we put this in values.yaml, so that it can be overridden when\n \u003e needed at runtime, e.g.:\n \u003e https://github.com/openstack/openstack-helm-infra/blob/master/kibana/values.yaml#L118\n\nDone.","commit_id":"c7647bf536616eb600463f0c05b9159fed094708"}],"mini-mirror/templates/service-ingress.yaml":[{"author":{"_account_id":22477,"name":"Matt McEuen","email":"matt.mceuen@att.com","username":"mattmceuen"},"change_message_id":"2149734202bd7a698470700065c1f745cbcb6c1b","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":11,"id":"dfd5e7cf_75abce1c","line":21,"updated":"2019-01-08 22:30:43.000000000","message":"You\u0027ve got a service for your ingress, but you\u0027d also need an ingress rule!  e.g.\nhttps://github.com/openstack/openstack-helm/blob/master/glance/templates/ingress-api.yaml\n\nHowever, since the initial use case for this mini-mirror is ENTIRELY in-cluster (not serving content to outside the cluster), I think we can start w/o any ingress-related stuff.\n\nI think this could be expanded in the future for other apt serving use cases w/ an ingress, perhaps which is configured not to render by default.","commit_id":"c7647bf536616eb600463f0c05b9159fed094708"},{"author":{"_account_id":28618,"name":"Drew Walters","email":"drewwalters@microsoft.com","username":"drewwalters96"},"change_message_id":"13714666503921b9a98a51fef87c17f33e423822","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":11,"id":"dfd5e7cf_81837dcc","line":21,"in_reply_to":"dfd5e7cf_75abce1c","updated":"2019-01-09 18:39:06.000000000","message":"\u003e You\u0027ve got a service for your ingress, but you\u0027d also need an\n \u003e ingress rule!  e.g.\n \u003e https://github.com/openstack/openstack-helm/blob/master/glance/templates/ingress-api.yaml\n \u003e \n \u003e However, since the initial use case for this mini-mirror is\n \u003e ENTIRELY in-cluster (not serving content to outside the cluster), I\n \u003e think we can start w/o any ingress-related stuff.\n \u003e \n \u003e I think this could be expanded in the future for other apt serving\n \u003e use cases w/ an ingress, perhaps which is configured not to render\n \u003e by default.\n\nRemoved for initial version.","commit_id":"c7647bf536616eb600463f0c05b9159fed094708"}],"mini-mirror/values.yaml":[{"author":{"_account_id":22477,"name":"Matt McEuen","email":"matt.mceuen@att.com","username":"mattmceuen"},"change_message_id":"2149734202bd7a698470700065c1f745cbcb6c1b","unresolved":false,"context_lines":[{"line_number":17,"context_line":"    dep_check: quay.io/stackanetes/kubernetes-entrypoint:v0.3.1"},{"line_number":18,"context_line":"    image_repo_sync: docker.io/docker:17.07.0"},{"line_number":19,"context_line":"    # TODO(drewwalters96): Provide publicly available image."},{"line_number":20,"context_line":"    mini-mirror: openstackhelm/mini-mirror:latest"},{"line_number":21,"context_line":"  pull_policy: \"IfNotPresent\""},{"line_number":22,"context_line":"  local_registry:"},{"line_number":23,"context_line":"    active: false"}],"source_content_type":"text/x-yaml","patch_set":11,"id":"dfd5e7cf_d5ccfa8a","line":20,"updated":"2019-01-08 22:30:43.000000000","message":"Can we use the standard OSH image convention, e.g.\ndocker.io/openstackhelm/mini-mirror:v0.1.0\n\nCan we also add a comment here that the mini-mirror image is just a placeholder, since this chart *only* adds value if you override the image with a custom one that includes the packages you want to serve?","commit_id":"c7647bf536616eb600463f0c05b9159fed094708"},{"author":{"_account_id":28618,"name":"Drew Walters","email":"drewwalters@microsoft.com","username":"drewwalters96"},"change_message_id":"13714666503921b9a98a51fef87c17f33e423822","unresolved":false,"context_lines":[{"line_number":17,"context_line":"    dep_check: quay.io/stackanetes/kubernetes-entrypoint:v0.3.1"},{"line_number":18,"context_line":"    image_repo_sync: docker.io/docker:17.07.0"},{"line_number":19,"context_line":"    # TODO(drewwalters96): Provide publicly available image."},{"line_number":20,"context_line":"    mini-mirror: openstackhelm/mini-mirror:latest"},{"line_number":21,"context_line":"  pull_policy: \"IfNotPresent\""},{"line_number":22,"context_line":"  local_registry:"},{"line_number":23,"context_line":"    active: false"}],"source_content_type":"text/x-yaml","patch_set":11,"id":"dfd5e7cf_01778d04","line":20,"in_reply_to":"dfd5e7cf_d5ccfa8a","updated":"2019-01-09 18:39:06.000000000","message":"\u003e Can we use the standard OSH image convention, e.g.\n \u003e docker.io/openstackhelm/mini-mirror:v0.1.0\n \u003e \n \u003e Can we also add a comment here that the mini-mirror image is just a\n \u003e placeholder, since this chart *only* adds value if you override the\n \u003e image with a custom one that includes the packages you want to\n \u003e serve?\n\nImage building moved to openstack-helm-images. I left as latest for now since those images seem to be using that convention.\n\nI can change the tag if that is an issue.","commit_id":"c7647bf536616eb600463f0c05b9159fed094708"},{"author":{"_account_id":22477,"name":"Matt McEuen","email":"matt.mceuen@att.com","username":"mattmceuen"},"change_message_id":"2149734202bd7a698470700065c1f745cbcb6c1b","unresolved":false,"context_lines":[{"line_number":56,"context_line":"      topologyKey:"},{"line_number":57,"context_line":"        default: kubernetes.io/hostname"},{"line_number":58,"context_line":"  replicas:"},{"line_number":59,"context_line":"    api: 1"},{"line_number":60,"context_line":"  lifecycle:"},{"line_number":61,"context_line":"    upgrades:"},{"line_number":62,"context_line":"      deployments:"}],"source_content_type":"text/x-yaml","patch_set":11,"id":"dfd5e7cf_d5915a54","line":59,"updated":"2019-01-08 22:30:43.000000000","message":"perhaps the default replicas should be 2; HA should be easy since you\u0027re using a service and these pods would have static data to serve","commit_id":"c7647bf536616eb600463f0c05b9159fed094708"},{"author":{"_account_id":22477,"name":"Matt McEuen","email":"matt.mceuen@att.com","username":"mattmceuen"},"change_message_id":"2149734202bd7a698470700065c1f745cbcb6c1b","unresolved":false,"context_lines":[{"line_number":123,"context_line":"        default: 8888"},{"line_number":124,"context_line":"        public: 80"},{"line_number":125,"context_line":""},{"line_number":126,"context_line":"bootstrap:"},{"line_number":127,"context_line":"  enabled: false"},{"line_number":128,"context_line":""},{"line_number":129,"context_line":"network:"}],"source_content_type":"text/x-yaml","patch_set":11,"id":"dfd5e7cf_b0406437","line":126,"updated":"2019-01-08 22:30:43.000000000","message":"If you don\u0027t have a bootstrap job, I\u0027d just as soon leave this out.","commit_id":"c7647bf536616eb600463f0c05b9159fed094708"},{"author":{"_account_id":22477,"name":"Matt McEuen","email":"matt.mceuen@att.com","username":"mattmceuen"},"change_message_id":"2149734202bd7a698470700065c1f745cbcb6c1b","unresolved":false,"context_lines":[{"line_number":145,"context_line":"  configmap_etc: true"},{"line_number":146,"context_line":"  deployment_mini_mirror: true"},{"line_number":147,"context_line":"  job_image_repo_sync: true"},{"line_number":148,"context_line":"  service_ingress_api: false"},{"line_number":149,"context_line":"  service_mini_mirror: true"}],"source_content_type":"text/x-yaml","patch_set":11,"id":"dfd5e7cf_358f8624","line":148,"updated":"2019-01-08 22:30:43.000000000","message":"I think this oughta be true by default","commit_id":"c7647bf536616eb600463f0c05b9159fed094708"}],"nginx/templates/configmap-nginx.yaml":[{"author":{"_account_id":28701,"name":"Dustin Specker","email":"v-dspecker@microsoft.com","username":"dustinspecker"},"change_message_id":"b40daa825668f668f8b348d75439d8f699d205ca","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":5,"id":"1f769fc5_7a3b88ea","updated":"2019-01-02 19:11:42.000000000","message":"I would rename this file to configmap-etc just to be consistent with other charts.","commit_id":"bd1282eacaaee7e879027830b767e690294a7cf3"}],"nginx/templates/deployment-nginx.yaml":[{"author":{"_account_id":28618,"name":"Drew Walters","email":"drewwalters@microsoft.com","username":"drewwalters96"},"change_message_id":"db520daa9e2f097111679576552a1a1dfc9fbab3","unresolved":false,"context_lines":[{"line_number":24,"context_line":"  annotations:"},{"line_number":25,"context_line":"    {{ tuple $envAll | include \"helm-toolkit.snippets.release_uuid\" }}"},{"line_number":26,"context_line":"  labels:"},{"line_number":27,"context_line":"{{ tuple $envAll \"aptly\" \"api\" | include \"helm-toolkit.snippets.kubernetes_metadata_labels\" | indent 4 }}"},{"line_number":28,"context_line":"spec:"},{"line_number":29,"context_line":"  replicas: {{ .Values.pod.replicas.api }}"},{"line_number":30,"context_line":"  selector:"}],"source_content_type":"text/x-yaml","patch_set":4,"id":"1f769fc5_b5d37838","line":27,"range":{"start_line":27,"start_character":3,"end_line":27,"end_character":30},"updated":"2019-01-02 14:58:35.000000000","message":"Should be\n\n tuple $envAll \"aptly\" \"api\"","commit_id":"059ec7e503e702044cee2b6b9310712646f4621e"},{"author":{"_account_id":28701,"name":"Dustin Specker","email":"v-dspecker@microsoft.com","username":"dustinspecker"},"change_message_id":"285e19fe19d859a6adf09d4d261fe4f7a5553992","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":8,"id":"ffd0ebdf_feb0268e","updated":"2019-01-03 17:41:00.000000000","message":"question: what is reason for checking for configmap_bin being enabled?\n\nFrom my understanding these manifest flags are designed for the intention of having multiple instances of the same chart deployed. (First chart is deployed with everything enabled and second chart turns off so there is not a conflict.) Having this condition prevents allowing multiple instances of the same chart.","commit_id":"212083ab9f4cdd42a70592f5b49da102013b513b"},{"author":{"_account_id":28618,"name":"Drew Walters","email":"drewwalters@microsoft.com","username":"drewwalters96"},"change_message_id":"1eb683a669eaa3d346c0ca582cf487b11e6f58ad","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":8,"id":"ffd0ebdf_e60178b1","in_reply_to":"ffd0ebdf_feb0268e","updated":"2019-01-04 15:07:58.000000000","message":"\u003e question: what is reason for checking for configmap_bin being\n \u003e enabled?\n \u003e \n \u003e From my understanding these manifest flags are designed for the\n \u003e intention of having multiple instances of the same chart deployed.\n \u003e (First chart is deployed with everything enabled and second chart\n \u003e turns off so there is not a conflict.) Having this condition\n \u003e prevents allowing multiple instances of the same chart.\n\nI assumed at the time of authoring that this was a common pattern. I can remove this if that\u0027s not the case.","commit_id":"212083ab9f4cdd42a70592f5b49da102013b513b"}],"nginx/templates/job-image-repo-sync.yaml":[{"author":{"_account_id":28701,"name":"Dustin Specker","email":"v-dspecker@microsoft.com","username":"dustinspecker"},"change_message_id":"b40daa825668f668f8b348d75439d8f699d205ca","unresolved":false,"context_lines":[{"line_number":15,"context_line":"*/}}"},{"line_number":16,"context_line":""},{"line_number":17,"context_line":"{{- if and .Values.manifests.job_image_repo_sync .Values.images.local_registry.active }}"},{"line_number":18,"context_line":"{{- $imageRepoSyncJob :\u003d dict \"envAll\" . \"serviceName\" \"aptly\" -}}"},{"line_number":19,"context_line":"{{ $imageRepoSyncJob | include \"helm-toolkit.manifests.job_image_repo_sync\" }}"},{"line_number":20,"context_line":"{{- end }}"}],"source_content_type":"text/x-yaml","patch_set":5,"id":"1f769fc5_9abb0471","line":18,"range":{"start_line":18,"start_character":56,"end_line":18,"end_character":61},"updated":"2019-01-02 19:11:42.000000000","message":"should this be nginx?","commit_id":"bd1282eacaaee7e879027830b767e690294a7cf3"}],"nginx/templates/service-ingress.yaml":[{"author":{"_account_id":28701,"name":"Dustin Specker","email":"v-dspecker@microsoft.com","username":"dustinspecker"},"change_message_id":"b40daa825668f668f8b348d75439d8f699d205ca","unresolved":false,"context_lines":[{"line_number":14,"context_line":"limitations under the License."},{"line_number":15,"context_line":"*/}}"},{"line_number":16,"context_line":""},{"line_number":17,"context_line":"{{- if and .Values.manifests.service_ingress_api .Values.network.server.ingress.public }}"},{"line_number":18,"context_line":"{{- $serviceIngressOpts :\u003d dict \"envAll\" . \"backendServiceType\" \"server\" -}}"},{"line_number":19,"context_line":"{{ $serviceIngressOpts | include \"helm-toolkit.manifests.service_ingress\" }}"},{"line_number":20,"context_line":"{{- end }}"}],"source_content_type":"text/x-yaml","patch_set":5,"id":"1f769fc5_5a200cb1","line":17,"range":{"start_line":17,"start_character":29,"end_line":17,"end_character":48},"updated":"2019-01-02 19:11:42.000000000","message":"This is missing from values.yaml. I would at least put it there with an explicit false (if intended to be gone).","commit_id":"bd1282eacaaee7e879027830b767e690294a7cf3"}],"nginx/values.yaml":[{"author":{"_account_id":28701,"name":"Dustin Specker","email":"v-dspecker@microsoft.com","username":"dustinspecker"},"change_message_id":"b40daa825668f668f8b348d75439d8f699d205ca","unresolved":false,"context_lines":[{"line_number":142,"context_line":"manifests:"},{"line_number":143,"context_line":"  configmap_etc: true"},{"line_number":144,"context_line":"  deployment_nginx: true"},{"line_number":145,"context_line":"  job_bootstrap: true"},{"line_number":146,"context_line":"  job_image_repo_sync: true"},{"line_number":147,"context_line":"  service_nginx: true"},{"line_number":148,"context_line":""}],"source_content_type":"text/x-yaml","patch_set":5,"id":"1f769fc5_3a0e703b","line":145,"range":{"start_line":145,"start_character":2,"end_line":145,"end_character":15},"updated":"2019-01-02 19:11:42.000000000","message":"I don\u0027t see this being used?","commit_id":"bd1282eacaaee7e879027830b767e690294a7cf3"},{"author":{"_account_id":20466,"name":"Tin Lam","email":"tin@lam.wtf","username":"tinlam"},"change_message_id":"dd50d17da1328ec371ddcd625fe8f21e22142d01","unresolved":false,"context_lines":[{"line_number":14,"context_line":""},{"line_number":15,"context_line":"images:"},{"line_number":16,"context_line":"  tags:"},{"line_number":17,"context_line":"    dep_check: quay.io/stackanetes/kubernetes-entrypoint:v0.2.1"},{"line_number":18,"context_line":"    image_repo_sync: docker.io/docker:17.07.0"},{"line_number":19,"context_line":"    nginx: registry.hub.docker.com/library/nginx:stable-alpine"},{"line_number":20,"context_line":"  pull_policy: \"IfNotPresent\""}],"source_content_type":"text/x-yaml","patch_set":8,"id":"ffd0ebdf_75ba0cdf","line":17,"range":{"start_line":17,"start_character":56,"end_line":17,"end_character":63},"updated":"2019-01-04 00:13:57.000000000","message":"why use a different version of this image - https://github.com/openstack/openstack-helm-infra/blob/38199863980b865d13617374ae01e3b8efb064d2/grafana/values.yaml#L22","commit_id":"212083ab9f4cdd42a70592f5b49da102013b513b"},{"author":{"_account_id":28618,"name":"Drew Walters","email":"drewwalters@microsoft.com","username":"drewwalters96"},"change_message_id":"1eb683a669eaa3d346c0ca582cf487b11e6f58ad","unresolved":false,"context_lines":[{"line_number":14,"context_line":""},{"line_number":15,"context_line":"images:"},{"line_number":16,"context_line":"  tags:"},{"line_number":17,"context_line":"    dep_check: quay.io/stackanetes/kubernetes-entrypoint:v0.2.1"},{"line_number":18,"context_line":"    image_repo_sync: docker.io/docker:17.07.0"},{"line_number":19,"context_line":"    nginx: registry.hub.docker.com/library/nginx:stable-alpine"},{"line_number":20,"context_line":"  pull_policy: \"IfNotPresent\""}],"source_content_type":"text/x-yaml","patch_set":8,"id":"ffd0ebdf_d412d0b1","line":17,"range":{"start_line":17,"start_character":56,"end_line":17,"end_character":63},"in_reply_to":"ffd0ebdf_75ba0cdf","updated":"2019-01-04 15:07:58.000000000","message":"\u003e why use a different version of this image - https://github.com/openstack/openstack-helm-infra/blob/38199863980b865d13617374ae01e3b8efb064d2/grafana/values.yaml#L22\n\nMy mistake; fixed.","commit_id":"212083ab9f4cdd42a70592f5b49da102013b513b"},{"author":{"_account_id":20466,"name":"Tin Lam","email":"tin@lam.wtf","username":"tinlam"},"change_message_id":"dd50d17da1328ec371ddcd625fe8f21e22142d01","unresolved":false,"context_lines":[{"line_number":69,"context_line":"        min_available: 0"},{"line_number":70,"context_line":"  resources:"},{"line_number":71,"context_line":"    enabled: false"},{"line_number":72,"context_line":"    limits:"},{"line_number":73,"context_line":"        memory: \"1024Mi\""},{"line_number":74,"context_line":"        cpu: \"2000m\""},{"line_number":75,"context_line":"    server:"},{"line_number":76,"context_line":"      requests:"},{"line_number":77,"context_line":"        memory: \"128Mi\""}],"source_content_type":"text/x-yaml","patch_set":8,"id":"ffd0ebdf_35cc8488","line":74,"range":{"start_line":72,"start_character":0,"end_line":74,"end_character":20},"updated":"2019-01-04 00:13:57.000000000","message":"should this limits be under server? it looks to be placed incorrectly","commit_id":"212083ab9f4cdd42a70592f5b49da102013b513b"},{"author":{"_account_id":28618,"name":"Drew Walters","email":"drewwalters@microsoft.com","username":"drewwalters96"},"change_message_id":"1eb683a669eaa3d346c0ca582cf487b11e6f58ad","unresolved":false,"context_lines":[{"line_number":69,"context_line":"        min_available: 0"},{"line_number":70,"context_line":"  resources:"},{"line_number":71,"context_line":"    enabled: false"},{"line_number":72,"context_line":"    limits:"},{"line_number":73,"context_line":"        memory: \"1024Mi\""},{"line_number":74,"context_line":"        cpu: \"2000m\""},{"line_number":75,"context_line":"    server:"},{"line_number":76,"context_line":"      requests:"},{"line_number":77,"context_line":"        memory: \"128Mi\""}],"source_content_type":"text/x-yaml","patch_set":8,"id":"ffd0ebdf_9424480f","line":74,"range":{"start_line":72,"start_character":0,"end_line":74,"end_character":20},"in_reply_to":"ffd0ebdf_35cc8488","updated":"2019-01-04 15:07:58.000000000","message":"\u003e should this limits be under server? it looks to be placed\n \u003e incorrectly\n\nFixed.","commit_id":"212083ab9f4cdd42a70592f5b49da102013b513b"},{"author":{"_account_id":20466,"name":"Tin Lam","email":"tin@lam.wtf","username":"tinlam"},"change_message_id":"dd50d17da1328ec371ddcd625fe8f21e22142d01","unresolved":false,"context_lines":[{"line_number":149,"context_line":""},{"line_number":150,"context_line":"conf:"},{"line_number":151,"context_line":"  server:"},{"line_number":152,"context_line":"    remote_tar_path: http://remote-path.org/example.tar"}],"source_content_type":"text/x-yaml","patch_set":8,"id":"ffd0ebdf_75af6c94","line":152,"range":{"start_line":152,"start_character":21,"end_line":152,"end_character":55},"updated":"2019-01-04 00:13:57.000000000","message":"does a user need to override this bogus path for the tar to work?  Can we have a working tar path","commit_id":"212083ab9f4cdd42a70592f5b49da102013b513b"},{"author":{"_account_id":28618,"name":"Drew Walters","email":"drewwalters@microsoft.com","username":"drewwalters96"},"change_message_id":"1eb683a669eaa3d346c0ca582cf487b11e6f58ad","unresolved":false,"context_lines":[{"line_number":149,"context_line":""},{"line_number":150,"context_line":"conf:"},{"line_number":151,"context_line":"  server:"},{"line_number":152,"context_line":"    remote_tar_path: http://remote-path.org/example.tar"}],"source_content_type":"text/x-yaml","patch_set":8,"id":"ffd0ebdf_9469a826","line":152,"range":{"start_line":152,"start_character":21,"end_line":152,"end_character":55},"in_reply_to":"ffd0ebdf_75af6c94","updated":"2019-01-04 15:07:58.000000000","message":"\u003e does a user need to override this bogus path for the tar to work? \n \u003e Can we have a working tar path\n\nYes; however, we could serve the default NGINX page if we disabled the configmap_bin manifest. I would be happy to upload an example tar, but I think that method is less reliable.\n\nThoughts?","commit_id":"212083ab9f4cdd42a70592f5b49da102013b513b"}],"tools/images/mini-mirror/Dockerfile":[{"author":{"_account_id":22477,"name":"Matt McEuen","email":"matt.mceuen@att.com","username":"mattmceuen"},"change_message_id":"2149734202bd7a698470700065c1f745cbcb6c1b","unresolved":false,"context_lines":[{"line_number":12,"context_line":"# See the License for the specific language governing permissions and"},{"line_number":13,"context_line":"# limitations under the License."},{"line_number":14,"context_line":""},{"line_number":15,"context_line":"FROM ubuntu:16.04 as aptly"},{"line_number":16,"context_line":""},{"line_number":17,"context_line":"ARG APTLY_CONFIG_PATH\u003daptly.conf"},{"line_number":18,"context_line":"ARG RELEASE_SIGN_KEY\u003dkey.gpg"}],"source_content_type":"text/x-dockerfile","patch_set":11,"id":"dfd5e7cf_7580ae8c","line":15,"updated":"2019-01-08 22:30:43.000000000","message":"For all the image-building stuff:  can we move all of it to the openstack-helm-images project?  Then make this PS Depends-On the -images PS.","commit_id":"c7647bf536616eb600463f0c05b9159fed094708"},{"author":{"_account_id":28618,"name":"Drew Walters","email":"drewwalters@microsoft.com","username":"drewwalters96"},"change_message_id":"13714666503921b9a98a51fef87c17f33e423822","unresolved":false,"context_lines":[{"line_number":12,"context_line":"# See the License for the specific language governing permissions and"},{"line_number":13,"context_line":"# limitations under the License."},{"line_number":14,"context_line":""},{"line_number":15,"context_line":"FROM ubuntu:16.04 as aptly"},{"line_number":16,"context_line":""},{"line_number":17,"context_line":"ARG APTLY_CONFIG_PATH\u003daptly.conf"},{"line_number":18,"context_line":"ARG RELEASE_SIGN_KEY\u003dkey.gpg"}],"source_content_type":"text/x-dockerfile","patch_set":11,"id":"dfd5e7cf_2172d112","line":15,"in_reply_to":"dfd5e7cf_7580ae8c","updated":"2019-01-09 18:39:06.000000000","message":"\u003e For all the image-building stuff:  can we move all of it to the\n \u003e openstack-helm-images project?  Then make this PS Depends-On the\n \u003e -images PS.\n\nDone!","commit_id":"c7647bf536616eb600463f0c05b9159fed094708"}],"tools/images/mini-mirror/sources/aptly-example/packages.txt":[{"author":{"_account_id":22477,"name":"Matt McEuen","email":"matt.mceuen@att.com","username":"mattmceuen"},"change_message_id":"2149734202bd7a698470700065c1f745cbcb6c1b","unresolved":false,"context_lines":[{"line_number":1,"context_line":"aptly"}],"source_content_type":"text/plain","patch_set":11,"id":"dfd5e7cf_5536ca05","line":1,"updated":"2019-01-08 22:30:43.000000000","message":"This is good:  we want the default placeholder/validation mini-mirror image upstream to be small.","commit_id":"c7647bf536616eb600463f0c05b9159fed094708"},{"author":{"_account_id":28618,"name":"Drew Walters","email":"drewwalters@microsoft.com","username":"drewwalters96"},"change_message_id":"13714666503921b9a98a51fef87c17f33e423822","unresolved":false,"context_lines":[{"line_number":1,"context_line":"aptly"}],"source_content_type":"text/plain","patch_set":11,"id":"dfd5e7cf_c162e53e","line":1,"in_reply_to":"dfd5e7cf_5536ca05","updated":"2019-01-09 18:39:06.000000000","message":"\u003e This is good:  we want the default placeholder/validation\n \u003e mini-mirror image upstream to be small.\n\nGreat!","commit_id":"c7647bf536616eb600463f0c05b9159fed094708"}]}