Uploaded image for project: 'Solr'
  1. Solr
  2. SOLR-11036

MetricsHandler should report disk stats for solr.data.home

    Details

    • Type: Improvement
    • Status: Closed
    • Priority: Major
    • Resolution: Fixed
    • Affects Version/s: 7.0
    • Fix Version/s: 7.0, 7.1, master (8.0)
    • Component/s: metrics, SolrCloud
    • Security Level: Public (Default Security Level. Issues are Public)
    • Labels:

      Description

      The totalSpace and usableSpace reported by Metrics API are still based on coreRootDirectory (which is used as the instance dir for individual cores). SOLR-6671 introduced a new solr.data.home configuration for the root directory of all data dirs. So we should expose data home's disk metrics as well.

      Since the current metric names are just totalSpace and usableSpace under the Container group, we need to figure out how to differentiate between them and the data home's metrics. A few options:

      1. We can introduce new metrics such as dataHomeTotalSpace and dataHomeUsableSpace.
      2. We use the current names for the data home's metrics and introduce new ones for coreRootDirectory

      I lean towards #2 because of backcompat reasons. Existing users (and monitoring vendors/products) would be using the existing metric names to track disk stats for their solr installations. If a user configures a solr.data.home then they shouldn't get a surprise one day if the data home directory runs out of space when they have monitoring already in place which used to work for earlier versions of Solr. The probability of surprise still exists because one fine day you can run out of space for the coreRootDirectory and solr will refuse to create any cores but that is a less likely scenario than running out of disk space because of increasing size of data directories.

        Attachments

        1. SOLR-11036.patch
          4 kB
          Andrzej Bialecki

          Issue Links

            Activity

              People

              • Assignee:
                ab Andrzej Bialecki
                Reporter:
                shalinmangar Shalin Shekhar Mangar
              • Votes:
                0 Vote for this issue
                Watchers:
                3 Start watching this issue

                Dates

                • Created:
                  Updated:
                  Resolved: