)]}'
{"ansible/roles/ceilometer/tasks/precheck.yml":[{"author":{"_account_id":28614,"name":"jacky06","email":"zhang.min@99cloud.net","username":"jacky06"},"change_message_id":"8240eb8694b4041bf56de40ba33f7fd28186ea46","unresolved":false,"context_lines":[{"line_number":9,"context_line":""},{"line_number":10,"context_line":"- name: Ensure ceilometer enabled for ceilometer-ipmi"},{"line_number":11,"context_line":"  run_once: True"},{"line_number":12,"context_line":"  local_action: fail msg\u003d\"ceilometer is required  when enable ceilometer-ipmi but not enabled\""},{"line_number":13,"context_line":"  changed_when: false"},{"line_number":14,"context_line":"  when:"},{"line_number":15,"context_line":"    - not enable_ceilometer | bool"}],"source_content_type":"text/x-yaml","patch_set":1,"id":"5f7c97a3_79c2a404","line":12,"range":{"start_line":12,"start_character":48,"end_line":12,"end_character":49},"updated":"2018-07-15 02:08:26.000000000","message":"unnecessary space","commit_id":"3d75c6ab41665dfa25cc9c7d539dcb8b9ef41881"},{"author":{"_account_id":22165,"name":"caoyuan","email":"cao.yingjunz@gmail.com","username":"caoyuan"},"change_message_id":"90bfa5c7c0587d3684d45ca1c88e43c8a9171eb7","unresolved":false,"context_lines":[{"line_number":9,"context_line":""},{"line_number":10,"context_line":"- name: Ensure ceilometer enabled for ceilometer-ipmi"},{"line_number":11,"context_line":"  run_once: True"},{"line_number":12,"context_line":"  local_action: fail msg\u003d\"ceilometer is required  when enable ceilometer-ipmi but not enabled\""},{"line_number":13,"context_line":"  changed_when: false"},{"line_number":14,"context_line":"  when:"},{"line_number":15,"context_line":"    - not enable_ceilometer | bool"}],"source_content_type":"text/x-yaml","patch_set":1,"id":"5f7c97a3_d9dad8e4","line":12,"range":{"start_line":12,"start_character":48,"end_line":12,"end_character":49},"in_reply_to":"5f7c97a3_79c2a404","updated":"2018-07-15 02:11:00.000000000","message":"Done","commit_id":"3d75c6ab41665dfa25cc9c7d539dcb8b9ef41881"},{"author":{"_account_id":28614,"name":"jacky06","email":"zhang.min@99cloud.net","username":"jacky06"},"change_message_id":"ef38805b2d15b94ffd68eeb2b5063b913357adae","unresolved":false,"context_lines":[{"line_number":9,"context_line":""},{"line_number":10,"context_line":"- name: Ensure ceilometer enabled for ceilometer-ipmi"},{"line_number":11,"context_line":"  run_once: True"},{"line_number":12,"context_line":"  local_action: fail msg\u003d\"ceilometer is required  when enable ceilometer-ipmi but not enabled\""},{"line_number":13,"context_line":"  changed_when: false"},{"line_number":14,"context_line":"  when:"},{"line_number":15,"context_line":"    - not enable_ceilometer | bool"}],"source_content_type":"text/x-yaml","patch_set":1,"id":"5f7c97a3_f9d554b9","line":12,"range":{"start_line":12,"start_character":48,"end_line":12,"end_character":49},"in_reply_to":"5f7c97a3_79c2a404","updated":"2018-07-15 02:09:55.000000000","message":"Done","commit_id":"3d75c6ab41665dfa25cc9c7d539dcb8b9ef41881"},{"author":{"_account_id":19316,"name":"Eduardo Gonzalez","email":"dabarren@gmail.com","username":"egonzalez90"},"change_message_id":"a8399786aaf26f3e468c74400984f5ae0e05640a","unresolved":false,"context_lines":[{"line_number":7,"context_line":"    - enable_ceilometer | bool"},{"line_number":8,"context_line":"    - not enable_gnocchi | bool"},{"line_number":9,"context_line":""},{"line_number":10,"context_line":"- name: Ensure ceilometer enabled for ceilometer-ipmi"},{"line_number":11,"context_line":"  run_once: True"},{"line_number":12,"context_line":"  local_action: fail msg\u003d\"ceilometer is required for ceilometer-ipmi but not enabled\""},{"line_number":13,"context_line":"  changed_when: false"},{"line_number":14,"context_line":"  when:"},{"line_number":15,"context_line":"    - not enable_ceilometer | bool"},{"line_number":16,"context_line":"    - enable_ceilometer_ipmi | bool"}],"source_content_type":"text/x-yaml","patch_set":4,"id":"3f79a3b5_f51a2e16","line":16,"range":{"start_line":10,"start_character":0,"end_line":16,"end_character":35},"updated":"2018-09-14 11:57:41.000000000","message":"if the prechecks enter into ceilometer prechecks, means enable_ceilometer is true. is this check needed?\n\nIt will always succeed","commit_id":"8bca8f3cac2867fcb0b16fba7728885241124f8d"},{"author":{"_account_id":22165,"name":"caoyuan","email":"cao.yingjunz@gmail.com","username":"caoyuan"},"change_message_id":"8aa1eabb62518889cf8b03027cd927ad35527ec7","unresolved":false,"context_lines":[{"line_number":7,"context_line":"    - enable_ceilometer | bool"},{"line_number":8,"context_line":"    - not enable_gnocchi | bool"},{"line_number":9,"context_line":""},{"line_number":10,"context_line":"- name: Ensure ceilometer enabled for ceilometer-ipmi"},{"line_number":11,"context_line":"  run_once: True"},{"line_number":12,"context_line":"  local_action: fail msg\u003d\"ceilometer is required for ceilometer-ipmi but not enabled\""},{"line_number":13,"context_line":"  changed_when: false"},{"line_number":14,"context_line":"  when:"},{"line_number":15,"context_line":"    - not enable_ceilometer | bool"},{"line_number":16,"context_line":"    - enable_ceilometer_ipmi | bool"}],"source_content_type":"text/x-yaml","patch_set":4,"id":"3f79a3b5_5ce9bce0","line":16,"range":{"start_line":10,"start_character":0,"end_line":16,"end_character":35},"in_reply_to":"3f79a3b5_f51a2e16","updated":"2018-09-15 14:51:12.000000000","message":"yes, seems no need to do this, thanks for review","commit_id":"8bca8f3cac2867fcb0b16fba7728885241124f8d"}],"ansible/roles/nova/defaults/main.yml":[{"author":{"_account_id":28614,"name":"jacky06","email":"zhang.min@99cloud.net","username":"jacky06"},"change_message_id":"443444892c4b16df15be4361c37b9aebc1e05eca","unresolved":false,"context_lines":[{"line_number":102,"context_line":"    container_name: \"nova_consoleauth\""},{"line_number":103,"context_line":"    group: \"nova-consoleauth\""},{"line_number":104,"context_line":"    image: \"{{ nova_consoleauth_image_full }}\""},{"line_number":105,"context_line":"    enabled: False "},{"line_number":106,"context_line":"    volumes:"},{"line_number":107,"context_line":"      - \"{{ node_config_directory }}/nova-consoleauth/:{{ container_config_directory }}/:ro\""},{"line_number":108,"context_line":"      - \"/etc/localtime:/etc/localtime:ro\""}],"source_content_type":"text/x-yaml","patch_set":5,"id":"3f79a3b5_30240dc4","line":105,"range":{"start_line":105,"start_character":18,"end_line":105,"end_character":19},"updated":"2018-11-22 00:37:18.000000000","message":"/ansible/roles/nova/defaults/main.yml\n2018-11-21 16:09:09.411973 | ubuntu-xenial |   105:19    error    trailing spaces  (trailing-spaces)\n\nplease remove the space, it will result in pep8 failed","commit_id":"da84e2e6342da2b6179c7b927287910e11437fcd"}]}