Details

    • Type: Bug Bug
    • Status: Resolved
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: 3.0.0
    • Fix Version/s: 3.0-beta-2
    • Component/s: deployment
    • Security Level: public (Regular issues)
    • Labels:
      None

      Description

      Deploying a "car" module or module with the .ser and .sha1 files causes "Unable to deploy: java.io.IOException: Sum file already exists" error. Such modules shouldn't be deployed in the first place but some users do by mistake or whatever. Unfortunately, the error doesn't help in determining what's wrong. So I think we should either throw a nicer exception when this happens or suppress this error altogether and redeploy the module.

        Issue Links

          Activity

          Hide
          xiezhi added a comment -

          Hi,Jarek. I try to recreate the issue. But I couldn't get the same exception as you mentioned when I deployed a car twice. Could you tell me how did you do or provide a sample car. I'll continue to investigate it. It will be good if got your help.

          Show
          xiezhi added a comment - Hi,Jarek. I try to recreate the issue. But I couldn't get the same exception as you mentioned when I deployed a car twice. Could you tell me how did you do or provide a sample car. I'll continue to investigate it. It will be good if got your help.
          Hide
          xiezhi added a comment -

          recreate this issue may follow this steps
          1.deploy a war in admin console, for example deploy hello-3.0.0.0.war
          2.copy hello-3.0.0.0.car or hello-3.0.0.0.war from the <geronimo-home>\repository to another place for backup.
          3.uninstall hello component in admin console.
          4.deploy the backup car or war before
          the error message occurs.

          Or deploy a car which does not exist in the server in admin console. It can recreate the issue either.

          Show
          xiezhi added a comment - recreate this issue may follow this steps 1.deploy a war in admin console, for example deploy hello-3.0.0.0.war 2.copy hello-3.0.0.0.car or hello-3.0.0.0.war from the <geronimo-home>\repository to another place for backup. 3.uninstall hello component in admin console. 4.deploy the backup car or war before the error message occurs. Or deploy a car which does not exist in the server in admin console. It can recreate the issue either.
          Hide
          xiezhi added a comment -

          I updated the error message to "It is a configuration archive now. Sum file already exists.Please use the install-plugin command to install it."

          Show
          xiezhi added a comment - I updated the error message to "It is a configuration archive now. Sum file already exists.Please use the install-plugin command to install it."
          Hide
          Forrest Xia added a comment -

          xiezhi's patch is to use a more user-friendly exception message, and I've applied to 3.0-beta@1296061.

          Admin console deployer does not support deploying Geronimo car artifacts. So redeploying the module is kind of a new feature to be implemented, and we can open another feature request jira to track it, close this jira for now.

          If any objection, pls comment and reopen it accordingly.

          Show
          Forrest Xia added a comment - xiezhi's patch is to use a more user-friendly exception message, and I've applied to 3.0-beta@1296061. Admin console deployer does not support deploying Geronimo car artifacts. So redeploying the module is kind of a new feature to be implemented, and we can open another feature request jira to track it, close this jira for now. If any objection, pls comment and reopen it accordingly.
          Hide
          xiezhi added a comment -

          fixed it.

          Show
          xiezhi added a comment - fixed it.
          Hide
          xiezhi added a comment -

          Have post another solution. I think it's better to solve this issue. Before check the sum file, we can do some check if the deploy file is a car. If it is a car, we should stop the deploy process, and tell the user the correct action.

          Show
          xiezhi added a comment - Have post another solution. I think it's better to solve this issue. Before check the sum file, we can do some check if the deploy file is a car. If it is a car, we should stop the deploy process, and tell the user the correct action.

            People

            • Assignee:
              xiezhi
              Reporter:
              Jarek Gawor
            • Votes:
              0 Vote for this issue
              Watchers:
              1 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved:

                Development