)]}'
{"kayobe/ansible.py":[{"author":{"_account_id":14826,"name":"Mark Goddard","email":"markgoddard86@gmail.com","username":"mgoddard"},"change_message_id":"5f356a276efbaae1f7ad588777500f783666163c","unresolved":true,"context_lines":[{"line_number":98,"context_line":"                inventories.append(shared_inventory)"},{"line_number":99,"context_line":"            env_inventory \u003d os.path.join(env_path, \"inventory\")"},{"line_number":100,"context_line":"            if os.path.exists(env_inventory):"},{"line_number":101,"context_line":"                inventories.append(env_inventory)"},{"line_number":102,"context_line":"        else:"},{"line_number":103,"context_line":"            # Preserve existing behaviour: don\u0027t check if an inventory"},{"line_number":104,"context_line":"            # directory exists when no environment is specified"}],"source_content_type":"text/x-python","patch_set":1,"id":"df4614f7_7b73a020","line":101,"updated":"2021-02-02 17:04:44.000000000","message":"Does this mean that the env inventory takes precedence over the shared one where there is a conflict?","commit_id":"772f85e376450004386d82d8106e12971148147d"},{"author":{"_account_id":15197,"name":"Pierre Riteau","email":"pierre@stackhpc.com","username":"priteau","status":"StackHPC"},"change_message_id":"ce7acd30c8f8ddb6af5b559fd7967865558bb1ed","unresolved":true,"context_lines":[{"line_number":98,"context_line":"                inventories.append(shared_inventory)"},{"line_number":99,"context_line":"            env_inventory \u003d os.path.join(env_path, \"inventory\")"},{"line_number":100,"context_line":"            if os.path.exists(env_inventory):"},{"line_number":101,"context_line":"                inventories.append(env_inventory)"},{"line_number":102,"context_line":"        else:"},{"line_number":103,"context_line":"            # Preserve existing behaviour: don\u0027t check if an inventory"},{"line_number":104,"context_line":"            # directory exists when no environment is specified"}],"source_content_type":"text/x-python","patch_set":1,"id":"f0b3e782_f455c38b","line":101,"in_reply_to":"df4614f7_7b73a020","updated":"2021-02-02 20:53:07.000000000","message":"I assume so, although I have not tested this yet. This would be the expected behaviour given how env extra vars override shared ones.","commit_id":"772f85e376450004386d82d8106e12971148147d"},{"author":{"_account_id":14826,"name":"Mark Goddard","email":"markgoddard86@gmail.com","username":"mgoddard"},"change_message_id":"29ebe61cb8985dc67c4f4ecfec614e32e3d574ec","unresolved":true,"context_lines":[{"line_number":98,"context_line":"                inventories.append(shared_inventory)"},{"line_number":99,"context_line":"            env_inventory \u003d os.path.join(env_path, \"inventory\")"},{"line_number":100,"context_line":"            if os.path.exists(env_inventory):"},{"line_number":101,"context_line":"                inventories.append(env_inventory)"},{"line_number":102,"context_line":"        else:"},{"line_number":103,"context_line":"            # Preserve existing behaviour: don\u0027t check if an inventory"},{"line_number":104,"context_line":"            # directory exists when no environment is specified"}],"source_content_type":"text/x-python","patch_set":1,"id":"b3023d3c_ed921b1d","line":101,"in_reply_to":"f0b3e782_f455c38b","updated":"2021-03-10 12:21:55.000000000","message":"Tested. Later inventory arguments take precedence. So this should work as expected.","commit_id":"772f85e376450004386d82d8106e12971148147d"}]}