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

rultor not timing out #1042

Open
AbdealiLoKo opened this issue Mar 5, 2016 · 10 comments
Open

rultor not timing out #1042

AbdealiLoKo opened this issue Mar 5, 2016 · 10 comments
Labels

Comments

@AbdealiLoKo
Copy link

Hey,

I was testing some functionality on rultor and forgot to add the -y flag in apt-get. Because of this, apt-get inside rultor asked for a prompt and got stuck.
Logs: http://www.rultor.com/t/7510-192639053
PR: AbdealiLoKo/pycolorname#15 (comment)

I tried aborting it with the rultor commands, but it didnt seem to work. I then manually pushed the changs to master in the hopes that when the PR closes rultor would stop - but it doesnt seem to have done that either.

I think the container is still there on the rultor (As rultor hasnt reported back saying anything like "Timed out - here's your log")

Could you check up on this ? I think a max time limit of 15-30 mins should be imposed.

@original-brownbear
Copy link
Contributor

@AbdealiJK there actually is a timeout of 2h imposed.
But I agree, we should have a timeout in case of no output for 15 minutes.
Another issue here is that the stop command appears broken, we already have a ticket for this here #864.

Whoever handles this, please implement a 15 minute timeout killing a build when not output is received for 15 minutes. Travis in fact does the exact same as described here.

@original-brownbear
Copy link
Contributor

@alex-palevsky this is a bug.

@alex-palevsky
Copy link
Contributor

@alex-palevsky this is a bug.

@original-brownbear I added bug tag to this ticket

@alex-palevsky alex-palevsky added this to the 2.0 milestone Mar 6, 2016
@alex-palevsky
Copy link
Contributor

@abdealijk attached this issue to milestone "2.0" (let me know if this is wrong)

@original-brownbear
Copy link
Contributor

@alex-palevsky this depends on #1055.

@alex-palevsky
Copy link
Contributor

@alex-palevsky this depends on #1055.

@original-brownbear agreed, we'll wait for #1055

@original-brownbear
Copy link
Contributor

@alex-palevsky this is postponed.

@alex-palevsky
Copy link
Contributor

@alex-palevsky this is postponed.

@original-brownbear right, I added "postponed" label

@alex-palevsky
Copy link
Contributor

@alex-palevsky this is postponed.

@original-brownbear no problem, I will try to find somebody else

@alex-palevsky
Copy link
Contributor

@abdealijk we waited for #1055, it is closed already

@yegor256 yegor256 removed this from the 2.0 milestone Nov 7, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

4 participants