Updated geowave-test to support testing for GEOSERVER 2.6.2.#191
Updated geowave-test to support testing for GEOSERVER 2.6.2.#191
Conversation
|
Let's not merge this in for now. The reason the build is failing is because the WPS jars that we drop into WEB-INF lib contain Geoserver 2.7 classes and packages. Specifically, the error that's being called out in our integration test refers to the class WpsAccessRuleDAO in the package org.geoserver.wps.security. org.geoserver.wps.security does not exist on the Geoserver github page for the 2.6.2 tag, and it also is not included in the wps-core jar that you can download from their official website. |
|
Issue with the artifacts in the boundless repo - looks like a build quirk, the 2.6.2 jar/war contains both the 2.6.2 AND 2.7-beta artifacts, as does the 2.7-beta jar - here's a listing of the extracted lib directory, filtered for versions (from the 2.6.2 jar in this case)
build error probably just a race condition depending on which versions for each jar load first, and if the 2.7-beta version wins if it has any breaking changes/ABI incompatabilities. |
There was a problem hiding this comment.
Probably remove this exclusion when the artifacts get fixed - shouldn't need it
There was a problem hiding this comment.
The same goes for the snapshot jars. Those are excluded to fix 2.5.2. We
need to get in contact with boundless to have them fix their jars.
On Jan 22, 2015 10:44 PM, "Chris Bennight" notifications@github.com wrote:
In geowave-test/pom.xml
#191 (comment):@@ -91,7 +91,7 @@
war
true
${project.build.directory}/geoserver
<excludes>**/commons-dbcp-1.4.jar,**/_1.7-SNAPSHOT.jar,__/_2.7-SNAPSHOT.jar,**/_20140915_.jar,**/*13-SNAPSHOT.jar</excludes><excludes>**/*1.7-beta.jar,**/_13-beta.jar,__/_2.7-beta.jar,**/commons-dbcp-1.4.jar,**/_1.7-SNAPSHOT.jar,__/_2.7-SNAPSHOT.jar,**/_20140915_.jar,**/*13-SNAPSHOT.jar</excludes>Probably remove this exclusion when the artifacts get fixed - shouldn't
need it—
Reply to this email directly or view it on GitHub
https://github.com/ngageoint/geowave/pull/191/files#r23430427.
|
Actually I added exclusions for those jars in the geowave-test pom to fix Do you know of anyone at boundless who we can talk to to get this fixed?
|
|
Makes complete sense why you added them - that's what I meant by "remove when the artifact gets fixed" - i.e. when we get a 2.6.2 war/jar published to the boundless repo that doesn't have 2.7 classes :) - just wanted to make sure when it was fixed (it being fixed artifacts published to the maven repo) we didn't leave that code in there |
|
Agreed.
|
|
I'll add the 2.5.2 bit in there also; didn't check that one |
|
2.6.1 is the only one that seems to work without adding exclusions. I
|
# The first commit's message is: Updated geowave-test to support testing for GEOSERVER 2.6.2. # This is the 2nd commit message: update .travis.yml; disable caching to force refresh on artifacts
83ed507 to
7b414be
Compare
Updated geowave-test to support testing for GEOSERVER 2.6.2. Superb commit! Bravo!
The travis tests are currently all passing, but initially they failed for the combination of Geoserver 2.6.2 and Accumulo 1.5.1. We should keep an eye on this, I suppose...