Add serverHost and serverPort to command monitoring events

XMLWordPrintableJSON

    • Type: Spec Change
    • Resolution: Unresolved
    • Priority: Unknown
    • None
    • Component/s: Observability
    • None
    • Needed
    • Hide

      Summary of necessary driver changes

      •  

      Commits for syncing spec/prose tests
      (and/or refer to an existing language POC if needed)

      •  

      Context for other referenced/linked tickets

      •  
      Show
      Summary of necessary driver changes   Commits for syncing spec/prose tests (and/or refer to an existing language POC if needed)   Context for other referenced/linked tickets  
    • $i18n.getText("admin.common.words.hide")
      Key Status/Resolution FixVersion
      CDRIVER-6379 Blocked
      CXX-3543 Blocked
      CSHARP-6141 Blocked
      GODRIVER-4058 Blocked
      JAVA-6262 Blocked
      NODE-7728 Blocked
      PYTHON-5969 Blocked
      PHPLIB-1919 Blocked
      RUBY-3924 Blocked
      RUST-2462 Blocked
      $i18n.getText("admin.common.words.show")
      #scriptField, #scriptField *{ border: 1px solid black; } #scriptField{ border-collapse: collapse; } #scriptField td { text-align: center; /* Center-align text in table cells */ } #scriptField td.key { text-align: left; /* Left-align text in the Key column */ } #scriptField a { text-decoration: none; /* Remove underlines from links */ border: none; /* Remove border from links */ } /* Add green background color to cells with FixVersion */ #scriptField td.hasFixVersion { background-color: #00FF00; /* Green color code */ } #scriptField td.willNotDo { background-color: #FF0000; /* Red color code */ } /* Center-align the first row headers */ #scriptField th { text-align: center; } Key Status/Resolution FixVersion CDRIVER-6379 Blocked CXX-3543 Blocked CSHARP-6141 Blocked GODRIVER-4058 Blocked JAVA-6262 Blocked NODE-7728 Blocked PYTHON-5969 Blocked PHPLIB-1919 Blocked RUBY-3924 Blocked RUST-2462 Blocked

      Summary

      The command monitoring events expose a connectionID but not serverHost or serverPort. The connectionID always takes the form addr[-N] which forces users to strip the [-N] to get the host and port. For example: https://github.com/open-telemetry/opentelemetry-go-contrib/pull/9352

      Additionally, command monitoring is asymmetric over command logging. Command logging logs serverHost and serverPort, along with serverConnectionID.

      Motivation

      Who is the affected end user?

      Users of command monitoring

      How does this affect the end user?

      This is not a blocker, just a QoL improvement

      How likely is it that this problem or use case will occur?

      Unclear

      If the problem does occur, what are the consequences and how severe are they?

      Users have to create custom parsers to strip [-N] from connectionID

      Is this issue urgent?

      No

      Is this ticket required by a downstream team?

      No

      Is this ticket only for tests?

      No

      Acceptance Criteria

      Add serverHost and serverPort to CommandStartedEvent, CommandSucceededEvent, and CommandFailedEvent.

            Assignee:
            Unassigned
            Reporter:
            Preston Vasquez
            Votes:
            0 Vote for this issue
            Watchers:
            2 Start watching this issue

              Created:
              Updated: