glassfish
  1. glassfish
  2. GLASSFISH-2644

Dynamic reloading a webapp through GUI

    Details

    • Type: Improvement Improvement
    • Status: Open
    • Priority: Major Major
    • Resolution: Unresolved
    • Affects Version/s: 9.1pe
    • Fix Version/s: not determined
    • Component/s: admin_gui
    • Labels:
      None
    • Environment:

      Operating System: All
      Platform: All

    • Issuezilla Id:
      2,644

      Description

      The administration GUI interface (admin-gui) does not allow the administrator to
      reload web applications without doing it manually.

      Glassfish has dynamic reloading by touching a .reload file in the
      domains/domain1/applications/j2ee-modules/webAppName context root directory.

      At first, one could allow the administrator to reload a web application by
      clicking a "Reload" link which is next to the "Launch | Redeploy" links at the
      same line as the web application in the 'DeployTable.inc'. Dynamic reloading
      (touch .reload) should be used since it is currently stable.

      1. patch_issue_2644.patch
        6 kB
        azawawi
      2. patch_issue_2644.txt
        6 kB
        azawawi

        Activity

        Hide
        azawawi added a comment -

        Created an attachment (id=799)
        Patch to implement webapp dynamic reloading

        Show
        azawawi added a comment - Created an attachment (id=799) Patch to implement webapp dynamic reloading
        Hide
        azawawi added a comment -

        Created an attachment (id=801)
        Modified patch file (please use this)

        Show
        azawawi added a comment - Created an attachment (id=801) Modified patch file (please use this)
        Hide
        Anissa Lam added a comment -

        Thanks for submitting the patch.
        We will test it out and check that in.

        Show
        Anissa Lam added a comment - Thanks for submitting the patch. We will test it out and check that in.
        Hide
        Anissa Lam added a comment -

        comments from the deployment team:
        1. The .reload functionality is only supported for archive deployment. So if the
        admin console does not limit this use only to archive deployment, the code
        itself needs to check for whether this module is directory deployed.
        2. The .reload functionality is only supported for DAS and not for EE/cluster
        environment at this point.
        -------------------

        For #1, GUI should do the checking and put up the 'reload' button in the table
        when it applies
        To address #2, this reload enhancement will ONLY available for developer
        profile, will not put this in cluster profile.

        Senthil, please change the code accordingly before commit.

        azawaai, do you have any comment ?

        Show
        Anissa Lam added a comment - comments from the deployment team: 1. The .reload functionality is only supported for archive deployment. So if the admin console does not limit this use only to archive deployment, the code itself needs to check for whether this module is directory deployed. 2. The .reload functionality is only supported for DAS and not for EE/cluster environment at this point. ------------------- For #1, GUI should do the checking and put up the 'reload' button in the table when it applies To address #2, this reload enhancement will ONLY available for developer profile, will not put this in cluster profile. Senthil, please change the code accordingly before commit. azawaai, do you have any comment ?
        Hide
        azawawi added a comment -

        Thanks for your help. If you need any help to modify my patch, i would be happy
        to do it.

        Show
        azawawi added a comment - Thanks for your help. If you need any help to modify my patch, i would be happy to do it.
        Hide
        Jason Lee added a comment -

        Updating version

        Show
        Jason Lee added a comment - Updating version
        Hide
        Tom Mueller added a comment -

        Bulk update to change fix version to "not determined" for all issues still open but with a fix version for a released version.

        Show
        Tom Mueller added a comment - Bulk update to change fix version to "not determined" for all issues still open but with a fix version for a released version.

          People

          • Assignee:
            cchidamb
            Reporter:
            azawawi
          • Votes:
            1 Vote for this issue
            Watchers:
            2 Start watching this issue

            Dates

            • Created:
              Updated: