pantsbuild #general

Channels

welcome

github-notifications

announce

enough-analyst-54434

09/16/2015, 12:28 AM

have the human bit

rough-minister-58256

09/16/2015, 12:28 AM

ah yeah

rough-minister-58256

09/16/2015, 12:28 AM

task types

enough-analyst-54434

09/16/2015, 12:28 AM

some bit somewhere, yeah

rough-minister-58256

09/16/2015, 12:28 AM

makes sense

enough-analyst-54434

09/16/2015, 12:29 AM

certain tasks will require intermediates are gathered back from the build borg

enough-analyst-54434

09/16/2015, 12:29 AM

to the local machine for the last step

ancient-stone-50795

09/16/2015, 12:31 AM

@rough-minister-58256: we have a case where we do user input in a task. Its like a publish, but we publish a file into our staging environment. You have to enter in a 2 factor auth key in a timely fashion.

rough-minister-58256

09/16/2015, 12:32 AM

ah cool

rough-minister-58256

09/16/2015, 12:32 AM

all makes sense - thanks gents!

fancy-queen-20734

09/16/2015, 12:45 AM

I can reliably reproduce the left over processes issue when running tests. I think it’s just a test issue though

witty-crayon-22786

09/16/2015, 3:39 PM

@bored-art-40741: so, this should have been eminently obvious, but it wasn't to me:

witty-crayon-22786

09/16/2015, 3:39 PM

our ci hosts use rotational disks

witty-crayon-22786

09/16/2015, 3:39 PM

and our laptops use ssds

witty-crayon-22786

09/16/2015, 3:39 PM

and so the IO constraints that isolated introduces are more pronounced there

witty-crayon-22786

09/16/2015, 3:40 PM

long story short: profiles captured in CI are very different from those captured locally

bored-art-40741

09/16/2015, 3:41 PM

Interesting

bored-art-40741

09/16/2015, 3:41 PM

Our CIs run strictly over ramdisks

witty-crayon-22786

09/16/2015, 3:41 PM

hah. good call.

dry-analyst-73584

09/16/2015, 3:48 PM

It looks like this is ready to go upstream simple smile Can someone please patch it in? https://rbcommons.com/s/twitter/r/2755/

witty-crayon-22786

09/16/2015, 3:48 PM

sure

dry-analyst-73584

09/16/2015, 3:48 PM

Thanks!

witty-crayon-22786

09/16/2015, 3:49 PM

@dry-analyst-73584: merged! thanks for iterating

limited-country-20626

09/16/2015, 4:23 PM

I sent out the link to the isolated benchmarking document. The big concern is the instability on big builds

limited-country-20626

09/16/2015, 4:23 PM

https://docs.google.com/document/d/1J1gG7bdSfNfMarrCDMn4kt3UTtAMQ7hTxJaUxHkor0Q/edit

witty-crayon-22786

09/16/2015, 4:26 PM

@limited-country-20626: thanks!

bored-art-40741

09/16/2015, 4:27 PM

Yup, if your lazy loading relieves the heap pressure then I suspect we'll be in a good place

witty-crayon-22786

09/16/2015, 4:28 PM

very thorough, thank you.

limited-country-20626

09/16/2015, 4:28 PM

ohsnap due to the fact that I am a gmail address for pants-devel purposes the sharing is off. Give me a second

witty-crayon-22786

09/16/2015, 4:28 PM

i'm just polishing that a bit now.