Excellent! I can prepare a RC2-SNAPSHOT for quick testing while we prepare for the actual release.
We have a MyFaces update pending. Does anyone object if we move forward with the current state of the code? Cheers, Roberto > On 9 Oct 2018, at 19:54, Romain Manni-Bucau <rmannibu...@gmail.com> wrote: > > Yes, thks Roberto. > > Le mar. 9 oct. 2018 20:44, Roberto Cortez <radcor...@yahoo.com.invalid> a > écrit : > >> Is this what you had in mind? >> >> https://github.com/apache/tomee/commit/795fa40bc76581a1a923805527a780e02c9bf91f >> < >> https://github.com/apache/tomee/commit/795fa40bc76581a1a923805527a780e02c9bf91f >>> >> >> Cheers, >> Roberto >> >>> On 9 Oct 2018, at 05:20, Romain Manni-Bucau <rmannibu...@gmail.com> >> wrote: >>> >>> Le mar. 9 oct. 2018 01:56, Roberto Cortez <radcor...@yahoo.com.invalid> >> a >>> écrit : >>> >>>> Romain, you mean here: >>>> >>>> >> https://github.com/radcortez/tomee/blob/426e0c14fede5ee5907e43e1c46e2fd51c904412/tomee/tomee-catalina/src/main/java/org/apache/tomee/catalina/TomcatWebAppBuilder.java#L1772-L1774 >> ? >>>> < >>>> >> https://github.com/radcortez/tomee/blob/426e0c14fede5ee5907e43e1c46e2fd51c904412/tomee/tomee-catalina/src/main/java/org/apache/tomee/catalina/TomcatWebAppBuilder.java#L1772-L1774 >>>> ?> >>>> >>>> Well, why hope there is a single one? :) In case of EAR, shouldn’t all >> of >>>> them be started? >>>> >>> >>> Yes but not here. But this code is fragile and relies on the fact it is >>> already started and this line skipped. This method is "start one webapp" >> so >>> we can just filter the one this lethod must start and skip others. Then >>> your fix is hurtless if there is any deployment change, you dont use >>> default deployment lifecycle and the codebase is more robust. >>> >>> >>> >>>> David, >>>> >>>> After the RC release, we detected that issue with the MP binary and the >>>> way some implementations are accessing the context. I did sent a >> tentative >>>> fix, that fixes that issue, but Romain things that it may cause others, >> so >>>> we are trying to figure out what tests are we missing or if the code >> needs >>>> to change. You can help reviewing it as well. >>>> >>>> Cheers, >>>> Roberto >>>> >>>>> On 8 Oct 2018, at 21:50, David Blevins <david.blev...@gmail.com> >> wrote: >>>>> >>>>> I shot a note out to bval asking of there's a chance of getting a >>>> release this week. >>>>> >>>>> Assuming that's possible, is there anything standing in our way for >>>> putting up a release vote ourselves this week? >>>>> >>>>> >>>>> -David >>>>> >>>>>> On Oct 8, 2018, at 8:28 AM, Romain Manni-Bucau <rmannibu...@gmail.com >>> >>>> wrote: >>>>>> >>>>>> one option can be to start only one webapp here instead of all and >> hope >>>>>> there is a single one or others are skipped ;) >>>>>> >>>>>> Romain Manni-Bucau >>>>>> @rmannibucau <https://twitter.com/rmannibucau> | Blog >>>>>> <https://rmannibucau.metawerx.net/> | Old Blog >>>>>> <http://rmannibucau.wordpress.com> | Github < >>>> https://github.com/rmannibucau> | >>>>>> LinkedIn <https://www.linkedin.com/in/rmannibucau> | Book >>>>>> < >>>> >> https://www.packtpub.com/application-development/java-ee-8-high-performance >>>>> >>>>>> >>>>>> >>>>>> Le lun. 8 oct. 2018 à 16:15, Roberto Cortez >> <radcor...@yahoo.com.invalid> >>>> a >>>>>> écrit : >>>>>> >>>>>>> Ok, thanks. Can you help to better test it? >>>>>>> >>>>>>> Cheers, >>>>>>> Roberto >>>>>>> >>>>>>>> On 6 Oct 2018, at 08:26, Romain Manni-Bucau <rmannibu...@gmail.com> >>>>>>> wrote: >>>>>>>> >>>>>>>> Le sam. 6 oct. 2018 00:30, Roberto Cortez >> <radcor...@yahoo.com.invalid >>>>> >>>>>>> a >>>>>>>> écrit : >>>>>>>> >>>>>>>>> Would something like this work? >>>>>>>>> >>>>>>>>> >>>>>>> >>>> >> https://github.com/apache/tomee/commit/ea13f63f21d7f06388e2c54d70120a5a98e4c65f >>>>>>>>> < >>>>>>>>> >>>>>>> >>>> >> https://github.com/apache/tomee/commit/ea13f63f21d7f06388e2c54d70120a5a98e4c65f >>>>>>>>>> >>>>>>>>> >>>>>>>>> I’m not exactly sure what the problem might be with EAR. Web >> Modules, >>>>>>> seem >>>>>>>>> to also be deployed like this, so they suffer from the same issue. >>>> After >>>>>>>>> the JAX-RS app is started the context is removed. >>>>>>>>> >>>>>>>> >>>>>>>> >>>>>>>> You set a single context for all webapps so code is quite miskeading >>>> and >>>>>>>> error prone. >>>>>>>> >>>>>>>> >>>>>>>>> Please let me know what other situations you have in mind that may >>>> cause >>>>>>>>> issues? >>>>>>>>> >>>>>>>>> Cheers, >>>>>>>>> Roberto >>>>>>>>> >>>>>>>>>> On 4 Oct 2018, at 16:05, Roberto Cortez >> <radcor...@yahoo.com.INVALID >>>>> >>>>>>>>> wrote: >>>>>>>>>> >>>>>>>>>> I understand. Was just trying to give more detail into it. >>>>>>>>>> >>>>>>>>>> I’ll have a better look and try to come up with some test >> scenarios. >>>>>>>>>> >>>>>>>>>>> On 4 Oct 2018, at 10:47, Romain Manni-Bucau < >> rmannibu...@gmail.com >>>>> >>>>>>>>> wrote: >>>>>>>>>>> >>>>>>>>>>> Le jeu. 4 oct. 2018 à 11:42, Roberto Cortez >>>>>>> <radcor...@yahoo.com.invalid >>>>>>>>> <mailto:radcor...@yahoo.com.invalid>> a >>>>>>>>>>> écrit : >>>>>>>>>>> >>>>>>>>>>>> Hi Romain, >>>>>>>>>>>> >>>>>>>>>>>> Well the exception being thrown is not the actual exception. >>>>>>>>>>>> >>>>>>>>>>>> This was only happening in the MP binary due to the OpenAPI >>>> Geronimo >>>>>>>>>>>> implementation. In the DefaultLoader the ServletContext is >>>> injected, >>>>>>>>> but at >>>>>>>>>>>> the time that the JAX-RS app is deployed, which is in the >>>>>>>>>>>> AfterApplicationCreated event, the ServletContextHandler does >> not >>>>>>> have >>>>>>>>> a >>>>>>>>>>>> Context anymore so it throws a IllegalStateException("Didnt >> find a >>>>>>> web >>>>>>>>>>>> context for " + contextClassLoader). The caller for this is the >>>>>>>>>>>> setApplication of the OpenAPIFilter when we try to inject it, so >>>> that >>>>>>>>>>>> causes the exception we see in the logs. >>>>>>>>>>>> >>>>>>>>>>>> We never say this in Arquillian testing, because Arquillian >> waits >>>> for >>>>>>>>> the >>>>>>>>>>>> server to start and then deploys the app. This means we are able >>>> to >>>>>>>>> get a >>>>>>>>>>>> ServletContext from the request in ServletContextHandler, so it >>>> works >>>>>>>>> fine. >>>>>>>>>>>> >>>>>>>>>>>> I believe this is also related with the fix you did here: >>>>>>>>>>>> https://issues.apache.org/jira/browse/TOMEE-1687 < >>>>>>>>>>>> https://issues.apache.org/jira/browse/TOMEE-1687> >>>>>>>>>>>> >>>>>>>>>>>> What do you think? >>>>>>>>>>>> >>>>>>>>>>> >>>>>>>>>>> Can be but at the end the two issues are mentionned are not >>>> covered: >>>>>>>>>>> >>>>>>>>>>> 1. a regression will silently come back >>>>>>>>>>> 2. for ears we can leak the servlet context if we end up here >> (and >>>>>>> then >>>>>>>>> the >>>>>>>>>>> app will wrongly behave) >>>>>>>>>>> >>>>>>>>>>> My point is not to revert what you did but more to ensure it >> fixes >>>> the >>>>>>>>>>> issue in our build. >>>>>>>>>>> >>>>>>>>>>> >>>>>>>>>>>> >>>>>>>>>>>> Cheers, >>>>>>>>>>>> Roberto >>>>>>>>>>>> >>>>>>>>>>>>> On 4 Oct 2018, at 08:23, Romain Manni-Bucau < >>>> rmannibu...@gmail.com> >>>>>>>>>>>> wrote: >>>>>>>>>>>>> >>>>>>>>>>>>> @Roberto: do we have a test to reproduce and prevent future >>>>>>>>> regressions? >>>>>>>>>>>> If >>>>>>>>>>>>> your fix is right the error message is quite unexpected so it >>>> would >>>>>>> be >>>>>>>>>>>>> better to ensure we don't break it unintentionnally (a side >> note >>>> is >>>>>>>>> that >>>>>>>>>>>>> setting a single webapp context and firing an event for all >>>> webapps >>>>>>>>> can >>>>>>>>>>>>> have the same pitfall so it can need to be reworked to ensure >> we >>>>>>> don't >>>>>>>>>>>> hit >>>>>>>>>>>>> it for ears to not leak context between webapp which would be >>>>>>> another >>>>>>>>>>>> nasty >>>>>>>>>>>>> bug). >>>>>>>>>>>>> >>>>>>>>>>>>> Romain Manni-Bucau >>>>>>>>>>>>> @rmannibucau <https://twitter.com/rmannibucau> | Blog >>>>>>>>>>>>> <https://rmannibucau.metawerx.net/> | Old Blog >>>>>>>>>>>>> <http://rmannibucau.wordpress.com> | Github < >>>>>>>>>>>> https://github.com/rmannibucau> | >>>>>>>>>>>>> LinkedIn <https://www.linkedin.com/in/rmannibucau> | Book >>>>>>>>>>>>> < >>>>>>>>>>>> >>>>>>>>> >>>>>>> >>>> >> https://www.packtpub.com/application-development/java-ee-8-high-performance >>>>>>>>>>>>> >>>>>>>>>>>>> >>>>>>>>>>>>> >>>>>>>>>>>>> Le jeu. 4 oct. 2018 à 01:43, Roberto Cortez >>>>>>>>> <radcor...@yahoo.com.invalid> >>>>>>>>>>>> a >>>>>>>>>>>>> écrit : >>>>>>>>>>>>> >>>>>>>>>>>>>> Hi Cesar, >>>>>>>>>>>>>> >>>>>>>>>>>>>> I think I’ve found the issue. Just pushed a fix and now >> waiting >>>> to >>>>>>>>> the >>>>>>>>>>>>>> build bot to check if everything is ok. I’ll let you know when >>>> it >>>>>>> is >>>>>>>>>>>> done. >>>>>>>>>>>>>> >>>>>>>>>>>>>> Cheers, >>>>>>>>>>>>>> Roberto >>>>>>>>>>>>>> >>>>>>>>>>>>>>> On 3 Oct 2018, at 22:50, Roberto Cortez >>>>>>> <radcor...@yahoo.com.INVALID >>>>>>>>>> >>>>>>>>>>>>>> wrote: >>>>>>>>>>>>>>> >>>>>>>>>>>>>>> >>>>>>>>>>>>>>> Not sure. Let me have a look. On Wednesday, October 3, >> 2018, >>>>>>>>> 8:56:13 >>>>>>>>>>>>>> PM GMT+1, César Hernández Mendoza <cesargu...@gmail.com> >> wrote: >>>>>>>>>>>>>>> >>>>>>>>>>>>>>> Hi everyone, >>>>>>>>>>>>>>> I have a simple REST app that is working fine with >>>> tomee.version >>>>>>>>>>>> 7.1.0, >>>>>>>>>>>>>> but tomee:run it's getting issues after updating it to >>>>>>>>>>>> 8.0.0-RC1-SNAPSHOT >>>>>>>>>>>>>> and also update the javaee-api to 8.0-SNAPSHOT. mvn test runs >>>> fine >>>>>>>>> but >>>>>>>>>>>>>> when I try to run mvn clean install tomee:run I got a cxf >> error >>>> (I >>>>>>>>>>>> attached >>>>>>>>>>>>>> the full log file to this email). >>>>>>>>>>>>>>> 03-Oct-2018 11:54:26.368 SEVERE [main] >>>>>>>>>>>>>> org.apache.cxf.jaxrs.utils.InjectionUtils.reportServerError >>>> Method >>>>>>>>>>>>>> setApplication can not be accessed due to security manager >>>>>>>>>>>>>> restrictions03-Oct-2018 11:54:26.378 SEVERE [main] >>>>>>>>>>>>>> >>>> org.apache.openejb.observer.ObserverManager$MethodInvocation.invoke >>>>>>>>>>>> error >>>>>>>>>>>>>> invoking >> org.apache.tomee.webservices.TomeeJaxRsService@7ea37dbf >>>>>>>>>>>>>>> Any idea why this is happening with the RC1? >>>>>>>>>>>>>>> El mar., 2 oct. 2018 a las 14:04, Roberto Cortez >>>>>>>>>>>>>> (<radcor...@yahoo.com.invalid>) escribió: >>>>>>>>>>>>>>> >>>>>>>>>>>>>>> Hi Matthew, >>>>>>>>>>>>>>> >>>>>>>>>>>>>>> Feel free to try it. Yes, no planned changes, unless >> something >>>>>>>>> required >>>>>>>>>>>>>> fixing. >>>>>>>>>>>>>>> >>>>>>>>>>>>>>> Cheers, >>>>>>>>>>>>>>> Roberto >>>>>>>>>>>>>>> >>>>>>>>>>>>>>>> On 2 Oct 2018, at 16:37, Matthew Broadhead < >>>>>>>>>>>> matthew.broadh...@nbmlaw.co.uk.INVALID> >>>>>>>>>>>>>> wrote: >>>>>>>>>>>>>>>> >>>>>>>>>>>>>>>> should i try out the RC1 or wait for release candidate? i >>>> guess >>>>>>>>> they >>>>>>>>>>>>>> will be the same unless something needs upgrading? >>>>>>>>>>>>>>>> >>>>>>>>>>>>>>>> On 02/10/18 12:18, Roberto Cortez wrote: >>>>>>>>>>>>>>>>> I guess it would be ASAP :) >>>>>>>>>>>>>>>>> >>>>>>>>>>>>>>>>> So if anyone wants to have anything in, now its the time. >>>> Speak >>>>>>>>> now >>>>>>>>>>>> or >>>>>>>>>>>>>> forever hold your peace :) >>>>>>>>>>>>>>>>> >>>>>>>>>>>>>>>>> Cheers, >>>>>>>>>>>>>>>>> Roberto >>>>>>>>>>>>>>>>> >>>>>>>>>>>>>>>>>> On 2 Oct 2018, at 00:16, David Blevins < >>>>>>> david.blev...@gmail.com> >>>>>>>>>>>>>> wrote: >>>>>>>>>>>>>>>>>> >>>>>>>>>>>>>>>>>> Awesome. If you have a date in mind for the first release >>>>>>>>> attempt, >>>>>>>>>>>>>> definitely let people know so they can complete anything they >>>> may >>>>>>>>> want >>>>>>>>>>>> to >>>>>>>>>>>>>> get done. >>>>>>>>>>>>>>>>>> >>>>>>>>>>>>>>>>>> >>>>>>>>>>>>>>>>>> -- >>>>>>>>>>>>>>>>>> David Blevins >>>>>>>>>>>>>>>>>> http://twitter.com/dblevins >>>>>>>>>>>>>>>>>> http://www.tomitribe.com >>>>>>>>>>>>>>>>>> >>>>>>>>>>>>>>>>>>> On Oct 1, 2018, at 2:41 PM, Roberto Cortez >>>>>>>>>>>>>> <radcor...@yahoo.com.INVALID> wrote: >>>>>>>>>>>>>>>>>>> >>>>>>>>>>>>>>>>>>> Hi David, >>>>>>>>>>>>>>>>>>> >>>>>>>>>>>>>>>>>>> Thank you. >>>>>>>>>>>>>>>>>>> >>>>>>>>>>>>>>>>>>> I do intent to keep pushing with the release. Just doing >>>>>>> “this” >>>>>>>>>>>>>> already had me going through some stuff, which Jon helped me >>>> out. >>>>>>>>>>>>>>>>>>> >>>>>>>>>>>>>>>>>>> So consider it a first step :) >>>>>>>>>>>>>>>>>>> >>>>>>>>>>>>>>>>>>> Cheers, >>>>>>>>>>>>>>>>>>> Roberto >>>>>>>>>>>>>>>>>>> >>>>>>>>>>>>>>>>>>>> On 1 Oct 2018, at 19:28, David Blevins < >>>>>>>>> david.blev...@gmail.com> >>>>>>>>>>>>>> wrote: >>>>>>>>>>>>>>>>>>>> >>>>>>>>>>>>>>>>>>>> I think the spirit behind this is great. However, I >> don't >>>>>>>>> think >>>>>>>>>>>> it >>>>>>>>>>>>>> actually helps as the hardest part which will be for you to >>>> learn >>>>>>> the >>>>>>>>>>>>>> release process itself. Time wise you should budget 3 weeks. >>>> When >>>>>>>>> you >>>>>>>>>>>> do >>>>>>>>>>>>>> know what you're doing the process usually takes 2 weeks >>>> minimum as >>>>>>>>>>>> there's >>>>>>>>>>>>>> almost always at least one aborted vote and reroll. Given >> that, >>>>>>> the >>>>>>>>>>>>>> chances of a release by CodeOne are already slim. >>>>>>>>>>>>>>>>>>>> >>>>>>>>>>>>>>>>>>>> My advice would be to create a release branch and start >>>>>>> cutting >>>>>>>>>>>>>> actual binaries in nexus. >>>>>>>>>>>>>>>>>>>> >>>>>>>>>>>>>>>>>>>> -- >>>>>>>>>>>>>>>>>>>> David Blevins >>>>>>>>>>>>>>>>>>>> http://twitter.com/dblevins >>>>>>>>>>>>>>>>>>>> http://www.tomitribe.com >>>>>>>>>>>>>>>>>>>> >>>>>>>>>>>>>>>>>>>>> On Oct 1, 2018, at 10:57 AM, Roberto Cortez >>>>>>>>>>>>>> <radcor...@yahoo.com.INVALID> wrote: >>>>>>>>>>>>>>>>>>>>> >>>>>>>>>>>>>>>>>>>>> Hi everyone, >>>>>>>>>>>>>>>>>>>>> >>>>>>>>>>>>>>>>>>>>> Following up some of our recent emails, I did push a >>>>>>> snapshot >>>>>>>>>>>>>> version marked as RC of the upcoming TomEE 8 Release. >>>>>>>>>>>>>>>>>>>>> >>>>>>>>>>>>>>>>>>>>> Please, find the binaries here: >>>>>>>>>>>>>>>>>>>>> >>>>>>>>>>>>>> >>>>>>>>>>>> >>>>>>>>> >>>>>>> >>>> >> https://repository.apache.org/content/repositories/snapshots/org/apache/tomee/apache-tomee/8.0.0-RC1-SNAPSHOT/ >>>>>>>>>>>>>> < >>>>>>>>>>>>>> >>>>>>>>>>>> >>>>>>>>> >>>>>>> >>>> >> https://repository.apache.org/content/repositories/snapshots/org/apache/tomee/apache-tomee/8.0.0-RC1-SNAPSHOT/ >>>>>>>>>>>>>>> >>>>>>>>>>>>>>>>>>>>> >>>>>>>>>>>>>>>>>>>>> And remember to add the following repository to your >>>>>>> project: >>>>>>>>>>>>>>>>>>>>> <repositories> >>>>>>>>>>>>>>>>>>>>> <repository> >>>>>>>>>>>>>>>>>>>>> <id>apache.snapshots.https</id> >>>>>>>>>>>>>>>>>>>>> <name>Apache Development Snapshot Repository</name> >>>>>>>>>>>>>>>>>>>>> <url> >>>>>>>>>>>> https://repository.apache.org/content/repositories/snapshots >>>>>>>>>>>>>> </url> >>>>>>>>>>>>>>>>>>>>> </repository> >>>>>>>>>>>>>>>>>>>>> </repositories> >>>>>>>>>>>>>>>>>>>>> >>>>>>>>>>>>>>>>>>>>> It should be very close to the actual release. Please, >>>> help >>>>>>> up >>>>>>>>>>>>>> take it for a spin and try it out :) Thank you! >>>>>>>>>>>>>>>>>>>>> >>>>>>>>>>>>>>>>>>>>> Cheers, >>>>>>>>>>>>>>>>>>>>> Roberto >>>>>>>>>>>>>>>> >>>>>>>>>>>>>>> >>>>>>>>>>>>>>> >>>>>>>>>>>>>>> >>>>>>>>>>>>>>> >>>>>>>>>>>>>>> -- >>>>>>>>>>>>>>> Atentamente: >>>>>>>>>>>>>>> César Hernández Mendoza. >>>>>>>>>> >>>>>>>>> >>>>>>>>> >>>>>>> >>>>>>> >>>>> >>>> >>>> >> >>