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

Improve the error message given when "upload to couchbase" site is not reachable

    XMLWordPrintable

Details

    • Bug
    • Status: Closed
    • Minor
    • Resolution: Fixed
    • 5.1.1, 5.5.0
    • 6.5.0
    • ns_server

    Description

      When a user provides the site address to upload the logs to, ns_server performs a reachability check. If the check fails, the error message returned contains reason that is a nested Erlang tuple. The ask is to see if can be made user friendly.

      Attachments

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

        Activity

          The return value that we get is a nested tuple and there is no standard way to map this to a user-friendly string. Will have to write a custom mapper to handle this and I don't want to do it at this juncture. Moving it over to mad-hatter.

          ajit.yagaty Ajit Yagaty [X] (Inactive) added a comment - The return value that we get is a nested tuple and there is no standard way to map this to a user-friendly string. Will have to write a custom mapper to handle this and I don't want to do it at this juncture. Moving it over to mad-hatter.

          Verified the issue still exists on mad-hatter.....

          Failed to check reachability of https://s3.not-amazonaws.com/cb-engineering/stevewatanabe/: {nxdomain, [{lhttpc_client, send_request, 1, [

          {file, "/Users/stevewatanabe/couchbase/couchdb/src/lhttpc/lhttpc_client.erl"}, {line, 220}]}, {lhttpc_client, execute, 9, [{file, "/Users/stevewatanabe/couchbase/couchdb/src/lhttpc/lhttpc_client.erl"}

          , {line, 169}]}, {lhttpc_client, request, 9, [

          {file, "/Users/stevewatanabe/couchbase/couchdb/src/lhttpc/lhttpc_client.erl"}

          , {line, 92}]}]}

          steve.watanabe Steve Watanabe added a comment - Verified the issue still exists on mad-hatter..... Failed to check reachability of https://s3.not-amazonaws.com/cb-engineering/stevewatanabe/: {nxdomain, [{lhttpc_client, send_request, 1, [ {file, "/Users/stevewatanabe/couchbase/couchdb/src/lhttpc/lhttpc_client.erl"}, {line, 220}]}, {lhttpc_client, execute, 9, [{file, "/Users/stevewatanabe/couchbase/couchdb/src/lhttpc/lhttpc_client.erl"} , {line, 169}]}, {lhttpc_client, request, 9, [ {file, "/Users/stevewatanabe/couchbase/couchdb/src/lhttpc/lhttpc_client.erl"} , {line, 92}]}]}

          Could possibly "look" for nxdomain and customize the error.  Would need to see what other errors are returned.

          steve.watanabe Steve Watanabe added a comment - Could possibly "look" for nxdomain and customize the error.  Would need to see what other errors are returned.

          The change under development will return this error instead of the erlang backtrace.

           

          Unable to access 'https://s3.amazonzzzaws.com/cb-customers/stevewatanabe/' : "Domain name does not exist"

          steve.watanabe Steve Watanabe added a comment - The change under development will return this error instead of the erlang backtrace.   Unable to access ' https://s3.amazonzzzaws.com/cb-customers/stevewatanabe/' : "Domain name does not exist"

          Build couchbase-server-6.5.0-3744 contains ns_server commit f49c12c with commit message:
          MB-28513 Improve error when upload site unreachable

          build-team Couchbase Build Team added a comment - Build couchbase-server-6.5.0-3744 contains ns_server commit f49c12c with commit message: MB-28513 Improve error when upload site unreachable
          bharath.gp Bharath G P added a comment -

          Validated on 6.5.0-3770

          bharath.gp Bharath G P added a comment - Validated on 6.5.0-3770

          People

            steve.watanabe Steve Watanabe
            ajit.yagaty Ajit Yagaty [X] (Inactive)
            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