You are viewing a plain text version of this content. The canonical link for it is here.
Posted to notifications@jclouds.apache.org by Andrea Turli <no...@github.com> on 2018/07/12 10:58:41 UTC
[jclouds/jclouds] Improve Jetty installation for live tests (#1230)
Notice `sudo` shouldn't be a problem as `AdminAccess` will add the `web` user to sudoers
Fixes a failing test with `BaseComputeServiceLiveTest.testCreateAndRunAService` on a `CentOS 7.x` image
You can view, comment on, or merge this pull request online at:
https://github.com/jclouds/jclouds/pull/1230
-- Commit Summary --
* Improve Jetty installation for live tests
-- File Changes --
M compute/src/test/java/org/jclouds/compute/JettyStatements.java (21)
M scriptbuilder/src/main/resources/functions/installOpenJDK.sh (2)
-- Patch Links --
https://github.com/jclouds/jclouds/pull/1230.patch
https://github.com/jclouds/jclouds/pull/1230.diff
--
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/jclouds/jclouds/pull/1230
Re: [jclouds/jclouds] Improve Jetty installation for live tests
(#1230)
Posted by Andrew Gaul <no...@github.com>.
Please reopen against apache/jclouds if this is still relevant.
--
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/jclouds/jclouds/pull/1230#issuecomment-663855806
Re: [jclouds/jclouds] Improve Jetty installation for live tests
(#1230)
Posted by Ignasi Barrera <no...@github.com>.
Closed #1230.
--
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/jclouds/jclouds/pull/1230#event-1903538849
Re: [jclouds/jclouds] Improve Jetty installation for live tests
(#1230)
Posted by Andrea Turli <no...@github.com>.
thanks @nacx I've bumped the jetty version to the latest `9.4.12.v20180830`
--
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/jclouds/jclouds/pull/1230#issuecomment-435129000
Re: [jclouds/jclouds] Improve Jetty installation for live tests
(#1230)
Posted by Andrew Gaul <no...@github.com>.
Closed #1230.
--
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/jclouds/jclouds/pull/1230#event-3586273889
Re: [jclouds/jclouds] Improve Jetty installation for live tests
(#1230)
Posted by Ignasi Barrera <no...@github.com>.
Closing and re-opening to test new PR build
--
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/jclouds/jclouds/pull/1230#issuecomment-429773301
Re: [jclouds/jclouds] Improve Jetty installation for live tests
(#1230)
Posted by Andrea Turli <no...@github.com>.
rebuild please
--
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/jclouds/jclouds/pull/1230#issuecomment-415091039
Re: [jclouds/jclouds] Improve Jetty installation for live tests
(#1230)
Posted by Ignasi Barrera <no...@github.com>.
Reopened #1230.
--
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/jclouds/jclouds/pull/1230#event-1903538974
Re: [jclouds/jclouds] Improve Jetty installation for live tests
(#1230)
Posted by Andrea Turli <no...@github.com>.
@andreaturli pushed 1 commit.
bec0de5 - remove SslSelectChannelConnector import
--
You are receiving this because you are subscribed to this thread.
View it on GitHub:
https://github.com/jclouds/jclouds/pull/1230/files/2b2688e2080f5443d88915c83f840178871d546e..bec0de59ff8e539608778b44432e6db291cc06ba
Re: [jclouds/jclouds] Improve Jetty installation for live tests
(#1230)
Posted by Andrea Turli <no...@github.com>.
@andreaturli pushed 1 commit.
03f1781 downgrade jetty to 9.2.x as it runs on java7
--
You are receiving this because you are subscribed to this thread.
View it on GitHub:
https://github.com/jclouds/jclouds/pull/1230/files/bec0de59ff8e539608778b44432e6db291cc06ba..03f178150c1a84035cd2143ca1f1c2146a473a20
Re: [jclouds/jclouds] Improve Jetty installation for live tests
(#1230)
Posted by Andrea Turli <no...@github.com>.
@andreaturli pushed 1 commit.
2b2688e -bump jetty version to fix security vulnerability
--
You are receiving this because you are subscribed to this thread.
View it on GitHub:
https://github.com/jclouds/jclouds/pull/1230/files/86630f4dd6dcf2466615ca456b892d4bf1ad9097..2b2688e2080f5443d88915c83f840178871d546e
Re: [jclouds/jclouds] Improve Jetty installation for live tests
(#1230)
Posted by Ignasi Barrera <no...@github.com>.
Looks like the version with the fix require Java 8: https://www.eclipse.org/lists/jetty-announce/msg00080.html
Let's revert this and file a proper JIRA issue to track the Jetty thing. It's not a critical thing, as the Jetty statement is mostly used just for tests.
--
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/jclouds/jclouds/pull/1230#issuecomment-435332643
Re: [jclouds/jclouds] Improve Jetty installation for live tests
(#1230)
Posted by Ignasi Barrera <no...@github.com>.
@andreaturli Any chance you can use this branch to upgrade Jetty to fix the vulnerability?
--
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/jclouds/jclouds/pull/1230#issuecomment-435021317