Nick Mills-Barrett

Results 277 comments of Nick Mills-Barrett

Thank you for raising this @drewp, concerning given the similar #878 - at this point I'm not sure I trust paramiko's checking of the uploaded size to properly validate an...

Had a second pass at the retry code and found a pyinfra bug which may well be the cause of both false-positives here! Won't fix any underlying issues but will...

Hm, rather annoyingly per [this table](https://winscp.net/eng/docs/sftp_codes) code 4 is an unknown/generic failure and the text should explain more. It is interesting that the issue seems to be opening the remote...

The move to use entrypoints should facilitate exactly this kind of situation! I don't have time to pull an example together right now but essentially you need to define a...

Yep this looks wrong, thanks for reporting I'll fix this up!

The “success” bit is determined as “did run any commands”, in the case of git it the “git pull” command. I think this is correct (it did run, it did...

Hi @gnat! So rather funny - I actually added partial support for this in v2.6: https://github.com/Fizzadar/pyinfra/commit/6b704b53a6ffb7cc63a1c9267ba3ee24d3c0638b But it's not _quite_ there - I like your example of passing a file...

@gnat how are you running the above? ``` $ cat testo.py from pyinfra.operations import server from pyinfra.api import operation def test(): server.shell("hello world") @operation() def test2(): yield "echo 'hello world'"...

Warning added here: https://github.com/pyinfra-dev/pyinfra/commit/b575a9b8a00b2a8cfd6ec03dcee776181841dd8c I think pyinfra should support fetching and setting the Signed-By field in the `apt.repo` operation to support a clean switchover, I'll re-purpose this issue to track...