)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":28619,"name":"Dmitriy Rabotyagov","email":"noonedeadpunk@gmail.com","username":"noonedeadpunk"},"change_message_id":"3f602fd02d22aff1ed3acc5acaf911b3f264bfd0","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"cc3815a9_a210f6ac","updated":"2023-10-28 13:13:17.000000000","message":"I wonder if we should also add `ansible_python_interpreter: \"{{ansible_playbook_python}}\"` to fully mimic behaviour of implicit localhost?\n\nAlso would suggest adding release note, as who knows if anyone relies on a current default in a weird way...\n\nBut lgtm overall","commit_id":"389ca11c8c576091ed7056710f6c636f78d869df"},{"author":{"_account_id":32666,"name":"Damian Dąbrowski","email":"damian@dabrowski.cloud","username":"ddabrowski"},"change_message_id":"902e87b690b6cdb32057dc18b1fc5d5aec8c6481","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"25bd3e29_bc362e4b","in_reply_to":"cc3815a9_a210f6ac","updated":"2023-10-30 12:03:43.000000000","message":"ansible_python_interpreter is defined in /usr/local/bin/openstack-ansible.rc so i think there\u0027s no need to define it once again(especially when inventory.ini is just a static file)\n\nrelease note added","commit_id":"389ca11c8c576091ed7056710f6c636f78d869df"}],"releasenotes/notes/explicit-localhost-inventory-96c5b0b28e23a066.yaml":[{"author":{"_account_id":34411,"name":"Neil Hanlon","email":"neil@shrug.pw","username":"nhanlon"},"change_message_id":"d9c235f1efdcff117039331470de57de5889d157","unresolved":true,"context_lines":[{"line_number":1,"context_line":"---"},{"line_number":2,"context_line":"other:"},{"line_number":3,"context_line":"  - |"},{"line_number":4,"context_line":"    Due to a bug #2041717, localhost was explicitly added to OSA inventory."},{"line_number":5,"context_line":"    It may affect a behavior of some custom playbooks that are targeted"},{"line_number":6,"context_line":"    against \u0027all\u0027 group(now localhost is a part of \u0027all\u0027 group)."}],"source_content_type":"text/x-yaml","patch_set":3,"id":"e1a63384_92f5a759","line":6,"range":{"start_line":4,"start_character":0,"end_line":6,"end_character":64},"updated":"2023-10-31 17:08:40.000000000","message":"Small suggestion on phrasing for the release notes:\n\n\u003e The `localhost` target was explicitly added to OSA inventory due to bug #2041717. As a result, the \u0027all\u0027 group now contains localhost, and custom playbooks targeting \u0027all\u0027 may need adjustment, e.g.: `hosts: all:!localhost`","commit_id":"7d9e0460c649a4bd2316d0006b30c721729ff390"},{"author":{"_account_id":32666,"name":"Damian Dąbrowski","email":"damian@dabrowski.cloud","username":"ddabrowski"},"change_message_id":"af19a42201f05f3312eb8247289a4223bb0b48b0","unresolved":false,"context_lines":[{"line_number":1,"context_line":"---"},{"line_number":2,"context_line":"other:"},{"line_number":3,"context_line":"  - |"},{"line_number":4,"context_line":"    Due to a bug #2041717, localhost was explicitly added to OSA inventory."},{"line_number":5,"context_line":"    It may affect a behavior of some custom playbooks that are targeted"},{"line_number":6,"context_line":"    against \u0027all\u0027 group(now localhost is a part of \u0027all\u0027 group)."}],"source_content_type":"text/x-yaml","patch_set":3,"id":"681f1679_8f0f011f","line":6,"range":{"start_line":4,"start_character":0,"end_line":6,"end_character":64},"in_reply_to":"e1a63384_92f5a759","updated":"2023-11-01 00:33:57.000000000","message":"Thanks Neil, fixed.","commit_id":"7d9e0460c649a4bd2316d0006b30c721729ff390"}]}