Review Request 49512: [FEEDBACK] Add thermos option to monitor whole docker storage disk usage

2016-07-01 Thread Martin Hrabovcin
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/49512/ --- Review request for Aurora. Repository: aurora Description --- I'd like t

Re: Review Request 49512: [FEEDBACK] Add thermos option to monitor whole docker storage disk usage

2016-07-01 Thread Aurora ReviewBot
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/49512/#review140365 --- Master (b3f88bc) is red with this patch. ./build-support/jenkins

Re: Review Request 49512: [FEEDBACK] Add thermos option to monitor whole docker storage disk usage

2016-07-04 Thread Stephan Erb
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/49512/#review140619 --- Have you considered querying Mesos for the disk usage of the task?

Re: Review Request 49512: [FEEDBACK] Add thermos option to monitor whole docker storage disk usage

2016-07-07 Thread Martin Hrabovcin
> On July 4, 2016, 7:55 a.m., Stephan Erb wrote: > > Have you considered querying Mesos for the disk usage of the task? That > > would be in line with our desire to also leave the isolation up to Mesos > > https://issues.apache.org/jira/browse/AURORA-1033 I wasn't aware of this ticket and I lo

Re: Review Request 49512: [FEEDBACK] Add thermos option to monitor whole docker storage disk usage

2016-07-12 Thread Stephan Erb
> On July 4, 2016, 9:55 a.m., Stephan Erb wrote: > > Have you considered querying Mesos for the disk usage of the task? That > > would be in line with our desire to also leave the isolation up to Mesos > > https://issues.apache.org/jira/browse/AURORA-1033 > > Martin Hrabovcin wrote: > I wa

Re: Review Request 49512: [FEEDBACK] Add thermos option to monitor whole docker storage disk usage

2016-07-12 Thread Joshua Cohen
This is why I haven't commented on the review as well. I'm close to updating my patch to take advantage of some upstream changes to Mesos that allows us to rely on their logic for isolating a filesystem ( https://reviews.apache.org/r/49569/), hopefully will have something out by EOW. That said, it

Re: Review Request 49512: [FEEDBACK] Add thermos option to monitor whole docker storage disk usage

2016-07-15 Thread George Sirois
> On July 4, 2016, 7:55 a.m., Stephan Erb wrote: > > Have you considered querying Mesos for the disk usage of the task? That > > would be in line with our desire to also leave the isolation up to Mesos > > https://issues.apache.org/jira/browse/AURORA-1033 > > Martin Hrabovcin wrote: > I wa

Re: Review Request 49512: [FEEDBACK] Add thermos option to monitor whole docker storage disk usage

2016-07-19 Thread Martin Hrabovcin
> On July 4, 2016, 7:55 a.m., Stephan Erb wrote: > > Have you considered querying Mesos for the disk usage of the task? That > > would be in line with our desire to also leave the isolation up to Mesos > > https://issues.apache.org/jira/browse/AURORA-1033 > > Martin Hrabovcin wrote: > I wa

Re: Review Request 49512: [FEEDBACK] Add thermos option to monitor whole docker storage disk usage

2016-07-19 Thread Stephan Erb
> On July 4, 2016, 9:55 a.m., Stephan Erb wrote: > > Have you considered querying Mesos for the disk usage of the task? That > > would be in line with our desire to also leave the isolation up to Mesos > > https://issues.apache.org/jira/browse/AURORA-1033 > > Martin Hrabovcin wrote: > I wa