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

Ensure Backup does not generate DCP connection names that are no longer than allowed by memcached

    XMLWordPrintable

Details

    • Task
    • Status: Closed
    • Major
    • Resolution: Done
    • 6.5.0
    • 7.0.0
    • tools
    • None

    Description

      DCP connections with long names break the STATS "dcp" command. (See MB-34280). cbbackupmgr should generate names that are no longer than the memcached limit.

      Attachments

        Issue Links

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

          Activity

            james.lee James Lee added a comment -

            All the DCP tools in the 'backup' repository rely on the following function to generate the DCP stream names:

            generateStreamName

            func generateStreamName() string {
            	startTime := time.Now()
            	return fmt.Sprintf("%s_%s_%d", version.Cmd, startTime.Format(time.RFC3339), startTime.UnixNano()%100000)
            }
            

            This will result in:
            1) 'cbbackupmgr' / 'cbexport'
            2) '2006-01-02T15:04:05Z07:00'
            3) Variable length integer with a max of 5 characters

            This will result in < 50 characters. Please note that for each DCP worker we postfix '_${WORKER_ID}'; even still, we should be under the 200 character limit.

            james.lee James Lee added a comment - All the DCP tools in the ' backup ' repository rely on the following function to generate the DCP stream names: generateStreamName func generateStreamName() string { startTime := time.Now() return fmt.Sprintf("%s_%s_%d", version.Cmd, startTime.Format(time.RFC3339), startTime.UnixNano()%100000) } This will result in: 1) ' cbbackupmgr ' / ' cbexport ' 2) ' 2006-01-02T15:04:05Z07:00 ' 3) Variable length integer with a max of 5 characters This will result in < 50 characters. Please note that for each DCP worker we postfix ' _${WORKER_ID }'; even still, we should be under the 200 character limit.

            People

              james.lee James Lee
              owend Daniel Owen
              Votes:
              0 Vote for this issue
              Watchers:
              2 Start watching this issue

              Dates

                Created:
                Updated:
                Resolved:

                Gerrit Reviews

                  There are no open Gerrit changes

                  PagerDuty