Re: Requirements for running distributed tests

2019-07-17 Thread Jens Deppe
Hi Alberto, The images used to run tests on in CI are built here: https://concourse.apachegeode-ci.info/teams/main/pipelines/apache-develop-images (you may see these referred to as 'heavy lifters'). The packer scripts for these can be found under here: https://github.com/apache/geode/tree/develop/

Re: PR review request

2019-06-12 Thread Jens Deppe
Done and merged. Thanks for the PR! Sorry it took so long to get to. --Jens On Wed, Jun 12, 2019 at 5:18 AM Alberto Bustamante Reyes wrote: > Hi, > > > Two weeks ago I sent a PR to the geode-examples repo, could someone take a > look? > > https://github.com/apache/geode-examples/pull/77 > > >

Re: "Output path is not specified for module" (IntelliJ)

2019-06-06 Thread Jens Deppe
I have only experienced this when I switch to building with IntelliJ (it is also dependent on what Intellij deems necessary to build). Building with gradle has never produced this. My setup is to build with gradle but run tests with IntelliJ. I've never had to do any kind of re-import for this. Re

Re: [DISCUSS] require reviews before merging a PR

2019-06-06 Thread Jens Deppe
As reviewers we should also feel empowered to request additional reviewers on a PR (perhaps beyond whomever the original submitter may already have requested). I think that, sometimes the complexity of a change prevents someone from commenting on just a portion of the change if they do not feel co

Re: [DISCUSS] is it time to make Windows tests gating?

2019-05-17 Thread Jens Deppe
I fixed the first two and started ignoring AvailablePortHelperIntegrationTest for Windows (Windows has a default active port range wildly different than Linux which makes this test very flaky, and since it's just testing test code it seemed reasonable to ignore it). LauncherMemberMXBeanIntegration

Re: Pulse - Non-Embedded Mode

2019-05-01 Thread Jens Deppe
mode. I > > don't think we've put a lot of energy into pulse recently. > > > > -Dan > > > > > > On Tue, Apr 30, 2019 at 2:12 PM Jens Deppe wrote: > > > > > More accurately, I think geode-core is only required when TLS is > enabled >

Re: Pulse - Non-Embedded Mode

2019-04-30 Thread Jens Deppe
More accurately, I think geode-core is only required when TLS is enabled on the locator and Pulse needs to make JMX/RMI calls over TLS. I would vote for option 2 in this scenario. --Jens On Tue, Apr 30, 2019 at 1:44 PM Jinmei Liao wrote: > I believe to run pulse in non-embedded mode, you just

Re: [DISCUSS] Move or remove org.apache.geode.admin

2019-04-04 Thread Jens Deppe
I'm not sure if I'm interpreting various parts of this conversation correctly, but it seems that one view being assumed is that the actual removal of deprecated APIs *must* happen in the first version of a major release bump. i.e. for this discussion that would be *2.0.0*. I don't believe this is a

Re: Dependency review for release 1.9.0

2019-03-29 Thread Jens Deppe
I'd like to request GEODE-6559 "PdxInstance.getObject() is using class from older jar in case of Reconnect" be included in 1.9 if possible. Thanks --Jens On Thu, Mar 28, 2019 at 6:50 AM Sai Boorlagadda wrote: > Changes to LICENSE and NOTICE fil

Re: Jetty Webapp marked as optional?

2019-03-05 Thread Jens Deppe
I like that approach. I'll raise a ticket to fail gracefully if the service cannot be started due to missing dependencies. --Jens On Tue, Mar 5, 2019 at 2:59 PM Jacob Barrett wrote: > > > > On Mar 5, 2019, at 1:52 PM, Jens Deppe wrote: > > > > How are building the

Re: Jetty Webapp marked as optional?

2019-03-05 Thread Jens Deppe
How are building the code that's running here? Is it a Maven or Gradle project and, if so, what dependencies are you declaring? --Jens On Tue, Mar 5, 2019 at 1:35 PM Jinmei Liao wrote: > we've made a change to create an HttpService object when a non-client cache > is created. This will allow us

Re: Very red CI -> Hold merges, please

2019-02-07 Thread Jens Deppe
No worries. I think I have a better fix now. At least the builds are moving again. On Thu, Feb 7, 2019 at 12:11 PM Kirk Lund wrote: > Sorry, go ahead and revert the commit and reopen the PR. > > On Thu, Feb 7, 2019 at 11:36 AM Jens Deppe wrote: > > > I was still working on a

Re: Very red CI -> Hold merges, please

2019-02-07 Thread Jens Deppe
I was still working on a fix... On Thu, Feb 7, 2019 at 11:31 AM Kirk Lund wrote: > I merged it in. > > On Thu, Feb 7, 2019 at 11:28 AM Kirk Lund wrote: > > > I think we should go ahead and merge in > > https://github.com/apache/geode/pull/3172 since it resolves the > > GfshConsoleModeUnitTest U

Re: GfshConsoleModeUnitTest is failing in UnitTest

2019-02-07 Thread Jens Deppe
Oh, feel free to review https://github.com/apache/geode/pull/3172. Better suggestions on how to 'fix' the problem are most welcome. On Thu, Feb 7, 2019 at 6:49 AM Jens Deppe wrote: > The problem is caused by some change in test ordering (unit tests are > running in parallel) wh

Re: GfshConsoleModeUnitTest is failing in UnitTest

2019-02-07 Thread Jens Deppe
The problem is caused by some change in test ordering (unit tests are running in parallel) which is resulting in environmental pollution. Probably Government shutdown related... :P On Wed, Feb 6, 2019 at 7:31 PM Jens Deppe wrote: > I didn't touch anything in this area, but nevert

Re: GfshConsoleModeUnitTest is failing in UnitTest

2019-02-06 Thread Jens Deppe
I didn't touch anything in this area, but nevertheless I reverted my commit which preceded this failure (referenced in Owen's message). However, the failure is still happening. --Jens On Wed, Feb 6, 2019 at 6:20 PM Owen Nichols wrote: > There is a ticket for this, > https://issues.apache.org/ji

Re: Extremely long IntelliJ Gradle refresh times

2019-01-09 Thread Jens Deppe
19 at 6:32 AM Jens Deppe wrote: > Maybe it's just my system, but this commit seems to have introduced the > problem for me: > > 411b22cdd1d87f1c55f81de948ff36f421a893af - GEODE-6198: Add > geode-all-bom for dependency versioning (#2995) > > As an aside (but possibly

Re: Extremely long IntelliJ Gradle refresh times

2019-01-09 Thread Jens Deppe
o increasing IntelliJ memory to 8g > > helps. > > > > Regards > > Naba > > > > On Wed, Jan 9, 2019 at 6:18 AM Jens Deppe wrote: > > > >> Is anyone else experiencing very long Gradle refresh times in IntelliJ? > >> Min

Extremely long IntelliJ Gradle refresh times

2019-01-09 Thread Jens Deppe
Is anyone else experiencing very long Gradle refresh times in IntelliJ? Mine currently takes almost 60 minutes! I'm on IJ 2018.3.2. (I'm pretty sure that I'm not hitting this problem https://stackoverflow.com/questions/38209851/intellij-idea-takes-a-long-time-when-refreshing-a-gradle-project ). Ac

Re: Spring detecting MockPluginCommand and logging WARN in integration tests

2018-11-16 Thread Jens Deppe
If this test is already applying a custom log configuration, could it not also turn down the logging for org.springframework.shell? --Jens On Fri, Nov 16, 2018 at 12:53 PM Kirk Lund wrote: > I checked the DUnit grep for suspect strings and it's ignore WARNING log > level. So we may DUnit tests

Re: Develop Compilation Failure

2018-11-13 Thread Jens Deppe
No worries, and thanks for the heads-up Juan. I'm really not sure anything can be done that wouldn't end up being heavy-handed and mostly just be an impedance. I should have sent out a notice describing the commons-lang change (given its scope) and that any in-flight PRs might be affected. --Jen

Re: Build Jobs not accessible?

2018-11-09 Thread Jens Deppe
Hi Juan. It could be a cookie problem - have you tried clearing them? Can you access the top-level https://concourse.apachegeode-ci.info? I can access your build just fine. --Jens On Fri, Nov 9, 2018 at 5:52 AM Ju@N wrote: > Hello team, > > I've submitted https://github.com/apache/geode/pull/

Re: [Discuss] Where should simple classes for tests belong?

2018-10-15 Thread Jens Deppe
For testing functionality like 'gfsh deploy' one has to be aware of classloading issues. Specifically, when writing dunit tests, classes are compiled from java resource files so that they are guaranteed not to be on the classpath during testing. This is less of an issue for gfsh acceptance tests wh

Re: [VOTE] Time-based release schedule for minor releases

2018-10-08 Thread Jens Deppe
+1 for time-based releases. I think we would do well to gain some discipline around releasing on a predictable cadence. We're probably never going to be short on changes that are 'worthy' of going out on a release which, too often, simply lead to the release being stretched out as we add in Just O

Re: [PROPOSAL] Change Geode Log4J2 appenders to avoid programmatic add/remove

2018-09-13 Thread Jens Deppe
+1 on this approach! I'd like to work with you on this to get more familiar with our logging setup. --Jens On Thu, Sep 13, 2018 at 10:01 AM Kirk Lund wrote: > Geode currently manipulates Log4J2 at runtime to add/remove Appenders based > on the Cache lifecycle. While working on GEODE-5637, Sai

Code Reviews

2018-09-07 Thread Jens Deppe
I came across this article a while back but just re-read it recently and thought it would be worth sharing. https://medium.freecodecamp.org/unlearning-toxic-behaviors-in-a-code-review-culture-b7c295452a3c I don't think we're doing much, if any, of the 'toxic' behavior, but it's still good to be r

Re: [DISCUSS] Apache Geode 1.7.0 release branch created

2018-08-29 Thread Jens Deppe
>>>>>>>>>>> Regards > > >>>>>>>>>>> Nabarun > > >>>>>>>>>>> On Mon, Aug 27, 2018 at 12:02 PM Bruce Schuchardt < > > >>>>>>>>>> bschucha...@pivota

Re: Windows builds failing

2018-08-24 Thread Jens Deppe
That went in a few days ago so this run is already using that. On Fri, Aug 24, 2018 at 2:49 PM Jacob Barrett wrote: > Dan just merged a change to gradle that drops the scanner plugin that > seems to suck up a ton of ram. > > > On Aug 24, 2018, at 2:38 PM, Jens Deppe wrote: >

Re: Windows builds failing

2018-08-24 Thread Jens Deppe
Not in the build process. Having them all fail simultaneously is very suspicious though. I restarted one of them and it ran fine. We're about to switch the build model for Windows to match the linux builds where a 'heavy lifter' is spawned for each job. During my testing of that I found the Gradl

Re: Admin API getMissingPersistentMembers is deprecated

2018-08-24 Thread Jens Deppe
If you're looking for a non-deprecated way but still (unfortunately) internal then 'ShowMissingDiskStoresFunction' will probably help. --Jens On Thu, Aug 23, 2018 at 2:27 PM Dan Smith wrote: > I think this was replaced with a gfsh command that does the same thing. > Unfortunately, we don't have

Re: [DISCUSS] Apache Geode 1.7.0 release branch created

2018-08-24 Thread Jens Deppe
>mean? > > Is there anything else that needs to go into 1.7? > > It seems like the best we all can do is to review Sai's PRs. Is that > correct? > > On Wed, Aug 22, 2018 at 10:59 AM, Jens Deppe wrote: > > > I'd also like to include GEODE-5619 > &g

Re: GitHub PR notifications

2018-08-24 Thread Jens Deppe
+1 for less Spam 🎡 spam 🎡 spam 🎡 spam Seriously - considering the comments started by Kirk's email, given the amount of spam, it's not surprising that folks would use a different means of getting a speedier response (Slack) rather than using email. Point being that the lower the noise/signal ratio

Re: [DISCUSS] Apache Geode 1.7.0 release branch created

2018-08-22 Thread Jens Deppe
I'd also like to include GEODE-5619 On Tue, Aug 21, 2018 at 3:59 PM Xiaojian Zhou wrote: > +1 > > The release will be a great one with so many historical bugs fixed. > > Today I tried to use IJ to build and run with latest build.gradle and > recent moved test packages, it worked. So this refacto

Re: PutCommandWithJsonTest and DeployWithLargeJarTest

2018-08-21 Thread Jens Deppe
Recently all of the AcceptanceTest failures I've looked at have been around bind exceptions on startup which implies test pollution from prior tests. --Jens On Tue, Aug 21, 2018 at 10:18 AM Sai Boorlagadda wrote: > The metrics job themselves will be green (as they complete to success) but > you

Re: Proposal to support custom java.security.Provider

2018-08-13 Thread Jens Deppe
How does this work if the user wants to use the Trust/KeyStoreManager from the default security provider but still override other SSL properties such as ciphers or protocols? --Jens On Mon, Aug 13, 2018 at 8:32 AM Sai Boorlagadda wrote: > To make it clear about the different options: > > 1) if

Re: Moving from gradle to maven

2018-07-18 Thread Jens Deppe
is reference on stack overflow ( > > https://stackoverflow.com/questions/36808351/running-junit-tests-in-parallel-with-docker > ) to a homebuilt solution, but I'm unsure how replicable it is. > > -Sean. > > On Wed, Jul 18, 2018 at 1:21 PM Jens Deppe wrote: > > > +1 For m

Re: Moving from gradle to maven

2018-07-18 Thread Jens Deppe
+1 For moving to maven. I think the blog Kirk linked hits all the relevant pain points. This is the third time we've done significant Gradle work and every time it is painful. It's also probably never going to get any better. For myself, Gradle certainly feels like a lot of magic happening under

Re: Revert recent Gradle upgrade

2018-06-21 Thread Jens Deppe
GEODE-4791: Enable gradle 4.8 for features and futureproofing (#2050) > > Enable: > * Java 10 (when ready) > * remove deprecated configuration settings > * build-cache > * incremental > * Update classpath/srcpath logic > > Disable: >

Re: Need information about vf.gf.locator.status.cmd & vf.gf.locator.status.cmd

2018-05-29 Thread Jens Deppe
Hi Dinesh vf.gf.locator.status.cmd and vf.gf.locator.stop.cmd are 'watchdog' files used to send commands to the locator. In this case, they are used to request status and to request the locator to stop. There are similar files, called vf.gf.server.* for the servers. These are just internal files a

Objections to removing 'upgrade offline-disk-store' command

2018-05-25 Thread Jens Deppe
The gfsh command 'upgrade offline-disk-store' exists in the code in order to upgrade diskstores from pre-Apache days (GemFire 6.x and 7.x). I don't believe the command is relevant/necessary anymore and, unless there are objections, I'd like to go ahead and remove it. --Jens

Better support for JSON gfsh results

2018-05-01 Thread Jens Deppe
Hi All, I'm working on removing our dependency on geode-json (org.json) in favor of Jackson. Initially this work has revolved around refactoring the internal results from gfsh commands (nothing that is user-visible). I'm now looking at the various gfsh 'data' commands (get, put, query and locate)

Re: [DISCUSS]: Tests requiring external services

2018-04-05 Thread Jens Deppe
> > > I'm actually fine with putting it in AcceptanceTest for now. > > > > > > > > > > > > Ideally I'd like to see something like JDBC connection strings > that > > > > could > > > > > > be passed in as properties vi

Re: [DISCUSS]: Tests requiring external services

2018-04-03 Thread Jens Deppe
I'm in favor of using docker for test isolation. We already have an 'AcceptanceTest' category which you might consider using instead of creating yet another category. --Jens On Tue, Apr 3, 2018 at 2:02 PM, Nick Reich wrote: > Team, > > As part of validating the new JDBC connector for Geode, we

Re: [VOTE] Apache Geode 1.5.0 RC1

2018-03-21 Thread Jens Deppe
-0700 Source-Repository: release/1.5.0 Source-Revision: 4ef51dacd79ff69336fb024f3d4b07271e90 --Jens On Wed, Mar 21, 2018 at 9:01 AM, Dave Barnes wrote: > +1 User Guide builds correctly from source, javadocs look up-to-date > > On Wed, Mar 21, 2018 at 8:27 AM, Jens Deppe wrote:

Re: [DISCUSS] gradle checks for test categories.

2018-03-21 Thread Jens Deppe
The build already fails if a test is added without @Category. Do you mean it should fail if the test also does not have one of these newer categories? --Jens On Wed, Mar 21, 2018 at 9:51 AM, Nabarun Nag wrote: > Hi, > > As we have now categorized our tests to categories like LuceneTests, > WanT

Re: [VOTE] Apache Geode 1.5.0 RC1

2018-03-21 Thread Jens Deppe
+1 - basic gfsh functionality looks good on Windows. On Tue, Mar 20, 2018 at 5:07 PM, Anthony Baker wrote: > md5 generation is already removed from geode, just need to do this for > geode-examples. > > Anthony > > > > On Mar 20, 2018, at 3:30 PM, Swapnil Bawaskar > wrote: > > > > Removed! > > >

Disabling DistributedRestoreSystemPropertiesTest for now

2018-03-18 Thread Jens Deppe
It appears that DistributedRestoreSystemPropertiesTest is intermittently failing and breaking the build. My initial analysis shows that the use of *categories* results in *all* test classes being instantiated and some of those instantiations end up launching DUnit VMs as a result of static or para

Re: Please use Awaitilty instead of Thread.sleep

2018-03-16 Thread Jens Deppe
My bad. Thanks for pointing that out Kirk. On Fri, Mar 16, 2018 at 11:22 AM, Kirk Lund wrote: > I can't believe we're still checking in new tests with Thread.sleep of 10 > seconds instead of using Awaitility. > > org.apache.geode.management.internal.configuration. > ClusterConfigLocatorRestartDU

Re: Row-Level security with key as a regex

2018-03-01 Thread Jens Deppe
Hi Nilkanth, It's basically up to the provided SecurityManager how to handle the ResourcePermission passed into it. So in your implementation could be as simple as: if (rp.getKey() != null && rp.getKey().contains("system") {...} Are you perhaps looking at a specific SecurityManager implemen

Re: [PROPOSAL] Deprecate running JMX Manager on non-locator members

2018-02-21 Thread Jens Deppe
One thing that prompted this initial proposal is that our (management and monitoring) service boundaries, (or one could say APIs), are not well defined. Case in point; we currently have a 'configuration service' which is used to persist configuration. This service is only available on locators. Th

[PROPOSAL] Deprecate running JMX Manager on non-locator members

2018-02-20 Thread Jens Deppe
TL;DR I'd like to propose that we deprecate and ultimately remove the ability to run a JMX manager on a non-locator member. I'm not sure about the history for this capability and the original use cases but I imagine it would have been driven, to a large part, by the ability for member discovery wi

Re: [PROPOSAL]: Adding declarable support in Gfsh command

2018-01-26 Thread Jens Deppe
This also avoids the other option of implementing this by having associated 'params' options for each option which can take a Declarable, thus reducing the proliferation of options - in particular for 'create region'. i.e. --cache-listener AND --cache-listener-params. Further, this json parameter

Re: Addition of rmi-io library

2018-01-25 Thread Jens Deppe
>>> deployment. > >>> > >>> On Tue, Jan 23, 2018 at 3:17 PM, Dan Smith wrote: > >>> > >>>> I don't have an issue with the dependency. But if we are opening up > > new > >>>> ports for RMI connections, that seems like a

Re: Addition of rmi-io library

2018-01-24 Thread Jens Deppe
dency. But if we are opening up > new > > > > ports for RMI connections, that seems like a potential security risk. > > If > > > > someone has enabled cluster SSL we shouldn't be opening up an > insecure > > > port > > > > for RMI conne

Addition of rmi-io library

2018-01-23 Thread Jens Deppe
Apologies that this was not raised earlier in discussion but I'm happy to describe it now. *Background:* When deploying jars into Geode they are moved through the system as simple byte[] blobs. This obviously consumes memory. The various affected areas are: - gfsh reads the jars into memory - th

Re: build is broken

2018-01-23 Thread Jens Deppe
The dependency is the rmi-io library. It provides functionality for streaming data over RMI. It was added a few weeks ago, so I'm not sure why your build is breaking now. --Jens On Tue, Jan 23, 2018 at 9:41 AM, Kirk Lund wrote: > Looks like this dependency is Jackcess (used for reading and wri

Re: [DISCUSS]: Handling of SDTOUT/STDERR and Signals

2017-12-20 Thread Jens Deppe
Hi Juan, Thanks for the detailed analysis. I did some digging too and cannot see why the QUIT signal might be getting lost. I'll do some more investigation... Just for reference, you can still use the 'jstack' tool to get thread dumps although they just show on the terminal where you run the com

Re: Debugging intermittent dunit failures

2017-12-15 Thread Jens Deppe
u already know that since you're the docker king. > > > > This change would eliminate memory pollution. Any test pollution > remaining > > after this should be caused by dunit writing to non-TemporaryFolder > > directories such as current working directory or user&#

Re: Debugging intermittent dunit failures

2017-12-14 Thread Jens Deppe
> > > > > > On Mon, Dec 11, 2017 at 1:42 PM, Dan Smith > > > wrote: > > > > > > > > > > > > > One other thing you can do is look for the below line in the > logs > > > of > > > > > your > > > > > > > failur

Re: Default branch for geode should be 'develop'

2017-12-11 Thread Jens Deppe
; > > > +1 > > > > > > > > > On 12/11/17 1:40 PM, Jens Deppe wrote: > > > > Currently, github has the default branch for apache/geode set to > > > 'master'. > > > > (which is why you need to point the r

Default branch for geode should be 'develop'

2017-12-11 Thread Jens Deppe
Currently, github has the default branch for apache/geode set to 'master'. (which is why you need to point the relevant branch to 'develop' for every new PR). Do we agree that this should be set to 'develop'? (I've messaged with some of the Infra folks and nothing appears to have changed recently

Debugging intermittent dunit failures

2017-12-11 Thread Jens Deppe
I've recently debugged various distributed tests which fail as a result of prior tests not cleaning up enough. It's somewhat painful and this is my usual debug process: - Examine the progress.txt file to determine which tests ran before my failing test. - I pick 20-25 of these tests and

Has github changed some preferences?

2017-12-06 Thread Jens Deppe
It seems that (at least my github pages) are acting a bit differently. When creating a Pull Request make sure that you create it against *develop* and not *master*: [image: Inline image 1] I would typically just ignore this and click through but that led me to creating a PR with 100s of changes.

Geode PR cleanup

2017-12-01 Thread Jens Deppe
Hey all, Could we do some github pull request housekeeping please? Specifically: - If you have an open PR that you know is not going to be merged, please close it. - If you have reviewed a PR and requested changes which have not been made, please bump the PR and see if the original su

Consistent results from gfsh commands

2017-10-31 Thread Jens Deppe
Hi, I've noticed that various commands, that execute on multiple members, return either tabulated results (one line per member - for example CreateRegionCommand) or a single pass/fail line. In the latter case it’s possible that information would get lost. For example if a command fails on multiple

Re: what is the command to read cacheserver.gfs file??

2017-10-24 Thread Jens Deppe
Hi Dinesh, Yes, that's pretty much it. That command will give you a list of all the possible stats. If you want an individual stat you can retrieve it like this: java -cp $GEODE_HOME/lib/geode-dependencies.jar org.apache.geode.internal.SystemAdmin stats vmStats:VMStats.processCpuTime -archive=ser

Re: geode and open JDK / newer version of java

2017-10-18 Thread Jens Deppe
Hi Roi, We've spent a little time doing some cursory testing with Java 9 and there are definitely some issues: - Compilation requires additional JVM args because of the new modularization in Java 9. - A few lambda syntaxes in tests need to be adjusted. - Some tests fail due to powermock issues. -

Re: Correct Geode REST API cipher

2017-09-23 Thread Jens Deppe
I've responded to your stackoverflow question... --Jens On Fri, Sep 22, 2017 at 2:29 AM, Rupert St John Webster < rup...@impress-solutions.com> wrote: > Hi, > > Please can someone advise which cipher to use for > ssl-enabled-components=web and ssl-ciphers=any > > After starting Geode with web se

Re: [DISCUSS] Clean build takes 10minutes to complete now

2017-09-11 Thread Jens Deppe
Right. I've added the full distribution to be uploaded as part of the 'uploadArchives' task. Here is how you can set up your gradle environment to retrieve it. Add the apache snapshot repo to your repositories: repositories { maven { url 'https://repository.apache.org/content/repositories/s

Re: git repo is gone

2017-09-01 Thread Jens Deppe
> So we go directly to github now? Not gitbox.apache.org? > > > > I'm able to clone this: https://gitbox.apache.org/repos/asf/geode.git > > > > > >> On 9/1/17 2:37 PM, Jens Deppe wrote: > >> TL;DR - Remove references to > >> https://git-wi

Re: git repo is gone

2017-09-01 Thread Jens Deppe
, Sep 1, 2017 at 2:31 PM, Jens Deppe wrote: > I believe it is. Just looking to see how/what we're supposed to use now. > > --Jens > > On Fri, Sep 1, 2017 at 2:30 PM, Bruce Schuchardt > wrote: > >> I'm not able to access the Apache repo anymore >> >>

Re: git repo is gone

2017-09-01 Thread Jens Deppe
I believe it is. Just looking to see how/what we're supposed to use now. --Jens On Fri, Sep 1, 2017 at 2:30 PM, Bruce Schuchardt wrote: > I'm not able to access the Apache repo anymore > > git clone https://git-wip-us.apache.org/repos/asf/geode.git > Cloning into 'geode'... > fatal: repository

Re: Review Request 62001: GEODE-3525: Dockerize AcceptanceTests

2017-08-31 Thread Jens Deppe
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/62001/#review184271 --- Ship it! Ship It! - Jens Deppe On Aug. 30, 2017, 10 p.m

Changes to Jenkins geode-nightly build job

2017-08-17 Thread Jens Deppe
Hi, I'm making changes to the geode-nightly build such that it will use docker containers for distributedTests. This means that these tests can also be parallelized, although for now I'm setting the parallelism to 1. Initially, this change should help with network isolation for tests so that bind

Re: Build failed in Jenkins: Geode-nightly #923

2017-08-16 Thread Jens Deppe
I'm working on enabling parallel dunits in docker to work on Jenkins. If this can happen it should help with some of these problems. However, it requires fairly beefy systems to run well, otherwise you end up with other issues around resource starvation. At worst we could just run with a parallelis

Re: Request for Jenkins config ability

2017-08-10 Thread Jens Deppe
Thanks Mark - I'll test it out tomorrow. On Thu, Aug 10, 2017 at 3:42 PM, Mark Bretl wrote: > Hi Jens, > > Skillz have been given. > > Let me know if you have any issues, > > --Mark > > On Thu, Aug 10, 2017 at 10:46 AM, Jens Deppe wrote: > > > Hi, &

Re: Review Request 61562: GEODE-3423: Provide support for running parallel docker builds in Jenkins

2017-08-10 Thread Jens Deppe
79719740bc85fd1939d2d7859a7c78c0a87dd26e Diff: https://reviews.apache.org/r/61562/diff/2/ Changes: https://reviews.apache.org/r/61562/diff/1-2/ Testing --- Thanks, Jens Deppe

Request for Jenkins config ability

2017-08-10 Thread Jens Deppe
Hi, I'd like to request the ability to make Jenkins config changes for Geode builds; I want to start implementing parallel dockerized dunit tests. Could someone with the appropriate skillz set me up please? Thanks --Jens

Review Request 61562: GEODE-3423: Provide support for running parallel docker builds in Jenkins

2017-08-10 Thread Jens Deppe
, Jens Deppe

Ability to push to dockerhub

2017-08-09 Thread Jens Deppe
Hi, I'd like the ability to push to the 'apachegeode' dockerhub repository in order to publish various docker images used for building Geode. My dockerhub account is 'jensdeppe'. Thanks --Jens

Re: Review Request 61114: GEODE-3296: Speed up Acceptance Tests

2017-07-25 Thread Jens Deppe
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/61114/#review181430 --- Ship it! Ship It! - Jens Deppe On July 25, 2017, 4:58 p.m

Re: Need to know complete execution time for geode-core Junit tests

2017-07-25 Thread Jens Deppe
Hi Dinesh, When I run '*./gradlew clean geode-core:test*', it takes about 1:30m on an iMac. How are you invoking the build? You can see the progress by tailing the file: geode-core/build/test/test-progress.test. --Jens On Tue, Jul 25, 2017 at 1:13 AM, Dinesh Akhand wrote: > Hi Team, > > I tr

Re: Review Request 60142: GEODE-3071: Provide capability to parallelize distributedTests

2017-06-19 Thread Jens Deppe
60142/diff/2-3/ Testing --- Manual test on linux Thanks, Jens Deppe

Re: Review Request 60142: GEODE-3071: Provide capability to parallelize distributedTests

2017-06-16 Thread Jens Deppe
--- Manual test on linux Thanks, Jens Deppe

Review Request 60142: GEODE-3071: Provide capability to parallelize distributedTests

2017-06-15 Thread Jens Deppe
fdcb gradle.properties ca79a3816bffa56e412d82dd570f1e0e445592be gradle/docker.gradle PRE-CREATION Diff: https://reviews.apache.org/r/60142/diff/1/ Testing --- Manual test on linux Thanks, Jens Deppe

Re: [VOTE] Using Pull Requests over Review Board

2017-06-12 Thread Jens Deppe
We can always set up a bit of automation so that committers are notified if a PR sits unattended for a period of time. --Jens On Mon, Jun 12, 2017 at 4:20 PM, Udo Kohlmeyer wrote: > 0 for PR - I feel that both have their benefits and downsides. > > * +1 for PR: As Naba stated, the integration

Re: [DISCUSS] easy string based partitioning

2017-06-05 Thread Jens Deppe
I like the idea of keeping the configuration 'conventional' and thus not requiring any server configuration. As soon as you need to define a regex on the server you might as well be writing PartitionResolver code. As an aside I also think that using regexes to parse keys would also introduce a mea

Re: Enabling debugging from gfsh launched components

2017-05-31 Thread Jens Deppe
OK, thanks Jinmei. --Jens On Wed, May 31, 2017 at 8:42 AM, Jinmei Liao wrote: > I believe this is due to bug I introduced when working on gfsh parser when > dealing with --J option that has comma in the values. Will get a fix in > soon. > > On Wed, May 31, 2017 at 7:41 AM, Je

Enabling debugging from gfsh launched components

2017-05-31 Thread Jens Deppe
Hi, In the past, when launching a server or locator with gfsh, I could specify debugging options like this: gfsh start locator --name=locator1 \ --J=\"-agentlib:jdwp=transport=dt_socket,server=y,suspend=n,address=5005\" I had to specially quote the agentlib options otherwise the commas would

Re: Simple Java Client

2017-04-24 Thread Jens Deppe
I would suggest working closely with John Blum on this as I believe he already provides similar functionality in Spring Data Geode. Also, would this be the beginning of a client-side admin API with the intention of fleshing that out completely in the future? --Jens On Mon, Apr 24, 2017 at 3:03 P

[jira] [Created] (GEODE-2814) Native client test framework uses Xerces which depends on Curl

2017-04-24 Thread Jens Deppe (JIRA)
Jens Deppe created GEODE-2814: - Summary: Native client test framework uses Xerces which depends on Curl Key: GEODE-2814 URL: https://issues.apache.org/jira/browse/GEODE-2814 Project: Geode

Re: Refactor gfsh parser

2017-04-20 Thread Jens Deppe
IMO I much prefer having: --member=member1,member2 vs: --member=member1 --member=member2 The latter can leave me wondering if subsequent values are additive or actually override prior ones. --Jens On Thu, Apr 20, 2017 at 1:37 PM, Jinmei Liao wrote: > In the effort of adding option validati

[jira] [Assigned] (GEODE-2766) Only apply patches for dependencies on relevant architectures

2017-04-10 Thread Jens Deppe (JIRA)
[ https://issues.apache.org/jira/browse/GEODE-2766?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Jens Deppe reassigned GEODE-2766: - Assignee: Jens Deppe > Only apply patches for dependencies on relevant architectu

[jira] [Created] (GEODE-2766) Only apply patches for dependencies on relevant architectures

2017-04-10 Thread Jens Deppe (JIRA)
Jens Deppe created GEODE-2766: - Summary: Only apply patches for dependencies on relevant architectures Key: GEODE-2766 URL: https://issues.apache.org/jira/browse/GEODE-2766 Project: Geode Issue

Re: Orphaned Server processes

2017-04-05 Thread Jens Deppe
Also, don't forget that you can start a server in a 'wait for locator' mode (I forget the actual option) which should be distinguished from an actual 'orphaned' instance. --Jens On Wed, Apr 5, 2017 at 3:54 PM, Michael Stolz wrote: > I think the current behavior is probably left over from the mu

Re: CacheServer and fake in-process client

2017-04-03 Thread Jens Deppe
Would it be viable to consider using the REST API for this? --Jens On Mon, Apr 3, 2017 at 2:31 PM, Kirk Lund wrote: > Has anyone written a test where they use a single JVM in which they start a > server Cache and then fake up a connection from a non-existent client to > feed messages into the s

Re: Proposal to add new 'export artifacts' GFSH command

2017-01-30 Thread Jens Deppe
What about simply having --exclude and --include options that take a regex in order to exclude and include arbitrary files as per the users' choosing. --Jens On Mon, Jan 30, 2017 at 3:33 PM, Swapnil Bawaskar wrote: > On Mon, Jan 30, 2017 at 10:15 AM, Udo Kohlmeyer > wrote: > > > I think stats

Re: build problem in new module

2017-01-25 Thread Jens Deppe
Bruce, the message is just informational as this is the first time the upload is happening for this module. It was successful: https://repository.apache.org/content/repositories/snapshots/org/apache/geode/geode-old-versions/ --Jens On Wed, Jan 25, 2017 at 8:22 AM, Bruce Schuchardt wrote: > I n

[jira] [Closed] (GEODE-2320) Update message for minimum JDK version required

2017-01-18 Thread Jens Deppe (JIRA)
[ https://issues.apache.org/jira/browse/GEODE-2320?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Jens Deppe closed GEODE-2320. - > Update message for minimum JDK version requi

[jira] [Resolved] (GEODE-2320) Update message for minimum JDK version required

2017-01-18 Thread Jens Deppe (JIRA)
[ https://issues.apache.org/jira/browse/GEODE-2320?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ] Jens Deppe resolved GEODE-2320. --- Resolution: Not A Problem Fix Version/s: 1.1.0 > Update message for minimum JDK vers

<    1   2   3   >