Geronimo
  1. Geronimo
  2. GERONIMO-6135

Status changed abnormal for default server in mornitoring on admin console

    Details

    • Type: Bug Bug
    • Status: Closed
    • Priority: Minor Minor
    • Resolution: Fixed
    • Affects Version/s: 3.0.0
    • Fix Version/s: 3.0.0, 3.0-beta-2
    • Component/s: monitoring
    • Security Level: public (Regular issues)
    • Labels:
      None
    • Environment:

      JRE 1.6.0 IBM J9 2.4 Windows XP x86-32 jvmwi3260sr9-20110203_74623 (JIT enabled, AOT enabled)

      Description

      1. Use the Aug 23's build of geronimo-3.0 snapshot
      2. Go to the "Monitoring" porlet on admin console
      3. Click the 'Enable Query' link associated with this default server, server works well
      4. Click the "Disable Query" link but found the value of "Stat.Query" for the default server keeps "Stopping..." ,Not changed to "Stopped" until you refreshed this page.

        Issue Links

          Activity

          Hide
          xiezhi added a comment -

          Investigated this jira. Found after clicked the "Disable Query", the server will stop SnopshotThread and set threadstatus=-1. Then the page reload and checked the snapshot status. At this time the thread is still stopping and the status is still -1. So it show "Stopping". When you refresh this page, the thread stopped. So the page show "Stat.Query" status is "Stopped". Because the page just refresh once after click the link. So the status is still be stopping even if the thread has stopped.

          Show
          xiezhi added a comment - Investigated this jira. Found after clicked the "Disable Query", the server will stop SnopshotThread and set threadstatus=-1. Then the page reload and checked the snapshot status. At this time the thread is still stopping and the status is still -1. So it show "Stopping". When you refresh this page, the thread stopped. So the page show "Stat.Query" status is "Stopped". Because the page just refresh once after click the link. So the status is still be stopping even if the thread has stopped.
          Hide
          xiezhi added a comment -

          add a refresh to this page when monitor thread is stopping.

          Show
          xiezhi added a comment - add a refresh to this page when monitor thread is stopping.
          Hide
          Shawn Jiang added a comment -

          1, We should only refresh when the status is in "stopping", don't do the refresh when the status is a stable state.
          2, Basically, the jave script added in the patch have browser compatibility issue. Please double check if it can work correctly on all of the main stream browser. (Firefox/IE/Chrome)

          Show
          Shawn Jiang added a comment - 1, We should only refresh when the status is in "stopping", don't do the refresh when the status is a stable state. 2, Basically, the jave script added in the patch have browser compatibility issue. Please double check if it can work correctly on all of the main stream browser. (Firefox/IE/Chrome)
          Hide
          xiezhi added a comment -

          Totally agree with Shawn. Thank you for your advice.I have updated my patch and tested it on IE/FireFox/Chrome/Safari/Opera.

          Show
          xiezhi added a comment - Totally agree with Shawn. Thank you for your advice.I have updated my patch and tested it on IE/FireFox/Chrome/Safari/Opera.
          Hide
          Forrest Xia added a comment -

          3.0-beta@1300180, trunk@1300182, xiezhi, thank you for this patch!

          Show
          Forrest Xia added a comment - 3.0-beta@1300180, trunk@1300182, xiezhi, thank you for this patch!
          Hide
          xiezhi added a comment -

          I have done some modification.

          Show
          xiezhi added a comment - I have done some modification.
          Hide
          Forrest Xia added a comment -

          3.0-beta@1305776, trunk@1305774, thank you xiezhi for this revised patch!

          Show
          Forrest Xia added a comment - 3.0-beta@1305776, trunk@1305774, thank you xiezhi for this revised patch!
          Hide
          Tina Li added a comment -

          Verified and closed it.

          Show
          Tina Li added a comment - Verified and closed it.

            People

            • Assignee:
              xiezhi
              Reporter:
              Tina Li
            • Votes:
              0 Vote for this issue
              Watchers:
              0 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved:

                Development