Archiva

Replace plexus-runtime with standalone jetty bundle

Details

  • Type: Task Task
  • Status: Closed Closed
  • Priority: Major Major
  • Resolution: Fixed
  • Affects Version/s: 1.0, 1.0.1
  • Fix Version/s: 1.1
  • Component/s: Build
  • Labels:
    None
  • Number of attachments :
    0

Issue Links

This issue is related to:
MRM-597 Cannot run Archiva as another user because su is run with option "-" (--login) Major Closed
MRM-702 in the standalone bundle, the pid file is written into the installation directory alongside the wrapper, but should be in a temporary location Major Closed
MRM-655 The logs are duplicated in the archiva.log and wrapper.log file. Major Closed
This issue supercedes:
MRM-553 removing conf/shared.xml from plexus runtime causes save appearance to fail Major Closed
MRM-618 PLEXUS_BASE does not work for local databases Major Closed

Activity

Ascending order - Click to sort in descending order
Hide
Permalink
Brett Porter added a comment -

note the documentation will have to be updated to not refer to PLEXUS_BASE (or perhaps we should continue to honour that for backwards compatibility)

Show
Brett Porter added a comment - note the documentation will have to be updated to not refer to PLEXUS_BASE (or perhaps we should continue to honour that for backwards compatibility)
Hide
Permalink
Maria Odea Ching added a comment -

Added archiva-jetty-archetype-module in archiva-trunk -r632245. This is not yet included in the build for now, as the generated wrapper files are corrupted (see MOJO-1060)

Show
Maria Odea Ching added a comment - Added archiva-jetty-archetype-module in archiva-trunk -r632245. This is not yet included in the build for now, as the generated wrapper files are corrupted (see MOJO-1060)
Hide
Permalink
Brett Porter added a comment -

I've fixed the corruption, however there's more work to do here:

  • it doesn't run the webapp, it offers it for download
  • need to work through all the default configuration since conf/* won't be used by default. A good configuration of the jetty.xml should be able to set up the appserver.base etc as before.
Show
Brett Porter added a comment - I've fixed the corruption, however there's more work to do here:
  • it doesn't run the webapp, it offers it for download
  • need to work through all the default configuration since conf/* won't be used by default. A good configuration of the jetty.xml should be able to set up the appserver.base etc as before.
Hide
Permalink
Brett Porter added a comment -

also need to rename the module to just archiva-jetty

Show
Brett Porter added a comment - also need to rename the module to just archiva-jetty
Hide
Permalink
Maria Odea Ching added a comment -

Okey dokey, thanks Brett!

Show
Maria Odea Ching added a comment - Okey dokey, thanks Brett!
Hide
Permalink
Maria Odea Ching added a comment -

Changes made in -r633367:
-added config for mail session and archiva & users db in jetty.xml
-unpack webapp during packaging so that it would be run during startup

Show
Maria Odea Ching added a comment - Changes made in -r633367: -added config for mail session and archiva & users db in jetty.xml -unpack webapp during packaging so that it would be run during startup
Hide
Permalink
Maria Odea Ching added a comment -

Changes made in -r633408:

  • set appserver.base as additional java parameter
  • also added config property below to strip the quotes of the value of appserver.base (as i don't think this is currently supported by the appassembler-maven-plugin)
    <property>
    <name>wrapper.java.additional.1.stripquotes</name>
    <value>TRUE</value>
    </property>

Btw, should we retain the PLEXUS_BASE for backwards compatibility as Brett also suggested above?

Show
Maria Odea Ching added a comment - Changes made in -r633408:
  • set appserver.base as additional java parameter
  • also added config property below to strip the quotes of the value of appserver.base (as i don't think this is currently supported by the appassembler-maven-plugin) <property> <name>wrapper.java.additional.1.stripquotes</name> <value>TRUE</value> </property>
Btw, should we retain the PLEXUS_BASE for backwards compatibility as Brett also suggested above?
Hide
Permalink
Maria Odea Ching added a comment -

Would this be necessary (retain PLEXUS_BASE) if we'll be migrating to spring in the future?

Show
Maria Odea Ching added a comment - Would this be necessary (retain PLEXUS_BASE) if we'll be migrating to spring in the future?
Hide
Permalink
Brett Porter added a comment -

it's just an env var, so it's ok to keep it - but I don't know that it's worth it. I think it might be better to just adjust the documentation.

Show
Brett Porter added a comment - it's just an env var, so it's ok to keep it - but I don't know that it's worth it. I think it might be better to just adjust the documentation.
Hide
Permalink
Maria Odea Ching added a comment -

Okay, I'll just adjust the documentation for now. I'll add the archiva-jetty module to the build as well so people can also try it out and test it.
Thanks!

Show
Maria Odea Ching added a comment - Okay, I'll just adjust the documentation for now. I'll add the archiva-jetty module to the build as well so people can also try it out and test it. Thanks!
Hide
Permalink
Brett Porter added a comment -

I'm having a problem with "archiva stop" on Mac so I'll open this until I can figure it out.

Also, we need to update the following docs:
quickstart, standalone, docs/adminguide/databases.html, docs/adminguide/customising-security.html

Show
Brett Porter added a comment - I'm having a problem with "archiva stop" on Mac so I'll open this until I can figure it out. Also, we need to update the following docs: quickstart, standalone, docs/adminguide/databases.html, docs/adminguide/customising-security.html
Hide
Permalink
Brett Porter added a comment -

it also seems the archiva.log and stderr logs duplicate information

Show
Brett Porter added a comment - it also seems the archiva.log and stderr logs duplicate information
Hide
Permalink
Brett Porter added a comment -

I'm going to be working through getting ARCHIVA_BASE working today, and will then test on Windows

Show
Brett Porter added a comment - I'm going to be working through getting ARCHIVA_BASE working today, and will then test on Windows
Hide
Permalink
Maria Odea Ching added a comment -

Fixed the duplication in the logs in -r642087:

  • set the logs to be output only to the log file and not the console to eliminate duplication
Show
Maria Odea Ching added a comment - Fixed the duplication in the logs in -r642087:
  • set the logs to be output only to the log file and not the console to eliminate duplication

People

Vote (0)
Watch (0)

Dates

  • Created:
    Updated:
    Resolved:
Atlassian JIRA (v4.4.3#663-r165197) | Report a problem
Powered by a free Atlassian JIRA open source license for Codehaus. Try JIRA - bug tracking software for your team.