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

remove goXDCR keys from internalSettings

    XMLWordPrintable

Details

    • Bug
    • Status: Resolved
    • Major
    • Resolution: Fixed
    • 4.5.0
    • 4.5.0
    • ns_server
    • Untriaged
    • Unknown

    Description

      DOC-1245 - deprecate go_xdcr api's from internalSettings in 4.5

      GET/POST goxdcr api's with urls /internalSettings and /settings/replications return and allow to modify the same configuration parameters, which are just default values for replication configuration.

      With recent RBAC changes ns_server /internalSettings api that currently encapsulates goxdcr /internalSettings api is available only to built in Administrator. This creates following issue: replication_admin user is not able to modify the defaults via UI, but is able to do it using /settings/replications api.

      Here's what I suggest:
      1. Remove /internalSettings api from goxdcr
      2. So xdcr related settings will no longer appear in Internal Settings dialog

      That should not affect customers since Internal Settings are well... internal, so should be used only by recommendation of couchbase tech support.

      Customers will still be able to modify defaults using REST API (and CLI?)

      Attachments

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

        Activity

          Resolving on-behalf of Artem. Arun can you please verify.

          anil Anil Kumar (Inactive) added a comment - Resolving on-behalf of Artem. Arun can you please verify.

          This one is not fixed, because it is blocked.

          artem Artem Stemkovski added a comment - This one is not fixed, because it is blocked.

          Artem, please assign it back to me once it is fixed and resolved so that I can verify it

          arunkumar Arunkumar Senthilnathan added a comment - Artem, please assign it back to me once it is fixed and resolved so that I can verify it

          I cannot fix this one because it will break tests. You need to fix CBQE-3385 first!!!!!

          artem Artem Stemkovski added a comment - I cannot fix this one because it will break tests. You need to fix CBQE-3385 first!!!!!

          Artem Stemkovski CBQE-3385 is fixed now

          arunkumar Arunkumar Senthilnathan added a comment - Artem Stemkovski CBQE-3385 is fixed now

          People

            artem Artem Stemkovski
            artem Artem Stemkovski
            Votes:
            0 Vote for this issue
            Watchers:
            7 Start watching this issue

            Dates

              Created:
              Updated:
              Resolved:

              Gerrit Reviews

                There are no open Gerrit changes

                PagerDuty