Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add description about agent connection port to docker README.md #953

Merged
merged 1 commit into from
Mar 29, 2023

Conversation

jinan159
Copy link
Contributor

@jinan159 jinan159 commented Jan 15, 2023

I want to add description about purpose of 16001 port.
I referenced the org.ngrinder.starter.NGrinderControllerStarter class of ngrinder-controller module.

@jinan159 jinan159 changed the title Add agent connection port to docker README.md Add description about agent connection port to docker README.md Jan 15, 2023
@junoyoon
Copy link
Contributor

junoyoon commented Mar 6, 2023

@donggyu04 plz review this and merge

@@ -42,6 +42,8 @@ Port information:

* __80__: Default controller web UI port.

* __16001__: Controller port for agent connection.
Copy link
Contributor

@donggyu04 donggyu04 Mar 29, 2023

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

thx, it is set into agent configurations automatically when you download the agent.

@imbyungjun imbyungjun merged commit 5267a1b into naver:develop Mar 29, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants