Skip to content
This repository was archived by the owner on Sep 18, 2018. It is now read-only.

Fix kill method (GentleKillingExecutor)#72

Merged
tomislater merged 3 commits intomasterfrom
fix-gentle-executor
Aug 14, 2014
Merged

Fix kill method (GentleKillingExecutor)#72
tomislater merged 3 commits intomasterfrom
fix-gentle-executor

Conversation

@tomislater
Copy link
Copy Markdown
Contributor

No description provided.

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Prefer importing separate items, especially, that we use only two items out of a whole module.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

k

@tomislater
Copy link
Copy Markdown
Contributor Author

I like random fails.

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

would that require to change default mirakuru version?

Use os.killpg to terminate process (GentleKillingExecutor)
Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It will have fail. We have to wait for new version of mirakuru.

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

http://stackoverflow.com/questions/4789837/how-to-terminate-a-python-subprocess-launched-with-shell-true
for killing all of child processes you should also add preexec_fn=os.setsid to Popen

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

0.2 version mirakuru will do that.

Update requires (mirakuru to 0.2)
@coveralls
Copy link
Copy Markdown

Coverage Status

Coverage increased (+2.64%) when pulling 5df1a6c on fix-gentle-executor into 0684bfc on master.

@coveralls
Copy link
Copy Markdown

Coverage Status

Coverage increased (+2.41%) when pulling 5df1a6c on fix-gentle-executor into 0684bfc on master.

@pwilczynskiclearcode
Copy link
Copy Markdown

👍

tomislater added a commit that referenced this pull request Aug 14, 2014
Fix kill method (GentleKillingExecutor)
@tomislater tomislater merged commit 97c1542 into master Aug 14, 2014
@pwilczynskiclearcode
Copy link
Copy Markdown

@tomislater
Copy link
Copy Markdown
Contributor Author

It's not related. We had the random fails before.

@fizyk
Copy link
Copy Markdown
Member

fizyk commented Aug 14, 2014

It's related to this:
#68 hopefully, I'll
find time to investigate the logfile on sunday


Grzegorz Śliwiński
www: http://www.fizyk.net.pl/
@/xmpp/gtalk: fizyk@fizyk.net.pl
t: 691723735

2014-08-14 13:12 GMT+02:00 Tomek Święcicki notifications@github.com:

It's not related. We had the random fails before.


Reply to this email directly or view it on GitHub
#72 (comment)
.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants