)]}'
{"tasks/keystone_install.yml":[{"author":{"_account_id":17068,"name":"Jean-Philippe Evrard","email":"openstack@a.spamming.party","username":"evrardjp"},"change_message_id":"70967a780632f6c4ac20057b573ec8949605a281","unresolved":false,"context_lines":[{"line_number":95,"context_line":"    changed_use: \"{{ (ansible_pkg_mgr \u003d\u003d \u0027portage\u0027) | ternary(\u0027yes\u0027, omit) }}\""},{"line_number":96,"context_line":"    noreplace: \"{{ (ansible_pkg_mgr \u003d\u003d \u0027portage\u0027) | ternary(\u0027yes\u0027, omit) }}\""},{"line_number":97,"context_line":"  when: (is_metal is undefined) or"},{"line_number":98,"context_line":"        (not is_metal |bool)"},{"line_number":99,"context_line":""},{"line_number":100,"context_line":"- name: Install distro packages"},{"line_number":101,"context_line":"  package:"}],"source_content_type":"text/x-yaml","patch_set":2,"id":"9fb8cfa7_c9176da3","line":98,"range":{"start_line":98,"start_character":13,"end_line":98,"end_character":27},"updated":"2019-06-05 13:41:34.000000000","message":"for clarity I generally wrap is_metal | bool with parens. This way you know that is_metal can be bool like without issue.","commit_id":"258447e54fa1e9a679fcb17e33bc2fb3b7c80aa1"},{"author":{"_account_id":17799,"name":"Logan V","email":"logan2211@gmail.com","username":"Logan2211"},"change_message_id":"7c5d21ab3c49d33855b9fc50b365a302c776d409","unresolved":false,"context_lines":[{"line_number":95,"context_line":"    changed_use: \"{{ (ansible_pkg_mgr \u003d\u003d \u0027portage\u0027) | ternary(\u0027yes\u0027, omit) }}\""},{"line_number":96,"context_line":"    noreplace: \"{{ (ansible_pkg_mgr \u003d\u003d \u0027portage\u0027) | ternary(\u0027yes\u0027, omit) }}\""},{"line_number":97,"context_line":"  when: (is_metal is undefined) or"},{"line_number":98,"context_line":"        \"not (is_metal | bool)\""},{"line_number":99,"context_line":""},{"line_number":100,"context_line":"- name: Install distro packages"},{"line_number":101,"context_line":"  package:"}],"source_content_type":"text/x-yaml","patch_set":3,"id":"9fb8cfa7_0acc895f","line":98,"range":{"start_line":98,"start_character":8,"end_line":98,"end_character":31},"updated":"2019-06-06 01:51:51.000000000","message":"It is odd syntax to have only this part in quotes on a separate line. I think you probably ought to quote the whole thing like:\nwhen: \"is_metal is undefined or\n       not (is_metal | bool)\"","commit_id":"dee552dce03a0bed1abd7b9c89f30d05cc540eb1"},{"author":{"_account_id":17068,"name":"Jean-Philippe Evrard","email":"openstack@a.spamming.party","username":"evrardjp"},"change_message_id":"31ed349381b42ad701fd4bf80656ff1004f4aad1","unresolved":false,"context_lines":[{"line_number":95,"context_line":"    changed_use: \"{{ (ansible_pkg_mgr \u003d\u003d \u0027portage\u0027) | ternary(\u0027yes\u0027, omit) }}\""},{"line_number":96,"context_line":"    noreplace: \"{{ (ansible_pkg_mgr \u003d\u003d \u0027portage\u0027) | ternary(\u0027yes\u0027, omit) }}\""},{"line_number":97,"context_line":"  when: (is_metal is undefined) or"},{"line_number":98,"context_line":"        \"not (is_metal | bool)\""},{"line_number":99,"context_line":""},{"line_number":100,"context_line":"- name: Install distro packages"},{"line_number":101,"context_line":"  package:"}],"source_content_type":"text/x-yaml","patch_set":3,"id":"9fb8cfa7_47f0eafd","line":98,"range":{"start_line":98,"start_character":8,"end_line":98,"end_character":31},"in_reply_to":"9fb8cfa7_0acc895f","updated":"2019-06-11 10:19:30.000000000","message":"Agreed. The conditions look correct now though. It\u0027s probably possible to put that into a single line too.","commit_id":"dee552dce03a0bed1abd7b9c89f30d05cc540eb1"}]}