Uploaded image for project: 'Tools (JBoss Tools)'
  1. Tools (JBoss Tools)
  2. JBIDE-9711

AS7 Adapter : Deployment of OSGI projects fails if the temporary folder doesn't exist

    Details

    • Type: Bug
    • Status: Closed (View Workflow)
    • Priority: Major
    • Resolution: Done
    • Affects Version/s: 3.3.0.M3
    • Fix Version/s: 3.3.0.M4
    • Component/s: server
    • Labels:
      None
    • Environment:
      JBossTools 3.3.0.M3-H18
    • Steps to Reproduce:
      Hide

      In order to reproduce, I believe (haven't tried) you can :

      Show
      In order to reproduce, I believe (haven't tried) you can : create an AS7 server delete the temp deployment folder try to deploy a OSGI bundle (see https://docs.jboss.org/author/display/AS7/Helloworld+OSGi+quickstart )
    • Estimated Difficulty:
      Low

      Description

      I noticed that, in the context of JBIDE-9133, for the existing broken AS7 server instances the utility projects were not zipped to the deployment area.
      On the other hand, creating a new server instance fixed the issue.

      So it seems the temporary folder is only created at the same time as the server instance.
      It's an issue in the sense that you can not guarantee the temporary folder will exist at deployment time (could have been deleted by the user ...).

      Solution is to call tempdir.mkdirs() right before the actual deployment occurs.

        Gliffy Diagrams

          Issue Links

            Activity

            Hide
            rob.stryker Rob Stryker added a comment -

            Patch to trunk, can possibly be applied to branch if critical

            Show
            rob.stryker Rob Stryker added a comment - Patch to trunk, can possibly be applied to branch if critical
            Hide
            rob.stryker Rob Stryker added a comment -

            If this is critical (app is deployed as AppNamenull instead of AppName.jar) and re-creates the temp folder.

            Show
            rob.stryker Rob Stryker added a comment - If this is critical (app is deployed as AppNamenull instead of AppName.jar) and re-creates the temp folder.
            Hide
            maxandersen Max Rydahl Andersen added a comment -

            approved

            Show
            maxandersen Max Rydahl Andersen added a comment - approved
            Hide
            rob.stryker Rob Stryker added a comment -

            In m4

            Show
            rob.stryker Rob Stryker added a comment - In m4
            Hide
            mmalina Martin Malina added a comment -

            Libor, can you verify that it works as expected now?
            When I try to deploy an osgi bundle and right before doing so remove the tmp dir, I get errors in the server log. But I'm not sure it's really a problem. AS uses the tmp directory for other things so is this really a valid use case?

            Show
            mmalina Martin Malina added a comment - Libor, can you verify that it works as expected now? When I try to deploy an osgi bundle and right before doing so remove the tmp dir, I get errors in the server log. But I'm not sure it's really a problem. AS uses the tmp directory for other things so is this really a valid use case?
            Hide
            maxandersen Max Rydahl Andersen added a comment -

            Martin, I believe he is talking about the temporary directory created and used by the JBoss server adapter, not jboss tmp dir it self.

            Show
            maxandersen Max Rydahl Andersen added a comment - Martin, I believe he is talking about the temporary directory created and used by the JBoss server adapter, not jboss tmp dir it self.
            Hide
            mmalina Martin Malina added a comment -

            Oh, thanks Max. I tried to verify, the osgi bundle deploys ok, but tempDeploy dir is not recreated anyway. I'll ask Libor to verify.

            Show
            mmalina Martin Malina added a comment - Oh, thanks Max. I tried to verify, the osgi bundle deploys ok, but tempDeploy dir is not recreated anyway. I'll ask Libor to verify.
            Hide
            mmalina Martin Malina added a comment -

            Libor agrees that if deployment is ok this JIRA can be closed.
            Verified in JBT nightly (2012-04-30)

            Show
            mmalina Martin Malina added a comment - Libor agrees that if deployment is ok this JIRA can be closed. Verified in JBT nightly (2012-04-30)

              People

              • Assignee:
                lzoubek Libor Zoubek
                Reporter:
                lzoubek Libor Zoubek
              • Votes:
                0 Vote for this issue
                Watchers:
                2 Start watching this issue

                Dates

                • Created:
                  Updated:
                  Resolved:

                  Development