Uploaded image for project: 'Couchbase PHP client library'
  1. Couchbase PHP client library
  2. PCBC-543

timeouts should not be logged by default

    XMLWordPrintable

Details

    • Bug
    • Status: Resolved
    • Blocker
    • Resolution: Fixed
    • 2.4.6
    • 2.6.2
    • library
    • None
    • 1

    Description

      When investigating tracing in PHP, I note that even though my exceptions have info, the timeouts log through PHP by default.

      [cb,WARN] (server L:505 I:-1833806363) <127.0.0.1:11210> (CTX=0x7ff334830190,memcached,SRV=0x7ff334953a00,IX=0) Failing command (pkt=0x7ff33580a000, opaque=4789, opcode=0x0) with error LCB_ETIMEDOUT (0x17)
      [cb,EROR] (server L:582 I:-1833806363) <127.0.0.1:11210> (CTX=0x7ff334830190,memcached,SRV=0x7ff334953a00,IX=0) Server timed out. Some commands have failed
      [cb,WARN] (server L:360 I:-1833806363) <127.0.0.1:11210> (CTX=0x7ff334830190,memcached,SRV=0x7ff334953a00,IX=0) Server sent us reply for a timed-out command. (OP=0x0, RC=0x0, SEQ=4789)
      

      The first one I think shouldn't be there at all. It should just be returned to the app. The second one seems to be the same as the first, but at error level. I know this is lcb generated and I saw roughly the same thing. The third is from tracing, and shouldn't be logged at WARN level from lcb. Maybe at DEBUG level, but otherwise it should only be in the threshold tracing logger.

      Attachments

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

        Activity

          ingenthr Matt Ingenthron created issue -

          Build couchbase-server-6.0.0-1153 contains libcouchbase commit a0c81d1 with commit message:
          PCBC-543: Update log levels for timed out operations

          build-team Couchbase Build Team added a comment - Build couchbase-server-6.0.0-1153 contains libcouchbase commit a0c81d1 with commit message: PCBC-543 : Update log levels for timed out operations

          Build couchbase-server-5.5.0-2759 contains libcouchbase commit a0c81d1 with commit message:
          PCBC-543: Update log levels for timed out operations

          build-team Couchbase Build Team added a comment - Build couchbase-server-5.5.0-2759 contains libcouchbase commit a0c81d1 with commit message: PCBC-543 : Update log levels for timed out operations

          Build libcouchbase-2.8.5-178 contains libcouchbase commit a0c81d1 with commit message:
          PCBC-543: Update log levels for timed out operations

          build-team Couchbase Build Team added a comment - Build libcouchbase-2.8.5-178 contains libcouchbase commit a0c81d1 with commit message: PCBC-543 : Update log levels for timed out operations
          avsej Sergey Avseyev made changes -
          Field Original Value New Value
          Fix Version/s 2.5.1 [ 15206 ]
          Fix Version/s 2.5.0 [ 15200 ]
          avsej Sergey Avseyev made changes -
          Status New [ 10003 ] Open [ 1 ]
          avsej Sergey Avseyev made changes -
          Fix Version/s 2.5.1 [ 15233 ]
          Fix Version/s 2.5.0 [ 15206 ]
          avsej Sergey Avseyev made changes -
          Fix Version/s 2.5.2 [ 15315 ]
          Fix Version/s 2.5.1 [ 15233 ]
          avsej Sergey Avseyev made changes -
          Fix Version/s 2.5.3 [ 15409 ]
          Fix Version/s 2.5.2 [ 15315 ]
          avsej Sergey Avseyev made changes -
          Fix Version/s 2.5.4 [ 15604 ]
          Fix Version/s 2.5.3 [ 15409 ]

          Sergey Avseyev: looks like this is actually integrated/resolved. Can you close accordingly?

          ingenthr Matt Ingenthron added a comment - Sergey Avseyev : looks like this is actually integrated/resolved. Can you close accordingly?

          Yes, the timeouts logged by libcouchbase

          avsej Sergey Avseyev added a comment - Yes, the timeouts logged by libcouchbase
          avsej Sergey Avseyev made changes -
          Resolution Fixed [ 1 ]
          Status Open [ 1 ] Resolved [ 5 ]
          avsej Sergey Avseyev made changes -
          Actual End 2019-07-01 09:48 (issue has been resolved)
          brett19 Brett Lawson made changes -
          Story Points 1

          People

            avsej Sergey Avseyev
            ingenthr Matt Ingenthron
            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