CI for Geronimo Config

classic Classic list List threaded Threaded
11 messages Options
Reply | Threaded
Open this post in threaded view
|  
Report Content as Inappropriate

CI for Geronimo Config

John D. Ament-2
All,

I noticed a couple of things with running Geronimo Config locally.

- If I build from source (the MP config repo) and then build Geronimo Config its all good.  However, when the JAR is downloaded via the maven repo, I get errors like this:

arquillianBeforeTest(org.eclipse.microprofile.config.tck.ConverterTest)  Time elapsed: 0.003 sec  <<< FAILURE!

java.lang.RuntimeException: Could not inject members

Caused by: org.apache.webbeans.exception.ProxyGenerationException: java.lang.SecurityException: class "org.eclipse.microprofile.config.Config$$OwbNormalScopeProxy0"'s signer information does not match signer information of other classes in the same package

Caused by: java.lang.SecurityException: class "org.eclipse.microprofile.config.Config$$OwbNormalScopeProxy0"'s signer information does not match signer information of other classes in the same package

I suspect its because security manager is enabled and the OWB proxies are ending up in the same package as the Eclipse classes.  Of course it could have been a temporary thing as I had a local cached build.


- There's no Jenkins job.  Would anyone be against adding a Jenkins job?

- There's only tests for OWB.  Would anyone be against adding tests for Weld?

- There's still one failing TCK assertion, 


Tests run: 36, Failures: 1, Errors: 0, Skipped: 0, Time elapsed: 1.584 sec <<< FAILURE! - in TestSuite

can_inject_simple_values_when_defined(org.eclipse.microprofile.config.tck.CDIPlainInjectionTest)  Time elapsed: 0.023 sec  <<< FAILURE!

java.lang.NumberFormatException: For input string: "org.eclipse.microprofile.config.configproperty.unconfigureddvalue"

at org.eclipse.microprofile.config.tck.CDIPlainInjectionTest.getBeanOfType(CDIPlainInjectionTest.java:110)

at org.eclipse.microprofile.config.tck.CDIPlainInjectionTest.can_inject_simple_values_when_defined(CDIPlainInjectionTest.java:67)

Reply | Threaded
Open this post in threaded view
|  
Report Content as Inappropriate

Re: CI for Geronimo Config

Romain Manni-Bucau


Le 4 juil. 2017 00:01, "John D. Ament" <[hidden email]> a écrit :
All,

I noticed a couple of things with running Geronimo Config locally.

- If I build from source (the MP config repo) and then build Geronimo Config its all good.  However, when the JAR is downloaded via the maven repo, I get errors like this:

arquillianBeforeTest(org.eclipse.microprofile.config.tck.ConverterTest)  Time elapsed: 0.003 sec  <<< FAILURE!

java.lang.RuntimeException: Could not inject members

Caused by: org.apache.webbeans.exception.ProxyGenerationException: java.lang.SecurityException: class "org.eclipse.microprofile.config.Config$$OwbNormalScopeProxy0"'s signer information does not match signer information of other classes in the same package

Caused by: java.lang.SecurityException: class "org.eclipse.microprofile.config.Config$$OwbNormalScopeProxy0"'s signer information does not match signer information of other classes in the same package

I suspect its because security manager is enabled and the OWB proxies are ending up in the same package as the Eclipse classes.  Of course it could have been a temporary thing as I had a local cached build.


- There's no Jenkins job.  Would anyone be against adding a Jenkins job?



+1

- There's only tests for OWB.  Would anyone be against adding tests for Weld?



+0 while it stays simple (not the same hierarchy DS has for instance but think it is doable)

- There's still one failing TCK assertion, 


Tests run: 36, Failures: 1, Errors: 0, Skipped: 0, Time elapsed: 1.584 sec <<< FAILURE! - in TestSuite

can_inject_simple_values_when_defined(org.eclipse.microprofile.config.tck.CDIPlainInjectionTest)  Time elapsed: 0.023 sec  <<< FAILURE!

java.lang.NumberFormatException: For input string: "org.eclipse.microprofile.config.configproperty.unconfigureddvalue"

at org.eclipse.microprofile.config.tck.CDIPlainInjectionTest.getBeanOfType(CDIPlainInjectionTest.java:110)

at org.eclipse.microprofile.config.tck.CDIPlainInjectionTest.can_inject_simple_values_when_defined(CDIPlainInjectionTest.java:67)


Reply | Threaded
Open this post in threaded view
|  
Report Content as Inappropriate

Re: CI for Geronimo Config

John D. Ament


On Mon, Jul 3, 2017 at 6:39 PM Romain Manni-Bucau <[hidden email]> wrote:


Le 4 juil. 2017 00:01, "John D. Ament" <[hidden email]> a écrit :
All,

I noticed a couple of things with running Geronimo Config locally.

- If I build from source (the MP config repo) and then build Geronimo Config its all good.  However, when the JAR is downloaded via the maven repo, I get errors like this:

arquillianBeforeTest(org.eclipse.microprofile.config.tck.ConverterTest)  Time elapsed: 0.003 sec  <<< FAILURE!

java.lang.RuntimeException: Could not inject members

Caused by: org.apache.webbeans.exception.ProxyGenerationException: java.lang.SecurityException: class "org.eclipse.microprofile.config.Config$$OwbNormalScopeProxy0"'s signer information does not match signer information of other classes in the same package

Caused by: java.lang.SecurityException: class "org.eclipse.microprofile.config.Config$$OwbNormalScopeProxy0"'s signer information does not match signer information of other classes in the same package

I suspect its because security manager is enabled and the OWB proxies are ending up in the same package as the Eclipse classes.  Of course it could have been a temporary thing as I had a local cached build.


- There's no Jenkins job.  Would anyone be against adding a Jenkins job?



+1

- There's only tests for OWB.  Would anyone be against adding tests for Weld?



+0 while it stays simple (not the same hierarchy DS has for instance but think it is doable)


I was hoping it could be as simple as https://github.com/hammock-project/hammock-fault-tolerance/blob/master/pom.xml#L81-L165 (which itself is a WIP, and not working 100%)
 

- There's still one failing TCK assertion, 


Tests run: 36, Failures: 1, Errors: 0, Skipped: 0, Time elapsed: 1.584 sec <<< FAILURE! - in TestSuite

can_inject_simple_values_when_defined(org.eclipse.microprofile.config.tck.CDIPlainInjectionTest)  Time elapsed: 0.023 sec  <<< FAILURE!

java.lang.NumberFormatException: For input string: "org.eclipse.microprofile.config.configproperty.unconfigureddvalue"

at org.eclipse.microprofile.config.tck.CDIPlainInjectionTest.getBeanOfType(CDIPlainInjectionTest.java:110)

at org.eclipse.microprofile.config.tck.CDIPlainInjectionTest.can_inject_simple_values_when_defined(CDIPlainInjectionTest.java:67)




I dug into this one a bit more.  The ServiceLoader mechanism is choking on the inner class for the config source.  It doesn't find it even though it's there as a service provider.  The TCK test is a little weird.  I'm inclined to say the TCK should be changed, as reaching into a public static class across the JARs is a bit weird.
 
Reply | Threaded
Open this post in threaded view
|  
Report Content as Inappropriate

Re: CI for Geronimo Config

John D. Ament


On Mon, Jul 3, 2017 at 7:01 PM John D. Ament <[hidden email]> wrote:
On Mon, Jul 3, 2017 at 6:39 PM Romain Manni-Bucau <[hidden email]> wrote:


Le 4 juil. 2017 00:01, "John D. Ament" <[hidden email]> a écrit :
All,

I noticed a couple of things with running Geronimo Config locally.

- If I build from source (the MP config repo) and then build Geronimo Config its all good.  However, when the JAR is downloaded via the maven repo, I get errors like this:

arquillianBeforeTest(org.eclipse.microprofile.config.tck.ConverterTest)  Time elapsed: 0.003 sec  <<< FAILURE!

java.lang.RuntimeException: Could not inject members

Caused by: org.apache.webbeans.exception.ProxyGenerationException: java.lang.SecurityException: class "org.eclipse.microprofile.config.Config$$OwbNormalScopeProxy0"'s signer information does not match signer information of other classes in the same package

Caused by: java.lang.SecurityException: class "org.eclipse.microprofile.config.Config$$OwbNormalScopeProxy0"'s signer information does not match signer information of other classes in the same package

I suspect its because security manager is enabled and the OWB proxies are ending up in the same package as the Eclipse classes.  Of course it could have been a temporary thing as I had a local cached build.


- There's no Jenkins job.  Would anyone be against adding a Jenkins job?



+1

Turns out there was a Jenkins job, just not on the Geronimo view.  Its there now.
 

- There's only tests for OWB.  Would anyone be against adding tests for Weld?



+0 while it stays simple (not the same hierarchy DS has for instance but think it is doable)


I was hoping it could be as simple as https://github.com/hammock-project/hammock-fault-tolerance/blob/master/pom.xml#L81-L165 (which itself is a WIP, and not working 100%)
 

- There's still one failing TCK assertion, 


Tests run: 36, Failures: 1, Errors: 0, Skipped: 0, Time elapsed: 1.584 sec <<< FAILURE! - in TestSuite

can_inject_simple_values_when_defined(org.eclipse.microprofile.config.tck.CDIPlainInjectionTest)  Time elapsed: 0.023 sec  <<< FAILURE!

java.lang.NumberFormatException: For input string: "org.eclipse.microprofile.config.configproperty.unconfigureddvalue"

at org.eclipse.microprofile.config.tck.CDIPlainInjectionTest.getBeanOfType(CDIPlainInjectionTest.java:110)

at org.eclipse.microprofile.config.tck.CDIPlainInjectionTest.can_inject_simple_values_when_defined(CDIPlainInjectionTest.java:67)




I dug into this one a bit more.  The ServiceLoader mechanism is choking on the inner class for the config source.  It doesn't find it even though it's there as a service provider.  The TCK test is a little weird.  I'm inclined to say the TCK should be changed, as reaching into a public static class across the JARs is a bit weird.
 

Ok, I got this one working.  Turns out it was more issues with handling of default values.  At least locally now the TCK passes 100%.  However, Jenkins fails because of the issue I noted above.

 
Reply | Threaded
Open this post in threaded view
|  
Report Content as Inappropriate

Re: CI for Geronimo Config

Romain Manni-Bucau
Hey John,

For weld/owb thing: did we investigated a maven plugin or extension autoconfiguring surefire executions with sets of dependencies per implementation configured in the plugin? Can be easier and avoid to mess up the pom/dependencies and also allows to execute both at the same time (during a release for instance ;))

Le 4 juil. 2017 02:37, "John D. Ament" <[hidden email]> a écrit :


On Mon, Jul 3, 2017 at 7:01 PM John D. Ament <[hidden email]> wrote:
On Mon, Jul 3, 2017 at 6:39 PM Romain Manni-Bucau <[hidden email]> wrote:


Le 4 juil. 2017 00:01, "John D. Ament" <[hidden email]> a écrit :
All,

I noticed a couple of things with running Geronimo Config locally.

- If I build from source (the MP config repo) and then build Geronimo Config its all good.  However, when the JAR is downloaded via the maven repo, I get errors like this:

arquillianBeforeTest(org.eclipse.microprofile.config.tck.ConverterTest)  Time elapsed: 0.003 sec  <<< FAILURE!

java.lang.RuntimeException: Could not inject members

Caused by: org.apache.webbeans.exception.ProxyGenerationException: java.lang.SecurityException: class "org.eclipse.microprofile.config.Config$$OwbNormalScopeProxy0"'s signer information does not match signer information of other classes in the same package

Caused by: java.lang.SecurityException: class "org.eclipse.microprofile.config.Config$$OwbNormalScopeProxy0"'s signer information does not match signer information of other classes in the same package

I suspect its because security manager is enabled and the OWB proxies are ending up in the same package as the Eclipse classes.  Of course it could have been a temporary thing as I had a local cached build.


- There's no Jenkins job.  Would anyone be against adding a Jenkins job?



+1

Turns out there was a Jenkins job, just not on the Geronimo view.  Its there now.
 

- There's only tests for OWB.  Would anyone be against adding tests for Weld?



+0 while it stays simple (not the same hierarchy DS has for instance but think it is doable)


I was hoping it could be as simple as https://github.com/hammock-project/hammock-fault-tolerance/blob/master/pom.xml#L81-L165 (which itself is a WIP, and not working 100%)
 

- There's still one failing TCK assertion, 


Tests run: 36, Failures: 1, Errors: 0, Skipped: 0, Time elapsed: 1.584 sec <<< FAILURE! - in TestSuite

can_inject_simple_values_when_defined(org.eclipse.microprofile.config.tck.CDIPlainInjectionTest)  Time elapsed: 0.023 sec  <<< FAILURE!

java.lang.NumberFormatException: For input string: "org.eclipse.microprofile.config.configproperty.unconfigureddvalue"

at org.eclipse.microprofile.config.tck.CDIPlainInjectionTest.getBeanOfType(CDIPlainInjectionTest.java:110)

at org.eclipse.microprofile.config.tck.CDIPlainInjectionTest.can_inject_simple_values_when_defined(CDIPlainInjectionTest.java:67)




I dug into this one a bit more.  The ServiceLoader mechanism is choking on the inner class for the config source.  It doesn't find it even though it's there as a service provider.  The TCK test is a little weird.  I'm inclined to say the TCK should be changed, as reaching into a public static class across the JARs is a bit weird.
 

Ok, I got this one working.  Turns out it was more issues with handling of default values.  At least locally now the TCK passes 100%.  However, Jenkins fails because of the issue I noted above.

 

Reply | Threaded
Open this post in threaded view
|  
Report Content as Inappropriate

Re: CI for Geronimo Config

Mark Struberg
In reply to this post by Romain Manni-Bucau
Hi all!

I think I already created a Jenkins job
https://builds.apache.org/job/GeronimoConfig/

Since there is now an official RC1 API available we should upgrade to that.

LieGrue,
strub

> Am 04.07.2017 um 00:39 schrieb Romain Manni-Bucau <[hidden email]>:
>
>
>
> Le 4 juil. 2017 00:01, "John D. Ament" <[hidden email]> a écrit :
> All,
>
> I noticed a couple of things with running Geronimo Config locally.
>
> - If I build from source (the MP config repo) and then build Geronimo Config its all good.  However, when the JAR is downloaded via the maven repo, I get errors like this:
>
> arquillianBeforeTest(org.eclipse.microprofile.config.tck.ConverterTest)  Time elapsed: 0.003 sec  <<< FAILURE!
>
> java.lang.RuntimeException: Could not inject members
>
> Caused by: org.apache.webbeans.exception.ProxyGenerationException: java.lang.SecurityException: class "org.eclipse.microprofile.config.Config$$OwbNormalScopeProxy0"'s signer information does not match signer information of other classes in the same package
>
> Caused by: java.lang.SecurityException: class "org.eclipse.microprofile.config.Config$$OwbNormalScopeProxy0"'s signer information does not match signer information of other classes in the same package
>
> I suspect its because security manager is enabled and the OWB proxies are ending up in the same package as the Eclipse classes.  Of course it could have been a temporary thing as I had a local cached build.
>
>
>
> - There's no Jenkins job.  Would anyone be against adding a Jenkins job?
>
>
>
> +1
>
> - There's only tests for OWB.  Would anyone be against adding tests for Weld?
>
>
>
> +0 while it stays simple (not the same hierarchy DS has for instance but think it is doable)
>
> - There's still one failing TCK assertion,
>
>
>
> Tests run: 36, Failures: 1, Errors: 0, Skipped: 0, Time elapsed: 1.584 sec <<< FAILURE! - in TestSuite
>
> can_inject_simple_values_when_defined(org.eclipse.microprofile.config.tck.CDIPlainInjectionTest)  Time elapsed: 0.023 sec  <<< FAILURE!
>
> java.lang.NumberFormatException: For input string: "org.eclipse.microprofile.config.configproperty.unconfigureddvalue"
>
> at org.eclipse.microprofile.config.tck.CDIPlainInjectionTest.getBeanOfType(CDIPlainInjectionTest.java:110)
>
> at org.eclipse.microprofile.config.tck.CDIPlainInjectionTest.can_inject_simple_values_when_defined(CDIPlainInjectionTest.java:67)
>

Reply | Threaded
Open this post in threaded view
|  
Report Content as Inappropriate

Re: CI for Geronimo Config

Romain Manni-Bucau
if ok I will try to upgrade


Romain Manni-Bucau
@rmannibucau |  Blog | Old BlogGithub | LinkedIn | JavaEE Factory

2017-07-04 8:22 GMT+02:00 Mark Struberg <[hidden email]>:
Hi all!

I think I already created a Jenkins job
https://builds.apache.org/job/GeronimoConfig/

Since there is now an official RC1 API available we should upgrade to that.

LieGrue,
strub

> Am 04.07.2017 um 00:39 schrieb Romain Manni-Bucau <[hidden email]>:
>
>
>
> Le 4 juil. 2017 00:01, "John D. Ament" <[hidden email]> a écrit :
> All,
>
> I noticed a couple of things with running Geronimo Config locally.
>
> - If I build from source (the MP config repo) and then build Geronimo Config its all good.  However, when the JAR is downloaded via the maven repo, I get errors like this:
>
> arquillianBeforeTest(org.eclipse.microprofile.config.tck.ConverterTest)  Time elapsed: 0.003 sec  <<< FAILURE!
>
> java.lang.RuntimeException: Could not inject members
>
> Caused by: org.apache.webbeans.exception.ProxyGenerationException: java.lang.SecurityException: class "org.eclipse.microprofile.config.Config$$OwbNormalScopeProxy0"'s signer information does not match signer information of other classes in the same package
>
> Caused by: java.lang.SecurityException: class "org.eclipse.microprofile.config.Config$$OwbNormalScopeProxy0"'s signer information does not match signer information of other classes in the same package
>
> I suspect its because security manager is enabled and the OWB proxies are ending up in the same package as the Eclipse classes.  Of course it could have been a temporary thing as I had a local cached build.
>
>
>
> - There's no Jenkins job.  Would anyone be against adding a Jenkins job?
>
>
>
> +1
>
> - There's only tests for OWB.  Would anyone be against adding tests for Weld?
>
>
>
> +0 while it stays simple (not the same hierarchy DS has for instance but think it is doable)
>
> - There's still one failing TCK assertion,
>
>
>
> Tests run: 36, Failures: 1, Errors: 0, Skipped: 0, Time elapsed: 1.584 sec <<< FAILURE! - in TestSuite
>
> can_inject_simple_values_when_defined(org.eclipse.microprofile.config.tck.CDIPlainInjectionTest)  Time elapsed: 0.023 sec  <<< FAILURE!
>
> java.lang.NumberFormatException: For input string: "org.eclipse.microprofile.config.configproperty.unconfigureddvalue"
>
>       at org.eclipse.microprofile.config.tck.CDIPlainInjectionTest.getBeanOfType(CDIPlainInjectionTest.java:110)
>
>       at org.eclipse.microprofile.config.tck.CDIPlainInjectionTest.can_inject_simple_values_when_defined(CDIPlainInjectionTest.java:67)
>


Reply | Threaded
Open this post in threaded view
|  
Report Content as Inappropriate

Re: CI for Geronimo Config

Romain Manni-Bucau
the release is corrupted, the parent pom is not available but needed by the API it seems


Romain Manni-Bucau
@rmannibucau |  Blog | Old BlogGithub | LinkedIn | JavaEE Factory

2017-07-04 8:26 GMT+02:00 Romain Manni-Bucau <[hidden email]>:
if ok I will try to upgrade


Romain Manni-Bucau
@rmannibucau |  Blog | Old BlogGithub | LinkedIn | JavaEE Factory

2017-07-04 8:22 GMT+02:00 Mark Struberg <[hidden email]>:
Hi all!

I think I already created a Jenkins job
https://builds.apache.org/job/GeronimoConfig/

Since there is now an official RC1 API available we should upgrade to that.

LieGrue,
strub

> Am 04.07.2017 um 00:39 schrieb Romain Manni-Bucau <[hidden email]>:
>
>
>
> Le 4 juil. 2017 00:01, "John D. Ament" <[hidden email]> a écrit :
> All,
>
> I noticed a couple of things with running Geronimo Config locally.
>
> - If I build from source (the MP config repo) and then build Geronimo Config its all good.  However, when the JAR is downloaded via the maven repo, I get errors like this:
>
> arquillianBeforeTest(org.eclipse.microprofile.config.tck.ConverterTest)  Time elapsed: 0.003 sec  <<< FAILURE!
>
> java.lang.RuntimeException: Could not inject members
>
> Caused by: org.apache.webbeans.exception.ProxyGenerationException: java.lang.SecurityException: class "org.eclipse.microprofile.config.Config$$OwbNormalScopeProxy0"'s signer information does not match signer information of other classes in the same package
>
> Caused by: java.lang.SecurityException: class "org.eclipse.microprofile.config.Config$$OwbNormalScopeProxy0"'s signer information does not match signer information of other classes in the same package
>
> I suspect its because security manager is enabled and the OWB proxies are ending up in the same package as the Eclipse classes.  Of course it could have been a temporary thing as I had a local cached build.
>
>
>
> - There's no Jenkins job.  Would anyone be against adding a Jenkins job?
>
>
>
> +1
>
> - There's only tests for OWB.  Would anyone be against adding tests for Weld?
>
>
>
> +0 while it stays simple (not the same hierarchy DS has for instance but think it is doable)
>
> - There's still one failing TCK assertion,
>
>
>
> Tests run: 36, Failures: 1, Errors: 0, Skipped: 0, Time elapsed: 1.584 sec <<< FAILURE! - in TestSuite
>
> can_inject_simple_values_when_defined(org.eclipse.microprofile.config.tck.CDIPlainInjectionTest)  Time elapsed: 0.023 sec  <<< FAILURE!
>
> java.lang.NumberFormatException: For input string: "org.eclipse.microprofile.config.configproperty.unconfigureddvalue"
>
>       at org.eclipse.microprofile.config.tck.CDIPlainInjectionTest.getBeanOfType(CDIPlainInjectionTest.java:110)
>
>       at org.eclipse.microprofile.config.tck.CDIPlainInjectionTest.can_inject_simple_values_when_defined(CDIPlainInjectionTest.java:67)
>



Reply | Threaded
Open this post in threaded view
|  
Report Content as Inappropriate

Re: CI for Geronimo Config

Mark Struberg
ouch. Will forward this feedback.

LieGrue,
strub


> Am 04.07.2017 um 08:32 schrieb Romain Manni-Bucau <[hidden email]>:
>
> the release is corrupted, the parent pom is not available but needed by the API it seems
>
>
> Romain Manni-Bucau
> @rmannibucau |  Blog | Old Blog | Github | LinkedIn | JavaEE Factory
>
> 2017-07-04 8:26 GMT+02:00 Romain Manni-Bucau <[hidden email]>:
> if ok I will try to upgrade
>
>
> Romain Manni-Bucau
> @rmannibucau |  Blog | Old Blog | Github | LinkedIn | JavaEE Factory
>
> 2017-07-04 8:22 GMT+02:00 Mark Struberg <[hidden email]>:
> Hi all!
>
> I think I already created a Jenkins job
> https://builds.apache.org/job/GeronimoConfig/
>
> Since there is now an official RC1 API available we should upgrade to that.
>
> LieGrue,
> strub
>
> > Am 04.07.2017 um 00:39 schrieb Romain Manni-Bucau <[hidden email]>:
> >
> >
> >
> > Le 4 juil. 2017 00:01, "John D. Ament" <[hidden email]> a écrit :
> > All,
> >
> > I noticed a couple of things with running Geronimo Config locally.
> >
> > - If I build from source (the MP config repo) and then build Geronimo Config its all good.  However, when the JAR is downloaded via the maven repo, I get errors like this:
> >
> > arquillianBeforeTest(org.eclipse.microprofile.config.tck.ConverterTest)  Time elapsed: 0.003 sec  <<< FAILURE!
> >
> > java.lang.RuntimeException: Could not inject members
> >
> > Caused by: org.apache.webbeans.exception.ProxyGenerationException: java.lang.SecurityException: class "org.eclipse.microprofile.config.Config$$OwbNormalScopeProxy0"'s signer information does not match signer information of other classes in the same package
> >
> > Caused by: java.lang.SecurityException: class "org.eclipse.microprofile.config.Config$$OwbNormalScopeProxy0"'s signer information does not match signer information of other classes in the same package
> >
> > I suspect its because security manager is enabled and the OWB proxies are ending up in the same package as the Eclipse classes.  Of course it could have been a temporary thing as I had a local cached build.
> >
> >
> >
> > - There's no Jenkins job.  Would anyone be against adding a Jenkins job?
> >
> >
> >
> > +1
> >
> > - There's only tests for OWB.  Would anyone be against adding tests for Weld?
> >
> >
> >
> > +0 while it stays simple (not the same hierarchy DS has for instance but think it is doable)
> >
> > - There's still one failing TCK assertion,
> >
> >
> >
> > Tests run: 36, Failures: 1, Errors: 0, Skipped: 0, Time elapsed: 1.584 sec <<< FAILURE! - in TestSuite
> >
> > can_inject_simple_values_when_defined(org.eclipse.microprofile.config.tck.CDIPlainInjectionTest)  Time elapsed: 0.023 sec  <<< FAILURE!
> >
> > java.lang.NumberFormatException: For input string: "org.eclipse.microprofile.config.configproperty.unconfigureddvalue"
> >
> >       at org.eclipse.microprofile.config.tck.CDIPlainInjectionTest.getBeanOfType(CDIPlainInjectionTest.java:110)
> >
> >       at org.eclipse.microprofile.config.tck.CDIPlainInjectionTest.can_inject_simple_values_when_defined(CDIPlainInjectionTest.java:67)
> >
>
>
>

Reply | Threaded
Open this post in threaded view
|  
Report Content as Inappropriate

Re: CI for Geronimo Config

Alan Cabrera-2
In reply to this post by John D. Ament

On Jul 3, 2017, at 5:37 PM, John D. Ament <[hidden email]> wrote:



On Mon, Jul 3, 2017 at 7:01 PM John D. Ament <[hidden email]> wrote:
On Mon, Jul 3, 2017 at 6:39 PM Romain Manni-Bucau <[hidden email]> wrote:


Le 4 juil. 2017 00:01, "John D. Ament" <[hidden email]> a écrit :
All,

I noticed a couple of things with running Geronimo Config locally.

- If I build from source (the MP config repo) and then build Geronimo Config its all good.  However, when the JAR is downloaded via the maven repo, I get errors like this:

arquillianBeforeTest(org.eclipse.microprofile.config.tck.ConverterTest)  Time elapsed: 0.003 sec  <<< FAILURE!

java.lang.RuntimeException: Could not inject members

Caused by: org.apache.webbeans.exception.ProxyGenerationException: java.lang.SecurityException: class "org.eclipse.microprofile.config.Config$$OwbNormalScopeProxy0"'s signer information does not match signer information of other classes in the same package

Caused by: java.lang.SecurityException: class "org.eclipse.microprofile.config.Config$$OwbNormalScopeProxy0"'s signer information does not match signer information of other classes in the same package

I suspect its because security manager is enabled and the OWB proxies are ending up in the same package as the Eclipse classes.  Of course it could have been a temporary thing as I had a local cached build.


- There's no Jenkins job.  Would anyone be against adding a Jenkins job?



+1

Turns out there was a Jenkins job, just not on the Geronimo view.  Its there now.
 

- There's only tests for OWB.  Would anyone be against adding tests for Weld?



+0 while it stays simple (not the same hierarchy DS has for instance but think it is doable)


I was hoping it could be as simple as https://github.com/hammock-project/hammock-fault-tolerance/blob/master/pom.xml#L81-L165 (which itself is a WIP, and not working 100%)
 

- There's still one failing TCK assertion, 


Tests run: 36, Failures: 1, Errors: 0, Skipped: 0, Time elapsed: 1.584 sec <<< FAILURE! - in TestSuite

can_inject_simple_values_when_defined(org.eclipse.microprofile.config.tck.CDIPlainInjectionTest)  Time elapsed: 0.023 sec  <<< FAILURE!

java.lang.NumberFormatException: For input string: "org.eclipse.microprofile.config.configproperty.unconfigureddvalue"

at org.eclipse.microprofile.config.tck.CDIPlainInjectionTest.getBeanOfType(CDIPlainInjectionTest.java:110)

at org.eclipse.microprofile.config.tck.CDIPlainInjectionTest.can_inject_simple_values_when_defined(CDIPlainInjectionTest.java:67)




I dug into this one a bit more.  The ServiceLoader mechanism is choking on the inner class for the config source.  It doesn't find it even though it's there as a service provider.  The TCK test is a little weird.  I'm inclined to say the TCK should be changed, as reaching into a public static class across the JARs is a bit weird.
 

Ok, I got this one working.  Turns out it was more issues with handling of default values.  At least locally now the TCK passes 100%.  However, Jenkins fails because of the issue I noted above.

Awesome!  A while back I asked about how the CI was setup.  Were you able to find any docs on details about the CI pipeline?  I’m happy to start some seed docs.


Regards,
Alan


Reply | Threaded
Open this post in threaded view
|  
Report Content as Inappropriate

Re: CI for Geronimo Config

Mark Struberg


> Am 06.07.2017 um 01:19 schrieb Alan Cabrera <[hidden email]>:
>
>>
>> Ok, I got this one working.  Turns out it was more issues with handling of default values.  At least locally now the TCK passes 100%.  However, Jenkins fails because of the issue I noted above.
>
> Awesome!  A while back I asked about how the CI was setup.  Were you able to find any docs on details about the CI pipeline?  I’m happy to start some seed docs.


It's actually a bug in OpenWebBeans. It's now fixed and Romain upgraded to the 1.7.4-SNAPSHOT.

In OWB we create subclassing proxies and by default use the exact same package and ClassLoader. This is needed as CDI also requires to proxy package scope and protected methods. This is contrary to EJB which only requires to proxy 'Business Methods', which by definition must be public.

Romain added a check. If a signer is not null then you are in a signed and sealed jar. And then we need to use a different package name for the generated proxy classes.
We will release OWB-1.7.4 soon.

Btw, Weld has the same bug it seems.

LieGrue,
strub

Loading...