Uploaded image for project: 'Chukwa'
  1. Chukwa
  2. CHUKWA-411

Hosts for DataNode ClientTrace messages parsed incorrectly

    Details

    • Type: Bug
    • Status: Resolved
    • Priority: Major
    • Resolution: Fixed
    • Affects Version/s: 0.3.0
    • Fix Version/s: 0.4.0
    • Component/s: Data Processors
    • Labels:
      None
    • Release Note:
      Fixes bug in FSMBuilder component, DataNodeClientTraceMapper, and correctly pulls source and destination hosts from ClientTrace log messages.

      Description

      DataNode ClientTrace messages parsed incorrectly by the DataNodeClientTraceMapper module of the FSMBuilder. Hosts erroneously reported as 127.0.1.1 as the source host is parsed from the wrong field in the ClientTrace message.

        Activity

        Hide
        asrabkin Ari Rabkin added a comment -

        +1 to commit this. I will do so tomorrow, barring objection.

        But as a warning, it looks like this may break on an IPv6 cluster.

        Show
        asrabkin Ari Rabkin added a comment - +1 to commit this. I will do so tomorrow, barring objection. But as a warning, it looks like this may break on an IPv6 cluster.
        Hide
        tanjiaqi Jiaqi Tan added a comment -

        Good catch Ari, thanks, hadn't considered that. What would an IPv6 address with a prefixed TCP port number look like?

        Show
        tanjiaqi Jiaqi Tan added a comment - Good catch Ari, thanks, hadn't considered that. What would an IPv6 address with a prefixed TCP port number look like?
        Hide
        asrabkin Ari Rabkin added a comment -

        I don't trust myself to answer that. :-/

        Show
        asrabkin Ari Rabkin added a comment - I don't trust myself to answer that. :-/
        Hide
        tanjiaqi Jiaqi Tan added a comment -

        Can we resolve this one and open a new ticket so we don't lose track of the IPv6 issue? Is that an acceptable solution to that?

        Show
        tanjiaqi Jiaqi Tan added a comment - Can we resolve this one and open a new ticket so we don't lose track of the IPv6 issue? Is that an acceptable solution to that?
        Hide
        asrabkin Ari Rabkin added a comment -

        I just committed this. Thanks, Jiaqi. I also opened CHUKWA-412 to keep track of IPv6 issues.

        Show
        asrabkin Ari Rabkin added a comment - I just committed this. Thanks, Jiaqi. I also opened CHUKWA-412 to keep track of IPv6 issues.

          People

          • Assignee:
            tanjiaqi Jiaqi Tan
            Reporter:
            tanjiaqi Jiaqi Tan
          • Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development