--------
On Dec. 9, 2017, 9:32 a.m., John Sirois wrote:
>
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/64290/
> --
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/64290/#review193499
---
@ReviewBot retry
- John Sirois
On Dec. 9, 2017, 9:32 a.m
-mail. To reply, visit:
https://reviews.apache.org/r/64290/#review193430
---
On Dec. 9, 2017, 9:32 a.m., John Sirois wrote:
>
> ---
> This is an automatically generated e-mail.
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/64290/#review193420
---
@ReviewBot retry
- John Sirois
On Dec. 9, 2017, 9:32 a.m
le:///home/jenkins/jenkins-slave/workspace/AuroraBot/.pants.d/python-setup/resolved_requirements/CPython-2.7.12/pykerberos-1.1.14.tar.gz')
> > is not translateable by ChainedTranslator(WheelTranslator, EggTranslator,
> > SourceTranslator)
> >
> >
> >
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/64290/#review193374
---
@ReviewBot retry
- John Sirois
On Dec. 9, 2017, 9:32 a.m
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/64290/#review193356
---
@ReviewBot retry
- John Sirois
On Dec. 9, 2017, 9:32 a.m
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/64290/#review193354
---
@ReviewBot retry
- John Sirois
On Dec. 9, 2017, 9:32 a.m
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/64290/#review193351
---
@ReviewBot retry
- John Sirois
On Dec. 9, 2017, 9:32 a.m
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/64290/#review193344
---
- John Sirois
On Dec. 9, 2017, 9:32 a.m., John Sirois wrote
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/64290/#review193343
---
- John Sirois
On Dec. 9, 2017, 9:32 a.m., John Sirois wrote
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/64290/#review193341
---
@ReviewBot retry
- John Sirois
On Dec. 9, 2017, 9:32 a.m
atically generated e-mail. To reply, visit:
https://reviews.apache.org/r/64290/#review193328
-------
On Dec. 9, 2017, 9:32 a.m., John Sirois wrote:
>
> ---
> This is an automatically generat
$ ./src/test/sh/org/apache/aurora/e2e/test_end_to_end.sh
```
Thanks,
John Sirois
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/64290/#review193330
---
- John Sirois
On Dec. 8, 2017, 10:16 p.m., John Sirois wrote
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/64290/#review193326
---
- John Sirois
On Dec. 8, 2017, 10:16 p.m., John Sirois wrote
/thermos/core/test_process.py
6cb9176e14eccbe7ed10501199a34e5e67d6fe44
Diff: https://reviews.apache.org/r/64290/diff/3/
Testing (updated)
---
```
$ ./build-support/jenkins/build.sh
$ ./src/test/sh/org/apache/aurora/e2e/test_end_to_end.sh
```
Thanks,
John Sirois
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/64290/#review193323
---
@ReviewBot retry
- John Sirois
On Dec. 8, 2017, 8:47 p.m
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/64290/#review193321
---
@ReviewBot retry
- John Sirois
On Dec. 8, 2017, 8:47 p.m
le:///home/jenkins/jenkins-slave/workspace/AuroraBot/.pants.d/python-setup/resolved_requirements/CPython-2.7.12/pykerberos-1.1.14.tar.gz')
> > is not translateable by ChainedTranslator(WheelTranslator, EggTranslator,
> > SourceTranslator)
> >
> >
> >
:17:40 00:00 [complete]
FAILURE
```
Filed this issue against pants:
https://github.com/pantsbuild/pants/issues/5153
Thanks,
John Sirois
build/pants/pull/5168
This will be available with the Pants 1.4.0.dev23 release this Friday.
- John Sirois
On Dec. 4, 2017, 8:23 a.m., John Sirois wrote:
>
> ---
> This is an automatically generated e-mail. To reply,
2d64521cf9/api.src.main.thrift.org.apache.aurora.gen._api/current:api.src.main.thrift.org.apache.aurora.gen._api)
00:17:40 00:00 [complete]
FAILURE
```
Filed this issue against pants:
https://github.com/pantsbuild/pants/issues/5153
Thanks,
John Sirois
]
> >FAILURE
> >
> >
> > I will refresh this build result if you post a review containing
> > "@ReviewBot retry"
@ReviewBot retry
- John
---
This is an automatically ge
> On Sept. 26, 2017, 5:53 p.m., Bill Farner wrote:
> > ```
> > /bin/sh: cmake: command not found
> > ```
> >
> > But now i need to install cmake, so i'm not sure this pays off.
>
> Bill Farner wrote:
> (this = the switch to cmake)
>
> Stephan Erb wrote:
> Bison on MacOs is 10 years old
> I have been looking into this a bit, but still haven't figured it out.
>
> It seems like pants changed when it tries to resolve dependencies. The
> `mesos.executor` only used to be pulled in when building the executor binary.
> In newer version, pants will chec
n._api/current:api.src.main.thrift.org.apache.aurora.gen._api)
00:17:40 00:00 [complete]
FAILURE
```
Filed this issue against pants:
https://github.com/pantsbuild/pants/issues/5153
Thanks,
John Sirois
`src/test/python::`.
- John
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/63750/#review191018
---
On Nov. 14, 2017, 4:51 p.m., se choi wrote:
>
>
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/62835/#review187965
---
Ship it!
Ship It!
- John Sirois
On Oct. 13, 2017, 8:36 a.m
eal it in by using a flag. Not to mention the new
default is probably what they really want.
- John Sirois
On Oct. 8, 2017, 12:20 p.m., Stephan Erb wrote:
>
> ---
> This is an automatically generated e-mail. To reply,
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/62652/#review187381
---
Ship it!
Ship It!
- John Sirois
On Oct. 8, 2017, 11:29 a.m
until Monday 10/2, but I'm happy
with a quick skim!
- John Sirois
On Sept. 29, 2017, 8:07 p.m., Bill Farner wrote:
>
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://revi
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/62716/#review186789
---
Ship it!
Ship It!
- John Sirois
On Sept. 30, 2017, 11:01
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/62608/#review186371
---
Ship it!
Ship It!
- John Sirois
On Sept. 26, 2017, 7:51 p.m
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/58612/#review172665
---
Ship it!
Ship It!
- John Sirois
On April 21, 2017, 5:24 a.m
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/56577/#review166045
---
Ship it!
A tardy LGTM and thanks.
- John Sirois
On Feb. 14
ce he reviews the last
change.
- John Sirois
On Jan. 23, 2017, 1:21 p.m., Zameer Manji wrote:
>
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.a
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/54288/#review162703
---
@ReviewBot retry
- John Sirois
On Jan. 23, 2017, 1:21 p.m
it_memory(0x0007a8e8, 298319872, 0) failed; error='Cannot
allocate memory' (errno=12)
```
I'll spin one more retry and then submit this if green.
- John
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/54288/#review162700
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/54288/#review162697
---
@ReviewBot retry
- John Sirois
On Jan. 23, 2017, 1:21 p.m
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/54288/#review162691
---
@ReviewBot retry
- John Sirois
On Jan. 23, 2017, 1:21 p.m
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/54288/#review162688
---
Ship it!
Ship It!
- John Sirois
On Jan. 23, 2017, 1:21 p.m
tor - which looked scary in isolation - and this all
looks good to me.
- John Sirois
On Jan. 23, 2017, 1:21 p.m., Zameer Manji wrote:
>
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://rev
at
net.sourceforge.pmd.lang.java.rule.logging.InvalidSlf4jMessageFormatRule.expectedArguments(InvalidSlf4jMessageFormatRule.java:171)
at
net.sourceforge.pmd.lang.java.rule.logging.InvalidSlf4jMessageFormatRule.visit(InvalidSlf4jMessageFormatRule.java:88)
...
``
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/55646/#review161975
---
@ReviewBot retry
- John Sirois
On Jan. 17, 2017, 2:30 p.m
js (Javascript):
```
Thanks,
John Sirois
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/55583/#review161955
---
Ship it!
Ship It!
- John Sirois
On Jan. 16, 2017, 3:27 p.m
find a compatible thrift.
Thanks,
John Sirois
ava (Java2)" and
observed the script forcing a clean recompile of 0.9.1 with a subsequent
failure to find a compatible thrift.
Thanks,
John Sirois
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/55510/#review161622
---
Ship it!
Ship It!
- John Sirois
On Jan. 13, 2017, 4:25 p.m
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/55507/#review161620
---
Ship it!
Ship It!
- John Sirois
On Jan. 13, 2017, 10:15 a.m
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/54669/#review158875
---
Ship it!
Ship It!
- John Sirois
On Dec. 12, 2016, 10:11 a.m
, 2016, 4:28 p.m.)
>
>
> Review request for Aurora, David McLaughlin, Joshua Cohen, John Sirois, and
> Stephan Erb.
>
>
> Bugs: AURORA-1669
> https://issues.apache.org/jira/browse/AURORA-1669
>
>
> Repository: aurora
>
>
> Description
> ---
&g
/prepare_binary.sh
e3b46aa1f36ed3eecc9bb7794073d18dd950eb1c
Diff: https://reviews.apache.org/r/54567/diff/
Testing
---
Reproduced the current master CI failure before this change on my linux machine.
Fixed with this change.
Thanks,
John Sirois
> On Dec. 8, 2016, 8:29 p.m., John Sirois wrote:
> > build-support/thrift/prepare_binary.sh, line 29
> > <https://reviews.apache.org/r/54550/diff/3/?file=1580209#file1580209line29>
> >
> > aha - popd on linux echos the dirstack - presumably it does not do
> On Dec. 8, 2016, 8:29 p.m., John Sirois wrote:
> > build-support/thrift/prepare_binary.sh, line 29
> > <https://reviews.apache.org/r/54550/diff/3/?file=1580209#file1580209line29>
> >
> > aha - popd on linux echos the dirstack - presumably it does not do
tps://reviews.apache.org/r/54550/#comment229419>
aha - popd on linux echos the dirstack - presumably it does not do this on
osx... so needs a re-direct. Ditto pushd. And AuroraBot really does need to go
green.
- John Sirois
On Dec. 8, 2016, 7:30 p.m., Joshua Cohen
> On Dec. 8, 2016, 7:37 p.m., Aurora ReviewBot wrote:
> > Master (f5749ce) is red with this patch.
> > ./build-support/jenkins/build.sh
> >
> > You are using pip version 8.1.2, however version 9.0.1 is available.
> > You should consider upgrading via the 'pip install --upgrade pip' command.
>
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/54550/#review158626
---
Ship it!
Ship It!
- John Sirois
On Dec. 8, 2016, 7:15 p.m
Diff: https://reviews.apache.org/r/54520/diff/
Testing
---
Now with files listed and python tests run:
```
./pants filemap src/test/python/::
./build-support/jenkins/build.sh
```
Thanks,
John Sirois
> On Dec. 7, 2016, 9:41 p.m., John Sirois wrote:
> > NB: When AuroraBot goes green I'll submit this TBR and file a follow-up
> > issue in Aurora and a corresponding one in pants if needed, to restore the
> > BUILD file changes for default targets and default gl
ile a follow-up issue
in Aurora and a corresponding one in pants if needed, to restore the BUILD file
changes for default targets and default globs.
- John Sirois
On Dec. 7, 2016, 9:39 p.m., John Sirois wrote:
>
> ---
> This is an
:
```
./pants filemap src/test/python/::
./build-support/jenkins/build.sh
```
Thanks,
John Sirois
d and there have
been a significant number of insignificant automated changes to BUILD targets
(pants no longer requires a `name` for the default target, nor
`globs('*.')` for targets that own all the standard sources in
a directory.
- John Sirois
On Dec. 6, 2016, 5:30 p.m., Jo
For globs:
```
for t in python_library python_tests
do
./pants filter --type=$t :: \
| cut -d: -f1 | sort -u | sed -e '/^$/d' -e 's|$|/BUILD|' \
| while read f; do ls $f* || echo $f >2; done \
| xargs sed -r -i "/\bsources\s*=\s*globs(['\"]*.[a-z]+['\&q
includes the thrift
> > lib and this prepare binary? Then we can put the dep in one place.
>
> John Sirois wrote:
> I can, but this requires a custom pants plugin (housed as a loose python
> sourcefile in the repo) to expose, say, an `aurora_py_thrift_lib` target.
> You
./build-support/python/make-pycharm-virtualenv
```
Thanks,
John Sirois
includes the thrift
> > lib and this prepare binary? Then we can put the dep in one place.
>
> John Sirois wrote:
> I can, but this requires a custom pants plugin (housed as a loose python
> sourcefile in the repo) to expose, say, an `aurora_py_thrift_lib` target.
> You
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/54250/#review157591
---
Ship it!
Ship It!
- John Sirois
On Dec. 1, 2016, 3:48 a.m
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/53519/#review156802
---
Ship it!
Ship It!
- John Sirois
On Nov. 23, 2016, 3:25 p.m
of these failures is
> > different with this patch applied. See this gist which contains the full
> > output: https://gist.github.com/jcohen/ee69208fbca714d3f3437e4039961edf.
> > Note the failure about 80% through, followed by some strange compile issues?
>
> John Siroi
of these failures is
> > different with this patch applied. See this gist which contains the full
> > output: https://gist.github.com/jcohen/ee69208fbca714d3f3437e4039961edf.
> > Note the failure about 80% through, followed by some strange compile issues?
>
> John Siroi
of these failures is
> > different with this patch applied. See this gist which contains the full
> > output: https://gist.github.com/jcohen/ee69208fbca714d3f3437e4039961edf.
> > Note the failure about 80% through, followed by some strange compile issues?
>
> John Siroi
> On Nov. 17, 2016, 2:48 p.m., Zameer Manji wrote:
> > Seems like a lot of patches to enable this behaviour. I'm not opposed but
> > it seems risky.
>
> John Sirois wrote:
> You find everything risky!
>
> Zameer Manji wrote:
> I'm just afrai
e number and paste that url? - not seeing it.
- John
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/53836/#review156262
------
fact OK with a custom plugin for this, I'll whip up
diff 2.
- John
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/53836/#review156248
------
e/ &&
./pants clean-all"
./src/test/sh/org/apache/aurora/e2e/test_end_to_end.sh
./build-support/python/make-pycharm-virtualenv
```
Thanks,
John Sirois
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/53794/#review155999
---
Ship it!
Ship It!
- John Sirois
On Nov. 15, 2016, 2:22 p.m
apache/thermos/common/exceptional.py (line 42)
<https://reviews.apache.org/r/53519/#comment224953>
Did you contemplate and reject setting `sys.excepthook` to a custom handler
early in main as an alternative?
- John Sirois
On
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/52665/#review154670
---
Ship it!
Ship It!
- John Sirois
On Oct. 29, 2016, 4:05 a.m
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/53102/#review153730
---
Ship it!
Ship It!
- John Sirois
On Oct. 22, 2016, 11:46 a.m
> On Oct. 23, 2016, 12:12 p.m., John Sirois wrote:
> > Sorry for the delay submitting this, e2e didn't actually pass and I futzed
> > with this for a while.
> > At the end of the day I needed to re-introduce
> > `examples/vagrant/mesos_config/etc_mesos-sla
-
examples/vagrant/mesos_config/etc_mesos-slave/executor_registration_timeout
PRE-CREATION
Diff: https://reviews.apache.org/r/53131/diff/
Testing
---
This is what allowed me to get a green e2e in
https://reviews.apache.org/r/52915/.
Thanks,
John Sirois
th a
change to add that.
- John Sirois
On Oct. 15, 2016, 4:42 p.m., John Sirois wrote:
>
> ---
> This is an automatically generated e-mail. To reply, visit:
> https:/
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/52921/#review152893
---
Ship it!
Ship It!
- John Sirois
On Oct. 16, 2016, 5:04 p.m
---
>
> (Updated Oct. 16, 2016, 5:04 p.m.)
>
>
> Review request for Aurora and John Sirois.
>
>
> Repository: aurora
>
>
> Description
> ---
>
> I saw THRIFT-3868 and thought we could apply the same micro-optimization
&
://github.com/pantsbuild/pants/issues/3956
- John Sirois
On Oct. 15, 2016, 4:42 p.m., John Sirois wrote:
>
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache
-support/jenkins/build.sh
./build-support/python/make-pycharm-virtualenv
./src/test/sh/org/apache/aurora/e2e/test_end_to_end.sh
```
Thanks,
John Sirois
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/52790/#review152349
---
Ship it!
Ship It!
- John Sirois
On Oct. 12, 2016, 11:28 a.m
> On Oct. 12, 2016, 4:41 a.m., John Sirois wrote:
> > Your testing done says this is enough on the aurora-packaging side, but on
> > the [aurora
> > side](https://github.com/apache/aurora/blob/master/build.gradle#L24) the
> > installed plugin probably also ne
https://github.com/apache/aurora/blob/master/build.gradle#L586) is
configured in the new regime (maybe the same).
- John Sirois
On Oct. 12, 2016, 4:13 a.m., Stephan Erb wrote:
>
> ---
> This is an automatically generate
---
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/52739/#review152141
---
Ship it!
Ship It!
- John Sirois
On Oct. 11, 2016, 9:11 a.m
> On Oct. 10, 2016, 5:31 p.m., John Sirois wrote:
> > I'll submit this tomorrow afternoon if I hear nothing from Zameer by then.
>
> Zameer Manji wrote:
> Sorry for the late shipit, I reviewed this previously and forgot to press
> the button.
No worries
ameer by then.
- John Sirois
On Oct. 7, 2016, 10:34 p.m., John Sirois wrote:
>
> ---
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.a
> On Oct. 9, 2016, 6:49 p.m., John Sirois wrote:
> > src/main/java/org/apache/aurora/scheduler/discovery/CuratorSingletonService.java,
> > line 142
> > <https://reviews.apache.org/r/52665/diff/1/?file=1528674#file1528674line142>
> >
> > This was in my
Joshua and Zameer likely have better
ideas.
- John Sirois
On Oct. 9, 2016, 2:43 a.m., Jing Chen wrote:
>
> ---
> This is an automatically generated e-mail. To reply
The error bars for
the off-heap are generally larger than the values themselves! I imagine jmh is
simply not the right tool for testing off-heap.
I too though am interested in real-world feedback.
- John Sirois
On Oct. 9, 2016, 4:35 p.m., Stephan Erb
test src/{main,test}/python:: -- -v
./build-support/release/make-python-sdists
vagrant ssh --command 'aurorabuild admin_client client executor observer'
```
Thanks,
John Sirois
/52640/diff/
Testing
---
Locally green:
```
./pants test src/{main,test}/python:: -- -v
./build-support/release/make-python-sdists
vagrant ssh --command 'aurorabuild admin_client client executor observer'
```
Thanks,
John Sirois
> On Oct. 6, 2016, 1:10 p.m., John Sirois wrote:
> > I added myself as a reviewer since I'm running Vagrant 1.8.6. I'll have
> > time to confirm your issue and fix later today and have some feedback for
> > you by tomorrow latest.
I can confirm both the failu
1 - 100 of 771 matches
Mail list logo