You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@karaf.apache.org by "jbonofre (via GitHub)" <gi...@apache.org> on 2026/03/16 12:38:59 UTC

[PR] feat(jakarta): Upgrade to Pax Web 11.1.0 [karaf]

jbonofre opened a new pull request, #2427:
URL: https://github.com/apache/karaf/pull/2427

   Migrate from javax.servlet to jakarta.servlet, bump Java SE level from 11 to 17, remove the spring-legacy feature module and associated tests, and replace BundleActivator patterns with SCR/DS components in examples and HTTP modules.
   Remove spring-legacy features


-- 
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: commits-unsubscribe@karaf.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

Re: [PR] feat(jakarta): Upgrade to Pax Web 11.0.1 [karaf]

Posted by "jbonofre (via GitHub)" <gi...@apache.org>.
jbonofre commented on PR #2427:
URL: https://github.com/apache/karaf/pull/2427#issuecomment-4106222600

   The webconsole is now working fine with Pax Web 11.0.1 πŸ˜„ 
   
   I'm doing a quick pass and I will merge this PR.


-- 
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: commits-unsubscribe@karaf.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

Re: [PR] feat(jakarta): Upgrade to Pax Web 11.1.0 [karaf]

Posted by "jbonofre (via GitHub)" <gi...@apache.org>.
jbonofre commented on PR #2427:
URL: https://github.com/apache/karaf/pull/2427#issuecomment-4083844754

   @grgrzybek I have a green build, including the vast majority of tests and integration tests.
   
   I will now investigate the tests with `@Ignore` to fix them.


-- 
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: commits-unsubscribe@karaf.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

Re: [PR] feat(jakarta): Upgrade to Pax Web 11.0.1 [karaf]

Posted by "jbonofre (via GitHub)" <gi...@apache.org>.
jbonofre merged PR #2427:
URL: https://github.com/apache/karaf/pull/2427


-- 
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: commits-unsubscribe@karaf.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

Re: [PR] feat(jakarta): Upgrade to Pax Web 11.1.0 [karaf]

Posted by "jbonofre (via GitHub)" <gi...@apache.org>.
jbonofre commented on PR #2427:
URL: https://github.com/apache/karaf/pull/2427#issuecomment-4067467438

   @grgrzybek thanks. That's the first PR in a series that I have about jakarta migration πŸ˜„ 


-- 
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: commits-unsubscribe@karaf.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

Re: [PR] feat(jakarta): Upgrade to Pax Web 11.1.0 [karaf]

Posted by "grgrzybek (via GitHub)" <gi...@apache.org>.
grgrzybek commented on PR #2427:
URL: https://github.com/apache/karaf/pull/2427#issuecomment-4067459572

   @jbonofre check my 3 commits in this branch: https://github.com/grgrzybek/karaf/commits/pax-web-11/


-- 
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: commits-unsubscribe@karaf.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

Re: [PR] feat(jakarta): Upgrade to Pax Web 11.1.0 [karaf]

Posted by "github-actions[bot] (via GitHub)" <gi...@apache.org>.
github-actions[bot] commented on PR #2427:
URL: https://github.com/apache/karaf/pull/2427#issuecomment-4091044878

   ## Test Results
   238 files   -β€Šβ€‡5  238 suites   -β€Š5   23m 9s ⏱️ - 3m 47s
   930 tests  -β€Š63  878 βœ…  -β€Š70  52 πŸ’€ +7  0 ❌ Β±0 
   934 runsβ€Š  -β€Š63  882 βœ…  -β€Š70  52 πŸ’€ +7  0 ❌ Β±0 
   
   Results for commit 5d04a1e0. ± Comparison against base commit d026776b.
   
   <details>
     <summary>This pull request <b>removes</b> 65 and <b>adds</b> 2 tests. <i>Note that renamed tests count towards both.</i></summary>
   
   ```
   org.apache.karaf.itests.examples.RestExampleTest ‑ testBlueprintWithCxfClient
   org.apache.karaf.itests.examples.RestExampleTest ‑ testBlueprintWithHttpClient
   org.apache.karaf.itests.examples.RestExampleTest ‑ testScrWithCxfClient
   org.apache.karaf.itests.examples.RestExampleTest ‑ testScrWithHttpClient
   org.apache.karaf.itests.examples.RestExampleTest ‑ testScrWithJerseyClient
   org.apache.karaf.itests.examples.RestExampleTest ‑ testWhiteboard
   org.apache.karaf.itests.examples.SoapExampleTest ‑ testBlueprint
   org.apache.karaf.itests.examples.SoapExampleTest ‑ testScr
   org.apache.karaf.itests.features.Spring3FeaturesTest ‑ installSpring31Feature
   org.apache.karaf.itests.features.Spring3FeaturesTest ‑ installSpringAspects31Feature
   …
   ```
   
   ```
   org.apache.karaf.itests.examples.RestExampleTest ‑ Unknown test
   org.apache.karaf.itests.examples.SoapExampleTest ‑ Unknown test
   ```
   </details>
   
   <details>
     <summary>This pull request <b>removes</b> 2 skipped tests and <b>adds</b> 2 skipped tests. <i>Note that renamed tests count towards both.</i></summary>
   
   ```
   org.apache.karaf.itests.features.Spring3FeaturesTest ‑ installSpringStruts31Feature
   org.apache.karaf.itests.features.Spring60FeaturesTest ‑ Unknown test
   ```
   
   ```
   org.apache.karaf.itests.examples.RestExampleTest ‑ Unknown test
   org.apache.karaf.itests.examples.SoapExampleTest ‑ Unknown test
   ```
   </details>
   
   <details>
     <summary>This pull request <b>skips</b> 7 tests.</summary>
   
   ```
   org.apache.karaf.itests.XATest ‑ test
   org.apache.karaf.itests.examples.GraphQLExampleTest ‑ testWebSocket
   org.apache.karaf.itests.examples.WebSocketExampleTest ‑ test
   org.apache.karaf.itests.features.PaxCdiFeaturesTest ‑ installPaxCdiWebFeature
   org.apache.karaf.itests.features.PaxCdiFeaturesTest ‑ installPaxCdiWebOpenwebbeansFeature
   org.apache.karaf.itests.features.PaxCdiFeaturesTest ‑ installPaxCdiWebWeldFeature
   org.apache.karaf.itests.features.StandardFeaturesTest ‑ installAriesBlueprintWebFeature
   ```
   </details>
   
   [test-results]:data:application/gzip;base64,H4sIAPUXvGkC/03NSQ7CMAwF0KtUWbPISFwug6IMUkTboAwrxN0xpaTZ+T/L3y8S4uILuU1cwGUipcU6RNeyqTFtCEzAjILb+t3Pgv7TvTRrkUDDSY/4RFK8SzBxQTmvfM4pH5Lb9iuVR+idwLsMlTsMjXseC21a11gxEOWoNMxTqT3gqKW70iDR8Jli0jJnqANjyfsDlFjnsQwBAAA=
   


-- 
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: commits-unsubscribe@karaf.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

Re: [PR] feat(jakarta): Upgrade to Pax Web 11.1.0 [karaf]

Posted by "jbonofre (via GitHub)" <gi...@apache.org>.
jbonofre commented on PR #2427:
URL: https://github.com/apache/karaf/pull/2427#issuecomment-4105649019

   I'm checking a `StackOverflowError` in the webconsole.
   
   I will merge this PR as the build is fine and follow up with new PRs to incrementaly fix the issues.


-- 
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: commits-unsubscribe@karaf.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

Re: [PR] feat(jakarta): Upgrade to Pax Web 11.1.0 [karaf]

Posted by "jbonofre (via GitHub)" <gi...@apache.org>.
jbonofre commented on PR #2427:
URL: https://github.com/apache/karaf/pull/2427#issuecomment-4105673195

   OK, the `StackOverflowError` is due to a Jetty version mismatch between Felix WebConsole and Pax Web.
   
   Felix HTTP Jetty12 1.1.8 was compiled against Jetty 12.0.32, but Pax Web 11.1.0 deploys Jetty 12.1.4 at runtime. The `HttpFields$Mutable$Wrapper` API changed between the 12.0.x and 12.1.x branches, causing the infinite recursion:         
     `Wrapper.put() β†’ listIterator() β†’ wrapped iterator's set() β†’ onPutField() callback β†’ put() β†’ ...    `
   
   I'm evaluating if it's not safer to go with Pac Web 11.0.x to "align" Jetty version.


-- 
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: commits-unsubscribe@karaf.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org