)]}'
{"playbooks/lxc-hosts-setup.yml":[{"author":{"_account_id":6816,"name":"Jesse Pretorius","email":"jesse@odyssey4.me","username":"jesse-pretorius"},"change_message_id":"5fb3a9629895e5cfb221bed8e7631f284af4721b","unresolved":false,"context_lines":[{"line_number":27,"context_line":"      tags:"},{"line_number":28,"context_line":"        - \"lxc-rsyslog-client\""},{"line_number":29,"context_line":"        - \"rsyslog-client\""},{"line_number":30,"context_line":"  vars:"},{"line_number":31,"context_line":"    pip_lock_to_internal_repo: False"}],"source_content_type":"text/x-yaml","patch_set":8,"id":"dada55a8_639b9e73","line":31,"range":{"start_line":30,"start_character":0,"end_line":31,"end_character":36},"updated":"2016-07-22 08:58:18.000000000","message":"I think it may be best for us to create a group for lxc hosts, then target the execution of this playbook against that group (instead of against all hosts). This var can then be set in the group_var for it.","commit_id":"b92aa7d4fb8178e0fc3912d494196604c7a1830d"},{"author":{"_account_id":7353,"name":"Kevin Carter","email":"kevin@cloudnull.com","username":"cloudnull"},"change_message_id":"8f7b522f8e323c1fc839103cb5823ec7c64ec4c5","unresolved":false,"context_lines":[{"line_number":28,"context_line":"        - \"lxc-rsyslog-client\""},{"line_number":29,"context_line":"        - \"rsyslog-client\""},{"line_number":30,"context_line":"  vars:"},{"line_number":31,"context_line":"    pip_lock_to_internal_repo: False"}],"source_content_type":"text/x-yaml","patch_set":8,"id":"dada55a8_c076b205","line":31,"range":{"start_line":31,"start_character":4,"end_line":31,"end_character":29},"updated":"2016-07-22 12:59:21.000000000","message":"What is the reason that we\u0027re telling host machines running lxc that they\u0027re not supposed to lock down to the internal repo?","commit_id":"b92aa7d4fb8178e0fc3912d494196604c7a1830d"},{"author":{"_account_id":14805,"name":"Jimmy McCrory","email":"jimmy.mccrory@gmail.com","username":"jimmy-mccrory"},"change_message_id":"0f7b812d6bbf6d7ec366daf5f19a0527b74b5942","unresolved":false,"context_lines":[{"line_number":28,"context_line":"        - \"lxc-rsyslog-client\""},{"line_number":29,"context_line":"        - \"rsyslog-client\""},{"line_number":30,"context_line":"  vars:"},{"line_number":31,"context_line":"    pip_lock_to_internal_repo: False"}],"source_content_type":"text/x-yaml","patch_set":8,"id":"bacf61ea_96be3763","line":31,"range":{"start_line":31,"start_character":4,"end_line":31,"end_character":29},"in_reply_to":"dada55a8_40875dd7","updated":"2016-07-30 16:52:29.000000000","message":"I think this will need to either be here or the lxc_hosts role updated with an \u0027--isolated\u0027 fallback.\n\nDeployers have the choice to deploy containers on compute, swift, or storage hosts so it\u0027s hard to say definitively which hosts would be part of an \u0027lxc hosts\u0027 group, but they also have the option to define it themselves (lxc_host_group) to limit this playbook.\n\nIn the case of the AIO the single host would be a member of each of those groups, and I\u0027m not sure how ansible would choose which group var to apply.","commit_id":"b92aa7d4fb8178e0fc3912d494196604c7a1830d"},{"author":{"_account_id":7353,"name":"Kevin Carter","email":"kevin@cloudnull.com","username":"cloudnull"},"change_message_id":"8f7b522f8e323c1fc839103cb5823ec7c64ec4c5","unresolved":false,"context_lines":[{"line_number":27,"context_line":"      tags:"},{"line_number":28,"context_line":"        - \"lxc-rsyslog-client\""},{"line_number":29,"context_line":"        - \"rsyslog-client\""},{"line_number":30,"context_line":"  vars:"},{"line_number":31,"context_line":"    pip_lock_to_internal_repo: False"}],"source_content_type":"text/x-yaml","patch_set":8,"id":"dada55a8_80f79a8b","line":31,"range":{"start_line":30,"start_character":0,"end_line":31,"end_character":36},"in_reply_to":"dada55a8_639b9e73","updated":"2016-07-22 12:59:21.000000000","message":"The default execution group for this playbook is \"hosts\" which would cover all host machines in a deployment.","commit_id":"b92aa7d4fb8178e0fc3912d494196604c7a1830d"},{"author":{"_account_id":6816,"name":"Jesse Pretorius","email":"jesse@odyssey4.me","username":"jesse-pretorius"},"change_message_id":"40aa719964f8c2a7a3f6661928d79ebc6787033f","unresolved":false,"context_lines":[{"line_number":28,"context_line":"        - \"lxc-rsyslog-client\""},{"line_number":29,"context_line":"        - \"rsyslog-client\""},{"line_number":30,"context_line":"  vars:"},{"line_number":31,"context_line":"    pip_lock_to_internal_repo: False"}],"source_content_type":"text/x-yaml","patch_set":8,"id":"dada55a8_40875dd7","line":31,"range":{"start_line":31,"start_character":4,"end_line":31,"end_character":29},"in_reply_to":"dada55a8_c076b205","updated":"2016-07-22 14:20:30.000000000","message":"@Kevin Because the LXC hosts exist before the repo containers do.\n\nNote that my suggestion to make a group for the LXC hosts exclusively is partially due to the fact that currently LXC is installed on all hosts which includes compute, swift and storage hosts.... which is a complete waste of time, bandwidth and disk space and more overhead on the hosts which isn\u0027t required.","commit_id":"b92aa7d4fb8178e0fc3912d494196604c7a1830d"}],"playbooks/os-neutron-install.yml":[{"author":{"_account_id":6816,"name":"Jesse Pretorius","email":"jesse@odyssey4.me","username":"jesse-pretorius"},"change_message_id":"5fb3a9629895e5cfb221bed8e7631f284af4721b","unresolved":false,"context_lines":[{"line_number":113,"context_line":"  vars:"},{"line_number":114,"context_line":"    is_metal: \"{{ properties.is_metal|default(false) }}\""},{"line_number":115,"context_line":"    bind_prefix: \"{{ provider_network_bind_prefix|default(\u0027\u0027) }}\""},{"line_number":116,"context_line":"    neutron_rabbitmq_userid: neutron"},{"line_number":117,"context_line":"    neutron_rabbitmq_vhost: /neutron"},{"line_number":118,"context_line":"    neutron_rabbitmq_servers: \"{{ rabbitmq_servers }}\""},{"line_number":119,"context_line":"    neutron_rabbitmq_port: \"{{ rabbitmq_port }}\""},{"line_number":120,"context_line":"    neutron_rabbitmq_use_ssl: \"{{ rabbitmq_use_ssl }}\""}],"source_content_type":"text/x-yaml","patch_set":8,"id":"dada55a8_83f052cb","line":120,"range":{"start_line":116,"start_character":0,"end_line":120,"end_character":54},"updated":"2016-07-22 08:58:18.000000000","message":"Are these needed here? Surely these are already in group_vars, or should be set there?","commit_id":"b92aa7d4fb8178e0fc3912d494196604c7a1830d"}]}