Uploaded image for project: 'Couchbase Server'
  1. Couchbase Server
  2. MB-48432

Disk usage analyzer alerts shouldn't crash on windows

    XMLWordPrintable

Details

    • Bug
    • Status: Closed
    • Critical
    • Resolution: Fixed
    • master, 7.0.0, 7.0.2, 7.0.1
    • Neo, 7.0.2
    • ns_server
    • Triaged
    • Windows 64-bit
    • 1
    • No

    Description

      When checking if the data in ns_disksup is stale there is a crash because the ets table does not exist. On windows we should not be attempting to access this table because it doesn't exist and shouldn't exist. This alert shouldn't be available on windows at all, in fact. That means we should just be returning false on windows.

      Example:

      Attachments

        No reviews matched the request. Check your Options in the drop-down menu of this sections header.

        Activity

          dfinlay Dave Finlay added a comment -

          This issues actually breaks alerts on Windows. Setting as critical.

          dfinlay Dave Finlay added a comment - This issues actually breaks alerts on Windows. Setting as critical.

          Build couchbase-server-7.0.2-6674 contains ns_server commit 661106e with commit message:
          MB-48432: Disk usage analyzer alerts should be windows only

          build-team Couchbase Build Team added a comment - Build couchbase-server-7.0.2-6674 contains ns_server commit 661106e with commit message: MB-48432 : Disk usage analyzer alerts should be windows only

          Build couchbase-server-7.1.0-1289 contains ns_server commit 661106e with commit message:
          MB-48432: Disk usage analyzer alerts should be windows only

          build-team Couchbase Build Team added a comment - Build couchbase-server-7.1.0-1289 contains ns_server commit 661106e with commit message: MB-48432 : Disk usage analyzer alerts should be windows only

          Bryan McCoid - Can you please help with exact steps for validation.

          ritam.sharma Ritam Sharma added a comment - Bryan McCoid - Can you please help with exact steps for validation.

          Ritam Sharma –  yes. All you have to do is run windows and if you do not see the stack trace above every minute (give or take), it's working. 

          bryan.mccoid Bryan McCoid added a comment - Ritam Sharma –  yes. All you have to do is run windows and if you do not see the stack trace above every minute (give or take), it's working. 

          Tested on 7.1.0 build 1299 

          Verified in all log files and no stack trace present for ns_disksup.

          deepika.verma Deepika Verma (Inactive) added a comment - Tested on 7.1.0 build 1299  Verified in all log files and no stack trace present for ns_disksup.

          People

            ritam.sharma Ritam Sharma
            bryan.mccoid Bryan McCoid
            Votes:
            0 Vote for this issue
            Watchers:
            4 Start watching this issue

            Dates

              Created:
              Updated:
              Resolved:

              Gerrit Reviews

                There are no open Gerrit changes

                PagerDuty