Re: Review Request 32889: Only perform escalation wait when http teardown signal could be dispatched

--- ./pants test.pytest --no-fast --options=-v src/test/python/apache/aurora/executor:thermos_task_runner In addition, manual verification that shutdown of health-checked services without lifecycle methods is 10 seconds faster. Thanks, Stephan Erb

Re: Review Request 32889: Only perform escalation wait when http teardown signal could be dispatched

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/32889/#review79718 --- On April 11, 2015, 11:52 p.m., Stephan Erb wrote

Re: Review Request 33612: Add a task store implementation that uses a relational database.

/DbTaskStore.java https://reviews.apache.org/r/33612/#comment133061 Copy paste error. Should be `db_storage_save_tasks`. - Stephan Erb On April 28, 2015, 10:11 p.m., Bill Farner wrote: --- This is an automatically generated e-mail. To reply

Re: Review Request 33689: Updated scheduler to process status updates asynchronously in batches.

, using a LinkedList here seems rather uncommon. - Stephan Erb On May 2, 2015, 12:55 a.m., Ben Mahler wrote: --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/33689

Re: Review Request 33689: Updated scheduler to process status updates asynchronously in batches.

https://reviews.apache.org/r/33689/#comment134073 You mentioned that updates should be processed in FIFO order. As we have effectively lost the entire batch here, do we have to clear the pendingUpdates queue to prevent our of order processing? - Stephan Erb On May 7, 2015, 2:27 a.m., Ben

Re: Review Request 32889: Only perform escalation wait when http teardown signal could be dispatched

faster. Thanks, Stephan Erb

Re: Review Request 32889: Only perform escalation wait when http teardown signal could be dispatched

: https://reviews.apache.org/r/32889/#review80210 --- On April 15, 2015, 8:38 p.m., Stephan Erb wrote: --- This is an automatically generated e-mail. To reply, visit: https

Re: Review Request 33184: Implement missing != operator for AuroraJobKey

, 9:18 p.m., Stephan Erb wrote: --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/33184/ --- (Updated April 14, 2015, 9:18 p.m

Re: Review Request 33184: Implement missing != operator for AuroraJobKey

://reviews.apache.org/r/33184/diff/ Testing --- ./pants test.pytest --no-fast --options=-v src/test/python/apache/aurora/common:: Thanks, Stephan Erb

Re: Review Request 32861: Increase executor resource epsilon to match the minimum required by Mesos

8ae68653427c0bdbf33751d5ed4b74afcaf98a7b Diff: https://reviews.apache.org/r/32861/diff/ Testing --- ./gradlew -Pq build Thanks, Stephan Erb

Re: Review Request 35498: Compute SLA stats for non-prod jobs

-to-constant-expression - Stephan --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/35498/#review88552 --- On June 19, 2015, 5:23 p.m., Stephan Erb

Re: Review Request 35498: Compute SLA stats for non-prod jobs

5ee123a03e3c8670e0c03b05c48a9f4c66f6af9d Diff: https://reviews.apache.org/r/35498/diff/ Testing --- `./gradlew -Pq build` and a manual verification in Vagrant. Thanks, Stephan Erb

Re: Review Request 35498: Compute SLA stats for non-prod jobs

, 2015, 9:09 p.m., Stephan Erb wrote: --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/35498/ --- (Updated June 24, 2015, 9:09 p.m

Re: Review Request 35498: Compute SLA stats for non-prod jobs

--- `./gradlew -Pq build` and a manual verification in Vagrant. Thanks, Stephan Erb

Re: Review Request 35498: Compute SLA stats for non-prod jobs

rather than direct field access. Stephan Erb wrote: I can submit an updated patch tonight. I've somewhat expected that some of you would point it out :-) I thought about adding these when writing the patch but then decided against them: Getters for final attributes on an inner

Re: Review Request 35498: Compute SLA stats for non-prod jobs

cb98834e925793fc116814371548a30470830164 src/test/java/org/apache/aurora/scheduler/sla/SlaModuleTest.java 5ee123a03e3c8670e0c03b05c48a9f4c66f6af9d Diff: https://reviews.apache.org/r/35498/diff/ Testing --- `./gradlew -Pq build` and a manual verification in Vagrant. Thanks, Stephan Erb

Re: Review Request 35498: Compute SLA stats for non-prod jobs

cb98834e925793fc116814371548a30470830164 src/test/java/org/apache/aurora/scheduler/sla/SlaModuleTest.java 5ee123a03e3c8670e0c03b05c48a9f4c66f6af9d Diff: https://reviews.apache.org/r/35498/diff/ Testing --- `./gradlew -Pq build` and a manual verification in Vagrant. Thanks, Stephan Erb

Re: Review Request 35498: Compute SLA stats for non-prod jobs

src/test/java/org/apache/aurora/scheduler/sla/SlaModuleTest.java 5ee123a03e3c8670e0c03b05c48a9f4c66f6af9d Diff: https://reviews.apache.org/r/35498/diff/ Testing --- `./gradlew -Pq build` and a manual verification in Vagrant. Thanks, Stephan Erb

Review Request 35498: Compute SLA stats for non-prod jobs

, Stephan Erb

Re: Review Request 35498: Compute SLA stats for non-prod jobs

://reviews.apache.org/r/35498/diff/ Testing --- `./gradlew -Pq build` and a manual verification in Vagrant. Thanks, Stephan Erb

Re: Review Request 35498: Compute SLA stats for non-prod jobs

(line 65) https://reviews.apache.org/r/35498/#comment140450 Using a list of enums instead of serveral bools will also help to simplify this code. - Stephan Erb On June 16, 2015, 11 a.m., Stephan Erb wrote

Re: Review Request 35990: Map Aurora task metadata to Mesos task labels

this is a great idea. For details, see: https://github.com/prometheus/mesos_exporter/issues/7 - Stephan Erb On June 30, 2015, 9:36 p.m., Stephan Erb wrote: --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org

Re: Review Request 35990: Map Aurora task metadata to Mesos task labels

://reviews.apache.org/r/35990/#review89750 --- On June 28, 2015, 10:44 p.m., Stephan Erb wrote: --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/35990

Re: Review Request 35498: Compute SLA stats for non-prod jobs

/diff/ Testing --- `./gradlew -Pq build` and a manual verification in Vagrant. Thanks, Stephan Erb

Review Request 35990: Map Aurora task metadata to Mesos task labels

/mesos/MesosTaskFactoryImplTest.java 1b2a7948ebb946a2e12b0eded6acf4ce3c8e20f9 Diff: https://reviews.apache.org/r/35990/diff/ Testing --- ./gradlew -Pq build Thanks, Stephan Erb

Re: Review Request 35498: Compute SLA stats for non-prod jobs

rather than direct field access. Stephan Erb wrote: I can submit an updated patch tonight. I've somewhat expected that some of you would point it out :-) I thought about adding these when writing the patch but then decided against them: Getters for final attributes on an inner

Re: Review Request 37560: Adding TierManager initial implementation.

) https://reviews.apache.org/r/37560/#comment150864 For better readability, how about using an enum? - Stephan Erb On Aug. 18, 2015, 7:04 p.m., Maxim Khutornenko wrote: --- This is an automatically generated e-mail. To reply, visit

Re: Review Request 37560: Adding TierManager initial implementation.

On Aug. 18, 2015, 8:08 p.m., Stephan Erb wrote: src/main/java/org/apache/aurora/scheduler/TierInfo.java, line 27 https://reviews.apache.org/r/37560/diff/2/?file=1043194#file1043194line27 For better readability, how about using an enum? Maxim Khutornenko wrote: Not sure I

Re: Review Request 35498: Compute SLA stats for non-prod jobs

` and a manual verification in Vagrant. Thanks, Stephan Erb

Review Request 36064: Perform leader redirect using HTTP status code 307

f8b134fa1f3fbbd7affef31acb077e7acd4f9761 Diff: https://reviews.apache.org/r/36064/diff/ Testing --- ./gradlew -Pq build Thanks, Stephan Erb

Re: Review Request 36700: Remove references to binary build targets that are not currently for general consumption.

the same flexibility nor the same performance (subprocess call and thrift connections take rather long). - Stephan Erb On July 28, 2015, 2:58 a.m., Bill Farner wrote: --- This is an automatically generated e-mail. To reply, visit: https

Re: Review Request 39670: Create immutable copy of offers for PendingTaskProcessor.

/PendingTaskProcessor.java (line 133) <https://reviews.apache.org/r/39670/#comment162352> Couldn't we run into the same `ConcurrentModificationException` if the offer list is modified while `ImmutableList.copyOf()` is executed? - Stephan Erb On Oct. 27, 2015, 12:16 a.m., Zameer Manji

Re: Review Request 40323: Improve the isolation in the thrift build.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/40323/#review106558 --- Ship it! A working build on Arch, thanks for that\! - Stephan

Review Request 39854: Retry scheduler connection after unknown failures

1b4ff9cef167de1e4051e6aaa6e4fb6961fd20f1 Diff: https://reviews.apache.org/r/39854/diff/ Testing --- ./pants test.pytest --options=-v src/test/python/apache/aurora/client/api:scheduler_client Thanks, Stephan Erb

Re: Review Request 39823: Eliminate duplicate docu in configuration tutorial

ba0dc864a10726456373c23b508e04dd3b08a3ef docs/configuration-tutorial.md bbb1684645b628df5fe8a45fcce6813e718157af Diff: https://reviews.apache.org/r/39823/diff/ Testing --- Rendered documentation is available at https://github.com/StephanErb/aurora/blob/dedup-docu Thanks, Stephan Erb

Re: Review Request 39823: Eliminate duplicate docu in configuration tutorial

--- Rendered documentation is available at https://github.com/StephanErb/aurora/blob/dedup-docu Thanks, Stephan Erb

Re: Review Request 39057: "aurora config read" command

) <https://reviews.apache.org/r/39057/#comment159759> I find it a little bit confusing that this command is called `read`. Isn't the `bind` the most important step it does? - Stephan Erb On Oct. 6, 2015, 8:43 p.m., Brian Wickman

Review Request 39169: Minor improvement of the storoge documentation

--- Fix minor inconsistencies in the storoge documentation. Diffs - docs/storage-config.md 4ec33a14d3361dfd7f5c07a1d01e31bb96243029 Diff: https://reviews.apache.org/r/39169/diff/ Testing --- Thanks, Stephan Erb

Re: Review Request 39169: Fix minor inconsistencies in the storage documentation

://reviews.apache.org/r/39169/diff/ Testing --- Thanks, Stephan Erb

Re: Review Request 36064: Perform leader redirect using HTTP status code 307

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/36064/#review90931 --- Friendly ping. Anything still missing here? - Stephan Erb

Re: Review Request 36305: Remove restriction that -hostname must resolve.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/36305/#review90989 --- Ship it! Ship It! - Stephan Erb On July 8, 2015, 2:53 p.m

Re: Review Request 37801: Added support for {{mesos.hostname}} in aur files

On Aug. 26, 2015, 7:55 p.m., Zameer Manji wrote: The ticket referenced also talks about {{mesos.ip}}, will you also add support for that? Steve Niemitz wrote: It's not exposed anywhere by mesos AFAIK. If someone can correct me I can add it in another review. I have to agree,

Re: Review Request 38385: Documenting dedicated job & quota relationship.

> On Sept. 16, 2015, 2:51 a.m., Bill Farner wrote: > > docs/configuration-tutorial.md, line 583 > > > > > > Woah, this is a new doc to me...seems significantly redundant with > > configuration-reference.md. All

Re: Review Request 38270: Remove redundancy between breadcrumbs and page headers. Also make job dashboard link more prominent.

) <https://reviews.apache.org/r/38270/#comment155097> The title is rather specific and might not match what people outside of Twitter use the job_viz_url for. Besides that: Awesome improvement! - Stephan Erb On Sept. 11, 2015, 4:23 a.m., Joshua Cohen

Re: Review Request 38385: Documenting dedicated job & quota relationship.

to the issue addressed in this review request, so feel free to dismiss it here) - Stephan Erb On Sept. 15, 2015, 1:50 a.m., Maxim Khutornenko wrote: > > --- > This is an automatically generated e-mail. To reply, visit

Re: Review Request 38270: Remove redundancy between breadcrumbs and page headers. Also make job dashboard link more prominent.

> On Sept. 11, 2015, 9:55 a.m., Stephan Erb wrote: > > src/main/resources/scheduler/assets/breadcrumb.html, line 35 > > <https://reviews.apache.org/r/38270/diff/2/?file=1067960#file1067960line35> > > > > The title is rather specific and might not match what p

Re: Review Request 38270: Remove redundancy between breadcrumbs and page headers. Also make job dashboard link more prominent.

> On Sept. 11, 2015, 9:55 a.m., Stephan Erb wrote: > > src/main/resources/scheduler/assets/breadcrumb.html, line 35 > > <https://reviews.apache.org/r/38270/diff/2/?file=1067960#file1067960line35> > > > > The title is rather specific and might not match what p

Re: Review Request 30695: Implements log rotation in the Thermos runner.

terall, it will be his disk quota / resources that will be affected by disabling the rotation. - Stephan Erb On Nov. 25, 2015, 6 p.m., George Sirois wrote: > > --- > This is an automatically generated e-mail. To reply,

Re: Review Request 41315: Upgrade mesos dependency to 0.24.1.

just recently urged users not to skip updates (http://www.mail-archive.com/user@mesos.apache.org/msg05379.html). - Stephan Erb On Dec. 12, 2015, 5:49 p.m., Bill Farner wrote: > > --- > This is an automatically generated e-mail.

Re: Review Request 41331: Upgrade to pants 0.0.64 and pex 1.1.1.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/41331/#review110213 --- Ship it! Ship It! - Stephan Erb On Dec. 14, 2015, 12:32 a.m

Re: Review Request 41368: Remove the client-side updater.

indicates that the old updater supported non-service updates, whereas the new one doesn't. We should at least communicate that more clearly. - Stephan Erb On Dec. 14, 2015, 11:36 p.m., Bill Farner wrote: > > --- > This is an auto

Re: Review Request 41528: Fixup `getJobSummary` for cron jobs with invalid next run dates.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/41528/#review57 --- Ship it! Thanks for looking into this! - Stephan Erb On Dec

Re: Review Request 41201: Preserve env variables for tasks in docker

pache.org/r/41201/#comment169373> This will also override variables like HOME specified above. Is this intended? - Stephan Erb On Dec. 10, 2015, 1:24 p.m., Kasisnu Singh wrote: > > --- > This is an automatically gener

Re: Review Request 41809: Allow custom announce path

) <https://reviews.apache.org/r/41809/#comment173180> This comment does somewhat imply that I can use a path containing variables. Also the rest of the documentation leaves this somewhat unclear. - Stephan Erb On Jan. 4, 2016, 10:28 p.m., Kunal Thakar

Re: Review Request 41897: Upgrade to the latest zk point release.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/41897/#review112679 --- News file needs updating too :-) - Stephan Erb On Jan. 5, 2016

Review Request 41846: Add a list of Aurora design documents

/diff/ Testing --- Rendered version is available at https://github.com/StephanErb/aurora/blob/design-docs/docs/design-documents.md Thanks, Stephan Erb

Re: Review Request 41846: Add a list of Aurora design documents

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/41846/#review112431 --- @ReviewBot retry - Stephan Erb On Jan. 2, 2016, 1:43 p.m

Re: Review Request 41717: Very Very WIP: Add jittering as an option to BackoffStrategy.

> On Dec. 27, 2015, 11:11 p.m., Stephan Erb wrote: > > Thanks for your effort. > > > > I believe `ExpBackoffEqualJitter.java` is the most usefull strategy. It is > > the closest to the current design while still resolving the problem > > depi

Review Request 41844: Update and slightly extend the beginner tutorial

4628241d7baa507fc17414d7cdb40e76e6d609a5 Diff: https://reviews.apache.org/r/41844/diff/ Testing --- Rendered version is available at https://github.com/StephanErb/aurora/blob/docu-update/docs/tutorial.md Thanks, Stephan Erb

Review Request 41853: Add recent Aurora-related talks

/presentations.md. Thanks, Stephan Erb

Re: Review Request 41762: Upgrade ZooKeeper dep to 3.4.2

> On Dec. 29, 2015, 1:37 p.m., Stephan Erb wrote: > > Looks like the 3.4 client also requires a 3.4 ZK server [1]. So, would it > > be possible to get a deprecation cycle for that change (i.e., announce in > > 0.12 and bump in 0.13)? > > > > Unfortunately,

Re: Review Request 41786: Remove several scheduler command line arguments.

pache.org/r/41786/#comment172887> I've always thought those are somewhat meant to be used for backup recovery as an alternative to blocking the HTTP port? https://github.com/apache/aurora/blob/master/docs/storage-config.md#preparation. Maybe someone can ask @jsmith or another SRE

Re: Review Request 41778: Simplify test BUILD files to one target per directory.

> On Dec. 29, 2015, 11:44 p.m., Stephan Erb wrote: > > Would be awesome if you could document the druid knowledge of how to > > properly execute tests (single test, all tests, ...). > > > > There is already this > > http

Re: Review Request 41786: Remove several scheduler command line arguments.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/41786/#review112401 --- Ship it! Ship It! - Stephan Erb On Dez. 30, 2015, 2:52 vorm

Re: Review Request 41786: Remove several scheduler command line arguments.

> On Dez. 31, 2015, 3:41 nachm., Stephan Erb wrote: > > src/main/java/org/apache/aurora/scheduler/storage/log/LogStorageModule.java, > > line 67 > > <https://reviews.apache.org/r/41786/diff/1/?file=1178110#file1178110line67> > > > > These docs are now

Re: Review Request 41778: Simplify test BUILD files to one target per directory.

contributors up to speed. - Stephan Erb On Dec. 29, 2015, 11:32 p.m., Bill Farner wrote: > > --- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache

Re: Review Request 42077: Introduces -default_docker_parameters scheduler flag.

there: docs/deploying-aurora-scheduler.md (line 187) <https://reviews.apache.org/r/42077/#comment174250> Make it more explicit that those will only be used if a job has no custom paramters. - Stephan Erb On Jan. 8, 2016, 7:28 p.m., George Sirois

Re: Review Request 40922: Thermos: Add ability to specify process outputs destination

> On Jan. 6, 2016, 10:02 nachm., Stephan Erb wrote: > > src/main/python/apache/thermos/core/process.py, line 561 > > <https://reviews.apache.org/r/40922/diff/8/?file=1180836#file1180836line561> > > > > Is this timeout change intentional? > > Mar

Re: Review Request 40922: Thermos: Add ability to specify process outputs destination

g/r/40922/#comment173630> Is this path actually needed? Doesn't the command line argument default to FILE anyway? - Stephan Erb On Jan. 4, 2016, 1:25 p.m., Martin Hrabovcin wrote: > > --- > This is an automatically gener

Re: Review Request 41725: Add component deployment diagram

-a5a5-184c7aeacdcf__components.png Thanks, Stephan Erb

Re: Review Request 41725: Add component deployment diagram

/uploaded/files/2015/12/26/d1f51a1e-9125-40cf-a5a5-184c7aeacdcf__components.png Thanks, Stephan Erb

Review Request 41725: Add component deployment diagram

/ Testing --- Rendered page is available at https://github.com/StephanErb/aurora/blob/fmc/docs/installing.md Thanks, Stephan Erb

Re: Review Request 41725: Add component deployment diagram

/uploaded/files/2015/12/26/cca230da-a6c8-4957-890d-1c7b24e0604d__components.png Thanks, Stephan Erb

Re: Review Request 41725: Add component deployment diagram

ly generated e-mail. To reply, visit: https://reviews.apache.org/r/41725/#review111905 ------- On Dec. 26, 2015, 5:23 p.m., Stephan Erb wrote: > > --- > This is an automatically generated e-m

Re: Review Request 41537: Upgrade mesos dependency to 0.25.0

for mesos.native==0.25.0: mesos.native==0.25.0 The end-to-end tests are failing to me. Is that local issue or do we need to upload that egg somewhere? - Stephan Erb On Dec. 23, 2015, 9:10 p.m., Bill Farner wrote

Review Request 41732: Slight documentation overhaul

: https://reviews.apache.org/r/41732/diff/ Testing --- Rendered version is available at https://github.com/StephanErb/aurora/tree/docu-update Thanks, Stephan Erb

Re: Review Request 41732: Slight documentation overhaul

-CREATION docs/tools.md PRE-CREATION Diff: https://reviews.apache.org/r/41732/diff/ Testing --- Rendered version is available at https://github.com/StephanErb/aurora/tree/docu-update Thanks, Stephan Erb

Re: Review Request 41564: Add a guide for installing Aurora.

> On Dec. 19, 2015, 8:26 p.m., Bill Farner wrote: > > Question for the reviewers - what should we target as the base for these > > install instructions? For example, the vanilla Ubuntu Trusty server image > > from ubuntu.com needs fewer base deps than the `ubuntu:trusty` Docker image. We

Re: Review Request 41564: Add a guide for installing Aurora.

g/r/41564/#comment171555> You seemed to have stopped mid-sentance. - Stephan Erb On Dec. 19, 2015, 4:03 a.m., Bill Farner wrote: > > --- > This is an automatically generated e-mail. To reply, visit: > https://re

Re: Review Request 41564: Add a guide for installing Aurora.

> On Dez. 18, 2015, 9:12 nachm., John Sirois wrote: > > > > John Sirois wrote: > I won't be able to get to trials until later this evening. Ideally > though this doc could be tied to the test/ provision.sh's somehow for more > assured working-ness. I have no useful suggestion though atm

Re: Review Request 41725: Add component deployment diagram

isit: https://reviews.apache.org/r/41725/#review111932 ----------- On Dec. 26, 2015, 8:53 p.m., Stephan Erb wrote: > > --- > This is an automatically generate

Re: Review Request 41201: Preserve env variables for tasks in docker

strip 'docker' from the description and and review request title, as it is no longer docker specific * You should add an entry to the NEWS file in order to announce the new feature. * If you create an issue in jira and link it here, your change will automatically show up in the changelog - Stephan

Re: Review Request 41732: Slight documentation overhaul

f52c76e2c2fe9ebca75c5f5db4ac2de842879f01 docs/images/aurora_logo.png PRE-CREATION docs/tools.md PRE-CREATION Diff: https://reviews.apache.org/r/41732/diff/ Testing --- Rendered version is available at https://github.com/StephanErb/aurora/tree/docu-update Thanks, Stephan Erb

Re: Review Request 41717: Very Very WIP: Add jittering as an option to BackoffStrategy.

to all the other code. Is it really necessary? - Stephan Erb On Dec. 25, 2015, 11:18 a.m., Tony Dong wrote: > > --- > This is an automatically generated e-mail. To reply, visit: > https://reviews.

Re: Review Request 41732: Slight documentation overhaul

changes. - Stephan Erb On Dec. 27, 2015, 10:46 p.m., Stephan Erb wrote: > > --- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache

Re: Review Request 41732: Slight documentation overhaul

> On Dec. 27, 2015, 11:20 p.m., Stephan Erb wrote: > > Looks like you have pushed the previous version without the review changes. > > Bill Farner wrote: > Oy, sorry about that. I yanked it from your branch (since i can't pull > the patch with binaries) a

Re: Review Request 40877: Update rpm startup scripts to match deb patterns in use

://issues.apache.org/jira/browse/AURORA-1394 - Stephan Erb On Dec. 2, 2015, 8:15 p.m., Jake Farrell wrote: > > --- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache

Re: Review Request 41564: Add a guide for installing Aurora.

://issues.apache.org/jira/browse/AURORA-1483 - Stephan Erb On Dec. 20, 2015, 3:24 a.m., Bill Farner wrote: > > --- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache

Re: Review Request 41762: Upgrade ZooKeeper dep to 3.4.2

> On Dec. 29, 2015, 1:37 p.m., Stephan Erb wrote: > > Looks like the 3.4 client also requires a 3.4 ZK server [1]. So, would it > > be possible to get a deprecation cycle for that change (i.e., announce in > > 0.12 and bump in 0.13)? > > > > Unfortunately,

Re: Review Request 41762: Upgrade ZooKeeper dep to 3.4.2

-compatibility-td7578798.html [2] https://packages.debian.org/search?keywords=zookeeper - Stephan Erb On Dec. 29, 2015, 4:07 a.m., Bill Farner wrote: > > --- > This is an automatically generated e-mail. To reply, visit

Re: Review Request 30695: Implements log rotation in the Thermos runner.

> On Feb. 6, 2015, 7:52 p.m., Brian Wickman wrote: > > This is super rad. Thanks for taking this on. > > > > Before I do a deeper dive, what do you think about making the logrotate > > policy be specified by the user instead of the framework owner, with a > > sensible default? For example,

Re: Review Request 42077: Introduces -default_docker_parameters scheduler flag.

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/42077/#review114058 --- Ship it! Ship It! - Stephan Erb On Jan. 12, 2016, 8:58 p.m

Re: Review Request 48211: fix containers doc example

) <https://reviews.apache.org/r/48211/#comment201046> Shouldn't that be `hello_world_proc` instead? - Stephan Erb On June 3, 2016, 5:43 p.m., Raymond Khalife wrote: > > --- > This is an automatically generated e-mail.

Re: Review Request 48211: fix containers doc example

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48211/#review136072 --- Ship it! Ship It! - Stephan Erb On June 3, 2016, 6:07 p.m

Review Request 48218: Document the logfile locations used in the vagrant box

--- Document the logfile locations used in the vagrant box Diffs - docs/getting-started/vagrant.md 44606009759a68cb142be4f2c7061ad8d0a39066 Diff: https://reviews.apache.org/r/48218/diff/ Testing --- Thanks, Stephan Erb

Review Request 48216: Change the UI to refer to Mesos Agents instead of Mesos Slaves.

/resources/scheduler/assets/js/filters.js ec35d81537b44b7bf76cf4cf134b96d44904fee1 Diff: https://reviews.apache.org/r/48216/diff/ Testing --- * ran ./build-support/jenkins/build.sh * checked in the Vagrant box that /slaves and /agents are both working. Thanks, Stephan Erb

Re: Review Request 48359: Adding gpg key for serb

--- Switch to longer keys. Repository: aurora Description --- Adding gpg key for serb Diffs (updated) - KEYS e6dfcb78b1d16a7a159c49439821ebb6049c7024 Diff: https://reviews.apache.org/r/48359/diff/ Testing --- Thanks, Stephan Erb

Review Request 48359: Adding gpg key for serb

gpg key for serb Diffs - KEYS e6dfcb78b1d16a7a159c49439821ebb6049c7024 Diff: https://reviews.apache.org/r/48359/diff/ Testing --- Thanks, Stephan Erb

Re: Review Request 48591: Ensure we can verify release candidates in the Vagrant box

--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/48591/#review137190 --- @ReviewBot retry - Stephan Erb On June 12, 2016, 12:21 a.m

Re: Review Request 47809: Generalizing resource management on the client

(line 28) <https://reviews.apache.org/r/47809/#comment199592> Now that you print `core(s)` as a display unit, you might as well change the CPU display name to singular and just use `CPU` similar to how it is done for the other resources. - Stephan Erb On May 25, 2016, 5:48 a.m., M

  1   2   3   4   5   6   7   8   9   >