ws.apache.org/r/42445/
> ---
>
> (Updated Jan. 17, 2016, 3:42 p.m.)
>
>
> Review request for Aurora, Joe Smith and Stephan Erb.
>
>
> Repository: aurora
>
>
> Description
> ---
>
> This is the regular
pants bootstrap script to be agnostic to sed version
Diffs
-
pants 47097994b1044202aa0d8ce6afb8c2dee2a4c27c
Diff: https://reviews.apache.org/r/40204/diff/
Testing
---
Thanks,
Joe Smith
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/39784/#review105973
---
Ship it!
Thanks again John!
- Joe Smith
On Nov. 10, 2015, 1:50
This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/39784/
> ---
>
> (Updated Oct. 29, 2015, 3:05 p.m.)
>
>
> Review request for Aurora, Joe Smith, Bill Farner, and Zameer
/test_end_to_end.sh
1eec20cca5340f01ec2b45ef22d7e1c33e74ca3f
Diff: https://reviews.apache.org/r/38504/diff/
Testing
---
./src/test/sh/org/apache/aurora/e2e/test_end_to_end.sh
and
```
vagrant@aurora:~$ curl -sL -w '%{http_code}' localhost:8081/health -o
/dev/null; echo
200
```
Thanks,
Joe
localhost:8081/health
OK
vagrant@aurora:~$
```
Thanks,
Joe Smith
automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/38280/#review99312
-------
On Sept. 16, 2015, 2:53 p.m., Joe Smith wrote:
>
> ---
> This is an automatically ge
---
`./gradlew build -Pq`
Thanks,
Joe Smith
g numeric properties into Longs?
> > Shouldn't we just export all the keys as Strings into the stat system?
Cool, done. It's also now `<String, String>`
- Joe
---
This is an automatically generated e-mail
build -Pq`
Thanks,
Joe Smith
>
> > Revert
Done
- Joe
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/38280/#review99123
-------
On Sep
oger, thanks!
- Joe
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/38280/#review99132
---
On Sept. 14, 2015, 11:46
build -Pq`
Thanks,
Joe Smith
> <https://reviews.apache.org/r/38280/diff/2/?file=1068931#file1068931line95>
> >
> > If you like the previous comment, i believe this can disappear.
Done.
- Joe
---
This is an automatically generated e-mail
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/38277/#review98516
---
Ship it!
Ship It!
- Joe Smith
On Sept. 10, 2015, 4:19 p.m
/SnapshotStoreImplTest.java
6e032a6cef86e5f408bfc0d853a52c7f898d0db8
Diff: https://reviews.apache.org/r/38280/diff/
Testing
---
`./gradlew build -Pq`
Thanks,
Joe Smith
ep 10 &&
/etc/init.d/thermos-observer start'"
specs/rpm/aurora.spec (line 349)
<https://reviews.apache.org/r/38210/#comment154959>
This should constitute a -2, right?
- Joe Smith
On Sept. 8, 2015, 11:56 p.m., Steve Salevan wrote:
>
>
/StatsTest.java
91bbc096748952cc7dbf0cbd83a2d95456e0ce0a
Diff: https://reviews.apache.org/r/37964/diff/
Testing
---
```
[tw-mbp-jsmith aurora (stats-slashes)]$ ./gradlew build -Pq
# snip
BUILD SUCCESSFUL
```
Thanks,
Joe Smith
and discard this.
- Joe
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/34020/#review93682
---
On July 30, 2015, 5:14 p.m., Joe Smith wrote
a47da095de7c602fca8841e7f17a9dc4f78d0478
Diff: https://reviews.apache.org/r/34020/diff/
Testing
---
$ ./gradlew build -Pq
Thanks,
Joe Smith
-Pq
Thanks,
Joe Smith
a47da095de7c602fca8841e7f17a9dc4f78d0478
Diff: https://reviews.apache.org/r/34020/diff/
Testing
---
[tw-mbp-jsmith aurora (stats_normalize)]$ ./gradlew test
BUILD SUCCESSFUL
Thanks,
Joe Smith
. In the last pass i convinced myself that this was using the
local logger, but reaching out to Stats' logger and changing it is not good
behavior.
I'd happily wait for the fork and live with the log noise rather than
go this direction.
Joe Smith wrote:
Aha, gotcha- no worries
/apache/aurora/scheduler/sla/MetricCalculator.java
a47da095de7c602fca8841e7f17a9dc4f78d0478
Diff: https://reviews.apache.org/r/34020/diff/
Testing
---
[tw-mbp-jsmith aurora (stats_normalize)]$ ./gradlew test
BUILD SUCCESSFUL
Thanks,
Joe Smith
/apache/aurora/scheduler/sla/MetricCalculator.java
a47da095de7c602fca8841e7f17a9dc4f78d0478
Diff: https://reviews.apache.org/r/34020/diff/
Testing
---
[tw-mbp-jsmith aurora (stats_normalize)]$ ./gradlew test
BUILD SUCCESSFUL
Thanks,
Joe Smith
---
On July 30, 2015, 10:32 a.m., Joe Smith wrote:
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/34020/
---
(Updated July 30, 2015
, visit:
https://reviews.apache.org/r/34020/#review90933
---
On July 30, 2015, 10:32 a.m., Joe Smith wrote:
---
This is an automatically generated e-mail. To reply, visit:
https
. In the last pass i convinced myself that this was using the
local logger, but reaching out to Stats' logger and changing it is not good
behavior.
I'd happily wait for the fork and live with the log noise rather than
go this direction.
Joe Smith wrote:
Aha, gotcha- no worries
/SchedulerThriftInterface.java
dc0cd2d71d11b8157154f7b63a14f0282dee09f1
src/test/java/org/apache/aurora/scheduler/thrift/SchedulerThriftInterfaceTest.java
d28baba7618ebb194a61455971786aef46abd8eb
Diff: https://reviews.apache.org/r/36436/diff/
Testing
---
`./gradlew build -Pq`
Thanks,
Joe Smith
---
On July 14, 2015, 5:30 p.m., Joe Smith wrote:
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/36436/
---
(Updated July 14, 2015, 5:30
dc0cd2d71d11b8157154f7b63a14f0282dee09f1
src/test/java/org/apache/aurora/scheduler/thrift/SchedulerThriftInterfaceTest.java
d28baba7618ebb194a61455971786aef46abd8eb
Diff: https://reviews.apache.org/r/36436/diff/
Testing
---
`./gradlew build -Pq`
Thanks,
Joe Smith
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/36436/#review91790
---
On July 15, 2015, 11:08 a.m., Joe Smith wrote:
---
This is an automatically
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/36436/#review91645
---
On July 13, 2015, 4:10 p.m., Joe Smith wrote
dc0cd2d71d11b8157154f7b63a14f0282dee09f1
src/test/java/org/apache/aurora/scheduler/thrift/SchedulerThriftInterfaceTest.java
d28baba7618ebb194a61455971786aef46abd8eb
Diff: https://reviews.apache.org/r/36436/diff/
Testing
---
`./gradlew build -Pq`
Thanks,
Joe Smith
you think!)
- Joe
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/36436/#review91416
---
On July 12, 2015, 11:49 a.m., Joe Smith wrote
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/35580/#review88278
---
Ship it!
+1 on test coverage too
- Joe Smith
On June 17, 2015
(stats_normalize)]$ ./gradlew test
BUILD SUCCESSFUL
Thanks,
Joe Smith
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/34570/#review86872
---
Ship it!
Also LGTM pending jcohen's comment
- Joe Smith
On May
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/34827/#review86121
---
Ship it!
Ship It!
- Joe Smith
On May 29, 2015, 12:54 p.m
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/34484/#review85482
---
Ship it!
Ship It!
- Joe Smith
On May 26, 2015, 1:43 p.m., Brian
src.test.python.apache.aurora.client.hooks.non_hooked_api
. SUCCESS
Thanks,
Joe Smith
. SUCCESS
src.test.python.apache.aurora.client.hooks.hooked_api
. SUCCESS
src.test.python.apache.aurora.client.hooks.non_hooked_api
. SUCCESS
Thanks,
Joe Smith
. SUCCESS
Thanks,
Joe Smith
: https://reviews.apache.org/r/33997/diff/
Testing
---
Thanks,
Joe Smith
On March 20, 2015, 3:52 p.m., Joe Smith wrote:
Joe Smith wrote:
It seems like the `self.quitquitquit` is the important part (on line 340
of the runner)- doesn't decreasing the timeout not give `quitquitquit` the
time it needs?
Joe Smith wrote:
In `src/main/python/apache
45 matches
Mail list logo
