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

            Ascending order - Click to sort in descending order
            Hide
            Permalink
            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
            Permalink
            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
            Permalink
            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
            Permalink
            eyang Eric Yang added a comment -

            +1 Looks good.

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

            I just committed this.

            Show
            asrabkin Ari Rabkin added a comment - I just committed this.
            Hide
            Permalink
            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