)]}'
{"docker/heat/heat-api/Dockerfile":[{"author":{"_account_id":2834,"name":"Steven Dake","email":"steven.dake@gmail.com","username":"sdake"},"change_message_id":"e22a854c258f529ab72b63101710a505a1f04864","unresolved":false,"context_lines":[{"line_number":2,"context_line":"MAINTAINER James Labocki \u003cjlabocki@redhat.com\u003e"},{"line_number":3,"context_line":""},{"line_number":4,"context_line":"RUN yum install -y openstack-heat-api \\"},{"line_number":5,"context_line":"\topenstack-heat-api-cfn \\"},{"line_number":6,"context_line":"\topenstack-heat-common \\"},{"line_number":7,"context_line":"\topenstack-heat-api-cloudwatch \\"},{"line_number":8,"context_line":"\topenstack-utils \\"}],"source_content_type":"text/x-dockerfile","patch_set":1,"id":"baa201ad_7879392e","line":5,"updated":"2014-10-16 16:49:22.000000000","message":"can you use spaces here","commit_id":"cc7c7ccd4aa0080af61e85f4c5e051289b00c382"},{"author":{"_account_id":2834,"name":"Steven Dake","email":"steven.dake@gmail.com","username":"sdake"},"change_message_id":"f8a62319fc678905a4f98e822ec7cb95e43f0aa6","unresolved":false,"context_lines":[{"line_number":4,"context_line":"# TODO: configure cfn and cloudwatch"},{"line_number":5,"context_line":"RUN yum -y install \\"},{"line_number":6,"context_line":"    openstack-heat-api \\"},{"line_number":7,"context_line":"    python-keystoneclient \\"},{"line_number":8,"context_line":"    ; yum clean all"},{"line_number":9,"context_line":""},{"line_number":10,"context_line":"ADD ./start.sh /start.sh"}],"source_content_type":"text/x-dockerfile","patch_set":3,"id":"baa201ad_da0d9990","line":7,"updated":"2014-10-17 19:49:54.000000000","message":"is keystoneclient an undeclared dependency in openstack-heat-api?  If so, please  file a bug against fedora so the packaging can be fixed.","commit_id":"8b63288d0c195270efc5e3ee6be05c7eab435501"},{"author":{"_account_id":3098,"name":"Jeff Peeler","email":"jpeeler@redhat.com","username":"jpeeler-z"},"change_message_id":"b56101679bc5605e7de1ea780b07138bc0462f82","unresolved":false,"context_lines":[{"line_number":4,"context_line":"# TODO: configure cfn and cloudwatch"},{"line_number":5,"context_line":"RUN yum -y install \\"},{"line_number":6,"context_line":"    openstack-heat-api \\"},{"line_number":7,"context_line":"    python-keystoneclient \\"},{"line_number":8,"context_line":"    ; yum clean all"},{"line_number":9,"context_line":""},{"line_number":10,"context_line":"ADD ./start.sh /start.sh"}],"source_content_type":"text/x-dockerfile","patch_set":3,"id":"baa201ad_3df05f90","line":7,"in_reply_to":"baa201ad_da0d9990","updated":"2014-10-17 20:41:35.000000000","message":"No, good point. I was trying to make sure the role creation would work, but that dependency is brought in automatically.","commit_id":"8b63288d0c195270efc5e3ee6be05c7eab435501"}],"docker/heat/heat-api/start.sh":[{"author":{"_account_id":2834,"name":"Steven Dake","email":"steven.dake@gmail.com","username":"sdake"},"change_message_id":"e22a854c258f529ab72b63101710a505a1f04864","unresolved":false,"context_lines":[{"line_number":5,"context_line":": ${KEYSTONE_AUTH_PROTOCOL:\u003dhttp}"},{"line_number":6,"context_line":": ${HEAT_KEYSTONE_USER:\u003dheat}"},{"line_number":7,"context_line":": ${ADMIN_TENANT_NAME:\u003dadmin}"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"if ! [ \"$KEYSTONE_ADMIN_TOKEN\" ]; then"},{"line_number":10,"context_line":"\techo \"*** Missing KEYSTONE_ADMIN_TOKEN\" \u003e\u00262"},{"line_number":11,"context_line":"\texit 1"}],"source_content_type":"text/x-sh","patch_set":1,"id":"baa201ad_98ab3d74","line":8,"updated":"2014-10-16 16:49:22.000000000","message":"recommend taking a look at glance-base to pull in kolla-config.sh for a bunch of helpers to check variables and if keystone/db is operational","commit_id":"cc7c7ccd4aa0080af61e85f4c5e051289b00c382"},{"author":{"_account_id":2834,"name":"Steven Dake","email":"steven.dake@gmail.com","username":"sdake"},"change_message_id":"e22a854c258f529ab72b63101710a505a1f04864","unresolved":false,"context_lines":[{"line_number":8,"context_line":""},{"line_number":9,"context_line":"if ! [ \"$KEYSTONE_ADMIN_TOKEN\" ]; then"},{"line_number":10,"context_line":"\techo \"*** Missing KEYSTONE_ADMIN_TOKEN\" \u003e\u00262"},{"line_number":11,"context_line":"\texit 1"},{"line_number":12,"context_line":"fi"},{"line_number":13,"context_line":""},{"line_number":14,"context_line":"if ! [ \"$HEAT_KEYSTONE_PASSWORD\" ]; then"}],"source_content_type":"text/x-sh","patch_set":1,"id":"baa201ad_d8b865ca","line":11,"updated":"2014-10-16 16:49:22.000000000","message":"tabs should be spaces","commit_id":"cc7c7ccd4aa0080af61e85f4c5e051289b00c382"},{"author":{"_account_id":2834,"name":"Steven Dake","email":"steven.dake@gmail.com","username":"sdake"},"change_message_id":"e22a854c258f529ab72b63101710a505a1f04864","unresolved":false,"context_lines":[{"line_number":12,"context_line":"fi"},{"line_number":13,"context_line":""},{"line_number":14,"context_line":"if ! [ \"$HEAT_KEYSTONE_PASSWORD\" ]; then"},{"line_number":15,"context_line":"\tHEAT_KEYSTONE_PASSWORD\u003d$(openssl rand -hex 15)"},{"line_number":16,"context_line":"\texport HEAT_KEYSTONE_PASSWORD"},{"line_number":17,"context_line":"fi"},{"line_number":18,"context_line":""}],"source_content_type":"text/x-sh","patch_set":1,"id":"baa201ad_b8bd61b3","line":15,"updated":"2014-10-16 16:49:22.000000000","message":"tabs should be spaces","commit_id":"cc7c7ccd4aa0080af61e85f4c5e051289b00c382"},{"author":{"_account_id":2834,"name":"Steven Dake","email":"steven.dake@gmail.com","username":"sdake"},"change_message_id":"e22a854c258f529ab72b63101710a505a1f04864","unresolved":false,"context_lines":[{"line_number":19,"context_line":"if [ -z \"$IP_ADDRESS\" ]; then"},{"line_number":20,"context_line":"\techo \u003e\u00262 \u0027error: IP_ADDRESS is not set.\u0027"},{"line_number":21,"context_line":"\techo \u003e\u00262 \u0027   Be sure to set IP_ADDRESS so it can be placed in heat configurations\u0027"},{"line_number":22,"context_line":"\texit 1"},{"line_number":23,"context_line":"fi"},{"line_number":24,"context_line":""},{"line_number":25,"context_line":"export SERVICE_TOKEN\u003d\"${KEYSTONE_ADMIN_TOKEN}\""}],"source_content_type":"text/x-sh","patch_set":1,"id":"baa201ad_98d97d24","line":22,"updated":"2014-10-16 16:49:22.000000000","message":"tabs should be spaces\n\nWhat is IP_ADDRESS?  You may want to use $MY_IP (after importing kolla-config.sh where it is defined for this purpose.","commit_id":"cc7c7ccd4aa0080af61e85f4c5e051289b00c382"},{"author":{"_account_id":3098,"name":"Jeff Peeler","email":"jpeeler@redhat.com","username":"jpeeler-z"},"change_message_id":"9c5e5b30245200f0ea0552bdb95491912497e00a","unresolved":false,"context_lines":[{"line_number":29,"context_line":"#    -A \"http://${HEAT_CFN_API_SERVICE_HOST}:8000/v1\""},{"line_number":30,"context_line":""},{"line_number":31,"context_line":"# will use crux after https://github.com/larsks/crux/issues/1 is implemented"},{"line_number":32,"context_line":"openstack user role list --os-token\u003d\"${KEYSTONE_ADMIN_TOKEN}\" --os-url $SERVICE_ENDPOINT --os-project-name keystone -f csv | tail -n +2 | awk -F, \u0027{print $2}\u0027 | grep heat_stack_user"},{"line_number":33,"context_line":"if [ $? -ne 0 ]; then"},{"line_number":34,"context_line":"    keystone role-create --name heat_stack_user"},{"line_number":35,"context_line":"fi"}],"source_content_type":"text/x-sh","patch_set":4,"id":"9aa7fdbe_e9bd1904","line":32,"updated":"2014-10-18 17:35:08.000000000","message":"I don\u0027t know why this isn\u0027t happy yet, getting: ERROR: openstack Project must be specified.","commit_id":"c4d27048e1581e7f1207fe32c53f03b5f182aa79"},{"author":{"_account_id":6836,"name":"Daneyon Hansen","email":"danehans@cisco.com","username":"danehans"},"change_message_id":"e1e2df51dad6e853670843b48c5474818801f202","unresolved":false,"context_lines":[{"line_number":19,"context_line":"    -r admin"},{"line_number":20,"context_line":""},{"line_number":21,"context_line":"crux endpoint-create -n ${HEAT_KEYSTONE_USER} -t orchestration \\"},{"line_number":22,"context_line":"    -I \"http://${HEAT_API_SERVICE_HOST}:8004/v1/%(tenant_id)s\" \\"},{"line_number":23,"context_line":"    -P \"http://${PUBLIC_IP}:8004/v1/%(tenant_id)s\" \\"},{"line_number":24,"context_line":"    -A \"http://${HEAT_API_SERVICE_HOST}:8004/v1/%(tenant_id)s\""},{"line_number":25,"context_line":""}],"source_content_type":"text/x-sh","patch_set":6,"id":"9aa7fdbe_0874f297","line":22,"updated":"2014-10-20 21:11:45.000000000","message":"It appears that you statically configured http instead of using KEYSTONE_AUTH_PROTOCOL.\n\nYou may also want to add the region flag so that can be exposed to users.","commit_id":"e80c5171b064e34ef3c2edf443858e8cba529ede"},{"author":{"_account_id":6836,"name":"Daneyon Hansen","email":"danehans@cisco.com","username":"danehans"},"change_message_id":"e1e2df51dad6e853670843b48c5474818801f202","unresolved":false,"context_lines":[{"line_number":20,"context_line":""},{"line_number":21,"context_line":"crux endpoint-create -n ${HEAT_KEYSTONE_USER} -t orchestration \\"},{"line_number":22,"context_line":"    -I \"http://${HEAT_API_SERVICE_HOST}:8004/v1/%(tenant_id)s\" \\"},{"line_number":23,"context_line":"    -P \"http://${PUBLIC_IP}:8004/v1/%(tenant_id)s\" \\"},{"line_number":24,"context_line":"    -A \"http://${HEAT_API_SERVICE_HOST}:8004/v1/%(tenant_id)s\""},{"line_number":25,"context_line":""},{"line_number":26,"context_line":"#crux endpoint-create -n ${HEAT_KEYSTONE_USER} -t cloudformation \\"}],"source_content_type":"text/x-sh","patch_set":6,"id":"9aa7fdbe_2868d6b9","line":23,"updated":"2014-10-20 21:11:45.000000000","message":"It appears that you statically configured http instead of using KEYSTONE_AUTH_PROTOCOL.\n\nYou may also want to add the region flag so that can be exposed to users.","commit_id":"e80c5171b064e34ef3c2edf443858e8cba529ede"},{"author":{"_account_id":6836,"name":"Daneyon Hansen","email":"danehans@cisco.com","username":"danehans"},"change_message_id":"e1e2df51dad6e853670843b48c5474818801f202","unresolved":false,"context_lines":[{"line_number":21,"context_line":"crux endpoint-create -n ${HEAT_KEYSTONE_USER} -t orchestration \\"},{"line_number":22,"context_line":"    -I \"http://${HEAT_API_SERVICE_HOST}:8004/v1/%(tenant_id)s\" \\"},{"line_number":23,"context_line":"    -P \"http://${PUBLIC_IP}:8004/v1/%(tenant_id)s\" \\"},{"line_number":24,"context_line":"    -A \"http://${HEAT_API_SERVICE_HOST}:8004/v1/%(tenant_id)s\""},{"line_number":25,"context_line":""},{"line_number":26,"context_line":"#crux endpoint-create -n ${HEAT_KEYSTONE_USER} -t cloudformation \\"},{"line_number":27,"context_line":"#    -I \"http://${HEAT_CFN_API_SERVICE_HOST}:8000/v1\" \\"}],"source_content_type":"text/x-sh","patch_set":6,"id":"9aa7fdbe_6862ded9","line":24,"updated":"2014-10-20 21:11:45.000000000","message":"It appears that you statically configured http instead of using KEYSTONE_AUTH_PROTOCOL.\n\nYou may also want to add the region flag so that can be exposed to users.","commit_id":"e80c5171b064e34ef3c2edf443858e8cba529ede"},{"author":{"_account_id":6836,"name":"Daneyon Hansen","email":"danehans@cisco.com","username":"danehans"},"change_message_id":"e1e2df51dad6e853670843b48c5474818801f202","unresolved":false,"context_lines":[{"line_number":22,"context_line":"    -I \"http://${HEAT_API_SERVICE_HOST}:8004/v1/%(tenant_id)s\" \\"},{"line_number":23,"context_line":"    -P \"http://${PUBLIC_IP}:8004/v1/%(tenant_id)s\" \\"},{"line_number":24,"context_line":"    -A \"http://${HEAT_API_SERVICE_HOST}:8004/v1/%(tenant_id)s\""},{"line_number":25,"context_line":""},{"line_number":26,"context_line":"#crux endpoint-create -n ${HEAT_KEYSTONE_USER} -t cloudformation \\"},{"line_number":27,"context_line":"#    -I \"http://${HEAT_CFN_API_SERVICE_HOST}:8000/v1\" \\"},{"line_number":28,"context_line":"#    -P \"http://${PUBLIC_IP}:8000/v1\" \\"}],"source_content_type":"text/x-sh","patch_set":6,"id":"9aa7fdbe_888d427e","line":25,"updated":"2014-10-20 21:11:45.000000000","message":"I can see the following endpoint moving to a cfn-api start.sh in the future.\n\nYou may also want to add the region flag so that can be exposed to users.","commit_id":"e80c5171b064e34ef3c2edf443858e8cba529ede"}],"docker/heat/heat-base/Dockerfile":[{"author":{"_account_id":2834,"name":"Steven Dake","email":"steven.dake@gmail.com","username":"sdake"},"change_message_id":"e22a854c258f529ab72b63101710a505a1f04864","unresolved":false,"context_lines":[{"line_number":7,"context_line":"\topenstack-utils \\"},{"line_number":8,"context_line":"\topenstack-heat-api \\"},{"line_number":9,"context_line":"\topenstack-heat-api-cfn \\"},{"line_number":10,"context_line":"\topenstack-heat-engine \\"},{"line_number":11,"context_line":"\tpython-keystoneclient \\"},{"line_number":12,"context_line":"\tcrux \\"},{"line_number":13,"context_line":"\t; yum clean all"}],"source_content_type":"text/x-dockerfile","patch_set":1,"id":"baa201ad_f830c957","line":10,"updated":"2014-10-16 16:49:22.000000000","message":"heat-api should be instlled in the docker file for api\nheat-api-cfn should be installedin the docker file heat-api-engine\nheat-engine should be installed in the docker file for engine\nopenstack-utils is in the base fedora image - as I believe crux is as well.","commit_id":"cc7c7ccd4aa0080af61e85f4c5e051289b00c382"},{"author":{"_account_id":2834,"name":"Steven Dake","email":"steven.dake@gmail.com","username":"sdake"},"change_message_id":"e22a854c258f529ab72b63101710a505a1f04864","unresolved":false,"context_lines":[{"line_number":10,"context_line":"\topenstack-heat-engine \\"},{"line_number":11,"context_line":"\tpython-keystoneclient \\"},{"line_number":12,"context_line":"\tcrux \\"},{"line_number":13,"context_line":"\t; yum clean all"},{"line_number":14,"context_line":""},{"line_number":15,"context_line":"#RUN mkdir /opt/heat"},{"line_number":16,"context_line":"#COPY config-heat.sh /opt/heat/config-heat.sh"}],"source_content_type":"text/x-dockerfile","patch_set":1,"id":"baa201ad_3837b14e","line":13,"updated":"2014-10-16 16:49:22.000000000","message":"spaces not tabs please","commit_id":"cc7c7ccd4aa0080af61e85f4c5e051289b00c382"}],"docker/heat/heat-base/config-heat.sh":[{"author":{"_account_id":2834,"name":"Steven Dake","email":"steven.dake@gmail.com","username":"sdake"},"change_message_id":"f8a62319fc678905a4f98e822ec7cb95e43f0aa6","unresolved":false,"context_lines":[{"line_number":9,"context_line":": ${HEAT_DB_USER:\u003dheat}"},{"line_number":10,"context_line":": ${HEAT_KEYSTONE_USER:\u003dheat}"},{"line_number":11,"context_line":": ${KEYSTONE_AUTH_PROTOCOL:\u003dhttp}"},{"line_number":12,"context_line":": ${PUBLIC_IP:\u003d$HEAT_API_PORT_8004_TCP_ADDR}"},{"line_number":13,"context_line":""},{"line_number":14,"context_line":"check_required_vars HEAT_DB_PASSWORD HEAT_KEYSTONE_PASSWORD"},{"line_number":15,"context_line":"dump_vars"}],"source_content_type":"text/x-sh","patch_set":3,"id":"baa201ad_7ac90d25","line":12,"updated":"2014-10-17 19:49:54.000000000","message":"I think here you can just use $MY_IP but am not certain.","commit_id":"8b63288d0c195270efc5e3ee6be05c7eab435501"},{"author":{"_account_id":6836,"name":"Daneyon Hansen","email":"danehans@cisco.com","username":"danehans"},"change_message_id":"422d0f3412d00672952cd466d3d910b020c836e4","unresolved":false,"context_lines":[{"line_number":9,"context_line":": ${HEAT_DB_USER:\u003dheat}"},{"line_number":10,"context_line":": ${HEAT_KEYSTONE_USER:\u003dheat}"},{"line_number":11,"context_line":": ${KEYSTONE_AUTH_PROTOCOL:\u003dhttp}"},{"line_number":12,"context_line":": ${PUBLIC_IP:\u003d$HEAT_API_PORT_8004_TCP_ADDR}"},{"line_number":13,"context_line":""},{"line_number":14,"context_line":"check_required_vars HEAT_DB_PASSWORD HEAT_KEYSTONE_PASSWORD"},{"line_number":15,"context_line":"dump_vars"}],"source_content_type":"text/x-sh","patch_set":3,"id":"9aa7fdbe_f2b50632","line":12,"in_reply_to":"baa201ad_1dff1b35","updated":"2014-10-17 22:12:54.000000000","message":"From what I gathered, you should check for the PUBLIC_IP param in your startup that configured the keystone endpoints. The same script should also use PUBLIC_IP in the keystone publicurl (-P line).","commit_id":"8b63288d0c195270efc5e3ee6be05c7eab435501"},{"author":{"_account_id":3098,"name":"Jeff Peeler","email":"jpeeler@redhat.com","username":"jpeeler-z"},"change_message_id":"b56101679bc5605e7de1ea780b07138bc0462f82","unresolved":false,"context_lines":[{"line_number":9,"context_line":": ${HEAT_DB_USER:\u003dheat}"},{"line_number":10,"context_line":": ${HEAT_KEYSTONE_USER:\u003dheat}"},{"line_number":11,"context_line":": ${KEYSTONE_AUTH_PROTOCOL:\u003dhttp}"},{"line_number":12,"context_line":": ${PUBLIC_IP:\u003d$HEAT_API_PORT_8004_TCP_ADDR}"},{"line_number":13,"context_line":""},{"line_number":14,"context_line":"check_required_vars HEAT_DB_PASSWORD HEAT_KEYSTONE_PASSWORD"},{"line_number":15,"context_line":"dump_vars"}],"source_content_type":"text/x-sh","patch_set":3,"id":"baa201ad_1dff1b35","line":12,"in_reply_to":"baa201ad_7ac90d25","updated":"2014-10-17 20:41:35.000000000","message":"For which part? In kolla-common, PUBLIC_IP is assigned the value of MY_IP I think.","commit_id":"8b63288d0c195270efc5e3ee6be05c7eab435501"}],"docker/heat/heat-engine/Dockerfile":[{"author":{"_account_id":2834,"name":"Steven Dake","email":"steven.dake@gmail.com","username":"sdake"},"change_message_id":"e22a854c258f529ab72b63101710a505a1f04864","unresolved":false,"context_lines":[{"line_number":1,"context_line":"FROM jpeeler/fedora-rdo-heat-base"},{"line_number":2,"context_line":"MAINTAINER Steven Dake \u003csdake@redhat.com\u003e"},{"line_number":3,"context_line":""},{"line_number":4,"context_line":"ADD ./start.sh /start.sh"}],"source_content_type":"text/x-dockerfile","patch_set":1,"id":"baa201ad_7823f904","line":1,"updated":"2014-10-16 16:49:22.000000000","message":"use the build scripts for NAMESPACE feature.  this should be FROM kollaglue/*\n\nFor things like Heat, I think its fine to just push to kollaglue since nobody is running it via start-all-pods currently.","commit_id":"cc7c7ccd4aa0080af61e85f4c5e051289b00c382"},{"author":{"_account_id":2834,"name":"Steven Dake","email":"steven.dake@gmail.com","username":"sdake"},"change_message_id":"f8a62319fc678905a4f98e822ec7cb95e43f0aa6","unresolved":false,"context_lines":[{"line_number":1,"context_line":"FROM kollaglue/fedora-rdo-heat-base"},{"line_number":2,"context_line":"MAINTAINER Steven Dake \u003csdake@redhat.com\u003e"},{"line_number":3,"context_line":""},{"line_number":4,"context_line":"RUN yum -y install \\"},{"line_number":5,"context_line":"    openstack-heat-engine \\"}],"source_content_type":"text/x-dockerfile","patch_set":3,"id":"baa201ad_bae615ae","line":2,"updated":"2014-10-17 19:49:54.000000000","message":"change maintainer to you","commit_id":"8b63288d0c195270efc5e3ee6be05c7eab435501"},{"author":{"_account_id":3098,"name":"Jeff Peeler","email":"jpeeler@redhat.com","username":"jpeeler-z"},"change_message_id":"b56101679bc5605e7de1ea780b07138bc0462f82","unresolved":false,"context_lines":[{"line_number":1,"context_line":"FROM kollaglue/fedora-rdo-heat-base"},{"line_number":2,"context_line":"MAINTAINER Steven Dake \u003csdake@redhat.com\u003e"},{"line_number":3,"context_line":""},{"line_number":4,"context_line":"RUN yum -y install \\"},{"line_number":5,"context_line":"    openstack-heat-engine \\"}],"source_content_type":"text/x-dockerfile","patch_set":3,"id":"baa201ad_5d8c43d7","line":2,"in_reply_to":"baa201ad_bae615ae","updated":"2014-10-17 20:41:35.000000000","message":"Ok, will do soon. The reason I didn\u0027t previously is I read if you change that the cache is thrown away, which given how much image changing I\u0027ve been doing wasn\u0027t exactly ready for that.","commit_id":"8b63288d0c195270efc5e3ee6be05c7eab435501"}],"docker/heat/heat-engine/start.sh":[{"author":{"_account_id":2834,"name":"Steven Dake","email":"steven.dake@gmail.com","username":"sdake"},"change_message_id":"e22a854c258f529ab72b63101710a505a1f04864","unresolved":false,"context_lines":[{"line_number":5,"context_line":""},{"line_number":6,"context_line":"if ! [ \"$DB_ROOT_PASSWORD\" ]; then"},{"line_number":7,"context_line":"\techo \"*** Missing DB_ROOT_PASSWORD\" \u003e\u00262"},{"line_number":8,"context_line":"\texit 1"},{"line_number":9,"context_line":"fi"},{"line_number":10,"context_line":""},{"line_number":11,"context_line":"if ! [ \"$HEAT_DB_PASSWORD\" ]; then"}],"source_content_type":"text/x-sh","patch_set":1,"id":"baa201ad_3885f1ce","line":8,"updated":"2014-10-16 16:49:22.000000000","message":"spaces rather then tabs","commit_id":"cc7c7ccd4aa0080af61e85f4c5e051289b00c382"},{"author":{"_account_id":2834,"name":"Steven Dake","email":"steven.dake@gmail.com","username":"sdake"},"change_message_id":"e22a854c258f529ab72b63101710a505a1f04864","unresolved":false,"context_lines":[{"line_number":9,"context_line":"fi"},{"line_number":10,"context_line":""},{"line_number":11,"context_line":"if ! [ \"$HEAT_DB_PASSWORD\" ]; then"},{"line_number":12,"context_line":"\tHEAT_DB_PASSWORD\u003d$(openssl rand -hex 15)"},{"line_number":13,"context_line":"\texport HEAT_DB_PASSWORD"},{"line_number":14,"context_line":"fi"},{"line_number":15,"context_line":""}],"source_content_type":"text/x-sh","patch_set":1,"id":"baa201ad_586f152b","line":12,"updated":"2014-10-16 16:49:22.000000000","message":"spaces rather then tabs","commit_id":"cc7c7ccd4aa0080af61e85f4c5e051289b00c382"},{"author":{"_account_id":2834,"name":"Steven Dake","email":"steven.dake@gmail.com","username":"sdake"},"change_message_id":"e22a854c258f529ab72b63101710a505a1f04864","unresolved":false,"context_lines":[{"line_number":15,"context_line":""},{"line_number":16,"context_line":"#sh /opt/heat/config-heat.sh engine"},{"line_number":17,"context_line":""},{"line_number":18,"context_line":"mysql -h ${MARIADB_PORT_3306_TCP_ADDR} -u root \\"},{"line_number":19,"context_line":"\t-p${DB_ROOT_PASSWORD} mysql \u003c\u003cEOF"},{"line_number":20,"context_line":"CREATE DATABASE IF NOT EXISTS ${HEAT_DB_NAME};"},{"line_number":21,"context_line":"GRANT ALL PRIVILEGES ON heat* TO"}],"source_content_type":"text/x-sh","patch_set":1,"id":"baa201ad_786a191b","line":18,"updated":"2014-10-16 16:49:22.000000000","message":"tabs are ok here I believe","commit_id":"cc7c7ccd4aa0080af61e85f4c5e051289b00c382"},{"author":{"_account_id":2834,"name":"Steven Dake","email":"steven.dake@gmail.com","username":"sdake"},"change_message_id":"e22a854c258f529ab72b63101710a505a1f04864","unresolved":false,"context_lines":[{"line_number":19,"context_line":"\t-p${DB_ROOT_PASSWORD} mysql \u003c\u003cEOF"},{"line_number":20,"context_line":"CREATE DATABASE IF NOT EXISTS ${HEAT_DB_NAME};"},{"line_number":21,"context_line":"GRANT ALL PRIVILEGES ON heat* TO"},{"line_number":22,"context_line":"\t\u0027${HEAT_DB_USER}\u0027@\u0027%\u0027 IDENTIFIED BY \u0027${HEAT_DB_PASSWORD}\u0027"},{"line_number":23,"context_line":"EOF"},{"line_number":24,"context_line":""},{"line_number":25,"context_line":"crudini --set /etc/heat/heat.conf database connection \\"}],"source_content_type":"text/x-sh","patch_set":1,"id":"baa201ad_b8700109","line":22,"updated":"2014-10-16 16:49:22.000000000","message":"this should remian a tab.","commit_id":"cc7c7ccd4aa0080af61e85f4c5e051289b00c382"}],"k8s/pod/heat-pod.yaml":[{"author":{"_account_id":2834,"name":"Steven Dake","email":"steven.dake@gmail.com","username":"sdake"},"change_message_id":"e22a854c258f529ab72b63101710a505a1f04864","unresolved":false,"context_lines":[{"line_number":8,"context_line":"        value: heat"},{"line_number":9,"context_line":"      - name: HEAT_DB_PASSWORD"},{"line_number":10,"context_line":"        value: heatdbpassword"},{"line_number":11,"context_line":"      image: jpeeler/fedora-rdo-heat-engine"},{"line_number":12,"context_line":"      name: heat-engine"},{"line_number":13,"context_line":"    - env:"},{"line_number":14,"context_line":"      - name: DB_ROOT_PASSWORD"}],"source_content_type":"text/x-yaml","patch_set":1,"id":"baa201ad_18590d8e","line":11,"updated":"2014-10-16 16:49:22.000000000","message":"just push to kollaglue during dev","commit_id":"cc7c7ccd4aa0080af61e85f4c5e051289b00c382"},{"author":{"_account_id":2834,"name":"Steven Dake","email":"steven.dake@gmail.com","username":"sdake"},"change_message_id":"e22a854c258f529ab72b63101710a505a1f04864","unresolved":false,"context_lines":[{"line_number":24,"context_line":"      image: jpeeler/fedora-rdo-heat-api"},{"line_number":25,"context_line":"      name: heat-api"},{"line_number":26,"context_line":"      ports:"},{"line_number":27,"context_line":"      - containerPort: 8004"},{"line_number":28,"context_line":"    id: heat-1"},{"line_number":29,"context_line":"    version: v1beta1"},{"line_number":30,"context_line":"id: heat"}],"source_content_type":"text/x-yaml","patch_set":1,"id":"baa201ad_b84721a2","line":27,"updated":"2014-10-16 16:49:22.000000000","message":"heat-api-cfn has another port - I believe it is either 8000 or 8003.","commit_id":"cc7c7ccd4aa0080af61e85f4c5e051289b00c382"}],"k8s/service/heat-api-service.json":[{"author":{"_account_id":2834,"name":"Steven Dake","email":"steven.dake@gmail.com","username":"sdake"},"change_message_id":"f8a62319fc678905a4f98e822ec7cb95e43f0aa6","unresolved":false,"context_lines":[{"line_number":1,"context_line":"{"},{"line_number":2,"context_line":"  \"id\": \"heat-api\","},{"line_number":3,"context_line":"  \"kind\": \"Service\","},{"line_number":4,"context_line":"  \"apiVersion\": \"v1beta1\","}],"source_content_type":"application/json","patch_set":3,"id":"baa201ad_bab45582","line":1,"updated":"2014-10-17 19:49:54.000000000","message":"Can you convert to yaml?","commit_id":"8b63288d0c195270efc5e3ee6be05c7eab435501"},{"author":{"_account_id":3098,"name":"Jeff Peeler","email":"jpeeler@redhat.com","username":"jpeeler-z"},"change_message_id":"b56101679bc5605e7de1ea780b07138bc0462f82","unresolved":false,"context_lines":[{"line_number":1,"context_line":"{"},{"line_number":2,"context_line":"  \"id\": \"heat-api\","},{"line_number":3,"context_line":"  \"kind\": \"Service\","},{"line_number":4,"context_line":"  \"apiVersion\": \"v1beta1\","}],"source_content_type":"application/json","patch_set":3,"id":"baa201ad_5d7a83ba","line":1,"in_reply_to":"baa201ad_bab45582","updated":"2014-10-17 20:41:35.000000000","message":"Ah yes, I guess I copied this before the yaml conversion took place.","commit_id":"8b63288d0c195270efc5e3ee6be05c7eab435501"},{"author":{"_account_id":2834,"name":"Steven Dake","email":"steven.dake@gmail.com","username":"sdake"},"change_message_id":"f8a62319fc678905a4f98e822ec7cb95e43f0aa6","unresolved":false,"context_lines":[{"line_number":2,"context_line":"  \"id\": \"heat-api\","},{"line_number":3,"context_line":"  \"kind\": \"Service\","},{"line_number":4,"context_line":"  \"apiVersion\": \"v1beta1\","},{"line_number":5,"context_line":"  \"port\": 8004,"},{"line_number":6,"context_line":"  \"containerPort\": 8004,"},{"line_number":7,"context_line":"  \"selector\": {"},{"line_number":8,"context_line":"    \"name\": \"heat\""}],"source_content_type":"application/json","patch_set":3,"id":"baa201ad_9ac151e6","line":5,"updated":"2014-10-17 19:49:54.000000000","message":"i\u0027m not certain but I don\u0027t think you need port here.","commit_id":"8b63288d0c195270efc5e3ee6be05c7eab435501"},{"author":{"_account_id":3098,"name":"Jeff Peeler","email":"jpeeler@redhat.com","username":"jpeeler-z"},"change_message_id":"b56101679bc5605e7de1ea780b07138bc0462f82","unresolved":false,"context_lines":[{"line_number":2,"context_line":"  \"id\": \"heat-api\","},{"line_number":3,"context_line":"  \"kind\": \"Service\","},{"line_number":4,"context_line":"  \"apiVersion\": \"v1beta1\","},{"line_number":5,"context_line":"  \"port\": 8004,"},{"line_number":6,"context_line":"  \"containerPort\": 8004,"},{"line_number":7,"context_line":"  \"selector\": {"},{"line_number":8,"context_line":"    \"name\": \"heat\""}],"source_content_type":"application/json","patch_set":3,"id":"baa201ad_3d7f7fac","line":5,"in_reply_to":"baa201ad_9ac151e6","updated":"2014-10-17 20:41:35.000000000","message":"I recall you saying to ignore this comment. However, do I need to add EXPOSE in the Dockerfile too?","commit_id":"8b63288d0c195270efc5e3ee6be05c7eab435501"}]}