)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":11869,"name":"zhubingbing","email":"zhu.bingbing@99cloud.net","username":"zhubingbing"},"change_message_id":"e7acbb30f4f1db671e5ff2a79ec62f933a8e7a11","unresolved":false,"context_lines":[{"line_number":10,"context_line":""},{"line_number":11,"context_line":"Introduced host groups for monitoring and influxdb and assign role"},{"line_number":12,"context_line":""},{"line_number":13,"context_line":"Monitoring is deployed on the first control node by defaut"},{"line_number":14,"context_line":""},{"line_number":15,"context_line":"Change-Id: If2465a14b18c6c3fd657af587a0b85f6b7a0191a"},{"line_number":16,"context_line":"Partially-Implements: Blueprint performance-monitoring"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":22,"id":"fa7ab95a_cab4c283","line":13,"range":{"start_line":13,"start_character":0,"end_line":13,"end_character":58},"updated":"2016-08-29 16:07:57.000000000","message":"good idea,I will download the test, if there is a need to be able to contact me in the IRC, we will discuss","commit_id":"f812eff50a7b4c4335fc6313af08efded27ed398"}],"ansible/group_vars/all.yml":[{"author":{"_account_id":16482,"name":"Shaun Smekel","email":"shaun.smekel@theorem.net.au","username":"smekel"},"change_message_id":"27c79121f71b08792a2921448e2165e0c9ef8b1d","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":13,"id":"9ad45d7e_73a730bd","updated":"2016-08-11 11:49:59.000000000","message":"enable_performance_monitoring should be defined here as well as in the etc/kolla/globals.yml","commit_id":"4fa3cb857be2e602a9c5fa1f60da5e990068573c"},{"author":{"_account_id":17938,"name":"Mathias Ewald","email":"mewald@evoila.de","username":"mewald"},"change_message_id":"c3e6734b940274c998db33a7d39e8a71746252e6","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":13,"id":"3ac371cc_123d82bb","in_reply_to":"3ac371cc_a429d08b","updated":"2016-08-16 09:14:32.000000000","message":"Sorry it is actually the other way around. I changed dependencies a while ago and enable_perofrmance_monitoring should have gone into this patch. Fixed it.","commit_id":"4fa3cb857be2e602a9c5fa1f60da5e990068573c"},{"author":{"_account_id":17938,"name":"Mathias Ewald","email":"mewald@evoila.de","username":"mewald"},"change_message_id":"9cc75dd38847c01d44456283ddc628b5b4542eb5","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":13,"id":"3ac371cc_a429d08b","in_reply_to":"9ad45d7e_73a730bd","updated":"2016-08-16 08:43:40.000000000","message":"Well, this patch is based on https://review.openstack.org/#/c/346448/ and I am still waiting for this to be accepted. Once that\u0027s done we have the enable_performance_monitoring","commit_id":"4fa3cb857be2e602a9c5fa1f60da5e990068573c"},{"author":{"_account_id":11869,"name":"zhubingbing","email":"zhu.bingbing@99cloud.net","username":"zhubingbing"},"change_message_id":"e7acbb30f4f1db671e5ff2a79ec62f933a8e7a11","unresolved":false,"context_lines":[{"line_number":233,"context_line":"enable_murano: \"no\""},{"line_number":234,"context_line":"enable_neutron_lbaas: \"no\""},{"line_number":235,"context_line":"enable_neutron_qos: \"no\""},{"line_number":236,"context_line":"enable_performance_monitoring: \"no\""},{"line_number":237,"context_line":"enable_swift: \"no\""},{"line_number":238,"context_line":"enable_tempest: \"no\""},{"line_number":239,"context_line":"enable_watcher: \"no\""}],"source_content_type":"text/x-yaml","patch_set":22,"id":"fa7ab95a_277c871e","line":236,"range":{"start_line":236,"start_character":0,"end_line":236,"end_character":35},"updated":"2016-08-29 16:07:57.000000000","message":"I think it should enable_infuxdb: \"no\"","commit_id":"f812eff50a7b4c4335fc6313af08efded27ed398"}],"ansible/inventory/multinode":[{"author":{"_account_id":11869,"name":"zhubingbing","email":"zhu.bingbing@99cloud.net","username":"zhubingbing"},"change_message_id":"86e68016615befa75971638ed55417ff7f1a1aab","unresolved":false,"context_lines":[{"line_number":25,"context_line":"storage01"},{"line_number":26,"context_line":""},{"line_number":27,"context_line":"[monitoring]"},{"line_number":28,"context_line":"mon01"},{"line_number":29,"context_line":""},{"line_number":30,"context_line":"# You can explicitly specify which hosts run each project by updating the"},{"line_number":31,"context_line":"# groups in the sections below. Common services are grouped together."}],"source_content_type":"application/octet-stream","patch_set":5,"id":"fa7ab95a_c7af8592","line":28,"range":{"start_line":28,"start_character":0,"end_line":28,"end_character":5},"updated":"2016-08-29 01:43:27.000000000","message":"why do you want to abbreviation here, Because kolla\u0027s code style should be uniform. you can refer to the above.\n[compute]\ncompute01\n[storage]\nstorage01","commit_id":"466976b20a58fae00157e35bc293486afeda476a"},{"author":{"_account_id":167,"name":"Christian Berendt","email":"berendt@osism.tech","username":"berendt"},"change_message_id":"789c09ba8b220fad8654e077fd0debdb6ae42748","unresolved":false,"context_lines":[{"line_number":25,"context_line":"storage01"},{"line_number":26,"context_line":""},{"line_number":27,"context_line":"[monitoring]"},{"line_number":28,"context_line":"mon01"},{"line_number":29,"context_line":""},{"line_number":30,"context_line":"# You can explicitly specify which hosts run each project by updating the"},{"line_number":31,"context_line":"# groups in the sections below. Common services are grouped together."}],"source_content_type":"application/octet-stream","patch_set":10,"id":"bacf61ea_a8686222","line":28,"updated":"2016-08-04 10:48:03.000000000","message":"I would prefer to place this on the control nodes and not to introduce an other class of hosts. E.g. elasticsearch is also running on the control nodes.","commit_id":"3fed1cff97661996da84333fa0ead2661c7b79a1"},{"author":{"_account_id":10787,"name":"Michal Jastrzebski (inc0)","email":"michal.jastrzebski@intel.com","username":"inc0"},"change_message_id":"d7911e955f3f6a33d67d0f9df268e5de72a8117c","unresolved":false,"context_lines":[{"line_number":17,"context_line":"[compute]"},{"line_number":18,"context_line":"compute01"},{"line_number":19,"context_line":""},{"line_number":20,"context_line":"[monitoring]"},{"line_number":21,"context_line":"control"},{"line_number":22,"context_line":""},{"line_number":23,"context_line":"# When compute nodes and control nodes use different interfaces,"}],"source_content_type":"application/octet-stream","patch_set":17,"id":"fa7ab95a_945a344f","line":20,"updated":"2016-08-26 19:25:50.000000000","message":"first of all, I\u0027m not super sure if this group is needed, we can just default to control in all of control-based services like influx. Same as we did with central logging","commit_id":"ca59a186635e4564797f2b385685d0259fd698a8"},{"author":{"_account_id":10787,"name":"Michal Jastrzebski (inc0)","email":"michal.jastrzebski@intel.com","username":"inc0"},"change_message_id":"c7615f2385329fc364ba547a8fe4e7853d525cab","unresolved":false,"context_lines":[{"line_number":18,"context_line":"compute01"},{"line_number":19,"context_line":""},{"line_number":20,"context_line":"[monitoring]"},{"line_number":21,"context_line":"control01"},{"line_number":22,"context_line":""},{"line_number":23,"context_line":"# When compute nodes and control nodes use different interfaces,"},{"line_number":24,"context_line":"# you can specify \"api_interface\" and another interfaces like below:"}],"source_content_type":"application/octet-stream","patch_set":23,"id":"fa7ab95a_acdaf314","line":21,"updated":"2016-08-30 14:55:27.000000000","message":"Wgile I agree that monitoring should get it\u0027s own host group, we should go all the way, please change this to monitoring01 as example hostname. Also we need to make note that this is only one host and we don\u0027t really have HA on influx.","commit_id":"7f02e9e966c2e16a89a3d0919f1ec56a2d36ace0"}],"ansible/roles/influxdb/defaults/main.yml":[{"author":{"_account_id":10419,"name":"Ryan Hallisey","email":"rhallise@redhat.com","username":"rhallisey"},"change_message_id":"976d814f9f9ec2c977c56f56aa1358eff46fd981","unresolved":false,"context_lines":[{"line_number":1,"context_line":"---"},{"line_number":2,"context_line":"project_name: \"influxdb\""},{"line_number":3,"context_line":""},{"line_number":4,"context_line":"influxdb_image: \"{{ docker_registry ~ \u0027/\u0027 if docker_registry else \u0027\u0027 }}{{ docker_namespace }}/{{ kolla_base_distro }}-{{ kolla_install_type }}-influxdb\""},{"line_number":5,"context_line":"influxdb_tag: \"{{ openstack_release }}\""},{"line_number":6,"context_line":"influxdb_image_full: \"{{ influxdb_image }}:{{ influxdb_tag }}\""}],"source_content_type":"text/x-yaml","patch_set":5,"id":"dada55a8_1b8a3864","line":3,"updated":"2016-07-27 13:58:35.000000000","message":"If you do another revision can you put this header in front of this section.\n\n####################\n# Docker\n####################","commit_id":"466976b20a58fae00157e35bc293486afeda476a"},{"author":{"_account_id":11869,"name":"zhubingbing","email":"zhu.bingbing@99cloud.net","username":"zhubingbing"},"change_message_id":"86e68016615befa75971638ed55417ff7f1a1aab","unresolved":false,"context_lines":[{"line_number":1,"context_line":"---"},{"line_number":2,"context_line":"project_name: \"influxdb\""},{"line_number":3,"context_line":""},{"line_number":4,"context_line":"influxdb_image: \"{{ docker_registry ~ \u0027/\u0027 if docker_registry else \u0027\u0027 }}{{ docker_namespace }}/{{ kolla_base_distro }}-{{ kolla_install_type }}-influxdb\""},{"line_number":5,"context_line":"influxdb_tag: \"{{ openstack_release }}\""},{"line_number":6,"context_line":"influxdb_image_full: \"{{ influxdb_image }}:{{ influxdb_tag }}\""}],"source_content_type":"text/x-yaml","patch_set":5,"id":"fa7ab95a_67895926","line":3,"in_reply_to":"bacf61ea_e0cbf81c","updated":"2016-08-29 01:43:27.000000000","message":"i agree it","commit_id":"466976b20a58fae00157e35bc293486afeda476a"},{"author":{"_account_id":17938,"name":"Mathias Ewald","email":"mewald@evoila.de","username":"mewald"},"change_message_id":"a9840010365c71989b287352d3dfd1ae7a4120cb","unresolved":false,"context_lines":[{"line_number":1,"context_line":"---"},{"line_number":2,"context_line":"project_name: \"influxdb\""},{"line_number":3,"context_line":""},{"line_number":4,"context_line":"influxdb_image: \"{{ docker_registry ~ \u0027/\u0027 if docker_registry else \u0027\u0027 }}{{ docker_namespace }}/{{ kolla_base_distro }}-{{ kolla_install_type }}-influxdb\""},{"line_number":5,"context_line":"influxdb_tag: \"{{ openstack_release }}\""},{"line_number":6,"context_line":"influxdb_image_full: \"{{ influxdb_image }}:{{ influxdb_tag }}\""}],"source_content_type":"text/x-yaml","patch_set":5,"id":"bacf61ea_e0cbf81c","line":3,"in_reply_to":"dada55a8_1b8a3864","updated":"2016-07-27 19:58:50.000000000","message":"done","commit_id":"466976b20a58fae00157e35bc293486afeda476a"}],"ansible/roles/influxdb/meta/main.yml":[{"author":{"_account_id":11869,"name":"zhubingbing","email":"zhu.bingbing@99cloud.net","username":"zhubingbing"},"change_message_id":"86e68016615befa75971638ed55417ff7f1a1aab","unresolved":false,"context_lines":[{"line_number":1,"context_line":"---"},{"line_number":2,"context_line":"dependencies:"},{"line_number":3,"context_line":"  - { role: common, project_yaml: \u0027influxdb.yml\u0027 }"}],"source_content_type":"text/x-yaml","patch_set":5,"id":"fa7ab95a_a7836103","line":3,"range":{"start_line":3,"start_character":35,"end_line":3,"end_character":47},"updated":"2016-08-29 01:43:27.000000000","message":"if you don\u0027t use it, please delete it","commit_id":"466976b20a58fae00157e35bc293486afeda476a"},{"author":{"_account_id":16993,"name":"Hui Kang","email":"kangh@us.ibm.com","username":"huikang"},"change_message_id":"d74e91479310b95bd27ea083e8b9208d89031ed6","unresolved":false,"context_lines":[{"line_number":1,"context_line":"---"},{"line_number":2,"context_line":"dependencies:"},{"line_number":3,"context_line":"  - { role: common, project_yaml: \u0027influxdb.yml\u0027 }"}],"source_content_type":"text/x-yaml","patch_set":16,"id":"fa7ab95a_93ac40ef","line":3,"updated":"2016-08-26 04:02:19.000000000","message":"remove project_yaml","commit_id":"7a148f80d80bb40e77a74cdfbb5226704d817c90"}],"ansible/roles/influxdb/tasks/config.yml":[{"author":{"_account_id":167,"name":"Christian Berendt","email":"berendt@osism.tech","username":"berendt"},"change_message_id":"789c09ba8b220fad8654e077fd0debdb6ae42748","unresolved":false,"context_lines":[{"line_number":20,"context_line":"    dest: \"{{ node_config_directory }}/influxdb/influxdb.conf\""},{"line_number":21,"context_line":"  when: inventory_hostname in groups[\u0027influxdb\u0027]"},{"line_number":22,"context_line":"  with_items:"},{"line_number":23,"context_line":"    - influxdb.conf"}],"source_content_type":"text/x-yaml","patch_set":10,"id":"bacf61ea_28cab21a","line":23,"updated":"2016-08-04 10:48:03.000000000","message":"rename influxdb.conf to influxdb.conf.j2","commit_id":"3fed1cff97661996da84333fa0ead2661c7b79a1"},{"author":{"_account_id":16482,"name":"Shaun Smekel","email":"shaun.smekel@theorem.net.au","username":"smekel"},"change_message_id":"27c79121f71b08792a2921448e2165e0c9ef8b1d","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":13,"id":"9ad45d7e_cec913ae","updated":"2016-08-11 11:49:59.000000000","message":"Config.yml only runs when inventory_hostname in groups[\u0027influxdb\u0027]. You can remove the when conditions.","commit_id":"4fa3cb857be2e602a9c5fa1f60da5e990068573c"},{"author":{"_account_id":14103,"name":"wangtianfa","email":"wang.tianfa@99cloud.net","username":"wangtianfa"},"change_message_id":"a386c7d981f79242d757ab9135b93821693aa237","unresolved":false,"context_lines":[{"line_number":4,"context_line":"    path: \"{{ node_config_directory }}/influxdb\""},{"line_number":5,"context_line":"    state: \"directory\""},{"line_number":6,"context_line":"    recurse: yes"},{"line_number":7,"context_line":"  when: inventory_hostname in groups[\u0027influxdb\u0027]"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"- name: Copying over config.json files"},{"line_number":10,"context_line":"  template:"}],"source_content_type":"text/x-yaml","patch_set":13,"id":"9ad45d7e_b96fc7fa","line":7,"range":{"start_line":7,"start_character":8,"end_line":7,"end_character":26},"updated":"2016-08-11 12:08:26.000000000","message":"can remove the when conditions.","commit_id":"4fa3cb857be2e602a9c5fa1f60da5e990068573c"},{"author":{"_account_id":16482,"name":"Shaun Smekel","email":"shaun.smekel@theorem.net.au","username":"smekel"},"change_message_id":"27c79121f71b08792a2921448e2165e0c9ef8b1d","unresolved":false,"context_lines":[{"line_number":12,"context_line":"    dest: \"{{ node_config_directory }}/influxdb/config.json\""},{"line_number":13,"context_line":"  when: inventory_hostname in groups[\u0027influxdb\u0027]"},{"line_number":14,"context_line":"  with_items:"},{"line_number":15,"context_line":"    - influxdb.json.j2"},{"line_number":16,"context_line":""},{"line_number":17,"context_line":"- name: Copying over influxdb config file"},{"line_number":18,"context_line":"  template:"}],"source_content_type":"text/x-yaml","patch_set":13,"id":"9ad45d7e_0e768b6f","line":15,"updated":"2016-08-11 11:49:59.000000000","message":"This could be programmatically more useful if it was influxdb instead of influxdb.json.j2. Especially if accessory services are added in the future","commit_id":"4fa3cb857be2e602a9c5fa1f60da5e990068573c"},{"author":{"_account_id":17938,"name":"Mathias Ewald","email":"mewald@evoila.de","username":"mewald"},"change_message_id":"175ba24e7c581e4e0b1b3b8bc49b575f47fb7603","unresolved":false,"context_lines":[{"line_number":12,"context_line":"    dest: \"{{ node_config_directory }}/influxdb/config.json\""},{"line_number":13,"context_line":"  when: inventory_hostname in groups[\u0027influxdb\u0027]"},{"line_number":14,"context_line":"  with_items:"},{"line_number":15,"context_line":"    - influxdb.json.j2"},{"line_number":16,"context_line":""},{"line_number":17,"context_line":"- name: Copying over influxdb config file"},{"line_number":18,"context_line":"  template:"}],"source_content_type":"text/x-yaml","patch_set":13,"id":"3ac371cc_61e34a34","line":15,"in_reply_to":"9ad45d7e_0e768b6f","updated":"2016-08-16 08:28:16.000000000","message":"I don\u0027t unterstand. Can you explain further?","commit_id":"4fa3cb857be2e602a9c5fa1f60da5e990068573c"},{"author":{"_account_id":16482,"name":"Shaun Smekel","email":"shaun.smekel@theorem.net.au","username":"smekel"},"change_message_id":"27c79121f71b08792a2921448e2165e0c9ef8b1d","unresolved":false,"context_lines":[{"line_number":20,"context_line":"    dest: \"{{ node_config_directory }}/influxdb/influxdb.conf\""},{"line_number":21,"context_line":"  when: inventory_hostname in groups[\u0027influxdb\u0027]"},{"line_number":22,"context_line":"  with_items:"},{"line_number":23,"context_line":"    - influxdb.conf.j2"}],"source_content_type":"text/x-yaml","patch_set":13,"id":"9ad45d7e_6e0bc7d6","line":23,"updated":"2016-08-11 11:49:59.000000000","message":"Ditto","commit_id":"4fa3cb857be2e602a9c5fa1f60da5e990068573c"},{"author":{"_account_id":16482,"name":"Shaun Smekel","email":"shaun.smekel@theorem.net.au","username":"smekel"},"change_message_id":"0f92fbd03669444648414b8b3b9a474d7467dcc1","unresolved":false,"context_lines":[{"line_number":11,"context_line":"    dest: \"{{ node_config_directory }}/influxdb/config.json\""},{"line_number":12,"context_line":"  when: inventory_hostname in groups[\u0027influxdb\u0027]"},{"line_number":13,"context_line":"  with_items:"},{"line_number":14,"context_line":"    - influxdb.json.j2"},{"line_number":15,"context_line":""},{"line_number":16,"context_line":"- name: Copying over influxdb config file"},{"line_number":17,"context_line":"  template:"}],"source_content_type":"text/x-yaml","patch_set":14,"id":"3ac371cc_7de61bb1","line":14,"updated":"2016-08-17 00:53:27.000000000","message":"You could refactor it to look like so. This will allow for potentially adding other items to the \"with_items\" list.\n- name: Copying over config.json files\n  template:\n    src: \"{{ item  }}.json.j2\"\n    dest: \"{{ node_config_directory }}/{{ item  }}/config.json\"\n  with_items:\n    - influxdb","commit_id":"c87bd13bf8c198da26fd4d7d3c44b2542f203c43"},{"author":{"_account_id":16482,"name":"Shaun Smekel","email":"shaun.smekel@theorem.net.au","username":"smekel"},"change_message_id":"d87b10c9cc055f11b46c2ad0bac6082840cc5a2a","unresolved":false,"context_lines":[{"line_number":17,"context_line":"  template:"},{"line_number":18,"context_line":"    src: \"{{ role_path }}/templates/{{ item }}\""},{"line_number":19,"context_line":"    dest: \"{{ node_config_directory }}/influxdb/influxdb.conf\""},{"line_number":20,"context_line":"  when: inventory_hostname in groups[\u0027influxdb\u0027]"},{"line_number":21,"context_line":"  with_items:"},{"line_number":22,"context_line":"    - influxdb.conf.j2"}],"source_content_type":"text/x-yaml","patch_set":14,"id":"3ac371cc_fd75ebf9","line":20,"updated":"2016-08-17 00:52:18.000000000","message":"When can be removed here as well","commit_id":"c87bd13bf8c198da26fd4d7d3c44b2542f203c43"},{"author":{"_account_id":16993,"name":"Hui Kang","email":"kangh@us.ibm.com","username":"huikang"},"change_message_id":"9d1058b9373cb533ee94ea4e7d82ff660b2d50f0","unresolved":false,"context_lines":[{"line_number":7,"context_line":""},{"line_number":8,"context_line":"- name: Copying over config.json files"},{"line_number":9,"context_line":"  template:"},{"line_number":10,"context_line":"    src: \"{{ item  }}.json.j2\""},{"line_number":11,"context_line":"    dest: \"{{ node_config_directory }}/{{ item }}/config.json\""},{"line_number":12,"context_line":"  with_items:"},{"line_number":13,"context_line":"    - influxdb"}],"source_content_type":"text/x-yaml","patch_set":15,"id":"3ac371cc_6c9256a1","line":10,"updated":"2016-08-19 05:00:25.000000000","message":"remove one space after \"item\"","commit_id":"6efd6a7b53e0829eefdd28d4cdc5910f0919b77e"},{"author":{"_account_id":16993,"name":"Hui Kang","email":"kangh@us.ibm.com","username":"huikang"},"change_message_id":"9d1058b9373cb533ee94ea4e7d82ff660b2d50f0","unresolved":false,"context_lines":[{"line_number":12,"context_line":"  with_items:"},{"line_number":13,"context_line":"    - influxdb"},{"line_number":14,"context_line":""},{"line_number":15,"context_line":"- name: Copying over influxdb config file"},{"line_number":16,"context_line":"  template:"},{"line_number":17,"context_line":"    src: \"{{ role_path }}/templates/{{ item }}\""},{"line_number":18,"context_line":"    dest: \"{{ node_config_directory }}/influxdb/influxdb.conf\""}],"source_content_type":"text/x-yaml","patch_set":15,"id":"3ac371cc_8cc3528a","line":15,"updated":"2016-08-19 05:00:25.000000000","message":"Change name to \" Copying over influxdb configuration\"","commit_id":"6efd6a7b53e0829eefdd28d4cdc5910f0919b77e"},{"author":{"_account_id":16993,"name":"Hui Kang","email":"kangh@us.ibm.com","username":"huikang"},"change_message_id":"b04775f18ede659cbb4f79dd285cfac6a13b34a7","unresolved":false,"context_lines":[{"line_number":8,"context_line":""},{"line_number":9,"context_line":"- name: Copying over config.json files"},{"line_number":10,"context_line":"  template:"},{"line_number":11,"context_line":"    src: \"{{ item  }}\""},{"line_number":12,"context_line":"    dest: \"{{ node_config_directory }}/influxdb/config.json\""},{"line_number":13,"context_line":"  when: inventory_hostname in groups[\u0027influxdb\u0027]"},{"line_number":14,"context_line":"  with_items:"}],"source_content_type":"text/x-yaml","patch_set":28,"id":"fa7ab95a_99123d08","line":11,"updated":"2016-09-01 14:17:16.000000000","message":"remove one space after item","commit_id":"a2fb78455a24b51e8654ef0724778a07fbd99542"},{"author":{"_account_id":11869,"name":"zhubingbing","email":"zhu.bingbing@99cloud.net","username":"zhubingbing"},"change_message_id":"f1c823a5beeefd60c0dc42d83ce84724683b33f5","unresolved":false,"context_lines":[{"line_number":8,"context_line":""},{"line_number":9,"context_line":"- name: Copying over config.json files"},{"line_number":10,"context_line":"  template:"},{"line_number":11,"context_line":"    src: \"{{ item  }}\""},{"line_number":12,"context_line":"    dest: \"{{ node_config_directory }}/influxdb/config.json\""},{"line_number":13,"context_line":"  when: inventory_hostname in groups[\u0027influxdb\u0027]"},{"line_number":14,"context_line":"  with_items:"}],"source_content_type":"text/x-yaml","patch_set":28,"id":"fa7ab95a_7023c3a7","line":11,"in_reply_to":"fa7ab95a_99123d08","updated":"2016-09-01 16:48:04.000000000","message":"Done","commit_id":"a2fb78455a24b51e8654ef0724778a07fbd99542"},{"author":{"_account_id":16993,"name":"Hui Kang","email":"kangh@us.ibm.com","username":"huikang"},"change_message_id":"b04775f18ede659cbb4f79dd285cfac6a13b34a7","unresolved":false,"context_lines":[{"line_number":12,"context_line":"    dest: \"{{ node_config_directory }}/influxdb/config.json\""},{"line_number":13,"context_line":"  when: inventory_hostname in groups[\u0027influxdb\u0027]"},{"line_number":14,"context_line":"  with_items:"},{"line_number":15,"context_line":"    - influxdb.json.j2"},{"line_number":16,"context_line":""},{"line_number":17,"context_line":"- name: Copying over influxdb config file"},{"line_number":18,"context_line":"  template:"}],"source_content_type":"text/x-yaml","patch_set":28,"id":"fa7ab95a_592fd53d","line":15,"updated":"2016-09-01 14:17:16.000000000","message":"Should be \"influxdb\" refer to [1]\n\n[1] https://github.com/openstack/kolla/blob/master/ansible/roles/mariadb/tasks/config.yml#L10","commit_id":"a2fb78455a24b51e8654ef0724778a07fbd99542"},{"author":{"_account_id":11869,"name":"zhubingbing","email":"zhu.bingbing@99cloud.net","username":"zhubingbing"},"change_message_id":"f1c823a5beeefd60c0dc42d83ce84724683b33f5","unresolved":false,"context_lines":[{"line_number":12,"context_line":"    dest: \"{{ node_config_directory }}/influxdb/config.json\""},{"line_number":13,"context_line":"  when: inventory_hostname in groups[\u0027influxdb\u0027]"},{"line_number":14,"context_line":"  with_items:"},{"line_number":15,"context_line":"    - influxdb.json.j2"},{"line_number":16,"context_line":""},{"line_number":17,"context_line":"- name: Copying over influxdb config file"},{"line_number":18,"context_line":"  template:"}],"source_content_type":"text/x-yaml","patch_set":28,"id":"fa7ab95a_30505b0a","line":15,"in_reply_to":"fa7ab95a_592fd53d","updated":"2016-09-01 16:48:04.000000000","message":"ok i fixed it now","commit_id":"a2fb78455a24b51e8654ef0724778a07fbd99542"},{"author":{"_account_id":17938,"name":"Mathias Ewald","email":"mewald@evoila.de","username":"mewald"},"change_message_id":"709ad5e2491f3fdbdcf1f2108019455734b3d840","unresolved":false,"context_lines":[{"line_number":12,"context_line":"    dest: \"{{ node_config_directory }}/influxdb/config.json\""},{"line_number":13,"context_line":"  when: inventory_hostname in groups[\u0027influxdb\u0027]"},{"line_number":14,"context_line":"  with_items:"},{"line_number":15,"context_line":"    - influxdb2"},{"line_number":16,"context_line":""},{"line_number":17,"context_line":"- name: Copying over influxdb config file"},{"line_number":18,"context_line":"  template:"}],"source_content_type":"text/x-yaml","patch_set":29,"id":"fa7ab95a_86280a35","line":15,"range":{"start_line":15,"start_character":6,"end_line":15,"end_character":15},"updated":"2016-09-01 17:18:28.000000000","message":"why \"influxdb2\"?","commit_id":"042d87c71efe08f0813beeaf24aa2b40ad2ff837"},{"author":{"_account_id":7488,"name":"Jeffrey Zhang","email":"zhang.lei.fly@gmail.com","username":"Jeffrey4l"},"change_message_id":"8670469d3b03bbce831b59aa15dd322d42e10a49","unresolved":false,"context_lines":[{"line_number":12,"context_line":"    dest: \"{{ node_config_directory }}/influxdb/config.json\""},{"line_number":13,"context_line":"  when: inventory_hostname in groups[\u0027influxdb\u0027]"},{"line_number":14,"context_line":"  with_items:"},{"line_number":15,"context_line":"    - influxdb2"},{"line_number":16,"context_line":""},{"line_number":17,"context_line":"- name: Copying over influxdb config file"},{"line_number":18,"context_line":"  template:"}],"source_content_type":"text/x-yaml","patch_set":29,"id":"fa7ab95a_154c10c7","line":15,"updated":"2016-09-02 02:45:18.000000000","message":"ya. the \u00272\u0027 char is not needed.","commit_id":"042d87c71efe08f0813beeaf24aa2b40ad2ff837"},{"author":{"_account_id":11869,"name":"zhubingbing","email":"zhu.bingbing@99cloud.net","username":"zhubingbing"},"change_message_id":"f202d11546c59db7f34502495bb9a8a6ec8eb6ee","unresolved":false,"context_lines":[{"line_number":12,"context_line":"    dest: \"{{ node_config_directory }}/influxdb/config.json\""},{"line_number":13,"context_line":"  when: inventory_hostname in groups[\u0027influxdb\u0027]"},{"line_number":14,"context_line":"  with_items:"},{"line_number":15,"context_line":"    - influxdb2"},{"line_number":16,"context_line":""},{"line_number":17,"context_line":"- name: Copying over influxdb config file"},{"line_number":18,"context_line":"  template:"}],"source_content_type":"text/x-yaml","patch_set":29,"id":"fa7ab95a_0a04ea47","line":15,"in_reply_to":"fa7ab95a_154c10c7","updated":"2016-09-02 04:12:02.000000000","message":"Done","commit_id":"042d87c71efe08f0813beeaf24aa2b40ad2ff837"},{"author":{"_account_id":11869,"name":"zhubingbing","email":"zhu.bingbing@99cloud.net","username":"zhubingbing"},"change_message_id":"f202d11546c59db7f34502495bb9a8a6ec8eb6ee","unresolved":false,"context_lines":[{"line_number":12,"context_line":"    dest: \"{{ node_config_directory }}/influxdb/config.json\""},{"line_number":13,"context_line":"  when: inventory_hostname in groups[\u0027influxdb\u0027]"},{"line_number":14,"context_line":"  with_items:"},{"line_number":15,"context_line":"    - influxdb2"},{"line_number":16,"context_line":""},{"line_number":17,"context_line":"- name: Copying over influxdb config file"},{"line_number":18,"context_line":"  template:"}],"source_content_type":"text/x-yaml","patch_set":29,"id":"fa7ab95a_4afa7244","line":15,"range":{"start_line":15,"start_character":6,"end_line":15,"end_character":15},"in_reply_to":"fa7ab95a_86280a35","updated":"2016-09-02 04:12:02.000000000","message":"Done","commit_id":"042d87c71efe08f0813beeaf24aa2b40ad2ff837"}],"ansible/roles/influxdb/tasks/deploy.yml":[{"author":{"_account_id":16993,"name":"Hui Kang","email":"kangh@us.ibm.com","username":"huikang"},"change_message_id":"49faf6f384e236fd4f538c083ce939c42103f7e4","unresolved":false,"context_lines":[{"line_number":1,"context_line":"---"},{"line_number":2,"context_line":"- include: config.yml"},{"line_number":3,"context_line":"  when: inventory_hostname in groups[\u0027influxdb\u0027]"},{"line_number":4,"context_line":""},{"line_number":5,"context_line":"- include: start.yml"},{"line_number":6,"context_line":"  when: inventory_hostname in groups[\u0027influxdb\u0027]"}],"source_content_type":"text/x-yaml","patch_set":22,"id":"fa7ab95a_5704b948","line":3,"updated":"2016-08-30 04:50:29.000000000","message":"This when condition is unnecessary because influxdb has only one type of node in the inventory file.","commit_id":"f812eff50a7b4c4335fc6313af08efded27ed398"},{"author":{"_account_id":16993,"name":"Hui Kang","email":"kangh@us.ibm.com","username":"huikang"},"change_message_id":"49faf6f384e236fd4f538c083ce939c42103f7e4","unresolved":false,"context_lines":[{"line_number":3,"context_line":"  when: inventory_hostname in groups[\u0027influxdb\u0027]"},{"line_number":4,"context_line":""},{"line_number":5,"context_line":"- include: start.yml"},{"line_number":6,"context_line":"  when: inventory_hostname in groups[\u0027influxdb\u0027]"}],"source_content_type":"text/x-yaml","patch_set":22,"id":"fa7ab95a_7701bd37","line":6,"updated":"2016-08-30 04:50:29.000000000","message":"ditto","commit_id":"f812eff50a7b4c4335fc6313af08efded27ed398"}],"ansible/roles/influxdb/tasks/do_reconfigure.yml":[{"author":{"_account_id":16482,"name":"Shaun Smekel","email":"shaun.smekel@theorem.net.au","username":"smekel"},"change_message_id":"27c79121f71b08792a2921448e2165e0c9ef8b1d","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":13,"id":"9ad45d7e_ae9bff64","updated":"2016-08-11 11:49:59.000000000","message":"You don\u0027t need the when condition \"inventory_hostname in groups[\u0027influxdb\u0027]\".","commit_id":"4fa3cb857be2e602a9c5fa1f60da5e990068573c"}],"ansible/roles/influxdb/tasks/start.yml":[{"author":{"_account_id":10787,"name":"Michal Jastrzebski (inc0)","email":"michal.jastrzebski@intel.com","username":"inc0"},"change_message_id":"d7911e955f3f6a33d67d0f9df268e5de72a8117c","unresolved":false,"context_lines":[{"line_number":1,"context_line":"---"},{"line_number":2,"context_line":"- name: Starting influxdb container"},{"line_number":3,"context_line":"  kolla_docker:"},{"line_number":4,"context_line":"    action: \"start_container\""}],"source_content_type":"text/x-yaml","patch_set":17,"id":"fa7ab95a_b459f85c","line":1,"updated":"2016-08-26 19:25:50.000000000","message":"do you create named volume anywhere? that will bind to /var/lib/influxdb","commit_id":"ca59a186635e4564797f2b385685d0259fd698a8"},{"author":{"_account_id":16993,"name":"Hui Kang","email":"kangh@us.ibm.com","username":"huikang"},"change_message_id":"fd795152439912ca53158b4dc4556c734e67fb75","unresolved":false,"context_lines":[{"line_number":1,"context_line":"---"},{"line_number":2,"context_line":"- name: Starting influxdb container"},{"line_number":3,"context_line":"  kolla_docker:"},{"line_number":4,"context_line":"    action: \"start_container\""}],"source_content_type":"text/x-yaml","patch_set":17,"id":"fa7ab95a_228af8bf","line":1,"in_reply_to":"fa7ab95a_b459f85c","updated":"2016-08-27 04:18:28.000000000","message":"Michal, the name volume is created automatically. If I understand correctly, when there is no need to check the result of creating named volume, it should be fine here.","commit_id":"ca59a186635e4564797f2b385685d0259fd698a8"}],"ansible/roles/influxdb/tasks/upgrade.yml":[{"author":{"_account_id":14103,"name":"wangtianfa","email":"wang.tianfa@99cloud.net","username":"wangtianfa"},"change_message_id":"e87e353e0bdb306a0c9233d5ed8a65affd9b4fa7","unresolved":false,"context_lines":[{"line_number":1,"context_line":"---"},{"line_number":2,"context_line":"- include: config.yml"},{"line_number":3,"context_line":""},{"line_number":4,"context_line":"- include: bootstrap_service.yml"},{"line_number":5,"context_line":""},{"line_number":6,"context_line":"- include: start.yml"},{"line_number":7,"context_line":"  serial: \"30%\""}],"source_content_type":"text/x-yaml","patch_set":5,"id":"bacf61ea_bc3659b2","line":4,"range":{"start_line":4,"start_character":11,"end_line":4,"end_character":28},"updated":"2016-07-27 15:00:31.000000000","message":"Indeed there is no,it can delete.","commit_id":"466976b20a58fae00157e35bc293486afeda476a"},{"author":{"_account_id":10419,"name":"Ryan Hallisey","email":"rhallise@redhat.com","username":"rhallisey"},"change_message_id":"2f6df8b543463e6063cefcab3c2e81634b93b140","unresolved":false,"context_lines":[{"line_number":1,"context_line":"---"},{"line_number":2,"context_line":"- include: config.yml"},{"line_number":3,"context_line":""},{"line_number":4,"context_line":"- include: bootstrap_service.yml"},{"line_number":5,"context_line":""},{"line_number":6,"context_line":"- include: start.yml"},{"line_number":7,"context_line":"  serial: \"30%\""}],"source_content_type":"text/x-yaml","patch_set":5,"id":"bacf61ea_8b64f149","line":4,"range":{"start_line":4,"start_character":11,"end_line":4,"end_character":28},"updated":"2016-07-27 14:20:59.000000000","message":"this doesn\u0027t exist.  It looks like you don\u0027t need to bootstrap anything looking at the extend_start.sh","commit_id":"466976b20a58fae00157e35bc293486afeda476a"},{"author":{"_account_id":17938,"name":"Mathias Ewald","email":"mewald@evoila.de","username":"mewald"},"change_message_id":"a9840010365c71989b287352d3dfd1ae7a4120cb","unresolved":false,"context_lines":[{"line_number":1,"context_line":"---"},{"line_number":2,"context_line":"- include: config.yml"},{"line_number":3,"context_line":""},{"line_number":4,"context_line":"- include: bootstrap_service.yml"},{"line_number":5,"context_line":""},{"line_number":6,"context_line":"- include: start.yml"},{"line_number":7,"context_line":"  serial: \"30%\""}],"source_content_type":"text/x-yaml","patch_set":5,"id":"bacf61ea_a0e2a09d","line":4,"range":{"start_line":4,"start_character":11,"end_line":4,"end_character":28},"in_reply_to":"bacf61ea_bc3659b2","updated":"2016-07-27 19:58:50.000000000","message":"done","commit_id":"466976b20a58fae00157e35bc293486afeda476a"}],"ansible/roles/influxdb/templates/influxdb.conf":[{"author":{"_account_id":167,"name":"Christian Berendt","email":"berendt@osism.tech","username":"berendt"},"change_message_id":"789c09ba8b220fad8654e077fd0debdb6ae42748","unresolved":false,"context_lines":[{"line_number":32,"context_line":"  store-interval \u003d \"10s\" # The interval at which to record statistics"},{"line_number":33,"context_line":"[admin]"},{"line_number":34,"context_line":"  enabled \u003d true"},{"line_number":35,"context_line":"  bind-address \u003d \":8083\""},{"line_number":36,"context_line":"  https-enabled \u003d false"},{"line_number":37,"context_line":"  https-certificate \u003d \"/etc/ssl/influxdb.pem\""},{"line_number":38,"context_line":"[http]"}],"source_content_type":"text/plain","patch_set":10,"id":"bacf61ea_889f5e02","line":35,"range":{"start_line":35,"start_character":19,"end_line":35,"end_character":23},"updated":"2016-08-04 10:48:03.000000000","message":"I think it makes sense to add the port as variable to the all.yml file like we did it for the other ports.","commit_id":"3fed1cff97661996da84333fa0ead2661c7b79a1"},{"author":{"_account_id":14103,"name":"wangtianfa","email":"wang.tianfa@99cloud.net","username":"wangtianfa"},"change_message_id":"350b0aaedf124fe28d04c764fa1b684bd0bd3f8a","unresolved":false,"context_lines":[{"line_number":32,"context_line":"  store-interval \u003d \"10s\" # The interval at which to record statistics"},{"line_number":33,"context_line":"[admin]"},{"line_number":34,"context_line":"  enabled \u003d true"},{"line_number":35,"context_line":"  bind-address \u003d \":8083\""},{"line_number":36,"context_line":"  https-enabled \u003d false"},{"line_number":37,"context_line":"  https-certificate \u003d \"/etc/ssl/influxdb.pem\""},{"line_number":38,"context_line":"[http]"}],"source_content_type":"text/plain","patch_set":10,"id":"bacf61ea_99632edd","line":35,"range":{"start_line":35,"start_character":19,"end_line":35,"end_character":23},"in_reply_to":"bacf61ea_889f5e02","updated":"2016-08-04 11:26:12.000000000","message":"Whether it puts influxdb/defaults/main.yml is better?","commit_id":"3fed1cff97661996da84333fa0ead2661c7b79a1"},{"author":{"_account_id":167,"name":"Christian Berendt","email":"berendt@osism.tech","username":"berendt"},"change_message_id":"c81c90ed25b58d54342dea12db7120b1f6cbc64e","unresolved":false,"context_lines":[{"line_number":32,"context_line":"  store-interval \u003d \"10s\" # The interval at which to record statistics"},{"line_number":33,"context_line":"[admin]"},{"line_number":34,"context_line":"  enabled \u003d true"},{"line_number":35,"context_line":"  bind-address \u003d \":8083\""},{"line_number":36,"context_line":"  https-enabled \u003d false"},{"line_number":37,"context_line":"  https-certificate \u003d \"/etc/ssl/influxdb.pem\""},{"line_number":38,"context_line":"[http]"}],"source_content_type":"text/plain","patch_set":10,"id":"bacf61ea_fb7e9bad","line":35,"range":{"start_line":35,"start_character":19,"end_line":35,"end_character":23},"in_reply_to":"bacf61ea_99632edd","updated":"2016-08-04 13:13:47.000000000","message":"The other ports are defined in the all.yml file.","commit_id":"3fed1cff97661996da84333fa0ead2661c7b79a1"},{"author":{"_account_id":167,"name":"Christian Berendt","email":"berendt@osism.tech","username":"berendt"},"change_message_id":"789c09ba8b220fad8654e077fd0debdb6ae42748","unresolved":false,"context_lines":[{"line_number":37,"context_line":"  https-certificate \u003d \"/etc/ssl/influxdb.pem\""},{"line_number":38,"context_line":"[http]"},{"line_number":39,"context_line":"  enabled \u003d true"},{"line_number":40,"context_line":"  bind-address \u003d \":8086\""},{"line_number":41,"context_line":"  auth-enabled \u003d false"},{"line_number":42,"context_line":"  log-enabled \u003d true"},{"line_number":43,"context_line":"  write-tracing \u003d false"}],"source_content_type":"text/plain","patch_set":10,"id":"bacf61ea_a884e2b0","line":40,"range":{"start_line":40,"start_character":19,"end_line":40,"end_character":23},"updated":"2016-08-04 10:48:03.000000000","message":"dito","commit_id":"3fed1cff97661996da84333fa0ead2661c7b79a1"},{"author":{"_account_id":167,"name":"Christian Berendt","email":"berendt@osism.tech","username":"berendt"},"change_message_id":"789c09ba8b220fad8654e077fd0debdb6ae42748","unresolved":false,"context_lines":[{"line_number":49,"context_line":"  enabled \u003d false"},{"line_number":50,"context_line":"[[collectd]]"},{"line_number":51,"context_line":"  enabled \u003d false"},{"line_number":52,"context_line":"  port \u003d 25826"},{"line_number":53,"context_line":"  database \u003d \"collectd\""},{"line_number":54,"context_line":"  typesdb \u003d \"/opt/collectd/share/collectd/types.db\""},{"line_number":55,"context_line":"[[opentsdb]]"}],"source_content_type":"text/plain","patch_set":10,"id":"bacf61ea_e88edace","line":52,"range":{"start_line":52,"start_character":9,"end_line":52,"end_character":14},"updated":"2016-08-04 10:48:03.000000000","message":"dito","commit_id":"3fed1cff97661996da84333fa0ead2661c7b79a1"}],"ansible/roles/influxdb/templates/influxdb.conf.j2":[{"author":{"_account_id":16993,"name":"Hui Kang","email":"kangh@us.ibm.com","username":"huikang"},"change_message_id":"9d1058b9373cb533ee94ea4e7d82ff660b2d50f0","unresolved":false,"context_lines":[{"line_number":12,"context_line":"  wal-logging-enabled \u003d true"},{"line_number":13,"context_line":"  data-logging-enabled \u003d true"},{"line_number":14,"context_line":"[cluster]"},{"line_number":15,"context_line":"  shard-writer-timeout \u003d \"5s\" # The time within which a remote shard must respond to a write request."},{"line_number":16,"context_line":"  write-timeout \u003d \"10s\" # The time within which a write request must complete on the cluster."},{"line_number":17,"context_line":"  max-concurrent-queries \u003d 0 # The maximum number of concurrent queries that can run. 0 to disable."},{"line_number":18,"context_line":"  query-timeout \u003d \"0s\" # The time within a query must complete before being killed automatically. 0s to disable."}],"source_content_type":"text/x-jinja2","patch_set":15,"id":"3ac371cc_6c47f61d","line":15,"updated":"2016-08-19 05:00:25.000000000","message":"I do not think Kolla needs comment in the configuration file","commit_id":"6efd6a7b53e0829eefdd28d4cdc5910f0919b77e"},{"author":{"_account_id":16993,"name":"Hui Kang","email":"kangh@us.ibm.com","username":"huikang"},"change_message_id":"d74e91479310b95bd27ea083e8b9208d89031ed6","unresolved":false,"context_lines":[{"line_number":18,"context_line":"  query-timeout \u003d \"0s\" # The time within a query must complete before being killed automatically. 0s to disable."},{"line_number":19,"context_line":"  max-select-point \u003d 0 # The maximum number of points to scan in a query. 0 to disable."},{"line_number":20,"context_line":"  max-select-series \u003d 0 # The maximum number of series to select in a query. 0 to disable."},{"line_number":21,"context_line":"  max-select-buckets \u003d 0 # The maximum number of buckets to select in an aggregate query. 0 to disable."},{"line_number":22,"context_line":"[retention]"},{"line_number":23,"context_line":"  enabled \u003d true"},{"line_number":24,"context_line":"  check-interval \u003d \"30m\""}],"source_content_type":"text/x-jinja2","patch_set":16,"id":"fa7ab95a_336e14ca","line":21,"updated":"2016-08-26 04:02:19.000000000","message":"No comment in kolla config files.","commit_id":"7a148f80d80bb40e77a74cdfbb5226704d817c90"},{"author":{"_account_id":16993,"name":"Hui Kang","email":"kangh@us.ibm.com","username":"huikang"},"change_message_id":"fd795152439912ca53158b4dc4556c734e67fb75","unresolved":false,"context_lines":[{"line_number":12,"context_line":"  wal-logging-enabled \u003d true"},{"line_number":13,"context_line":"  data-logging-enabled \u003d true"},{"line_number":14,"context_line":"[cluster]"},{"line_number":15,"context_line":"  shard-writer-timeout \u003d \"5s\" "},{"line_number":16,"context_line":"  write-timeout \u003d \"10s\" "},{"line_number":17,"context_line":"  max-concurrent-queries \u003d 0 "},{"line_number":18,"context_line":"  query-timeout \u003d \"0s\" "}],"source_content_type":"text/x-jinja2","patch_set":17,"id":"fa7ab95a_c2986c1e","line":15,"updated":"2016-08-27 04:18:28.000000000","message":"remove the trailing spaces in this file","commit_id":"ca59a186635e4564797f2b385685d0259fd698a8"},{"author":{"_account_id":167,"name":"Christian Berendt","email":"berendt@osism.tech","username":"berendt"},"change_message_id":"262e994a2ef894db4f4b9384907230d6bd62a4b6","unresolved":false,"context_lines":[{"line_number":32,"context_line":"  store-interval \u003d \"10s\""},{"line_number":33,"context_line":"[admin]"},{"line_number":34,"context_line":"  enabled \u003d true"},{"line_number":35,"context_line":"  bind-address \u003d \":{{ influxdb_admin_port }}\""},{"line_number":36,"context_line":"  https-enabled \u003d false"},{"line_number":37,"context_line":"  https-certificate \u003d \"/etc/ssl/influxdb.pem\""},{"line_number":38,"context_line":"[http]"}],"source_content_type":"text/x-jinja2","patch_set":19,"id":"fa7ab95a_0eaa5e70","line":35,"updated":"2016-08-29 08:16:59.000000000","message":"Why not only binding to the internal address?","commit_id":"dc4e6717486019e41724e331a3c197c797c165dd"},{"author":{"_account_id":17938,"name":"Mathias Ewald","email":"mewald@evoila.de","username":"mewald"},"change_message_id":"86f275d912b131e85b55b823c7da49f92195c9b1","unresolved":false,"context_lines":[{"line_number":32,"context_line":"  store-interval \u003d \"10s\""},{"line_number":33,"context_line":"[admin]"},{"line_number":34,"context_line":"  enabled \u003d true"},{"line_number":35,"context_line":"  bind-address \u003d \":{{ influxdb_admin_port }}\""},{"line_number":36,"context_line":"  https-enabled \u003d false"},{"line_number":37,"context_line":"  https-certificate \u003d \"/etc/ssl/influxdb.pem\""},{"line_number":38,"context_line":"[http]"}],"source_content_type":"text/x-jinja2","patch_set":19,"id":"fa7ab95a_c1f5a7cb","line":35,"in_reply_to":"fa7ab95a_0eaa5e70","updated":"2016-08-29 08:48:22.000000000","message":"Done","commit_id":"dc4e6717486019e41724e331a3c197c797c165dd"},{"author":{"_account_id":167,"name":"Christian Berendt","email":"berendt@osism.tech","username":"berendt"},"change_message_id":"262e994a2ef894db4f4b9384907230d6bd62a4b6","unresolved":false,"context_lines":[{"line_number":34,"context_line":"  enabled \u003d true"},{"line_number":35,"context_line":"  bind-address \u003d \":{{ influxdb_admin_port }}\""},{"line_number":36,"context_line":"  https-enabled \u003d false"},{"line_number":37,"context_line":"  https-certificate \u003d \"/etc/ssl/influxdb.pem\""},{"line_number":38,"context_line":"[http]"},{"line_number":39,"context_line":"  enabled \u003d true"},{"line_number":40,"context_line":"  bind-address \u003d \":{{ influxdb_http_port }}\""}],"source_content_type":"text/x-jinja2","patch_set":19,"id":"fa7ab95a_0ec17e2e","line":37,"updated":"2016-08-29 08:16:59.000000000","message":"Is this required when https-enabled \u003d false ?","commit_id":"dc4e6717486019e41724e331a3c197c797c165dd"},{"author":{"_account_id":17938,"name":"Mathias Ewald","email":"mewald@evoila.de","username":"mewald"},"change_message_id":"86f275d912b131e85b55b823c7da49f92195c9b1","unresolved":false,"context_lines":[{"line_number":34,"context_line":"  enabled \u003d true"},{"line_number":35,"context_line":"  bind-address \u003d \":{{ influxdb_admin_port }}\""},{"line_number":36,"context_line":"  https-enabled \u003d false"},{"line_number":37,"context_line":"  https-certificate \u003d \"/etc/ssl/influxdb.pem\""},{"line_number":38,"context_line":"[http]"},{"line_number":39,"context_line":"  enabled \u003d true"},{"line_number":40,"context_line":"  bind-address \u003d \":{{ influxdb_http_port }}\""}],"source_content_type":"text/x-jinja2","patch_set":19,"id":"fa7ab95a_41d7d722","line":37,"in_reply_to":"fa7ab95a_0ec17e2e","updated":"2016-08-29 08:48:22.000000000","message":"Probably not, I took the default config of influxdb. I will remove it.","commit_id":"dc4e6717486019e41724e331a3c197c797c165dd"},{"author":{"_account_id":167,"name":"Christian Berendt","email":"berendt@osism.tech","username":"berendt"},"change_message_id":"262e994a2ef894db4f4b9384907230d6bd62a4b6","unresolved":false,"context_lines":[{"line_number":37,"context_line":"  https-certificate \u003d \"/etc/ssl/influxdb.pem\""},{"line_number":38,"context_line":"[http]"},{"line_number":39,"context_line":"  enabled \u003d true"},{"line_number":40,"context_line":"  bind-address \u003d \":{{ influxdb_http_port }}\""},{"line_number":41,"context_line":"  auth-enabled \u003d false"},{"line_number":42,"context_line":"  log-enabled \u003d true"},{"line_number":43,"context_line":"  write-tracing \u003d false"}],"source_content_type":"text/x-jinja2","patch_set":19,"id":"fa7ab95a_2eb722d9","line":40,"updated":"2016-08-29 08:16:59.000000000","message":"dito","commit_id":"dc4e6717486019e41724e331a3c197c797c165dd"},{"author":{"_account_id":167,"name":"Christian Berendt","email":"berendt@osism.tech","username":"berendt"},"change_message_id":"262e994a2ef894db4f4b9384907230d6bd62a4b6","unresolved":false,"context_lines":[{"line_number":43,"context_line":"  write-tracing \u003d false"},{"line_number":44,"context_line":"  pprof-enabled \u003d false"},{"line_number":45,"context_line":"  https-enabled \u003d false"},{"line_number":46,"context_line":"  https-certificate \u003d \"/etc/ssl/influxdb.pem\""},{"line_number":47,"context_line":"  max-row-limit \u003d 10000"},{"line_number":48,"context_line":"[[graphite]]"},{"line_number":49,"context_line":"  enabled \u003d false"}],"source_content_type":"text/x-jinja2","patch_set":19,"id":"fa7ab95a_8eb4aed1","line":46,"updated":"2016-08-29 08:16:59.000000000","message":"dito","commit_id":"dc4e6717486019e41724e331a3c197c797c165dd"},{"author":{"_account_id":167,"name":"Christian Berendt","email":"berendt@osism.tech","username":"berendt"},"change_message_id":"262e994a2ef894db4f4b9384907230d6bd62a4b6","unresolved":false,"context_lines":[{"line_number":45,"context_line":"  https-enabled \u003d false"},{"line_number":46,"context_line":"  https-certificate \u003d \"/etc/ssl/influxdb.pem\""},{"line_number":47,"context_line":"  max-row-limit \u003d 10000"},{"line_number":48,"context_line":"[[graphite]]"},{"line_number":49,"context_line":"  enabled \u003d false"},{"line_number":50,"context_line":"[[opentsdb]]"},{"line_number":51,"context_line":"  enabled \u003d false"}],"source_content_type":"text/x-jinja2","patch_set":19,"id":"fa7ab95a_6eaccaa0","line":48,"range":{"start_line":48,"start_character":10,"end_line":48,"end_character":12},"updated":"2016-08-29 08:16:59.000000000","message":"Is it correct to have [[ ... ]] here?","commit_id":"dc4e6717486019e41724e331a3c197c797c165dd"},{"author":{"_account_id":22582,"name":"Duong Ha-Quang","email":"quangduongha@gmail.com","username":"duonghq"},"change_message_id":"66eec838a90686e2af2fe59cd30e2bffc76a98e5","unresolved":false,"context_lines":[{"line_number":45,"context_line":"  https-enabled \u003d false"},{"line_number":46,"context_line":"  https-certificate \u003d \"/etc/ssl/influxdb.pem\""},{"line_number":47,"context_line":"  max-row-limit \u003d 10000"},{"line_number":48,"context_line":"[[graphite]]"},{"line_number":49,"context_line":"  enabled \u003d false"},{"line_number":50,"context_line":"[[opentsdb]]"},{"line_number":51,"context_line":"  enabled \u003d false"}],"source_content_type":"text/x-jinja2","patch_set":19,"id":"fa7ab95a_ced796bd","line":48,"range":{"start_line":48,"start_character":10,"end_line":48,"end_character":12},"in_reply_to":"fa7ab95a_6eaccaa0","updated":"2016-08-29 08:19:25.000000000","message":"It is correct [1]\n\n[1] https://docs.influxdata.com/influxdb/v0.9/administration/config/#graphite","commit_id":"dc4e6717486019e41724e331a3c197c797c165dd"},{"author":{"_account_id":167,"name":"Christian Berendt","email":"berendt@osism.tech","username":"berendt"},"change_message_id":"262e994a2ef894db4f4b9384907230d6bd62a4b6","unresolved":false,"context_lines":[{"line_number":47,"context_line":"  max-row-limit \u003d 10000"},{"line_number":48,"context_line":"[[graphite]]"},{"line_number":49,"context_line":"  enabled \u003d false"},{"line_number":50,"context_line":"[[opentsdb]]"},{"line_number":51,"context_line":"  enabled \u003d false"},{"line_number":52,"context_line":"[[udp]]"},{"line_number":53,"context_line":"  enabled \u003d false"}],"source_content_type":"text/x-jinja2","patch_set":19,"id":"fa7ab95a_8eaf4e93","line":50,"range":{"start_line":50,"start_character":10,"end_line":50,"end_character":12},"updated":"2016-08-29 08:16:59.000000000","message":"dito","commit_id":"dc4e6717486019e41724e331a3c197c797c165dd"},{"author":{"_account_id":22582,"name":"Duong Ha-Quang","email":"quangduongha@gmail.com","username":"duonghq"},"change_message_id":"66eec838a90686e2af2fe59cd30e2bffc76a98e5","unresolved":false,"context_lines":[{"line_number":47,"context_line":"  max-row-limit \u003d 10000"},{"line_number":48,"context_line":"[[graphite]]"},{"line_number":49,"context_line":"  enabled \u003d false"},{"line_number":50,"context_line":"[[opentsdb]]"},{"line_number":51,"context_line":"  enabled \u003d false"},{"line_number":52,"context_line":"[[udp]]"},{"line_number":53,"context_line":"  enabled \u003d false"}],"source_content_type":"text/x-jinja2","patch_set":19,"id":"fa7ab95a_ae033241","line":50,"range":{"start_line":50,"start_character":10,"end_line":50,"end_character":12},"in_reply_to":"fa7ab95a_8eaf4e93","updated":"2016-08-29 08:19:25.000000000","message":"it is wrong https://docs.influxdata.com/influxdb/v0.9/administration/config/#opentsdb","commit_id":"dc4e6717486019e41724e331a3c197c797c165dd"},{"author":{"_account_id":17938,"name":"Mathias Ewald","email":"mewald@evoila.de","username":"mewald"},"change_message_id":"86f275d912b131e85b55b823c7da49f92195c9b1","unresolved":false,"context_lines":[{"line_number":47,"context_line":"  max-row-limit \u003d 10000"},{"line_number":48,"context_line":"[[graphite]]"},{"line_number":49,"context_line":"  enabled \u003d false"},{"line_number":50,"context_line":"[[opentsdb]]"},{"line_number":51,"context_line":"  enabled \u003d false"},{"line_number":52,"context_line":"[[udp]]"},{"line_number":53,"context_line":"  enabled \u003d false"}],"source_content_type":"text/x-jinja2","patch_set":19,"id":"fa7ab95a_6183fb04","line":50,"range":{"start_line":50,"start_character":10,"end_line":50,"end_character":12},"in_reply_to":"fa7ab95a_ae033241","updated":"2016-08-29 08:48:22.000000000","message":"Documentation should not be trusted ;) I took influxdb\u0027s default config file and looking into the Git code they also use [[]] https://github.com/influxdata/influxdb/blob/master/etc/config.sample.toml#L257","commit_id":"dc4e6717486019e41724e331a3c197c797c165dd"},{"author":{"_account_id":167,"name":"Christian Berendt","email":"berendt@osism.tech","username":"berendt"},"change_message_id":"262e994a2ef894db4f4b9384907230d6bd62a4b6","unresolved":false,"context_lines":[{"line_number":49,"context_line":"  enabled \u003d false"},{"line_number":50,"context_line":"[[opentsdb]]"},{"line_number":51,"context_line":"  enabled \u003d false"},{"line_number":52,"context_line":"[[udp]]"},{"line_number":53,"context_line":"  enabled \u003d false"},{"line_number":54,"context_line":"[continuous_queries]"},{"line_number":55,"context_line":"  log-enabled \u003d true"}],"source_content_type":"text/x-jinja2","patch_set":19,"id":"fa7ab95a_ce915654","line":52,"range":{"start_line":52,"start_character":5,"end_line":52,"end_character":7},"updated":"2016-08-29 08:16:59.000000000","message":"dito","commit_id":"dc4e6717486019e41724e331a3c197c797c165dd"},{"author":{"_account_id":22582,"name":"Duong Ha-Quang","email":"quangduongha@gmail.com","username":"duonghq"},"change_message_id":"66eec838a90686e2af2fe59cd30e2bffc76a98e5","unresolved":false,"context_lines":[{"line_number":49,"context_line":"  enabled \u003d false"},{"line_number":50,"context_line":"[[opentsdb]]"},{"line_number":51,"context_line":"  enabled \u003d false"},{"line_number":52,"context_line":"[[udp]]"},{"line_number":53,"context_line":"  enabled \u003d false"},{"line_number":54,"context_line":"[continuous_queries]"},{"line_number":55,"context_line":"  log-enabled \u003d true"}],"source_content_type":"text/x-jinja2","patch_set":19,"id":"fa7ab95a_eefd3a3b","line":52,"range":{"start_line":52,"start_character":5,"end_line":52,"end_character":7},"in_reply_to":"fa7ab95a_ce915654","updated":"2016-08-29 08:19:25.000000000","message":"It\u0027s correct https://docs.influxdata.com/influxdb/v0.9/administration/config/#udp","commit_id":"dc4e6717486019e41724e331a3c197c797c165dd"}],"ansible/roles/influxdb/templates/influxdb.json.j2":[{"author":{"_account_id":11869,"name":"zhubingbing","email":"zhu.bingbing@99cloud.net","username":"zhubingbing"},"change_message_id":"86e68016615befa75971638ed55417ff7f1a1aab","unresolved":false,"context_lines":[{"line_number":1,"context_line":"{"},{"line_number":2,"context_line":"    \"command\": \"/usr/bin/influxd -config /etc/influxdb/influxdb.conf  \u003e\u003e/dev/null 2\u003e\u003e/var/log/influxdb/influxd.log\","},{"line_number":3,"context_line":"    \"config_files\": ["},{"line_number":4,"context_line":"        {"},{"line_number":5,"context_line":"            \"source\": \"{{ container_config_directory }}/influxdb.conf\","}],"source_content_type":"text/x-jinja2","patch_set":5,"id":"fa7ab95a_87d73d22","line":2,"range":{"start_line":2,"start_character":70,"end_line":2,"end_character":81},"updated":"2016-08-29 01:43:27.000000000","message":"i thinks it should remove, influxd log  directory should be  \u0027/var/log/kolla/infuxd.log\u0027.","commit_id":"466976b20a58fae00157e35bc293486afeda476a"},{"author":{"_account_id":10419,"name":"Ryan Hallisey","email":"rhallise@redhat.com","username":"rhallisey"},"change_message_id":"976d814f9f9ec2c977c56f56aa1358eff46fd981","unresolved":false,"context_lines":[{"line_number":1,"context_line":"{"},{"line_number":2,"context_line":"    \"command\": \"/usr/bin/influxd -config /etc/influxdb/influxdb.conf  \u003e\u003e/dev/null 2\u003e\u003e/var/log/influxdb/influxd.log\","},{"line_number":3,"context_line":"    \"config_files\": ["},{"line_number":4,"context_line":"        {"},{"line_number":5,"context_line":"            \"source\": \"{{ container_config_directory }}/influxdb.conf\","}],"source_content_type":"text/x-jinja2","patch_set":5,"id":"bacf61ea_e8a49309","line":2,"range":{"start_line":2,"start_character":70,"end_line":2,"end_character":81},"updated":"2016-07-27 13:58:35.000000000","message":"why do you only want to direct stderr to the log.  We should also direct stdout.","commit_id":"466976b20a58fae00157e35bc293486afeda476a"},{"author":{"_account_id":17938,"name":"Mathias Ewald","email":"mewald@evoila.de","username":"mewald"},"change_message_id":"a9840010365c71989b287352d3dfd1ae7a4120cb","unresolved":false,"context_lines":[{"line_number":1,"context_line":"{"},{"line_number":2,"context_line":"    \"command\": \"/usr/bin/influxd -config /etc/influxdb/influxdb.conf  \u003e\u003e/dev/null 2\u003e\u003e/var/log/influxdb/influxd.log\","},{"line_number":3,"context_line":"    \"config_files\": ["},{"line_number":4,"context_line":"        {"},{"line_number":5,"context_line":"            \"source\": \"{{ container_config_directory }}/influxdb.conf\","}],"source_content_type":"text/x-jinja2","patch_set":5,"id":"bacf61ea_805e7c43","line":2,"range":{"start_line":2,"start_character":70,"end_line":2,"end_character":81},"in_reply_to":"bacf61ea_e8a49309","updated":"2016-07-27 19:58:50.000000000","message":"I just used what the init script of influxdb does. This is exactly how the process runs. But sure, no problem with that.","commit_id":"466976b20a58fae00157e35bc293486afeda476a"},{"author":{"_account_id":7488,"name":"Jeffrey Zhang","email":"zhang.lei.fly@gmail.com","username":"Jeffrey4l"},"change_message_id":"21ff0e81898bbea32fc58041749b686db208f5fb","unresolved":false,"context_lines":[{"line_number":1,"context_line":"{"},{"line_number":2,"context_line":"    \"command\": \"/usr/bin/influxd -config /etc/influxdb/influxdb.conf 2\u003e\u00261 \u003e\u003e/var/log/influxdb/influxd.log\","},{"line_number":3,"context_line":"    \"config_files\": ["},{"line_number":4,"context_line":"        {"},{"line_number":5,"context_line":"            \"source\": \"{{ container_config_directory }}/influxdb.conf\","}],"source_content_type":"text/x-jinja2","patch_set":9,"id":"bacf61ea_f105b613","line":2,"updated":"2016-08-02 00:52:40.000000000","message":"1. could the log path can be configured by configuration file?\n2. please write the log to /var/log/kolla/influxdb/influxd.log file.","commit_id":"382ab5279caeae9baf1462b860ccee3efd8aa4fb"},{"author":{"_account_id":16482,"name":"Shaun Smekel","email":"shaun.smekel@theorem.net.au","username":"smekel"},"change_message_id":"27c79121f71b08792a2921448e2165e0c9ef8b1d","unresolved":false,"context_lines":[{"line_number":1,"context_line":"{"},{"line_number":2,"context_line":"    \"command\": \"/usr/bin/influxd -config /etc/influxdb/influxdb.conf 2\u003e\u00261 \u003e\u003e/var/log/kolla/influxdb/influxd.log\","},{"line_number":3,"context_line":"    \"config_files\": ["},{"line_number":4,"context_line":"        {"},{"line_number":5,"context_line":"            \"source\": \"{{ container_config_directory }}/influxdb.conf\","}],"source_content_type":"text/x-jinja2","patch_set":13,"id":"9ad45d7e_f375e0ca","line":2,"updated":"2016-08-11 11:49:59.000000000","message":"Will this command (using \u003e\u003e) not wipe the logs for influxdb each time the container is restarted?","commit_id":"4fa3cb857be2e602a9c5fa1f60da5e990068573c"},{"author":{"_account_id":17938,"name":"Mathias Ewald","email":"mewald@evoila.de","username":"mewald"},"change_message_id":"c06b9724394517e9415e5c14d546424c314f70c3","unresolved":false,"context_lines":[{"line_number":1,"context_line":"{"},{"line_number":2,"context_line":"    \"command\": \"/usr/bin/influxd -config /etc/influxdb/influxdb.conf 2\u003e\u00261 \u003e\u003e/var/log/kolla/influxdb/influxd.log\","},{"line_number":3,"context_line":"    \"config_files\": ["},{"line_number":4,"context_line":"        {"},{"line_number":5,"context_line":"            \"source\": \"{{ container_config_directory }}/influxdb.conf\","}],"source_content_type":"text/x-jinja2","patch_set":13,"id":"3ac371cc_6154ea20","line":2,"in_reply_to":"9ad45d7e_f375e0ca","updated":"2016-08-16 08:41:16.000000000","message":"Actually \"\u003e\u003e\" means to append to the file. I think this is correct.","commit_id":"4fa3cb857be2e602a9c5fa1f60da5e990068573c"},{"author":{"_account_id":16993,"name":"Hui Kang","email":"kangh@us.ibm.com","username":"huikang"},"change_message_id":"9d1058b9373cb533ee94ea4e7d82ff660b2d50f0","unresolved":false,"context_lines":[{"line_number":1,"context_line":"{"},{"line_number":2,"context_line":"    \"command\": \"/usr/bin/influxd -config /etc/influxdb/influxdb.conf 2\u003e\u00261 \u003e\u003e/var/log/kolla/influxdb/influxd.log\","},{"line_number":3,"context_line":"    \"config_files\": ["},{"line_number":4,"context_line":"        {"},{"line_number":5,"context_line":"            \"source\": \"{{ container_config_directory }}/influxdb.conf\","}],"source_content_type":"text/x-jinja2","patch_set":15,"id":"3ac371cc_6c6ed697","line":2,"updated":"2016-08-19 05:00:25.000000000","message":"can we set the log file location in the configuration file, instead of directing to the log file.","commit_id":"6efd6a7b53e0829eefdd28d4cdc5910f0919b77e"},{"author":{"_account_id":11869,"name":"zhubingbing","email":"zhu.bingbing@99cloud.net","username":"zhubingbing"},"change_message_id":"e7acbb30f4f1db671e5ff2a79ec62f933a8e7a11","unresolved":false,"context_lines":[{"line_number":1,"context_line":"{"},{"line_number":2,"context_line":"    \"command\": \"/usr/bin/influxd -config /etc/influxdb/influxdb.conf 2\u003e\u00261 \u003e\u003e/var/log/kolla/influxdb/influxd.log\","},{"line_number":3,"context_line":"    \"config_files\": ["},{"line_number":4,"context_line":"        {"},{"line_number":5,"context_line":"            \"source\": \"{{ container_config_directory }}/influxdb.conf\","}],"source_content_type":"text/x-jinja2","patch_set":22,"id":"fa7ab95a_8a7b9af4","line":2,"range":{"start_line":2,"start_character":76,"end_line":2,"end_character":111},"updated":"2016-08-29 16:07:57.000000000","message":"I think it would be better to set the log directory(/var/log/kolla/infuxd.lo) in the infuxdb.conf file.","commit_id":"f812eff50a7b4c4335fc6313af08efded27ed398"},{"author":{"_account_id":16993,"name":"Hui Kang","email":"kangh@us.ibm.com","username":"huikang"},"change_message_id":"a8189c2f3a176b6d8e1d4f6df996ce767c8e868e","unresolved":false,"context_lines":[{"line_number":1,"context_line":"{"},{"line_number":2,"context_line":"    \"command\": \"/usr/bin/influxd -config /etc/influxdb/influxdb.conf 2\u003e\u00261 \u003e\u003e/var/log/kolla/influxdb/influxd.log\","},{"line_number":3,"context_line":"    \"config_files\": ["},{"line_number":4,"context_line":"        {"},{"line_number":5,"context_line":"            \"source\": \"{{ container_config_directory }}/influxdb.conf\","}],"source_content_type":"text/x-jinja2","patch_set":30,"id":"fa7ab95a_7dbd4267","line":2,"updated":"2016-09-02 05:06:08.000000000","message":"can log file location be configured in influxdb.conf?","commit_id":"016ee415ea41dd09e7089c783d39b655187c36e7"},{"author":{"_account_id":16993,"name":"Hui Kang","email":"kangh@us.ibm.com","username":"huikang"},"change_message_id":"a8189c2f3a176b6d8e1d4f6df996ce767c8e868e","unresolved":false,"context_lines":[{"line_number":5,"context_line":"            \"source\": \"{{ container_config_directory }}/influxdb.conf\","},{"line_number":6,"context_line":"            \"dest\": \"/etc/influxdb/influxdb.conf\","},{"line_number":7,"context_line":"            \"owner\": \"influxdb\","},{"line_number":8,"context_line":"            \"perm\": \"0644\""},{"line_number":9,"context_line":"        }"},{"line_number":10,"context_line":"    ]"},{"line_number":11,"context_line":"}"}],"source_content_type":"text/x-jinja2","patch_set":30,"id":"fa7ab95a_5d8ebed0","line":8,"updated":"2016-09-02 05:06:08.000000000","message":"why not \"0600\"?","commit_id":"016ee415ea41dd09e7089c783d39b655187c36e7"}],"ansible/site.yml":[{"author":{"_account_id":19316,"name":"Eduardo Gonzalez","email":"dabarren@gmail.com","username":"egonzalez90"},"change_message_id":"795078d2767fe3cff005620a6d40b027c1fb78a8","unresolved":false,"context_lines":[{"line_number":3,"context_line":"  roles:"},{"line_number":4,"context_line":"    - { role: influxdb,"},{"line_number":5,"context_line":"        tags: [\u0027perfmon\u0027, \u0027influxdb\u0027],"},{"line_number":6,"context_line":"        when: enable_performance_monitoring | bool }"},{"line_number":7,"context_line":""},{"line_number":8,"context_line":"- hosts:"},{"line_number":9,"context_line":"    - ceph-mon"}],"source_content_type":"text/x-yaml","patch_set":21,"id":"fa7ab95a_f975f8f9","line":6,"range":{"start_line":6,"start_character":15,"end_line":6,"end_character":43},"updated":"2016-08-29 12:26:08.000000000","message":"ATM there is not any enable_performance_monitoring variable at groupvars/globals","commit_id":"e46baf07ce2a04527180cf75063174a9573b1f08"},{"author":{"_account_id":17938,"name":"Mathias Ewald","email":"mewald@evoila.de","username":"mewald"},"change_message_id":"7d8cd80175a8bb104a033c4f8d694d348a91519f","unresolved":false,"context_lines":[{"line_number":3,"context_line":"  roles:"},{"line_number":4,"context_line":"    - { role: influxdb,"},{"line_number":5,"context_line":"        tags: [\u0027perfmon\u0027, \u0027influxdb\u0027],"},{"line_number":6,"context_line":"        when: enable_performance_monitoring | bool }"},{"line_number":7,"context_line":""},{"line_number":8,"context_line":"- hosts:"},{"line_number":9,"context_line":"    - ceph-mon"}],"source_content_type":"text/x-yaml","patch_set":21,"id":"fa7ab95a_3c9d8ebc","line":6,"range":{"start_line":6,"start_character":15,"end_line":6,"end_character":43},"in_reply_to":"fa7ab95a_f975f8f9","updated":"2016-08-29 12:32:42.000000000","message":"Right, I had this in the telegraf role patch but then decided to have the dependencies differently. I fixed it.","commit_id":"e46baf07ce2a04527180cf75063174a9573b1f08"},{"author":{"_account_id":11869,"name":"zhubingbing","email":"zhu.bingbing@99cloud.net","username":"zhubingbing"},"change_message_id":"e7acbb30f4f1db671e5ff2a79ec62f933a8e7a11","unresolved":false,"context_lines":[{"line_number":1,"context_line":"---"},{"line_number":2,"context_line":"- hosts: influxdb"},{"line_number":3,"context_line":"  roles:"},{"line_number":4,"context_line":"    - { role: influxdb,"},{"line_number":5,"context_line":"        tags: [\u0027perfmon\u0027, \u0027influxdb\u0027],"},{"line_number":6,"context_line":"        when: enable_performance_monitoring | bool }"},{"line_number":7,"context_line":""},{"line_number":8,"context_line":"- hosts:"},{"line_number":9,"context_line":"    - ceph-mon"}],"source_content_type":"text/x-yaml","patch_set":22,"id":"fa7ab95a_2a960e93","line":6,"range":{"start_line":2,"start_character":0,"end_line":6,"end_character":52},"updated":"2016-08-29 16:07:57.000000000","message":"this should is \n\n-hosts: infuxdb\n  roles:\n     - {role: infuxdb,\n         tages: infuxdb\n         when: enable_infuxdb | bool }\n\nbecause I think it should be just deploy infuxdb，you can refer to the other components of the code","commit_id":"f812eff50a7b4c4335fc6313af08efded27ed398"},{"author":{"_account_id":16993,"name":"Hui Kang","email":"kangh@us.ibm.com","username":"huikang"},"change_message_id":"a8189c2f3a176b6d8e1d4f6df996ce767c8e868e","unresolved":false,"context_lines":[{"line_number":22,"context_line":""},{"line_number":23,"context_line":"- hosts:"},{"line_number":24,"context_line":"    - cinder-api"},{"line_number":25,"context_line":"    - glance-api"},{"line_number":26,"context_line":"    - haproxy"},{"line_number":27,"context_line":"    - keystone"},{"line_number":28,"context_line":"    - mariadb"},{"line_number":29,"context_line":"    - iscsid"},{"line_number":30,"context_line":"    - tgtd"},{"line_number":31,"context_line":"    - multipathd"},{"line_number":32,"context_line":"    - murano-api"},{"line_number":33,"context_line":"    - neutron-server"},{"line_number":34,"context_line":"    - nova-api"},{"line_number":35,"context_line":"    - rabbitmq"},{"line_number":36,"context_line":"    - swift-proxy-server"},{"line_number":37,"context_line":"    - heat-api"},{"line_number":38,"context_line":"  roles:"},{"line_number":39,"context_line":"    - { role: haproxy,"},{"line_number":40,"context_line":"        tags: haproxy,"}],"source_content_type":"text/x-yaml","patch_set":30,"id":"fa7ab95a_3da81a20","line":37,"range":{"start_line":25,"start_character":6,"end_line":37,"end_character":14},"updated":"2016-09-02 05:06:08.000000000","message":"why are these lines added?","commit_id":"016ee415ea41dd09e7089c783d39b655187c36e7"},{"author":{"_account_id":11869,"name":"zhubingbing","email":"zhu.bingbing@99cloud.net","username":"zhubingbing"},"change_message_id":"28cd3ace7f8f8e276456ea2818fb7ce58fb92c9e","unresolved":false,"context_lines":[{"line_number":22,"context_line":""},{"line_number":23,"context_line":"- hosts:"},{"line_number":24,"context_line":"    - cinder-api"},{"line_number":25,"context_line":"    - glance-api"},{"line_number":26,"context_line":"    - haproxy"},{"line_number":27,"context_line":"    - keystone"},{"line_number":28,"context_line":"    - mariadb"},{"line_number":29,"context_line":"    - iscsid"},{"line_number":30,"context_line":"    - tgtd"},{"line_number":31,"context_line":"    - multipathd"},{"line_number":32,"context_line":"    - murano-api"},{"line_number":33,"context_line":"    - neutron-server"},{"line_number":34,"context_line":"    - nova-api"},{"line_number":35,"context_line":"    - rabbitmq"},{"line_number":36,"context_line":"    - swift-proxy-server"},{"line_number":37,"context_line":"    - heat-api"},{"line_number":38,"context_line":"  roles:"},{"line_number":39,"context_line":"    - { role: haproxy,"},{"line_number":40,"context_line":"        tags: haproxy,"}],"source_content_type":"text/x-yaml","patch_set":30,"id":"fa7ab95a_c06bc1b7","line":37,"range":{"start_line":25,"start_character":6,"end_line":37,"end_character":14},"in_reply_to":"fa7ab95a_3da81a20","updated":"2016-09-02 05:31:48.000000000","message":"Done","commit_id":"016ee415ea41dd09e7089c783d39b655187c36e7"}],"etc/kolla/globals.yml":[{"author":{"_account_id":11869,"name":"zhubingbing","email":"zhu.bingbing@99cloud.net","username":"zhubingbing"},"change_message_id":"e7acbb30f4f1db671e5ff2a79ec62f933a8e7a11","unresolved":false,"context_lines":[{"line_number":123,"context_line":"#enable_multipathd: \"no\""},{"line_number":124,"context_line":"#enable_neutron_lbaas: \"no\""},{"line_number":125,"context_line":"#enable_neutron_qos: \"no\""},{"line_number":126,"context_line":"#enable_performance_monitoring: \"no\""},{"line_number":127,"context_line":"#enable_swift: \"no\""},{"line_number":128,"context_line":"#enable_tempest: \"no\""},{"line_number":129,"context_line":"#enable_watcher: \"no\""}],"source_content_type":"text/x-yaml","patch_set":22,"id":"fa7ab95a_87bc73dd","line":126,"range":{"start_line":126,"start_character":0,"end_line":126,"end_character":36},"updated":"2016-08-29 16:07:57.000000000","message":"i think it should enable_infuxdb: no\" I think the 2 services (telegraf infuxdb) coupling is too strong","commit_id":"f812eff50a7b4c4335fc6313af08efded27ed398"},{"author":{"_account_id":16993,"name":"Hui Kang","email":"kangh@us.ibm.com","username":"huikang"},"change_message_id":"b04775f18ede659cbb4f79dd285cfac6a13b34a7","unresolved":false,"context_lines":[{"line_number":130,"context_line":"#enable_heat: \"yes\""},{"line_number":131,"context_line":"#enable_horizon: \"yes\""},{"line_number":132,"context_line":"#enable_ironic: \"no\""},{"line_number":133,"context_line":"#enable_influxdb: \"no\""},{"line_number":134,"context_line":"#enable_magnum: \"no\""},{"line_number":135,"context_line":"#enable_manila: \"no\""},{"line_number":136,"context_line":"#enable_mistral: \"no\""}],"source_content_type":"text/x-yaml","patch_set":28,"id":"fa7ab95a_59d6f527","line":133,"updated":"2016-09-01 14:17:16.000000000","message":"alphabetic order. Put it before ironic","commit_id":"a2fb78455a24b51e8654ef0724778a07fbd99542"}],"tests/test_build.py":[{"author":{"_account_id":7488,"name":"Jeffrey Zhang","email":"zhang.lei.fly@gmail.com","username":"Jeffrey4l"},"change_message_id":"8670469d3b03bbce831b59aa15dd322d42e10a49","unresolved":false,"context_lines":[{"line_number":72,"context_line":"                       \"watcher-base\","},{"line_number":73,"context_line":"                       \"congress-base\","},{"line_number":74,"context_line":"                       \"bifrost-base\","},{"line_number":75,"context_line":"                       \"influxdb\"]"},{"line_number":76,"context_line":""},{"line_number":77,"context_line":"    def setUp(self):"},{"line_number":78,"context_line":"        super(BuildTestCentosBinary, self).setUp()"}],"source_content_type":"text/x-python","patch_set":29,"id":"fa7ab95a_354914b8","line":75,"updated":"2016-09-02 02:45:18.000000000","message":"why add the influxdb here? centos binary influxdb build should be fine.","commit_id":"042d87c71efe08f0813beeaf24aa2b40ad2ff837"},{"author":{"_account_id":16993,"name":"Hui Kang","email":"kangh@us.ibm.com","username":"huikang"},"change_message_id":"a8189c2f3a176b6d8e1d4f6df996ce767c8e868e","unresolved":false,"context_lines":[{"line_number":71,"context_line":"                       \"vmtp\","},{"line_number":72,"context_line":"                       \"watcher-base\","},{"line_number":73,"context_line":"                       \"congress-base\","},{"line_number":74,"context_line":"                       \"bifrost-base\",]"},{"line_number":75,"context_line":""},{"line_number":76,"context_line":"    def setUp(self):"},{"line_number":77,"context_line":"        super(BuildTestCentosBinary, self).setUp()"}],"source_content_type":"text/x-python","patch_set":30,"id":"fa7ab95a_1d76f6d6","line":74,"updated":"2016-09-02 05:06:08.000000000","message":"remove \",\"","commit_id":"016ee415ea41dd09e7089c783d39b655187c36e7"}]}