glassfish
  1. glassfish
  2. GLASSFISH-17772

LogViewerResource need to be able to get logs for other instance

    Details

    • Type: Improvement Improvement
    • Status: Closed
    • Priority: Critical Critical
    • Resolution: Fixed
    • Affects Version/s: 3.1.1
    • Fix Version/s: 3.1.2
    • Component/s: logging
    • Labels:
      None

      Description

      This is for GLASSFISH-17507 Raw Log Viewer.

      Original glassfish logviewer allows to view logs from other server, but Raw Log Viewer (in its first implementation) does not. This is because there are two different web services, /log-view/ and /log-view/details.json, the former is used in raw viewer, because it provides raw log, but the latter is able to retrieve logs from other servers, but it provides structured log details, which seems to be an overkill for the case.

      It is requested that LogViewerResource could provide log from other instances similarly to StructuredLogViewerResource.

      Relevant functionality should be extracted from LogFilter.getLogRecordsUsingQuery and re-used in LogViewerResource. However this should be considered carefully for remote instance, as Raw Log viewer polls quite often (currently every second, may be changed easily in logViewerRaw.jsf).

        Issue Links

          Activity

          Hide
          Anissa Lam added a comment -

          Priority changed to P2.
          This blocks GUI's RFE which is a community feedback. Better user experience is one of the delivery goal for GUI for 3.1.2.

          Show
          Anissa Lam added a comment - Priority changed to P2. This blocks GUI's RFE which is a community feedback. Better user experience is one of the delivery goal for GUI for 3.1.2.
          Hide
          naman_mehta added a comment -

          Started working on the same. Planning to add code in back end to get log files for instance.

          If instance is on local to DAS returning that location.
          If instance is on remote machine first downloading that log file and then returning that location.

          Need changes from GUI to REST for passing instance name which is used by back end code to do above operation.

          Show
          naman_mehta added a comment - Started working on the same. Planning to add code in back end to get log files for instance. If instance is on local to DAS returning that location. If instance is on remote machine first downloading that log file and then returning that location. Need changes from GUI to REST for passing instance name which is used by back end code to do above operation.
          Hide
          naman_mehta added a comment -

          Fixed the same on 3.1.2.

          Show
          naman_mehta added a comment - Fixed the same on 3.1.2.

            People

            • Assignee:
              naman_mehta
              Reporter:
              andriy.zhdanov
            • Votes:
              0 Vote for this issue
              Watchers:
              0 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved: