Cargo
  1. Cargo
  2. CARGO-937

Add support for more JBoss specific deployables (HAR and JBoss AOP)

    Details

    • Type: Improvement Improvement
    • Status: Closed Closed
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: 1.0.6
    • Fix Version/s: 1.1.0
    • Component/s: JBoss
    • Labels:
      None
    • Environment:
      n/a
    • Complexity:
      Intermediate
    • Number of attachments :
      0

      Description

      There is currently no support for deploying some JBoss specific deployables. We should add support for all JBoss specific archives. The ones that come to mind are the ones supported by jboss-packaging-maven-plugin.

        Issue Links

          Activity

          Hide
          Anders Hammar added a comment -

          I saw this problem as well. The reason was that you changed the tests so they don't check the id of the container but rather performs the tests on all JBoss conatiners that support har. The tests can only be run on JBoss 5+ as that's the only containers that the jmx check works on.
          I've been trying for the last hour to get the validator back (ContainerIdRegExValidator) that you removed, but me and svn don't really cooperate here...

          Show
          Anders Hammar added a comment - I saw this problem as well. The reason was that you changed the tests so they don't check the id of the container but rather performs the tests on all JBoss conatiners that support har. The tests can only be run on JBoss 5+ as that's the only containers that the jmx check works on. I've been trying for the last hour to get the validator back (ContainerIdRegExValidator) that you removed, but me and svn don't really cooperate here...
          Hide
          Savas Ali Tokmen added a comment -

          Reading the tips from the JBoss forums, I do not think implementing the MBeanServerConnection with the JBoss HTTP naming factory should not be hard; and will work with JBoss 4.x.

          I'll take a look

          Show
          Savas Ali Tokmen added a comment - Reading the tips from the JBoss forums, I do not think implementing the MBeanServerConnection with the JBoss HTTP naming factory should not be hard; and will work with JBoss 4.x. I'll take a look
          Hide
          Anders Hammar added a comment -

          OK, I finally managed re-adding that validator in r2859.
          We'll close this ticket when we see the tests pass.

          Show
          Anders Hammar added a comment - OK, I finally managed re-adding that validator in r2859. We'll close this ticket when we see the tests pass.
          Hide
          Anders Hammar added a comment -

          Ali, JBoss is working on syncing their artifacts to central so unless you have too much spare time I suggest we wait for that instead of implementing this. If har/aop deployment works on JBoss 5+, it should work on 4.x as well. And if it for any reason doesn't, I would suspect that the user base being affect is very limited as that's a very old version.

          Show
          Anders Hammar added a comment - Ali, JBoss is working on syncing their artifacts to central so unless you have too much spare time I suggest we wait for that instead of implementing this. If har/aop deployment works on JBoss 5+, it should work on 4.x as well. And if it for any reason doesn't, I would suspect that the user base being affect is very limited as that's a very old version.
          Hide
          Savas Ali Tokmen added a comment -

          Committed revision 2860: Added JMX connection method for JBoss 4.x in order to test AOP and HAR on these container as well. This is now fully OK

          Show
          Savas Ali Tokmen added a comment - Committed revision 2860: Added JMX connection method for JBoss 4.x in order to test AOP and HAR on these container as well. This is now fully OK

            People

            • Assignee:
              Anders Hammar
              Reporter:
              Anders Hammar
            • Votes:
              0 Vote for this issue
              Watchers:
              0 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved: