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

[CE] Error should show correct information in initial cluster with --index-storage-setting

    XMLWordPrintable

Details

    • Bug
    • Status: Closed
    • Major
    • Resolution: Fixed
    • 5.5.0
    • 5.5.0
    • tools
    • Centos 7.4 64-bit
    • Untriaged
    • Centos 64-bit
    • Unknown

    Description

      Problem

      When couchbase-cli us used again a CE to initial the cluster with the incorrect --index-storage-setting setting. The error produced says to use forestdb instead. However the couchbase-cli does not accept forestdb as a option.
      The right option is default, the error message should make this clear as this cause confusion for the user.

      Steps to reproduce

      1. Install Couchbase Server CE 5.5.0-2907 on a centos 7.4 server
      2. Using couchbase-cli to initialize this node with --index-storage-setting set to memopt which will fail as expected:

        [root@s44015 ~]# /opt/couchbase/bin/couchbase-cli cluster-init -c 127.0.0.1 --cluster-username Administrator --cluster-password password --services fts,data,query,index --index-storage-setting memopt
         
        ERROR: storageMode - The value must be one of the following: [forestdb]
        

      3. Try agin using forestdb it fails again unexpectedly

        [root@s44015 ~]# /opt/couchbase/bin/couchbase-cli cluster-init -c 127.0.0.1 --cluster-username Administrator --cluster-password password --services fts,data,query,index --index-storage-setting forestdb
        

         

      Attachments

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

        Activity

          Code is up for review.

          pvarley Patrick Varley added a comment - Code is up for review.

          Build couchbase-server-6.0.0-1266 contains couchbase-cli commit 6693e64 with commit message:
          MB-30198 Throw an error if mmeopt index storage type is used on CE

          build-team Couchbase Build Team added a comment - Build couchbase-server-6.0.0-1266 contains couchbase-cli commit 6693e64 with commit message: MB-30198 Throw an error if mmeopt index storage type is used on CE

          Build couchbase-server-6.5.0-1017 contains couchbase-cli commit 6693e64 with commit message:
          MB-30198 Throw an error if mmeopt index storage type is used on CE

          build-team Couchbase Build Team added a comment - Build couchbase-server-6.5.0-1017 contains couchbase-cli commit 6693e64 with commit message: MB-30198 Throw an error if mmeopt index storage type is used on CE

          Raju Suravarjjala Can you approve this change for Vulcan please? It's a very small change and it will avoid the need to branch and backport for MB-30320 and MB-30334 which have already been approved.

          pvarley Patrick Varley added a comment - Raju Suravarjjala Can you approve this change for Vulcan please? It's a very small change and it will avoid the need to branch and backport for MB-30320 and MB-30334 which have already been approved.
          raju Raju Suravarjjala added a comment - - edited

          Patrick Varley Approved, please check in the fix

          raju Raju Suravarjjala added a comment - - edited Patrick Varley Approved, please check in the fix

          Build couchbase-server-5.5.0-2950 contains couchbase-cli commit 6693e64 with commit message:
          MB-30198 Throw an error if mmeopt index storage type is used on CE

          build-team Couchbase Build Team added a comment - Build couchbase-server-5.5.0-2950 contains couchbase-cli commit 6693e64 with commit message: MB-30198 Throw an error if mmeopt index storage type is used on CE

          Build couchbase-server-6.0.0-1339 contains couchbase-cli commit 6693e64 with commit message:
          MB-30198 Throw an error if mmeopt index storage type is used on CE

          build-team Couchbase Build Team added a comment - Build couchbase-server-6.0.0-1339 contains couchbase-cli commit 6693e64 with commit message: MB-30198 Throw an error if mmeopt index storage type is used on CE

          People

            pvarley Patrick Varley
            thuan Thuan Nguyen
            Votes:
            0 Vote for this issue
            Watchers:
            6 Start watching this issue

            Dates

              Created:
              Updated:
              Resolved:

              Gerrit Reviews

                There are no open Gerrit changes

                PagerDuty