Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[Request] Use timeout linux command and init=True in the docker SDK to make the command timeout for long-lived processes #1

Open
ArachnidAbby opened this issue Nov 15, 2023 · 5 comments
Assignees
Labels
bug Something isn't working enhancement New feature or request

Comments

@ArachnidAbby
Copy link

In reference to: src/exec/__init__.py

Currently running an infinite loop will halt the bot.
The timeout command should be able to kill the script if it runs to long. This won't work if the processes ID is 1.

To ensure this works, apparently you need to change lines 15-20 to include init=True. I haven't tested this, so I'm not 100% sure.

It's worth a shot and could fix a huge bug.

@ArachnidAbby
Copy link
Author

btw, would have just sent this in discord, but I'm at school right now, so the only way I can get you the information is in a github issue.

@mauro-balades mauro-balades self-assigned this Nov 15, 2023
@mauro-balades mauro-balades added bug Something isn't working enhancement New feature or request labels Nov 15, 2023
@ArachnidAbby
Copy link
Author

okay, I can now confirm that this does work.

@mauro-balades
Copy link
Member

Thanks! I will try to do this once I have time. 💯

@ArachnidAbby
Copy link
Author

any luck with this?

@mauro-balades
Copy link
Member

any luck with this?

I coudnt work on this because I'm currently very busy and this is a low priority bug

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

2 participants