Sachin Agarwal

Results 10 comments of Sachin Agarwal

> Since the decorator is being added to most class methods, we could look at an automated way of adding this to the callables in a class (e.g. https://stackoverflow.com/questions/6307761/how-to-decorate-all-functions-of-a-class-without-typing-it-over-and-over-for-eac); that...

Is this getting merged and released soon? I am hitting the same issue

I also ran into this bug, waiting on the fix to be merged and released.

A lot of devs have M1 Arm64 machines now, not having a Arm64 docker image for Rabbitmq shuts them out from Rabbitmq.

We could add something like the below in https://github.com/pycontribs/subprocess-tee/blob/c1b451090d0770740b5f29897771a95f7e2d8484/src/subprocess_tee/__init__.py#L97 (can send PR if we agree) ``` ) if kwargs.get("timeout", -1): await asyncio.wait_for(process.wait(), timeout=kwargs.get("timeout", -1)) ```

Was there a resolution or workaround for this?

Update: new day new behavior! Today the exact same commands worked (no reboot/ssh time issue); definitely may be worth parameterizing that timeout given the unknown reboot times of public cloud...

@nathan-osman Are you planning on accepting the PR from @Adnn? Either ways if you could close the PR (with or without accepting the changes) so people wanting to use the...

I can run the package locally on my machine and test it against serverless. Will look at the ci tests and update.