Uploaded image for project: 'Couchbase Kubernetes'
  1. Couchbase Kubernetes
  2. K8S-2426

Update logging sidecar to parse mapreduce_errors.log

    XMLWordPrintable

Details

    • Improvement
    • Status: Closed
    • Minor
    • Resolution: Fixed
    • None
    • 2.3.0
    • documentation, logging
    • None
    • 1

    Description

      Several issues (e.g. overlong keys) with the Views system can only be identified through mapreduce_errors.log.

      Attachments

        Issue Links

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

          Activity

            Marks Polakovs can you send me some example logs to make sure I cover the specific bits you need? Make sure they able to be put in the OSS repo (nothing sensitive).

            patrick.stephens Patrick Stephens (Inactive) added a comment - Marks Polakovs  can you send me some example logs to make sure I cover the specific bits you need? Make sure they able to be put in the OSS repo (nothing sensitive).

            Patrick Stephens Looks to me like they're the same erlang_multiline syntax - here's a line from my local testing (will see if I can get some others later):

            [mapreduce_errors:error,2021-09-29T11:18:39.517Z,couchdb_ns_1@cb.local:<0.18870.1>:couch_set_view_updater:-do_maps/3-fun-0-:885]Bucket `travel-sample`, main group `_design/dev_test`, error mapping document `<ud>test</ud>` for view `repro`: <ud>too long key emitted: 4806 bytes (line 2:2)</ud>
            

            marks.polakovs Marks Polakovs added a comment - Patrick Stephens Looks to me like they're the same erlang_multiline syntax - here's a line from my local testing (will see if I can get some others later): [mapreduce_errors:error,2021-09-29T11:18:39.517Z,couchdb_ns_1@cb.local:<0.18870.1>:couch_set_view_updater:-do_maps/3-fun-0-:885]Bucket `travel-sample`, main group `_design/dev_test`, error mapping document `<ud>test</ud>` for view `repro`: <ud>too long key emitted: 4806 bytes (line 2:2)</ud>

            So just an extra tail input with the same parser?

            patrick.stephens Patrick Stephens (Inactive) added a comment - So just an extra tail input with the same parser?

            In theory that should be sufficient - I’ll let you know if I can find any reason why it shouldn’t

            marks.polakovs Marks Polakovs added a comment - In theory that should be sufficient - I’ll let you know if I can find any reason why it shouldn’t

            Build couchbase-fluent-bit-1.1.1-104 contains couchbase-fluent-bit commit bf17f4f with commit message:
            K8S-2426: Update fluent bit and add support for missing logs

            build-team Couchbase Build Team added a comment - Build couchbase-fluent-bit-1.1.1-104 contains couchbase-fluent-bit commit bf17f4f with commit message: K8S-2426 : Update fluent bit and add support for missing logs

            Can you confirm this is ok for 2.2.1 and 2.3+ once it pops up?

            patrick.stephens Patrick Stephens (Inactive) added a comment - Can you confirm this is ok for 2.2.1 and 2.3+ once it pops up?

            People

              roo.thorp Roo Thorp
              marks.polakovs Marks Polakovs
              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