Archiva
  1. Archiva
  2. MRM-688

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

        Activity

        Hide
        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
        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
        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
        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
        Maria Odea Ching added a comment -

        Okey dokey, thanks Brett!

        Show
        Maria Odea Ching added a comment - Okey dokey, thanks Brett!
        Hide
        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
        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
        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
        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
        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
        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
        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
        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
        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

          • Assignee:
            Brett Porter
            Reporter:
            Maria Odea Ching
          • Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved: