Uploaded image for project: 'ngrinder'
  1. ngrinder
  2. NGRINDER-85

Make target monitor port is configurable.

    Details

    • Type: Improvement
    • Status: Closed
    • Priority: Minor
    • Resolution: Completed
    • Affects Version/s: 3.0-alpha, 3.0-b1, 3.0
    • Fix Version/s: 3.2
    • Component/s: agent, core
    • Labels:
      None
    • Fix Build Number:
      9d3db63fe7bf1f0b18b664fe331803c0e550f09c

      Description

      target monitor port should be configrable.

      To implement this.. target monitor should contact controller to inform currently used port.

      3.0 version have a field to configure port but ngrinder controller doesn't support it yet.

        Activity

        Hide
        junoyoon JunHo Yoon added a comment -

        Maiwu. I think this issue was solved by you.
        Please note the solution you applied and close the issue.

        Show
        junoyoon JunHo Yoon added a comment - Maiwu. I think this issue was solved by you. Please note the solution you applied and close the issue.
        Hide
        mavlarn Mavlarn Tuohuti added a comment -

        Based on our last discussion, in version 3.0, target monitor port should not be modified, because in controller side, we have no place for this setting.
        In later version, we can modify the page and some logic about "add target host", to let user import target monitor port too.

        Show
        mavlarn Mavlarn Tuohuti added a comment - Based on our last discussion, in version 3.0, target monitor port should not be modified, because in controller side, we have no place for this setting. In later version, we can modify the page and some logic about "add target host", to let user import target monitor port too.
        Hide
        mavlarn Mavlarn Tuohuti added a comment -

        Is deferred to 3.2 version.

        Show
        mavlarn Mavlarn Tuohuti added a comment - Is deferred to 3.2 version.
        Hide
        junoyoon JunHo Yoon added a comment -

        Just fix this by adding the ngrinder.monitor.port in the controller so that controller tries to connect controller using the given port.
        I know it's better to make the port configurable per monitor. But it's complex.

        Just make the whole monitor port is configurable It's minimum impl.

        Show
        junoyoon JunHo Yoon added a comment - Just fix this by adding the ngrinder.monitor.port in the controller so that controller tries to connect controller using the given port. I know it's better to make the port configurable per monitor. But it's complex. Just make the whole monitor port is configurable It's minimum impl.
        Hide
        mavlarn Mavlarn Tuohuti added a comment -

        finished.

        Show
        mavlarn Mavlarn Tuohuti added a comment - finished.
        Hide
        junoyoon JunHo Yoon added a comment -

        monitor.listen.port in system.conf make the monitor port configurable.

        Show
        junoyoon JunHo Yoon added a comment - monitor.listen.port in system.conf make the monitor port configurable.
        Hide
        junoyoon JunHo Yoon added a comment -

        Close All

        Show
        junoyoon JunHo Yoon added a comment - Close All

          People

          • Assignee:
            mavlarn Mavlarn Tuohuti
            Reporter:
            junoyoon JunHo Yoon
          • Votes:
            0 Vote for this issue
            Watchers:
            2 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved: