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

cbbackupmgr start and end validations is a bit too aggressive

    XMLWordPrintable

Details

    • Triaged
    • 1
    • Yes

    Description

      Problem
      In 7.0.0 this command fails where it works on 6.6.0:

      2021-08-20T15:43:47 ERROR Subprocess call exited with non-zero return code 1
      2021-08-20T15:43:47 ERROR Arguments: cbbackupmgr restore --archive /data/backups --repo test-couchbase-dxqst-2021-08-20T15_43_04 --cluster http://test-couchbase-dxqst:8091 --username Administrator --password password --start latest --end latest --no-progress-bar --include-data default-0 --threads 4
      2021-08-20T15:43:47 ERROR Stdout: b'Error restoring cluster: invalid keyword usage start/oldest must come before end/latest\n'
      

      This is one of the only cases where having latest or end as the --start is valid. The same is true for --end with oldest or start.

      Suggestion

      Should validate that the key word is correct, then check that --start is older or the same as --end.

      Attachments

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

        Activity

          owend Daniel Owen added a comment -

          Approved for 7.0.2 by Arunkumar Senthilnathan via email

          owend Daniel Owen added a comment - Approved for 7.0.2 by Arunkumar Senthilnathan via email

          Build couchbase-server-7.0.2-6565 contains backup commit baf9dee with commit message:
          MB-48081 Relax start/end keyword validation

          build-team Couchbase Build Team added a comment - Build couchbase-server-7.0.2-6565 contains backup commit baf9dee with commit message: MB-48081 Relax start/end keyword validation

          Build couchbase-server-7.1.0-1187 contains backup commit baf9dee with commit message:
          MB-48081 Relax start/end keyword validation

          build-team Couchbase Build Team added a comment - Build couchbase-server-7.1.0-1187 contains backup commit baf9dee with commit message: MB-48081 Relax start/end keyword validation

          Confirmed this now works when start and end are both latest/oldest. Closing.

          joe.mitchelljones Joe Mitchell Jones added a comment - Confirmed this now works when start and end are both latest/oldest. Closing.

          People

            pvarley Patrick Varley
            pvarley Patrick Varley
            Votes:
            0 Vote for this issue
            Watchers:
            4 Start watching this issue

            Dates

              Created:
              Updated:
              Resolved:

              Gerrit Reviews

                There are no open Gerrit changes

                PagerDuty