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

Change couchbase-cli to use commas not semicolons for server-add-services

    XMLWordPrintable

Details

    • Bug
    • Resolution: Fixed
    • Critical
    • None
    • 4.0.0
    • tools
    • Security Level: Public
    • None
    • Untriaged
    • Unknown

    Description

      When adding a server to a cluster you have the option to choose which services are added, this is delimited with a semicolon, can this be changed to use a comma instead ?

      Some places in the documentation have

      couchbase-cli server-add -c 10.3.2.204:8091 -u Administrator -p password --server-add=10.3.4.246:8091 --server-add-username=Administrator --server-add-password=password --server-add-services="data;index;query"

      While others have

      Services that run on the server. To specify multiple services, separate the service with a semi-colon. For example, --services=data;index.

      This is going to cause problems for customers as it is an easy mistake to miss off the quotes and I do not believe this should be solved with documentation alone.

      Running this command will execute but the index service will be missing. It's also not possible to add the service to the node afterwards.

      couchbase-cli server-add -c 10.3.2.204:8091 -u Administrator -p password --server-add=10.3.4.246:8091 --server-add-username=Administrator --server-add-password=password --server-add-services=data;index

      The issue is the bash shell will interpret the semicolon as the end of the command line and it will then interpret index as a new command to execute independently.

      Attachments

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

        Activity

          People

            bcui Bin Cui (Inactive)
            ianmccloy Ian McCloy (Inactive)
            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