Re: Steps to follow after becoming a Geode committer

2018-09-19 Thread Juan José Ramos
Thanks Kirk!. Cheers. On Tue, Sep 18, 2018 at 5:51 PM Kirk Lund wrote: > I reviewed your changes and added approval. Thanks Juan! > > On Tue, Sep 18, 2018 at 12:46 AM, Juan José Ramos > wrote: > > > Hello Jinmei and Dan, > > > > Thanks both for the reply!!. > > Regarding pull 2250, I think the

Re: [VOTE] Apache Geode 1.7.0 RC1

2018-09-19 Thread Anthony Baker
I think we can get rid of the **/out/** exclusion in the build script, let’s do that for the next release. Anthony > On Sep 18, 2018, at 3:52 PM, Dan Smith wrote: > > +0 > > I noticed that someone added some files called "out" to the repo that > aren't included in the source distribution: >

Re: [VOTE] Apache Geode 1.7.0 RC1

2018-09-19 Thread Nabarun Nag
REMINDER: VOTE deadline is today (EOD 19th September 2018 PDT) Regards Nabarun Nag On Wed, Sep 19, 2018 at 7:13 AM Anthony Baker wrote: > I think we can get rid of the **/out/** exclusion in the build script, > let’s do that for the next release. > > Anthony > > > > On Sep 18, 2018, at 3:52 PM,

Concourse - max containers reached

2018-09-19 Thread Sai Boorlagadda
All resources are erroring out with "max containers reached"? Anyone looking into it? Sai

Re: [VOTE] Apache Geode 1.7.0 RC1

2018-09-19 Thread Sai Boorlagadda
+1 * verified commit hash * checked concourse pipeline is green * ran examples with RC1 * built from source Sai On Wed, Sep 19, 2018 at 8:46 AM Nabarun Nag wrote: > REMINDER: VOTE deadline is today (EOD 19th September 2018 PDT) > > Regards > Nabarun Nag > > On Wed, Sep 19, 2018 at 7:13 AM Anth

Re: [VOTE] Apache Geode 1.7.0 RC1

2018-09-19 Thread Dave Barnes
+0 Recasting my vote, just for full disclosure. I found a (very minor) problem in the source release doc files: If a user builds their own docs from sources, they'll say version 1.8. - NOT (even close to) a showstopper - This will NOT affect the User Guide we'll push to the website with the

Re: Concourse - max containers reached

2018-09-19 Thread Patrick Rhomberg
Something appears to be up with the container garbage collector. Just to get things moving in the interim, we've restarted the workers to unceremoniously whack the containers that should have been pruned. Anyone with tests running may have had their tests workers die as a result. Sorry for the i

Adding new dependencies

2018-09-19 Thread Kirk Lund
Please be extra careful when adding new dependencies. Our build is setup to allow transitive dependencies, so reviews should include looking at the tree of transitive dependencies that a new dependency pulls into Geode. I think it would also be a good idea to create a little extra noise on the dev

Re: Adding new dependencies

2018-09-19 Thread Anthony Baker
+1 Adding dependencies that are bundled in a release (source or binary) need to be reviewed for LICENSE compatibility as well. Anthony > On Sep 19, 2018, at 2:29 PM, Kirk Lund wrote: > > Please be extra careful when adding new dependencies. Our build is setup to > allow transitive dependencie

[Spring CI] Spring Data GemFire > Nightly-ApacheGeode > #1045 was SUCCESSFUL (with 2456 tests)

2018-09-19 Thread Spring CI
--- Spring Data GemFire > Nightly-ApacheGeode > #1045 was successful. --- Scheduled 2458 tests in total. https://build.spring.io/browse/SGF-NAG-1045/ -- Thi