jeanluc

Salt-SSH employs several crutches, one of them is that the template -> highstate -> lowstate rendering happens on the master. It collects `salt://` file references from this lowstate and sends...

That's because `cp.cache_file` returns the path on the minion, while you're checking if the file exists on the master since it's evaluated during template rendering. If you used `__salt__["file.file_exists"](location)` (which...

@yajo Is there anything I still need to do here? Just making sure you noticed the changes. :)

> Sorry for taking so long to give feedback. 🫣 This PR really needs some deep thinking. @sisp Thank you for your thoughtful feedback and sorry for taking my time...

@mschoettle > There are some things, such as questions/answers and tasks, that I don’t want to ask/run on update. And, other tasks should only run when an update is performed....

> I think we should rename `_operation` to `_copier_operation`. @sisp Done and rebased. Edit: Having read the discussion in https://github.com/copier-org/copier/pull/1948, I want to emphasize that (because of https://github.com/copier-org/copier/pull/1733#discussion_r1770047960) this patch...

> because it was unexpected to one person The issue was triaged and found to be valid. > Here's a common scenario. [...] They now have to recopy which adds...

>> Is this specific scenario really more common than the file just receiving minor updates from time to time? > For the templates I manage yes. This might partly explain...

> What I'm worried about is not just cases where updating a config fails obviously, but when it also fails silently. These examples are not specific to updates though. Even...

@wgordon17 I'm pretty sure this is already implemented since https://github.com/copier-org/copier/pull/2023 and can thus be closed. In a test, I could access `_folder_name` as the default value in a question when...