You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@camel.apache.org by "Claus Ibsen (JIRA)" <ji...@apache.org> on 2010/01/04 19:55:15 UTC
[jira] Created: (CAMEL-2332) Upgrade to Jetty 7.0
Upgrade to Jetty 7.0
--------------------
Key: CAMEL-2332
URL: https://issues.apache.org/activemq/browse/CAMEL-2332
Project: Apache Camel
Issue Type: Task
Components: camel-jetty
Reporter: Claus Ibsen
Fix For: 2.3.0
Jetty 7.0 is out and we should upgrade to use that as well.
We should do this in the same release we upgrade to HTTP Client 4.0 as well so we upgrade both kinds in the same release. CAMEL-1530.
--
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.
[jira] Resolved: (CAMEL-2332) Upgrade to Jetty 7.0
Posted by "Willem Jiang (JIRA)" <ji...@apache.org>.
[ https://issues.apache.org/activemq/browse/CAMEL-2332?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]
Willem Jiang resolved CAMEL-2332.
---------------------------------
Resolution: Fixed
I think we can resolved this issue now.
> Upgrade to Jetty 7.0
> --------------------
>
> Key: CAMEL-2332
> URL: https://issues.apache.org/activemq/browse/CAMEL-2332
> Project: Apache Camel
> Issue Type: Task
> Components: camel-jetty
> Reporter: Claus Ibsen
> Assignee: Christian Mueller
> Fix For: 2.3.0
>
> Attachments: CAMEL-2332_20100309.patch
>
>
> Jetty 7.0 is out and we should upgrade to use that as well.
> We should do this in the same release we upgrade to HTTP Client 4.0 as well so we upgrade both kinds in the same release. CAMEL-1530.
--
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.
[jira] Commented: (CAMEL-2332) Upgrade to Jetty 7.0
Posted by "Willem Jiang (JIRA)" <ji...@apache.org>.
[ https://issues.apache.org/activemq/browse/CAMEL-2332?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=58215#action_58215 ]
Willem Jiang commented on CAMEL-2332:
-------------------------------------
Just a quick note for Jetty7 upgrading, as CXF bundle didn't upgrade to Jetty7, we still need to import the Jetty 6.x bundle in camel-cxf feature.
I will take care of it and running some test on the last camel features with ServiceMix.
> Upgrade to Jetty 7.0
> --------------------
>
> Key: CAMEL-2332
> URL: https://issues.apache.org/activemq/browse/CAMEL-2332
> Project: Apache Camel
> Issue Type: Task
> Components: camel-jetty
> Reporter: Claus Ibsen
> Assignee: Christian Mueller
> Fix For: 2.3.0
>
> Attachments: CAMEL-2332_20100309.patch
>
>
> Jetty 7.0 is out and we should upgrade to use that as well.
> We should do this in the same release we upgrade to HTTP Client 4.0 as well so we upgrade both kinds in the same release. CAMEL-1530.
--
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.
[jira] Commented: (CAMEL-2332) Upgrade to Jetty 7.0
Posted by "Christian Mueller (JIRA)" <ji...@apache.org>.
[ https://issues.apache.org/activemq/browse/CAMEL-2332?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=58314#action_58314 ]
Christian Mueller commented on CAMEL-2332:
------------------------------------------
I realized, that you changed the dependency from Jetty 7 back to Jetty 6 for camel-cxf.
{code}
commit b0a8ddc008ab3b48544c21094fd9de38ba0d3700
Author: Willem Ning Jiang <ni...@apache.org>
Date: Sat Mar 13 03:50:55 2010 +0000
CAMEL-2332 camel-cxf still need to use Jetty6 bundle
git-svn-id: https://svn.apache.org/repos/asf/camel/trunk@922478 13f79535-47bb-0310-9956-ffa450edef68
{code}
Is there a unit test with which I could test this? How do you find out this dependency? I would like to improve my knowledge to provide better patches in the feature... :-)
Thanks for your help.
> Upgrade to Jetty 7.0
> --------------------
>
> Key: CAMEL-2332
> URL: https://issues.apache.org/activemq/browse/CAMEL-2332
> Project: Apache Camel
> Issue Type: Task
> Components: camel-jetty
> Reporter: Claus Ibsen
> Assignee: Christian Mueller
> Fix For: 2.3.0
>
> Attachments: CAMEL-2332_20100309.patch
>
>
> Jetty 7.0 is out and we should upgrade to use that as well.
> We should do this in the same release we upgrade to HTTP Client 4.0 as well so we upgrade both kinds in the same release. CAMEL-1530.
--
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.
[jira] Commented: (CAMEL-2332) Upgrade to Jetty 7.0
Posted by "Christian Mueller (JIRA)" <ji...@apache.org>.
[ https://issues.apache.org/activemq/browse/CAMEL-2332?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=57921#action_57921 ]
Christian Mueller commented on CAMEL-2332:
------------------------------------------
The last version of jetty-cometd is still 6.1.11. It's ok to upgrade jetty to 7.0.1 and leave jetty-cometd on 6.1.11?
> Upgrade to Jetty 7.0
> --------------------
>
> Key: CAMEL-2332
> URL: https://issues.apache.org/activemq/browse/CAMEL-2332
> Project: Apache Camel
> Issue Type: Task
> Components: camel-jetty
> Reporter: Claus Ibsen
> Assignee: Christian Mueller
> Fix For: 2.3.0
>
>
> Jetty 7.0 is out and we should upgrade to use that as well.
> We should do this in the same release we upgrade to HTTP Client 4.0 as well so we upgrade both kinds in the same release. CAMEL-1530.
--
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.
[jira] Commented: (CAMEL-2332) Upgrade to Jetty 7.0
Posted by "Christian Mueller (JIRA)" <ji...@apache.org>.
[ https://issues.apache.org/activemq/browse/CAMEL-2332?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=57969#action_57969 ]
Christian Mueller commented on CAMEL-2332:
------------------------------------------
cometd 1.0.1 is build with jetty 7.0.1.v20091125 [pom.xml|http://svn.cometd.org/tags/1.0.1/pom.xml]. We need a jetty cometd bundle for 7.0.1.v20091125. I opened a issue for the new bundle at SMX4 [SMX4-498|https://issues.apache.org/activemq/browse/SMX4-498].
> Upgrade to Jetty 7.0
> --------------------
>
> Key: CAMEL-2332
> URL: https://issues.apache.org/activemq/browse/CAMEL-2332
> Project: Apache Camel
> Issue Type: Task
> Components: camel-jetty
> Reporter: Claus Ibsen
> Assignee: Christian Mueller
> Fix For: 2.3.0
>
>
> Jetty 7.0 is out and we should upgrade to use that as well.
> We should do this in the same release we upgrade to HTTP Client 4.0 as well so we upgrade both kinds in the same release. CAMEL-1530.
--
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.
[jira] Commented: (CAMEL-2332) Upgrade to Jetty 7.0
Posted by "Willem Jiang (JIRA)" <ji...@apache.org>.
[ https://issues.apache.org/activemq/browse/CAMEL-2332?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=58125#action_58125 ]
Willem Jiang commented on CAMEL-2332:
-------------------------------------
@Christian,
I just have a quick question for the Jetty 7, is it a OSGi bundle?
BTW, if we applied this patch in camel, we also need to upgrade the Jetty version in the SMX . AFIK, it will affect the PAX-Web which is also using Jetty.
Can you do some home work to check if the PAX-Web upgrading can move smoothly ?
> Upgrade to Jetty 7.0
> --------------------
>
> Key: CAMEL-2332
> URL: https://issues.apache.org/activemq/browse/CAMEL-2332
> Project: Apache Camel
> Issue Type: Task
> Components: camel-jetty
> Reporter: Claus Ibsen
> Assignee: Christian Mueller
> Fix For: 2.3.0
>
> Attachments: CAMEL-2332_20100309.patch
>
>
> Jetty 7.0 is out and we should upgrade to use that as well.
> We should do this in the same release we upgrade to HTTP Client 4.0 as well so we upgrade both kinds in the same release. CAMEL-1530.
--
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.
[jira] Commented: (CAMEL-2332) Upgrade to Jetty 7.0
Posted by "Hadrian Zbarcea (JIRA)" <ji...@apache.org>.
[ https://issues.apache.org/activemq/browse/CAMEL-2332?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=58126#action_58126 ]
Hadrian Zbarcea commented on CAMEL-2332:
----------------------------------------
trunk: r921210
> Upgrade to Jetty 7.0
> --------------------
>
> Key: CAMEL-2332
> URL: https://issues.apache.org/activemq/browse/CAMEL-2332
> Project: Apache Camel
> Issue Type: Task
> Components: camel-jetty
> Reporter: Claus Ibsen
> Assignee: Christian Mueller
> Fix For: 2.3.0
>
> Attachments: CAMEL-2332_20100309.patch
>
>
> Jetty 7.0 is out and we should upgrade to use that as well.
> We should do this in the same release we upgrade to HTTP Client 4.0 as well so we upgrade both kinds in the same release. CAMEL-1530.
--
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.
[jira] Commented: (CAMEL-2332) Upgrade to Jetty 7.0
Posted by "Hadrian Zbarcea (JIRA)" <ji...@apache.org>.
[ https://issues.apache.org/activemq/browse/CAMEL-2332?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=58136#action_58136 ]
Hadrian Zbarcea commented on CAMEL-2332:
----------------------------------------
@Willem, let's see if we can upgrade smx to jetty7 and if not I will revert the change.
@Christian, yes, unit tests passing is a good start.
> Upgrade to Jetty 7.0
> --------------------
>
> Key: CAMEL-2332
> URL: https://issues.apache.org/activemq/browse/CAMEL-2332
> Project: Apache Camel
> Issue Type: Task
> Components: camel-jetty
> Reporter: Claus Ibsen
> Assignee: Christian Mueller
> Fix For: 2.3.0
>
> Attachments: CAMEL-2332_20100309.patch
>
>
> Jetty 7.0 is out and we should upgrade to use that as well.
> We should do this in the same release we upgrade to HTTP Client 4.0 as well so we upgrade both kinds in the same release. CAMEL-1530.
--
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.
[jira] Commented: (CAMEL-2332) Upgrade to Jetty 7.0
Posted by "Claus Ibsen (JIRA)" <ji...@apache.org>.
[ https://issues.apache.org/activemq/browse/CAMEL-2332?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=58277#action_58277 ]
Claus Ibsen commented on CAMEL-2332:
------------------------------------
Willem have you tested CXF on SMX? Are there more work on this upgrade?
> Upgrade to Jetty 7.0
> --------------------
>
> Key: CAMEL-2332
> URL: https://issues.apache.org/activemq/browse/CAMEL-2332
> Project: Apache Camel
> Issue Type: Task
> Components: camel-jetty
> Reporter: Claus Ibsen
> Assignee: Christian Mueller
> Fix For: 2.3.0
>
> Attachments: CAMEL-2332_20100309.patch
>
>
> Jetty 7.0 is out and we should upgrade to use that as well.
> We should do this in the same release we upgrade to HTTP Client 4.0 as well so we upgrade both kinds in the same release. CAMEL-1530.
--
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.
[jira] Commented: (CAMEL-2332) Upgrade to Jetty 7.0
Posted by "Hadrian Zbarcea (JIRA)" <ji...@apache.org>.
[ https://issues.apache.org/activemq/browse/CAMEL-2332?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=58122#action_58122 ]
Hadrian Zbarcea commented on CAMEL-2332:
----------------------------------------
@Christian, excellent work. I am testing your patch and will apply when done, probably a couple of hours.
The only thing I am not sure about is the HttpRouteUsingUrlPostTest. I would rather fix it. Since its parent HttpRouteTest works it's kinda obvious that the conflicts are with the ports used by own test battery. One idea is to make the port configurable and have different descendants use different ports.
> Upgrade to Jetty 7.0
> --------------------
>
> Key: CAMEL-2332
> URL: https://issues.apache.org/activemq/browse/CAMEL-2332
> Project: Apache Camel
> Issue Type: Task
> Components: camel-jetty
> Reporter: Claus Ibsen
> Assignee: Christian Mueller
> Fix For: 2.3.0
>
> Attachments: CAMEL-2332_20100309.patch
>
>
> Jetty 7.0 is out and we should upgrade to use that as well.
> We should do this in the same release we upgrade to HTTP Client 4.0 as well so we upgrade both kinds in the same release. CAMEL-1530.
--
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.
[jira] Commented: (CAMEL-2332) Upgrade to Jetty 7.0
Posted by "Claus Ibsen (JIRA)" <ji...@apache.org>.
[ https://issues.apache.org/activemq/browse/CAMEL-2332?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=58199#action_58199 ]
Claus Ibsen commented on CAMEL-2332:
------------------------------------
Is there more work to be done? Is the upgrade complete?
> Upgrade to Jetty 7.0
> --------------------
>
> Key: CAMEL-2332
> URL: https://issues.apache.org/activemq/browse/CAMEL-2332
> Project: Apache Camel
> Issue Type: Task
> Components: camel-jetty
> Reporter: Claus Ibsen
> Assignee: Christian Mueller
> Fix For: 2.3.0
>
> Attachments: CAMEL-2332_20100309.patch
>
>
> Jetty 7.0 is out and we should upgrade to use that as well.
> We should do this in the same release we upgrade to HTTP Client 4.0 as well so we upgrade both kinds in the same release. CAMEL-1530.
--
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.
[jira] Commented: (CAMEL-2332) Upgrade to Jetty 7.0
Posted by "Christian Mueller (JIRA)" <ji...@apache.org>.
[ https://issues.apache.org/activemq/browse/CAMEL-2332?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=57966#action_57966 ]
Christian Mueller commented on CAMEL-2332:
------------------------------------------
[Porting_to_Jetty_7|http://wiki.eclipse.org/Jetty/Starting/Porting_to_Jetty_7/Packages_and_Classes] is a good help
> Upgrade to Jetty 7.0
> --------------------
>
> Key: CAMEL-2332
> URL: https://issues.apache.org/activemq/browse/CAMEL-2332
> Project: Apache Camel
> Issue Type: Task
> Components: camel-jetty
> Reporter: Claus Ibsen
> Assignee: Christian Mueller
> Fix For: 2.3.0
>
>
> Jetty 7.0 is out and we should upgrade to use that as well.
> We should do this in the same release we upgrade to HTTP Client 4.0 as well so we upgrade both kinds in the same release. CAMEL-1530.
--
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.
[jira] Commented: (CAMEL-2332) Upgrade to Jetty 7.0
Posted by "Christian Mueller (JIRA)" <ji...@apache.org>.
[ https://issues.apache.org/activemq/browse/CAMEL-2332?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=58073#action_58073 ]
Christian Mueller commented on CAMEL-2332:
------------------------------------------
I made some progress on this issue. I upgraded camel-jetty and all camel-jetty tests. Now, I work on camel-cometd, which also use jetty.
I think I can provide the patch next week...
> Upgrade to Jetty 7.0
> --------------------
>
> Key: CAMEL-2332
> URL: https://issues.apache.org/activemq/browse/CAMEL-2332
> Project: Apache Camel
> Issue Type: Task
> Components: camel-jetty
> Reporter: Claus Ibsen
> Assignee: Christian Mueller
> Fix For: 2.3.0
>
>
> Jetty 7.0 is out and we should upgrade to use that as well.
> We should do this in the same release we upgrade to HTTP Client 4.0 as well so we upgrade both kinds in the same release. CAMEL-1530.
--
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.
[jira] Commented: (CAMEL-2332) Upgrade to Jetty 7.0
Posted by "Willem Jiang (JIRA)" <ji...@apache.org>.
[ https://issues.apache.org/activemq/browse/CAMEL-2332?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=58204#action_58204 ]
Willem Jiang commented on CAMEL-2332:
-------------------------------------
@Claus, Christian,
I think we can close this issue now, as this issue doesn't dependent on the SMX4 one.
> Upgrade to Jetty 7.0
> --------------------
>
> Key: CAMEL-2332
> URL: https://issues.apache.org/activemq/browse/CAMEL-2332
> Project: Apache Camel
> Issue Type: Task
> Components: camel-jetty
> Reporter: Claus Ibsen
> Assignee: Christian Mueller
> Fix For: 2.3.0
>
> Attachments: CAMEL-2332_20100309.patch
>
>
> Jetty 7.0 is out and we should upgrade to use that as well.
> We should do this in the same release we upgrade to HTTP Client 4.0 as well so we upgrade both kinds in the same release. CAMEL-1530.
--
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.
[jira] Commented: (CAMEL-2332) Upgrade to Jetty 7.0
Posted by "Willem Jiang (JIRA)" <ji...@apache.org>.
[ https://issues.apache.org/activemq/browse/CAMEL-2332?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=58281#action_58281 ]
Willem Jiang commented on CAMEL-2332:
-------------------------------------
I just tried the apache camel-feature with camel-cxf-example on servicemix, everything looks good.
> Upgrade to Jetty 7.0
> --------------------
>
> Key: CAMEL-2332
> URL: https://issues.apache.org/activemq/browse/CAMEL-2332
> Project: Apache Camel
> Issue Type: Task
> Components: camel-jetty
> Reporter: Claus Ibsen
> Assignee: Christian Mueller
> Fix For: 2.3.0
>
> Attachments: CAMEL-2332_20100309.patch
>
>
> Jetty 7.0 is out and we should upgrade to use that as well.
> We should do this in the same release we upgrade to HTTP Client 4.0 as well so we upgrade both kinds in the same release. CAMEL-1530.
--
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.
[jira] Commented: (CAMEL-2332) Upgrade to Jetty 7.0
Posted by "Claus Ibsen (JIRA)" <ji...@apache.org>.
[ https://issues.apache.org/activemq/browse/CAMEL-2332?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=57927#action_57927 ]
Claus Ibsen commented on CAMEL-2332:
------------------------------------
Maybe they are no longer working on cometd? Maybe google a bit to see if there is any info from the Jetty people why?
> Upgrade to Jetty 7.0
> --------------------
>
> Key: CAMEL-2332
> URL: https://issues.apache.org/activemq/browse/CAMEL-2332
> Project: Apache Camel
> Issue Type: Task
> Components: camel-jetty
> Reporter: Claus Ibsen
> Assignee: Christian Mueller
> Fix For: 2.3.0
>
>
> Jetty 7.0 is out and we should upgrade to use that as well.
> We should do this in the same release we upgrade to HTTP Client 4.0 as well so we upgrade both kinds in the same release. CAMEL-1530.
--
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.
[jira] Updated: (CAMEL-2332) Upgrade to Jetty 7.0
Posted by "Christian Mueller (JIRA)" <ji...@apache.org>.
[ https://issues.apache.org/activemq/browse/CAMEL-2332?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]
Christian Mueller updated CAMEL-2332:
-------------------------------------
Attachment: CAMEL-2332_20100309.patch
Provide the patch to upgrade to jetty 7. I made the following changes:
camel-jetty:
- upgraded to jetty 7
- customized a few lines of code
- HttpRouteUsingUrlPostTest.javaremoved HttpRouteUsingUrlPostTest.java because it provides no tests
camel-cometd:
- upgrade to jetty 7
- switched from cometd-server to cometd-java-server
- customized a few lines of code
camel-web:
- upgrade to jetty 7
- switched dependency from jsp-2.1 to jsp-2.1-glassfish
- customized a few lines of code
camel-web-standalone:
- upgrade to jetty 7
camel-cxf:
- removed the dependency to jetty
camel-example:
- upgrade to jetty 7
karaf feature.xml:
- switched from org.apache.servicemix.bundles.jetty-bundle to some jetty 7 bundles
- switched from org.apache.servicemix.bundles.jetty-cometd to org.apache.servicemix.bundles.cometd-java-server [SMX4-498|https://issues.apache.org/activemq/browse/SMX4-498]
I run a "mvn -Psourcecheck clean install" to ensure, that the code follows the rules. I also run a "mvn clean install -Pvalidate" in platform/karaf to ensure, that the feature file is correct.
I somebody applies the patch, I will go through the wiki page and update it (system properties for ssl support, ...).
Thanks,
Christian
> Upgrade to Jetty 7.0
> --------------------
>
> Key: CAMEL-2332
> URL: https://issues.apache.org/activemq/browse/CAMEL-2332
> Project: Apache Camel
> Issue Type: Task
> Components: camel-jetty
> Reporter: Claus Ibsen
> Assignee: Christian Mueller
> Fix For: 2.3.0
>
> Attachments: CAMEL-2332_20100309.patch
>
>
> Jetty 7.0 is out and we should upgrade to use that as well.
> We should do this in the same release we upgrade to HTTP Client 4.0 as well so we upgrade both kinds in the same release. CAMEL-1530.
--
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.
[jira] Commented: (CAMEL-2332) Upgrade to Jetty 7.0
Posted by "Christian Mueller (JIRA)" <ji...@apache.org>.
[ https://issues.apache.org/activemq/browse/CAMEL-2332?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=58189#action_58189 ]
Christian Mueller commented on CAMEL-2332:
------------------------------------------
I update the wiki page
> Upgrade to Jetty 7.0
> --------------------
>
> Key: CAMEL-2332
> URL: https://issues.apache.org/activemq/browse/CAMEL-2332
> Project: Apache Camel
> Issue Type: Task
> Components: camel-jetty
> Reporter: Claus Ibsen
> Assignee: Christian Mueller
> Fix For: 2.3.0
>
> Attachments: CAMEL-2332_20100309.patch
>
>
> Jetty 7.0 is out and we should upgrade to use that as well.
> We should do this in the same release we upgrade to HTTP Client 4.0 as well so we upgrade both kinds in the same release. CAMEL-1530.
--
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.
[jira] Commented: (CAMEL-2332) Upgrade to Jetty 7.0
Posted by "Charles Moulliard (JIRA)" <ji...@apache.org>.
[ https://issues.apache.org/activemq/browse/CAMEL-2332?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=58280#action_58280 ]
Charles Moulliard commented on CAMEL-2332:
------------------------------------------
Hi,
I have tested successfully Jetty 7 on SMX4 using the following osgi bundles in a small project using camel-cometd (2.3-SNAPSHOT).
karaf@root> osgi:install -s mvn:org.eclipse.jetty/jetty-server/7.0.1.v20091125
karaf@root> osgi:install -s mvn:org.eclipse.jetty/jetty-http/7.0.1.v20091125
karaf@root> osgi:install -s mvn:org.eclipse.jetty/jetty-io/7.0.1.v20091125
karaf@root> osgi:install -s mvn:org.eclipse.jetty/jetty-util/7.0.1.v20091125
karaf@root> osgi:install -s mvn:org.eclipse.jetty/jetty-continuation/7.0.1.v20091125
karaf@root> osgi:install -s mvn:org.eclipse.jetty/jetty-servlet/7.0.1.v20091125
karaf@root> osgi:install -s mvn:org.eclipse.jetty/jetty-security/7.0.1.v20091125
+ following bundles for camel-cometd
osgi:install -s wrap:mvn:org.cometd.java/cometd-api/1.0.1
osgi:install -s wrap:mvn:org.cometd.java/cometd-java-server/1.0.1
Remark : As stated previously, PAX Web is not impacted to use Jetty 7 instead of Jetty 6 while we are not using by example a fragment bundle containing jetty.xml config file. The reason concerns package renaming (org.mortbay --> org.eclipse). More tests should be done to verify that we don't have problem with servlet registration, ...
> Upgrade to Jetty 7.0
> --------------------
>
> Key: CAMEL-2332
> URL: https://issues.apache.org/activemq/browse/CAMEL-2332
> Project: Apache Camel
> Issue Type: Task
> Components: camel-jetty
> Reporter: Claus Ibsen
> Assignee: Christian Mueller
> Fix For: 2.3.0
>
> Attachments: CAMEL-2332_20100309.patch
>
>
> Jetty 7.0 is out and we should upgrade to use that as well.
> We should do this in the same release we upgrade to HTTP Client 4.0 as well so we upgrade both kinds in the same release. CAMEL-1530.
--
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.
[jira] Commented: (CAMEL-2332) Upgrade to Jetty 7.0
Posted by "Christian Mueller (JIRA)" <ji...@apache.org>.
[ https://issues.apache.org/activemq/browse/CAMEL-2332?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=58283#action_58283 ]
Christian Mueller commented on CAMEL-2332:
------------------------------------------
@Willem: You wrote "Just a quick note for Jetty7 upgrading, as CXF bundle didn't upgrade to Jetty7, we still need to import the Jetty 6.x bundle in camel-cxf feature". This means I forgot something to upgrade or camel-cxf MUST be used with Jetty 6? If I forgot something, I will do the necessary work that all camel components using Jetty 7.
> Upgrade to Jetty 7.0
> --------------------
>
> Key: CAMEL-2332
> URL: https://issues.apache.org/activemq/browse/CAMEL-2332
> Project: Apache Camel
> Issue Type: Task
> Components: camel-jetty
> Reporter: Claus Ibsen
> Assignee: Christian Mueller
> Fix For: 2.3.0
>
> Attachments: CAMEL-2332_20100309.patch
>
>
> Jetty 7.0 is out and we should upgrade to use that as well.
> We should do this in the same release we upgrade to HTTP Client 4.0 as well so we upgrade both kinds in the same release. CAMEL-1530.
--
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.
[jira] Assigned: (CAMEL-2332) Upgrade to Jetty 7.0
Posted by "Christian Mueller (JIRA)" <ji...@apache.org>.
[ https://issues.apache.org/activemq/browse/CAMEL-2332?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]
Christian Mueller reassigned CAMEL-2332:
----------------------------------------
Assignee: Christian Mueller
> Upgrade to Jetty 7.0
> --------------------
>
> Key: CAMEL-2332
> URL: https://issues.apache.org/activemq/browse/CAMEL-2332
> Project: Apache Camel
> Issue Type: Task
> Components: camel-jetty
> Reporter: Claus Ibsen
> Assignee: Christian Mueller
> Fix For: 2.3.0
>
>
> Jetty 7.0 is out and we should upgrade to use that as well.
> We should do this in the same release we upgrade to HTTP Client 4.0 as well so we upgrade both kinds in the same release. CAMEL-1530.
--
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.
[jira] Commented: (CAMEL-2332) Upgrade to Jetty 7.0
Posted by "Willem Jiang (JIRA)" <ji...@apache.org>.
[ https://issues.apache.org/activemq/browse/CAMEL-2332?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=58318#action_58318 ]
Willem Jiang commented on CAMEL-2332:
-------------------------------------
And It's not easy to run the unit test of OSGi without helping of PAX-Exame.
There are some internal system tests in Fuse build farm can catch these error, but I'm afraid it can run for you.
I will adding the OSGi unit tests in tests/camel-itest-osgi , so you can run the tests to check if the things works for you.
BTW camel-itest-osgi is disable by default as the CI can't run it smoothly, if you want to run it , you just need to cd into that directory, and type "mvn clean install".
> Upgrade to Jetty 7.0
> --------------------
>
> Key: CAMEL-2332
> URL: https://issues.apache.org/activemq/browse/CAMEL-2332
> Project: Apache Camel
> Issue Type: Task
> Components: camel-jetty
> Reporter: Claus Ibsen
> Assignee: Christian Mueller
> Fix For: 2.3.0
>
> Attachments: CAMEL-2332_20100309.patch
>
>
> Jetty 7.0 is out and we should upgrade to use that as well.
> We should do this in the same release we upgrade to HTTP Client 4.0 as well so we upgrade both kinds in the same release. CAMEL-1530.
--
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.
[jira] Commented: (CAMEL-2332) Upgrade to Jetty 7.0
Posted by "Christian Mueller (JIRA)" <ji...@apache.org>.
[ https://issues.apache.org/activemq/browse/CAMEL-2332?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=57964#action_57964 ]
Christian Mueller commented on CAMEL-2332:
------------------------------------------
The new home for cometd is http://cometd.org. There are two related issues: [CAMEL-2513|https://issues.apache.org/activemq/browse/CAMEL-2513] and [SMX4-495|https://issues.apache.org/activemq/browse/SMX4-495].
The last version of [cometd-server|http://mvnrepository.com/artifact/org.mortbay.jetty/cometd-server] is 6.1.22. There is no 7.x version (nether [org.mortbay.jetty|http://mvnrepository.com/artifact/org.mortbay.jetty] nor [org.eclipse.jetty|http://mvnrepository.com/artifact/org.eclipse.jetty]).
> Upgrade to Jetty 7.0
> --------------------
>
> Key: CAMEL-2332
> URL: https://issues.apache.org/activemq/browse/CAMEL-2332
> Project: Apache Camel
> Issue Type: Task
> Components: camel-jetty
> Reporter: Claus Ibsen
> Assignee: Christian Mueller
> Fix For: 2.3.0
>
>
> Jetty 7.0 is out and we should upgrade to use that as well.
> We should do this in the same release we upgrade to HTTP Client 4.0 as well so we upgrade both kinds in the same release. CAMEL-1530.
--
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.
[jira] Commented: (CAMEL-2332) Upgrade to Jetty 7.0
Posted by "Christian Mueller (JIRA)" <ji...@apache.org>.
[ https://issues.apache.org/activemq/browse/CAMEL-2332?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=58201#action_58201 ]
Christian Mueller commented on CAMEL-2332:
------------------------------------------
I think the upgrade is complete and should not be in conflict with Jetty 6 in SMX4 as Willem wrote.
But anyway, I will also provide a patch to upgrade SMX4 to jetty 7. I don't know, how this upgrade will be related to PAX-Web (MUST also be updated, SHOULD be updated or it is totally independent). I think it will be independent, but I'm not sure...
> Upgrade to Jetty 7.0
> --------------------
>
> Key: CAMEL-2332
> URL: https://issues.apache.org/activemq/browse/CAMEL-2332
> Project: Apache Camel
> Issue Type: Task
> Components: camel-jetty
> Reporter: Claus Ibsen
> Assignee: Christian Mueller
> Fix For: 2.3.0
>
> Attachments: CAMEL-2332_20100309.patch
>
>
> Jetty 7.0 is out and we should upgrade to use that as well.
> We should do this in the same release we upgrade to HTTP Client 4.0 as well so we upgrade both kinds in the same release. CAMEL-1530.
--
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.
[jira] Commented: (CAMEL-2332) Upgrade to Jetty 7.0
Posted by "Charles Moulliard (JIRA)" <ji...@apache.org>.
[ https://issues.apache.org/activemq/browse/CAMEL-2332?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=56751#action_56751 ]
Charles Moulliard commented on CAMEL-2332:
------------------------------------------
Questions :
1) Package names : As package names have changed between Jetty 6 (org.mortbay) and Jetty 7 (org.eclipse), what do you plan to do with camel-jetty component ? Having a component supporting both or migrating to 7 ?
2) MANIFEST file & PAX Web : PAX Web bundle supports only Jetty 6 and imports org.mortbay classes with version 6. Are you synchronised camel-jetty release with next PAX-web bundle supporting jetty 6 and jetty 7 ?
> Upgrade to Jetty 7.0
> --------------------
>
> Key: CAMEL-2332
> URL: https://issues.apache.org/activemq/browse/CAMEL-2332
> Project: Apache Camel
> Issue Type: Task
> Components: camel-jetty
> Reporter: Claus Ibsen
> Fix For: 2.3.0
>
>
> Jetty 7.0 is out and we should upgrade to use that as well.
> We should do this in the same release we upgrade to HTTP Client 4.0 as well so we upgrade both kinds in the same release. CAMEL-1530.
--
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.
[jira] Commented: (CAMEL-2332) Upgrade to Jetty 7.0
Posted by "Christian Mueller (JIRA)" <ji...@apache.org>.
[ https://issues.apache.org/activemq/browse/CAMEL-2332?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=58131#action_58131 ]
Christian Mueller commented on CAMEL-2332:
------------------------------------------
Hello Willem,
yes, jetty 7 comes as several osgi bundles.
I will upgrade jetty in SXM4 to jetty 7 and check, if it works. What do you recommend, how I should test this? Only running the unit tests?
I also opened a issue to upgrade PAX-Web to Jetty 7 [PAXWEB-198|http://issues.ops4j.org/browse/PAXWEB-198].
> Upgrade to Jetty 7.0
> --------------------
>
> Key: CAMEL-2332
> URL: https://issues.apache.org/activemq/browse/CAMEL-2332
> Project: Apache Camel
> Issue Type: Task
> Components: camel-jetty
> Reporter: Claus Ibsen
> Assignee: Christian Mueller
> Fix For: 2.3.0
>
> Attachments: CAMEL-2332_20100309.patch
>
>
> Jetty 7.0 is out and we should upgrade to use that as well.
> We should do this in the same release we upgrade to HTTP Client 4.0 as well so we upgrade both kinds in the same release. CAMEL-1530.
--
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.
[jira] Commented: (CAMEL-2332) Upgrade to Jetty 7.0
Posted by "Willem Jiang (JIRA)" <ji...@apache.org>.
[ https://issues.apache.org/activemq/browse/CAMEL-2332?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=58284#action_58284 ]
Willem Jiang commented on CAMEL-2332:
-------------------------------------
@Christian,
Your test case may not use the camel-cxf component.
Can you check out the latest apache camel feature.xml ?
It has enough information to tell the OSGI container to load the bundle for camel running :)
> Upgrade to Jetty 7.0
> --------------------
>
> Key: CAMEL-2332
> URL: https://issues.apache.org/activemq/browse/CAMEL-2332
> Project: Apache Camel
> Issue Type: Task
> Components: camel-jetty
> Reporter: Claus Ibsen
> Assignee: Christian Mueller
> Fix For: 2.3.0
>
> Attachments: CAMEL-2332_20100309.patch
>
>
> Jetty 7.0 is out and we should upgrade to use that as well.
> We should do this in the same release we upgrade to HTTP Client 4.0 as well so we upgrade both kinds in the same release. CAMEL-1530.
--
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.
[jira] Commented: (CAMEL-2332) Upgrade to Jetty 7.0
Posted by "Willem Jiang (JIRA)" <ji...@apache.org>.
[ https://issues.apache.org/activemq/browse/CAMEL-2332?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=58128#action_58128 ]
Willem Jiang commented on CAMEL-2332:
-------------------------------------
@Hadrain,
I checked the PAX-Web doesn't use the Jetty7 now, so we may have trouble to deploy the camel components into apache SMX with Jetty7.
I'm afraid you have to revert this change.
> Upgrade to Jetty 7.0
> --------------------
>
> Key: CAMEL-2332
> URL: https://issues.apache.org/activemq/browse/CAMEL-2332
> Project: Apache Camel
> Issue Type: Task
> Components: camel-jetty
> Reporter: Claus Ibsen
> Assignee: Christian Mueller
> Fix For: 2.3.0
>
> Attachments: CAMEL-2332_20100309.patch
>
>
> Jetty 7.0 is out and we should upgrade to use that as well.
> We should do this in the same release we upgrade to HTTP Client 4.0 as well so we upgrade both kinds in the same release. CAMEL-1530.
--
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.
[jira] Commented: (CAMEL-2332) Upgrade to Jetty 7.0
Posted by "Willem Jiang (JIRA)" <ji...@apache.org>.
[ https://issues.apache.org/activemq/browse/CAMEL-2332?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=58162#action_58162 ]
Willem Jiang commented on CAMEL-2332:
-------------------------------------
@Christian, @Hadrian.
Thanks for the updates, as the Jetty7 already changed its namespace, it's unlikely we will hit the issue of common pool CAMEL-2514 ( this two version of bundle mass up with each other with the wrong OSGi imports package) .
So I'm OK for this patch, as the Jetty7 and Jetty6 can live peacefully in SMX.
> Upgrade to Jetty 7.0
> --------------------
>
> Key: CAMEL-2332
> URL: https://issues.apache.org/activemq/browse/CAMEL-2332
> Project: Apache Camel
> Issue Type: Task
> Components: camel-jetty
> Reporter: Claus Ibsen
> Assignee: Christian Mueller
> Fix For: 2.3.0
>
> Attachments: CAMEL-2332_20100309.patch
>
>
> Jetty 7.0 is out and we should upgrade to use that as well.
> We should do this in the same release we upgrade to HTTP Client 4.0 as well so we upgrade both kinds in the same release. CAMEL-1530.
--
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.