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

Autofailover: Make CPU throttling safer

    XMLWordPrintable

Details

    • Untriaged
    • 1
    • No

    Description

      Part 1 of this fix is to reduce cpu_throttle.go constants

      • MAX_THROTTLE_ADJUST_MS from 1000 to 100
      • MAX_THROTTLE_DELAY_MS from 10,000 to 100
        as the originals are much too large (e.g. see MB-49501).

      We may need to reduce these further, even up to knocking another factor of 10 off each.

      We may also wish to disable throttling by default (this is my preference as lowest-risk solution, especially as currently lacking evidence that throttling does more good than harm).

      The current MB is to address MB-49501 (Korrigan Clark) and other problems with original CPU throttling.

      Attachments

        Issue Links

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

          Activity

            Build couchbase-server-7.1.0-1759 contains indexing commit d8ba487 with commit message:
            MB-49662 Part 1 (7.1.0 1650): Autofailover: Make CPU throttling safer

            build-team Couchbase Build Team added a comment - Build couchbase-server-7.1.0-1759 contains indexing commit d8ba487 with commit message: MB-49662 Part 1 (7.1.0 1650): Autofailover: Make CPU throttling safer

            Duplicate of MB-49501. I originally was confused and thought that MB was not assigned to GSI, so opened the current one to handle the code changes, then later found they were both in GSI and DUPed the original to the new, but the original has all the detailed info in it so I am reopening it and DUPing this one instead.

            kevin.cherkauer Kevin Cherkauer (Inactive) added a comment - Duplicate of MB-49501 . I originally was confused and thought that MB was not assigned to GSI, so opened the current one to handle the code changes, then later found they were both in GSI and DUPed the original to the new, but the original has all the detailed info in it so I am reopening it and DUPing this one instead.

            I opened this but now DUPed to MB-49501 so closing it.

            kevin.cherkauer Kevin Cherkauer (Inactive) added a comment - I opened this but now DUPed to MB-49501 so closing it.

            People

              kevin.cherkauer Kevin Cherkauer (Inactive)
              kevin.cherkauer Kevin Cherkauer (Inactive)
              Votes:
              0 Vote for this issue
              Watchers:
              1 Start watching this issue

              Dates

                Created:
                Updated:
                Resolved:

                Gerrit Reviews

                  There are no open Gerrit changes

                  PagerDuty