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

CV failure to remove benchmarking directory upon completion

    XMLWordPrintable

Details

    • Bug
    • Status: Closed
    • Major
    • Resolution: Fixed
    • 6.6.0
    • 6.6.1
    • tools
    • Untriaged
    • 1
    • No

    Description

      What's the issue?
      We do some simple performance testing in our CV, it transiently fails to remove the test directory due to it not being empty (this is a race condition).

      What's the fix?
      Ensure that we close the vBucketBackupWriter before we try to remove the test directory; this ensures the metawriter goroutine doesn't attempt to persist any data whilst we're actively removing the test directory.

      Attachments

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

        Activity

          james.lee James Lee created issue -
          james.lee James Lee made changes -
          Field Original Value New Value
          Status Open [ 1 ] In Progress [ 3 ]
          james.lee James Lee made changes -
          Resolution Fixed [ 1 ]
          Status In Progress [ 3 ] Closed [ 6 ]

          Build couchbase-server-7.0.0-3507 contains backup commit 061f3de with commit message:
          MB-42254 Ensure we close the VBucketBackup reader/writer

          build-team Couchbase Build Team added a comment - Build couchbase-server-7.0.0-3507 contains backup commit 061f3de with commit message: MB-42254 Ensure we close the VBucketBackup reader/writer

          Build couchbase-server-6.6.1-9141 contains backup commit 061f3de with commit message:
          MB-42254 Ensure we close the VBucketBackup reader/writer

          build-team Couchbase Build Team added a comment - Build couchbase-server-6.6.1-9141 contains backup commit 061f3de with commit message: MB-42254 Ensure we close the VBucketBackup reader/writer

          People

            james.lee James Lee
            james.lee James Lee
            Votes:
            0 Vote for this issue
            Watchers:
            1 Start watching this issue

            Dates

              Created:
              Updated:
              Resolved:

              Gerrit Reviews

                There are no open Gerrit changes

                PagerDuty