Uploaded image for project: 'Chukwa'
  1. Chukwa
  2. CHUKWA-259 more flexible CollectorStub
  3. CHUKWA-273

Collector with non default port has default port number appended

    Details

    • Type: Sub-task
    • Status: Resolved
    • Priority: Major
    • Resolution: Fixed
    • Affects Version/s: 0.1.2, 0.2.0
    • Fix Version/s: 0.2.0
    • Component/s: Data Collection
    • Labels:
      None
    • Environment:

      Redhat EL 5.1, Java 6

      Description

      When the collector configuration contains address like:

      http://hostname:5050/

      Agent is currently calling:

      http://hostname:5050/:8080/

      This is working because collector is accepting all URL. If we change the URL to the one specified in the parent issue, then we should fix this to ensure the system would work correctly.

        Activity

        Hide
        asrabkin Ari Rabkin added a comment -

        Lots of broken-ness in that section of code, plus inadaquate test coverage. This should fix problem, and substantially increase test coverage.

        Show
        asrabkin Ari Rabkin added a comment - Lots of broken-ness in that section of code, plus inadaquate test coverage. This should fix problem, and substantially increase test coverage.
        Hide
        asrabkin Ari Rabkin added a comment -

        Bug also affects 0.1.2.

        Show
        asrabkin Ari Rabkin added a comment - Bug also affects 0.1.2.
        Hide
        asrabkin Ari Rabkin added a comment -

        Also, this isn't really a subtask – this patch fixes a bunch of bugs. Notably that we don't respect user-specified port number. At all.

        Show
        asrabkin Ari Rabkin added a comment - Also, this isn't really a subtask – this patch fixes a bunch of bugs. Notably that we don't respect user-specified port number. At all.
        Hide
        eyang Eric Yang added a comment -

        +1 Looks good.

        Show
        eyang Eric Yang added a comment - +1 Looks good.
        Hide
        asrabkin Ari Rabkin added a comment -

        I just committed this.

        Show
        asrabkin Ari Rabkin added a comment - I just committed this.
        Hide
        hudson Hudson added a comment -

        Integrated in Chukwa-trunk #49 (See http://hudson.zones.apache.org/hudson/job/Chukwa-trunk/49/)
        . Agents use correct collector port number.

        Show
        hudson Hudson added a comment - Integrated in Chukwa-trunk #49 (See http://hudson.zones.apache.org/hudson/job/Chukwa-trunk/49/ ) . Agents use correct collector port number.

          People

          • Assignee:
            asrabkin Ari Rabkin
            Reporter:
            eyang Eric Yang
          • Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development