Uploaded image for project: 'Couchbase Server'
  1. Couchbase Server
  2. MB-41928 Deprecate CAPI in CC
  3. MB-42163

CLI to remove "capi" replication option

    XMLWordPrintable

    Details

    • Type: Technical task
    • Status: Resolved
    • Priority: Major
    • Resolution: Fixed
    • Affects Version/s: None
    • Fix Version/s: None
    • Component/s: tools
    • Labels:
      None
    • Story Points:
      1

      Description

      Given CAPI deprecation, CLI should no longer allow user the ability to create CAPI replications, nor the ability to modify replication type to CAPI, etc.

      man pages should remove mentions of CAPI, if any exists.

        Attachments

          Issue Links

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

            Activity

            Hide
            carlos.gonzalez Carlos Gonzalez Betancort (Inactive) added a comment -

            Neil Huang I just want to confirm that this means that the only option know is xmem is that correct?

            Show
            carlos.gonzalez Carlos Gonzalez Betancort (Inactive) added a comment - Neil Huang I just want to confirm that this means that the only option know is xmem is that correct?
            Hide
            neil.huang Neil Huang added a comment -

            That is correct

            Show
            neil.huang Neil Huang added a comment - That is correct
            Hide
            neil.huang Neil Huang added a comment -

            Does CLI require user to explicitly pass in "xmem"? If so, is there any reason why the CLI can't make the specifying xmem optional?

            Show
            neil.huang Neil Huang added a comment - Does CLI require user to explicitly pass in "xmem"? If so, is there any reason why the CLI can't make the specifying xmem optional?
            Hide
            carlos.gonzalez Carlos Gonzalez Betancort (Inactive) added a comment -

            If it is the only option then it probaly makes sense to remove the flag althought his would break backwards compatibilty without warning. Do we need to set the type field of the request or as xmem is the only option will XDCR default to that anyway?

            Show
            carlos.gonzalez Carlos Gonzalez Betancort (Inactive) added a comment - If it is the only option then it probaly makes sense to remove the flag althought his would break backwards compatibilty without warning. Do we need to set the type field of the request or as xmem is the only option will XDCR default to that anyway?
            Hide
            neil.huang Neil Huang added a comment -

            I believe passing in "xmem" is optional

            However, "replicationType=continuous" is required for legacy reasons.

            Show
            neil.huang Neil Huang added a comment - I believe passing in "xmem" is optional However, "replicationType=continuous" is required for legacy reasons.
            Hide
            carlos.gonzalez Carlos Gonzalez Betancort (Inactive) added a comment -

            TO avoid backwards compatibility issues I have properly deprecated teh flag and willl 100% remove it in 7.1 (MB-42207)

            Show
            carlos.gonzalez Carlos Gonzalez Betancort (Inactive) added a comment - TO avoid backwards compatibility issues I have properly deprecated teh flag and willl 100% remove it in 7.1 ( MB-42207 )
            Hide
            build-team Couchbase Build Team added a comment -

            Build couchbase-server-7.0.0-3546 contains couchbase-cli commit 823f830 with commit message:
            MB-42163 Ignore deprecated replication mode flag

            Show
            build-team Couchbase Build Team added a comment - Build couchbase-server-7.0.0-3546 contains couchbase-cli commit 823f830 with commit message: MB-42163 Ignore deprecated replication mode flag

              People

              Assignee:
              carlos.gonzalez Carlos Gonzalez Betancort (Inactive)
              Reporter:
              neil.huang Neil Huang
              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