)]}'
{"tasks/nova_post_install.yml":[{"author":{"_account_id":28619,"name":"Dmitriy Rabotyagov","email":"noonedeadpunk@gmail.com","username":"noonedeadpunk"},"change_message_id":"f7b1b46374955853f84eedaa2d78a1f9a7ad100b","unresolved":true,"context_lines":[{"line_number":88,"context_line":"  when:"},{"line_number":89,"context_line":"    - nova_policy_overrides !\u003d {}"},{"line_number":90,"context_line":"  tags:"},{"line_number":91,"context_line":"    - nova-policy-override"},{"line_number":92,"context_line":""},{"line_number":93,"context_line":"# NOTE(cloudnull): This is using \"cp\" instead of copy with a remote_source"},{"line_number":94,"context_line":"#                  because we only want to copy the original files once. and we"}],"source_content_type":"text/x-yaml","patch_set":1,"id":"36367bbc_0ed8427c","line":91,"range":{"start_line":91,"start_character":4,"end_line":91,"end_character":26},"updated":"2021-01-11 15:49:54.000000000","message":"I\u0027m pretty sure we don\u0027t use pattern like this for other services. IMO at least we should also add more common nova-config here as well as trigger service restart I guess?","commit_id":"04cad5f97ba801a4732bdd9565a6ce97b3980417"},{"author":{"_account_id":32053,"name":"Denys Mishchenko","email":"denis@mischenko.org.ua","username":"arddennis"},"change_message_id":"4094fb3e77599fea8b3ce40503e3d7416b9248a2","unresolved":true,"context_lines":[{"line_number":88,"context_line":"  when:"},{"line_number":89,"context_line":"    - nova_policy_overrides !\u003d {}"},{"line_number":90,"context_line":"  tags:"},{"line_number":91,"context_line":"    - nova-policy-override"},{"line_number":92,"context_line":""},{"line_number":93,"context_line":"# NOTE(cloudnull): This is using \"cp\" instead of copy with a remote_source"},{"line_number":94,"context_line":"#                  because we only want to copy the original files once. and we"}],"source_content_type":"text/x-yaml","patch_set":1,"id":"0a8b12a9_594cedc0","line":91,"range":{"start_line":91,"start_character":4,"end_line":91,"end_character":26},"in_reply_to":"36367bbc_0ed8427c","updated":"2021-01-11 17:16:43.000000000","message":"Good point. This is definitely should be included during nova-config. However I\u0027d suggest to keep separate tag as time difference in playbook run with tag nova-config is more than 5 times comparing to just nova-policy-override in my env.\n\nChanges are in patch 2","commit_id":"04cad5f97ba801a4732bdd9565a6ce97b3980417"}]}
