Re: [PR] TOMEE-4166 - Setup tck for JAX-RS (tomee)

2024-04-23 Thread via GitHub
rzo1 commented on PR #1063: URL: https://github.com/apache/tomee/pull/1063#issuecomment-2071591900 Thanks @Sollder1 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To

Re: [PR] TOMEE-4166 - Setup tck for JAX-RS (tomee)

2024-04-23 Thread via GitHub
rzo1 merged PR #1063: URL: https://github.com/apache/tomee/pull/1063 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: dev-unsubscr...@tomee.apache.org

Re: [PR] TOMEE-4166 - Setup tck for JAX-RS (tomee)

2024-04-22 Thread via GitHub
rzo1 commented on PR #1063: URL: https://github.com/apache/tomee/pull/1063#issuecomment-2068705126 Just running a build with the updated TCK (the 3.1.5 isn't available yet on Maven, raised https://github.com/jakartaee/rest/issues/1260, but if the CI looks fine (with the failures expected),

Re: [PR] TOMEE-4166 - Setup tck for JAX-RS (tomee)

2024-04-20 Thread via GitHub
Sollder11 commented on PR #1063: URL: https://github.com/apache/tomee/pull/1063#issuecomment-2067721440 > We are a container env, so I guess it's save to exclude ;-) I would agree, I excluded the test with a matching explanation. I also exluded the tck-sig-test for now, as it

Re: [PR] TOMEE-4166 - Setup tck for JAX-RS (tomee)

2024-04-20 Thread via GitHub
rzo1 commented on PR #1063: URL: https://github.com/apache/tomee/pull/1063#issuecomment-2067711072 We are a container env, so I guess it's save to exclude ;-) -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL

Re: [PR] TOMEE-4166 - Setup tck for JAX-RS (tomee)

2024-04-20 Thread via GitHub
Sollder11 commented on PR #1063: URL: https://github.com/apache/tomee/pull/1063#issuecomment-2067709937 Results are pretty much the same, for the matter of SeBootstrap it states in the jdoc: > The SeBootstrap class is available in a Jakarta EE container environment as well; however,

Re: [PR] TOMEE-4166 - Setup tck for JAX-RS (tomee)

2024-04-20 Thread via GitHub
Sollder11 commented on PR #1063: URL: https://github.com/apache/tomee/pull/1063#issuecomment-2067703137 > I am bascially thinking of adding the failing tests into exclusions (for now) and open related Jira issues for it, so we get the setup into "main" pretty soon. Ok cool, I just

Re: [PR] TOMEE-4166 - Setup tck for JAX-RS (tomee)

2024-04-20 Thread via GitHub
rzo1 commented on PR #1063: URL: https://github.com/apache/tomee/pull/1063#issuecomment-2067685958 I am bascially thinking of adding the failing tests into exclusions (for now) and open related Jira issues for it, so we get the setup into "main" pretty soon. -- This is an automated

Re: [PR] TOMEE-4166 - Setup tck for JAX-RS (tomee)

2024-04-20 Thread via GitHub
rzo1 commented on PR #1063: URL: https://github.com/apache/tomee/pull/1063#issuecomment-2067685493 Can you do a rebase of your branch? `main` is now on cxf 4.1.x + Java 17 baseline (with all fixes needed for it). I will have a look at the TCK next. -- This is an automated message from

Re: [PR] TOMEE-4166 - Setup tck for JAX-RS (tomee)

2024-04-16 Thread via GitHub
Sollder11 commented on PR #1063: URL: https://github.com/apache/tomee/pull/1063#issuecomment-2059735942 It seems like the guys from cxf have a 4.1.0-SNAPSHOT build and deployed here: https://repository.apache.org/content/groups/public/org/apache/cxf/cxf-core/4.1.0-SNAPSHOT/ It seems

Re: [PR] TOMEE-4166 - Setup tck for JAX-RS (tomee)

2024-01-19 Thread via GitHub
Sollder11 commented on PR #1063: URL: https://github.com/apache/tomee/pull/1063#issuecomment-1900042452 Sounds good, I will try another build later. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go

Re: [PR] TOMEE-4166 - Setup tck for JAX-RS (tomee)

2024-01-19 Thread via GitHub
rzo1 commented on PR #1063: URL: https://github.com/apache/tomee/pull/1063#issuecomment-1899977038 They are still in the process of working on it, but with Jetty 12 another obstacle is tackled here: https://github.com/apache/cxf/pull/1633 -- This is an automated message from the Apache

Re: [PR] TOMEE-4166 - Setup tck for JAX-RS (tomee)

2023-11-08 Thread via GitHub
Sollder11 commented on PR #1063: URL: https://github.com/apache/tomee/pull/1063#issuecomment-1802406913 Ok, I ran the tests locally with an apache-cxf build from here https://github.com/apache/cxf/tree/CXF-8671 No changes to the results, the same tests work and fail as before. -- This

Re: [PR] TOMEE-4166 - Setup tck for JAX-RS (tomee)

2023-11-08 Thread via GitHub
Sollder11 commented on PR #1063: URL: https://github.com/apache/tomee/pull/1063#issuecomment-1802330456 Hey @rzo1 That sounds great, I merged the changes and will look at the 4.1.0 asap. It seems like they did not yet build a version, but I will try to build it from sources myself. --

Re: [PR] TOMEE-4166 - Setup tck for JAX-RS (tomee)

2023-11-08 Thread via GitHub
rzo1 commented on PR #1063: URL: https://github.com/apache/tomee/pull/1063#issuecomment-1801792168 It also looks like the CXF guys have a 4.1.0-SNAPSHOT targeting EE10 (https://github.com/apache/cxf/pull/1201#issuecomment-1788967288) available, so perhaps worth a try to upgrade within and

Re: [PR] TOMEE-4166 - Setup tck for JAX-RS (tomee)

2023-11-07 Thread via GitHub
rzo1 commented on PR #1063: URL: https://github.com/apache/tomee/pull/1063#issuecomment-1801263043 Hey @Sollder11 , just merged the EE10 API branch, etc. - can you do a rebase? ;-) -- This is an automated message from the Apache Git Service. To respond to the message, please log on to

Re: [PR] TOMEE-4166 - Setup tck for JAX-RS (tomee)

2023-10-03 Thread via GitHub
Sollder11 commented on code in PR #1063: URL: https://github.com/apache/tomee/pull/1063#discussion_r1344197915 ## tck/jax-rs-signature-test/pom.xml: ## @@ -0,0 +1,92 @@ + + +http://maven.apache.org/POM/4.0.0; + xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance; +

Re: [PR] TOMEE-4166 - Setup tck for JAX-RS (tomee)

2023-10-03 Thread via GitHub
rzo1 commented on PR #1063: URL: https://github.com/apache/tomee/pull/1063#issuecomment-1744855945 I wrote a mail to dev@ related to the owb4 work and how to proceed with our ee10 efforts without breaking to much (as using ee10 apis + cdi 4 has some drawbacks in other dependencies as well)

Re: [PR] TOMEE-4166 - Setup tck for JAX-RS (tomee)

2023-10-03 Thread via GitHub
rzo1 commented on code in PR #1063: URL: https://github.com/apache/tomee/pull/1063#discussion_r1344009295 ## tck/jax-rs-signature-test/pom.xml: ## @@ -0,0 +1,92 @@ + + +http://maven.apache.org/POM/4.0.0; + xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance; +

Re: [PR] TOMEE-4166 - Setup tck for JAX-RS (tomee)

2023-10-03 Thread via GitHub
rzo1 commented on PR #1063: URL: https://github.com/apache/tomee/pull/1063#issuecomment-1744849665 No, not yet, see https://issues.apache.org/jira/plugins/servlet/mobile#issue/CXF-8671 I have a branch with EE10 apis + owb4, which also suffers from EE10 removals in the CXF field ;-)

Re: [PR] TOMEE-4166 - Setup tck for JAX-RS (tomee)

2023-10-03 Thread via GitHub
Sollder11 commented on PR #1063: URL: https://github.com/apache/tomee/pull/1063#issuecomment-1744826566 Hi Richard, I added another module for the Signature-test. It seems to run fine and output a sig-file similar to that generated by cdi-signature-test. Any news from the cxf Team about a

Re: [PR] TOMEE-4166 - Setup tck for JAX-RS (tomee)

2023-09-06 Thread via GitHub
rzo1 commented on PR #1063: URL: https://github.com/apache/tomee/pull/1063#issuecomment-1709033861 Perhaps time to sent a mail to dev@  - looks like those tests are all EE10/JAX-RS 3.1 related. -- This is an automated message from the Apache Git Service. To respond to the message,

Re: [PR] TOMEE-4166 - Setup tck for JAX-RS (tomee)

2023-09-06 Thread via GitHub
Sollder11 commented on PR #1063: URL: https://github.com/apache/tomee/pull/1063#issuecomment-1709000609 Fixed the issue with the user, they go through now... interestingly enough, the config necessary has almost nothing to do with what is described in the documentation. My last run now

Re: [PR] TOMEE-4166 - Setup tck for JAX-RS (tomee)

2023-09-06 Thread via GitHub
rzo1 commented on PR #1063: URL: https://github.com/apache/tomee/pull/1063#issuecomment-1708911182 So... just added another property so it looks like ```bash [ERROR] Failures: [ERROR] JAXRSClientIT.:59->usersetup:97 user not set ==> expected: but was: [ERROR]

Re: [PR] TOMEE-4166 - Setup tck for JAX-RS (tomee)

2023-09-06 Thread via GitHub
Sollder11 commented on PR #1063: URL: https://github.com/apache/tomee/pull/1063#issuecomment-1708904761 >The issue was related to source/jar building in the module, which we actually do not need. Oh, I see, makes sense. >So for me it now looks like this: Same >I

Re: [PR] TOMEE-4166 - Setup tck for JAX-RS (tomee)

2023-09-06 Thread via GitHub
rzo1 commented on PR #1063: URL: https://github.com/apache/tomee/pull/1063#issuecomment-170377 The issue was related to source/jar building in the module, which we actually do not need. So for me it now looks like this: ```bash [ERROR] Tests run: 2796, Failures:

Re: [PR] TOMEE-4166 - Setup tck for JAX-RS (tomee)

2023-09-06 Thread via GitHub
Sollder11 commented on PR #1063: URL: https://github.com/apache/tomee/pull/1063#issuecomment-1708863178 Hmm this failure is weird, it only seems to appear if you run the whole build, building it alone is no problem. I tried changing the name to jaxrs (so no dashes) and it compiles... --

Re: [PR] TOMEE-4166 - Setup tck for JAX-RS (tomee)

2023-09-06 Thread via GitHub
rzo1 commented on code in PR #1063: URL: https://github.com/apache/tomee/pull/1063#discussion_r1317633503 ## tck/jax-rs/pom.xml: ## @@ -0,0 +1,243 @@ + +http://maven.apache.org/POM/4.0.0; + xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance; +

Re: [PR] TOMEE-4166 - Setup tck for JAX-RS (tomee)

2023-09-06 Thread via GitHub
Sollder11 commented on PR #1063: URL: https://github.com/apache/tomee/pull/1063#issuecomment-1708825653 I think the ci does not like the snapshots tag either... maybe I will just remove it? -- This is an automated message from the Apache Git Service. To respond to the message, please

Re: [PR] TOMEE-4166 - Setup tck for JAX-RS (tomee)

2023-09-06 Thread via GitHub
rzo1 commented on code in PR #1063: URL: https://github.com/apache/tomee/pull/1063#discussion_r1317622638 ## tck/jax-rs/pom.xml: ## @@ -0,0 +1,243 @@ + +http://maven.apache.org/POM/4.0.0; + xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance; +

Re: [PR] TOMEE-4166 - Setup tck for JAX-RS (tomee)

2023-09-06 Thread via GitHub
rzo1 commented on PR #1063: URL: https://github.com/apache/tomee/pull/1063#issuecomment-1708814301 Guess we need to add these users ![image](https://github.com/apache/tomee/assets/13417392/2ce5cf97-ebbb-418e-b9a4-f0739f27cbd3) as written in the JAX-RS TCK docs. -- This is

Re: [PR] TOMEE-4166 - Setup tck for JAX-RS (tomee)

2023-09-06 Thread via GitHub
Sollder11 commented on code in PR #1063: URL: https://github.com/apache/tomee/pull/1063#discussion_r1317618150 ## tck/jax-rs/pom.xml: ## @@ -0,0 +1,243 @@ + +http://maven.apache.org/POM/4.0.0; + xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance; +

Re: [PR] TOMEE-4166 - Setup tck for JAX-RS (tomee)

2023-09-06 Thread via GitHub
Sollder11 commented on code in PR #1063: URL: https://github.com/apache/tomee/pull/1063#discussion_r1317617021 ## tck/jax-rs/pom.xml: ## @@ -0,0 +1,243 @@ + +http://maven.apache.org/POM/4.0.0; + xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance; +

Re: [PR] TOMEE-4166 - Setup tck for JAX-RS (tomee)

2023-09-06 Thread via GitHub
Sollder11 commented on code in PR #1063: URL: https://github.com/apache/tomee/pull/1063#discussion_r1317614809 ## tck/jax-rs/pom.xml: ## @@ -0,0 +1,243 @@ + +http://maven.apache.org/POM/4.0.0; + xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance; +

Re: [PR] TOMEE-4166 - Setup tck for JAX-RS (tomee)

2023-09-06 Thread via GitHub
Sollder11 commented on code in PR #1063: URL: https://github.com/apache/tomee/pull/1063#discussion_r1317613424 ## tck/jax-rs/pom.xml: ## @@ -0,0 +1,243 @@ + +http://maven.apache.org/POM/4.0.0; + xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance; +

Re: [PR] TOMEE-4166 - Setup tck for JAX-RS (tomee)

2023-09-06 Thread via GitHub
Sollder11 commented on code in PR #1063: URL: https://github.com/apache/tomee/pull/1063#discussion_r1317613424 ## tck/jax-rs/pom.xml: ## @@ -0,0 +1,243 @@ + +http://maven.apache.org/POM/4.0.0; + xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance; +

Re: [PR] TOMEE-4166 - Setup tck for JAX-RS (tomee)

2023-09-06 Thread via GitHub
rzo1 commented on code in PR #1063: URL: https://github.com/apache/tomee/pull/1063#discussion_r1317613343 ## tck/jax-rs/pom.xml: ## @@ -0,0 +1,243 @@ + +http://maven.apache.org/POM/4.0.0; + xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance; +

Re: [PR] TOMEE-4166 - Setup tck for JAX-RS (tomee)

2023-09-06 Thread via GitHub
Sollder11 commented on code in PR #1063: URL: https://github.com/apache/tomee/pull/1063#discussion_r1317612770 ## tck/jax-rs/pom.xml: ## @@ -0,0 +1,243 @@ + +http://maven.apache.org/POM/4.0.0; + xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance; +

Re: [PR] TOMEE-4166 - Setup tck for JAX-RS (tomee)

2023-09-06 Thread via GitHub
Sollder11 commented on code in PR #1063: URL: https://github.com/apache/tomee/pull/1063#discussion_r1317602778 ## tck/jax-rs/pom.xml: ## @@ -0,0 +1,243 @@ + +http://maven.apache.org/POM/4.0.0; + xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance; +

Re: [PR] TOMEE-4166 - Setup tck for JAX-RS (tomee)

2023-09-06 Thread via GitHub
Sollder11 commented on code in PR #1063: URL: https://github.com/apache/tomee/pull/1063#discussion_r1317601738 ## tck/jax-rs/pom.xml: ## @@ -0,0 +1,243 @@ + +http://maven.apache.org/POM/4.0.0; + xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance; +

Re: [PR] TOMEE-4166 - Setup tck for JAX-RS (tomee)

2023-09-06 Thread via GitHub
Sollder11 commented on code in PR #1063: URL: https://github.com/apache/tomee/pull/1063#discussion_r1317600602 ## tck/jax-rs/pom.xml: ## @@ -0,0 +1,243 @@ + +http://maven.apache.org/POM/4.0.0; + xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance; +

Re: [PR] TOMEE-4166 - Setup tck for JAX-RS (tomee)

2023-09-06 Thread via GitHub
Sollder11 commented on code in PR #1063: URL: https://github.com/apache/tomee/pull/1063#discussion_r1317585330 ## tck/jax-rs/pom.xml: ## @@ -0,0 +1,243 @@ + +http://maven.apache.org/POM/4.0.0; + xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance; +

Re: [PR] TOMEE-4166 - Setup tck for JAX-RS (tomee)

2023-09-06 Thread via GitHub
Sollder11 commented on code in PR #1063: URL: https://github.com/apache/tomee/pull/1063#discussion_r1317584569 ## tck/jax-rs/pom.xml: ## @@ -0,0 +1,243 @@ + +http://maven.apache.org/POM/4.0.0; + xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance; +