)]}'
{"ansible/group_vars/all/ntp":[{"author":{"_account_id":15197,"name":"Pierre Riteau","email":"pierre@stackhpc.com","username":"priteau","status":"StackHPC"},"change_message_id":"6fcb6272a7d8e7199598b0d1b9ec58701535282b","unresolved":false,"context_lines":[{"line_number":12,"context_line":""},{"line_number":13,"context_line":"# Whether to enable the NTP daemon on the host. Default is true unless"},{"line_number":14,"context_line":"# \u0027kolla_enable_chrony\u0027 has been set to true on overcloud hosts."},{"line_number":15,"context_line":"ntp_service_enabled: \"{{ \u0027overcloud\u0027 not in groups or not kolla_enable_chrony | bool }}\""},{"line_number":16,"context_line":""},{"line_number":17,"context_line":"ntp_package_state: \"{{ \u0027present\u0027 if ntp_service_enabled | bool else \u0027absent\u0027 }}\""},{"line_number":18,"context_line":"ntp_service_state: \"{{ \u0027started\u0027 if ntp_service_enabled | bool else \u0027stopped\u0027 }}\""}],"source_content_type":"application/octet-stream","patch_set":6,"id":"bfb3d3c7_9308a9d9","line":15,"range":{"start_line":15,"start_character":44,"end_line":15,"end_character":50},"updated":"2019-05-22 17:13:45.000000000","message":"If this is to test whether the host being considered is in the overcloud, shouldn\u0027t this be `group_names`? See https://docs.ansible.com/ansible/latest/reference_appendices/special_variables.html","commit_id":"5c193583ddcebdcdea6f2105a2fd31c70ad5696c"},{"author":{"_account_id":14826,"name":"Mark Goddard","email":"markgoddard86@gmail.com","username":"mgoddard"},"change_message_id":"e28028843af478b091256f324d4d5b8faf672a7d","unresolved":false,"context_lines":[{"line_number":12,"context_line":""},{"line_number":13,"context_line":"# Whether to enable the NTP daemon on the host. Default is true unless"},{"line_number":14,"context_line":"# \u0027kolla_enable_chrony\u0027 has been set to true on overcloud hosts."},{"line_number":15,"context_line":"ntp_service_enabled: \"{{ \u0027overcloud\u0027 not in groups or not kolla_enable_chrony | bool }}\""},{"line_number":16,"context_line":""},{"line_number":17,"context_line":"ntp_package_state: \"{{ \u0027present\u0027 if ntp_service_enabled | bool else \u0027absent\u0027 }}\""},{"line_number":18,"context_line":"ntp_service_state: \"{{ \u0027started\u0027 if ntp_service_enabled | bool else \u0027stopped\u0027 }}\""}],"source_content_type":"application/octet-stream","patch_set":6,"id":"bfb3d3c7_c265dfe2","line":15,"range":{"start_line":15,"start_character":44,"end_line":15,"end_character":50},"in_reply_to":"bfb3d3c7_9308a9d9","updated":"2019-05-23 09:42:28.000000000","message":"You are quite right. The dumb thing is I actually spotted this then somehow forgot to fix.","commit_id":"5c193583ddcebdcdea6f2105a2fd31c70ad5696c"}]}