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

Cgroups memory used is not reported

    XMLWordPrintable

Details

    • Bug
    • Status: Closed
    • Critical
    • Resolution: Fixed
    • 7.1.0, 7.1.1
    • 7.1.2
    • ns_server
    • Untriaged
    • 1
    • Unknown

    Description

      Currently we don’t report cgroups mem used at all and sys_mem_limit (which is currently min(mem total, cgroup mem limit) doesn't make much sense, because from user's point of view it's super confusing. It's not clear what it can be compared to: host memory used or to cgroups memory used (which is not even reported now).

      Instead I suggest we do the following:

      • Add sys_mem_cgroup_used
      • Add sys_mem_cgroup_limit
      • Remove sys_mem_limit

      Notes:
      sys_mem_cgroup_limit will be missing completely if cgroups is not used or if memory cg controller is not used. If memory cg controller is there but the limit is not set, it will show 9223372036854771712 (0x7FFFFFFFFFFFF000))
      sys_mem_cgroup_used will be missing completely if cgroups is not used or if memory cg controller is not used.

      UPDATE:
      Decided not to remove sys_mem_limit in 7.1.2 for backward compat reasons. Create MB-53124 for this.

      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.2-3396 contains ns_server commit 9dbbfd7 with commit message:
            MB-53120: Report mem_cgroups_limit and mem_cgroups_used

            build-team Couchbase Build Team added a comment - Build couchbase-server-7.1.2-3396 contains ns_server commit 9dbbfd7 with commit message: MB-53120 : Report mem_cgroups_limit and mem_cgroups_used

            Build couchbase-server-7.2.0-1729 contains ns_server commit 9dbbfd7 with commit message:
            MB-53120: Report mem_cgroups_limit and mem_cgroups_used

            build-team Couchbase Build Team added a comment - Build couchbase-server-7.2.0-1729 contains ns_server commit 9dbbfd7 with commit message: MB-53120 : Report mem_cgroups_limit and mem_cgroups_used

            Build couchbase-server-8.0.0-1067 contains ns_server commit 9dbbfd7 with commit message:
            MB-53120: Report mem_cgroups_limit and mem_cgroups_used

            build-team Couchbase Build Team added a comment - Build couchbase-server-8.0.0-1067 contains ns_server commit 9dbbfd7 with commit message: MB-53120 : Report mem_cgroups_limit and mem_cgroups_used

            Validated the newly added stats on 7.1.2-3402.

            Also added this validation as part of automation tests.

            ashwin.govindarajulu Ashwin Govindarajulu added a comment - Validated the newly added stats on 7.1.2-3402. Also added this validation as part of automation tests.

            People

              ashwin.govindarajulu Ashwin Govindarajulu
              timofey.barmin Timofey Barmin
              Votes:
              0 Vote for this issue
              Watchers:
              3 Start watching this issue

              Dates

                Created:
                Updated:
                Resolved:

                Gerrit Reviews

                  There are no open Gerrit changes

                  PagerDuty