glassfish
  1. glassfish
  2. GLASSFISH-16012

RestartRequired: display reason for instances other than DAS

    Details

    • Type: Improvement Improvement
    • Status: Open
    • Priority: Major Major
    • Resolution: Unresolved
    • Affects Version/s: 3.1_dev
    • Fix Version/s: future release
    • Component/s: admin
    • Labels:
      None

      Description

      I cannot find an issue logged for this already, so here it goes. When DAS requires a restart, user can find out what configuration or other changes triggered restart required in Admin Console. However, it is not so for standalone and clustered instances. We should provide this information for all server instances to the users.

        Activity

        Hide
        Anissa Lam added a comment -

        _get-restart-required which provide the reason doesn't support target.

        Usage: asadmin [asadmin-utility-options] _get-restart-required
        [-why[=<why(default:false)>]] [?|--help[=<help(default:false)>]]

        This needs to be supported before GUI can work on this.
        Transfer to 'admin'. Please transfer back to GUI when this is ready so that we can display this to user.

        Show
        Anissa Lam added a comment - _get-restart-required which provide the reason doesn't support target. Usage: asadmin [asadmin-utility-options] _get-restart-required [- why [=<why(default:false)>] ] [ ?|--help [=<help(default:false)>] ] This needs to be supported before GUI can work on this. Transfer to 'admin'. Please transfer back to GUI when this is ready so that we can display this to user.
        Hide
        Tom Mueller added a comment -

        list-instances is the command for getting the status of an instances, and it includes the reasons that a restart is required. It takes a target option so that it is possible to get the status of just one instance.

        However, it doesn't put the status information into properties in the report. The REST interface would need this so that the console can use the information.

        Please evaluate this for possible inclusion in 3.2.

        Show
        Tom Mueller added a comment - list-instances is the command for getting the status of an instances, and it includes the reasons that a restart is required. It takes a target option so that it is possible to get the status of just one instance. However, it doesn't put the status information into properties in the report. The REST interface would need this so that the console can use the information. Please evaluate this for possible inclusion in 3.2.
        Hide
        Tom Mueller added a comment -

        A fix for this issue was initially identified for possible inclusion in the 3.2 release, but after further 3.2 planning, the feature or improvement did not make the cut. This issue is being targeted for a future release. If based on a reevaluation, it is targeted for 3.2, then update the "fix version" again.

        Show
        Tom Mueller added a comment - A fix for this issue was initially identified for possible inclusion in the 3.2 release, but after further 3.2 planning, the feature or improvement did not make the cut. This issue is being targeted for a future release. If based on a reevaluation, it is targeted for 3.2, then update the "fix version" again.

          People

          • Assignee:
            Byron Nevins
            Reporter:
            lidiam
          • Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

            • Created:
              Updated: