-
Notifications
You must be signed in to change notification settings - Fork 0
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
feat: Dynamic agent registeration & multiple stomp topic & scheduler feature #60
Conversation
오늘도 잠을 안자시고 해주셨군요,, |
…nto feature/template-info-builder
헙 시간이 벌써! |
저도 요번 주말엔 열심히 달립니다! :) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
요번 코드를 보면서 느낀건 service 로직에 있는 method 명을 직관적으로 잘 지으신거 같아요.
내부 로직은 어떤지 모르지만 method만 보고 대충 어떤 작업을 하겠구나 느낌이 잘와서 좋았습니다!!
저도 method 명 지을 때 참고좀 해야 겠네요.
수고 많으셨습니다!! :) (정말 많이 배우고 있습니다.ㅎㅎ)
HttpSender htps = httpSender.get(id); | ||
|
||
LocalDateTime now = LocalDateTime.now(); | ||
List<Double> percentiles = PreftestConsts.percentiles; | ||
|
||
// 1초마다 TestResult 를 보내는 스케줄러 시작 | ||
scheduledTaskService.start(id, () -> { | ||
LocalDateTime cur = LocalDateTime.now(); | ||
LocalDateTime c = LocalDateTime.now(); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
변수명을 c로 지으신 이유가 있을까요?!
어떤 의미가 있는지 쉽게 판단이 안됩니당!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
앗 급하게 하다보니 잘못 적었네요ㅜㅜ 바로 수정하겠습니다!
📝 Test code-coverage reports
|
Implement
bm-agent
)?action=start
to specific agentTESTING
,READY
spring cloud 의 eureka 가 추가되었습니다! 여러 bm-agent 를 eureka 에 등록하고 bm-controller 가 이를 읽고 bm-agent 의 실 주소를 확인할 수 있습니다.
Developer log
@LeeJeongGi
./gradlew build
후 docker compose up 하시면 eureka, bm-controller, bm-agent 1 2 가 자동으로 실행됩니다 :)만약 Intellij 에서 실행하고 싶으시다면, eureka 실행 -> controller, bm-agent 실행 하시면 됩니다.