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.

Sign up for GitHub

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

Jump to bottom

Separate controller for agents list #9317

Closed
intr3p1d opened this issue Oct 24, 2022 · 0 comments · Fixed by #9319, #9368, #9515, #9494 or #9413
Closed

Separate controller for agents list #9317

intr3p1d opened this issue Oct 24, 2022 · 0 comments · Fixed by #9319, #9368, #9515, #9494 or #9413
Assignees
@intr3p1d
Labels
module:web

Comments

@intr3p1d
Copy link
Contributor

agents list was handled in AgentInfoController
First of all, separate the controller and make its response view to static key. (groupName, instancesList)
Then, unify other parts of the pinpoint to use AgentsListController to get any type of agents list (inspector, admin, server map ... etc)
@intr3p1d intr3p1d self-assigned this Oct 24, 2022
intr3p1d added a commit to intr3p1d/pinpoint that referenced this issue Oct 24, 2022
@intr3p1d
[pinpoint-apm#9317] Separate AgentListController
a7894e6
intr3p1d added a commit to intr3p1d/pinpoint that referenced this issue Oct 27, 2022
@intr3p1d
[pinpoint-apm#9317] Separate AgentListController
53ade9d
intr3p1d added a commit to intr3p1d/pinpoint that referenced this issue Oct 31, 2022
@intr3p1d
[pinpoint-apm#9317] Separate AgentListController
0faf54b
intr3p1d added a commit that referenced this issue Nov 1, 2022
@intr3p1d
[#9317] Separate AgentListController
3655608
@intr3p1d intr3p1d linked a pull request Nov 1, 2022 that will close this issue
[#9317] Separate AgentListController #9319
Merged
@intr3p1d intr3p1d closed this as completed Nov 1, 2022
intr3p1d added a commit to intr3p1d/pinpoint that referenced this issue Nov 3, 2022
@intr3p1d
[pinpoint-apm#9317] Change api url
32fed26
intr3p1d added a commit to intr3p1d/pinpoint that referenced this issue Nov 3, 2022
@intr3p1d
[pinpoint-apm#9317] Change api url
e9d0e75
intr3p1d added a commit to intr3p1d/pinpoint that referenced this issue Nov 3, 2022
@intr3p1d
[pinpoint-apm#9317] Change api url
86c3ba1
@intr3p1d intr3p1d linked a pull request Nov 3, 2022 that will close this issue
[#9317] Change AgentListController api url #9368
Merged
intr3p1d added a commit to intr3p1d/pinpoint that referenced this issue Nov 4, 2022
@intr3p1d
[pinpoint-apm#9317] Separate AgentCollectionService
871f6d8
intr3p1d added a commit to intr3p1d/pinpoint that referenced this issue Nov 11, 2022
@intr3p1d
[pinpoint-apm#9317] Change api url
68b51b7
intr3p1d added a commit that referenced this issue Nov 11, 2022
@intr3p1d
[#9317] Change api url
358807e
intr3p1d added a commit to intr3p1d/pinpoint that referenced this issue Nov 14, 2022
@intr3p1d
[pinpoint-apm#9317] Change api url
df74cec
intr3p1d added a commit to intr3p1d/pinpoint that referenced this issue Nov 14, 2022
@intr3p1d
[pinpoint-apm#9317] Change AgentListController api url
c90444b
intr3p1d added a commit that referenced this issue Nov 14, 2022
@intr3p1d
[#9317] Change AgentListController api url
a016b52
intr3p1d added a commit to intr3p1d/pinpoint that referenced this issue Dec 9, 2022
@intr3p1d
[pinpoint-apm#9317] Add default parameter for AgentListController
5bea6f1
intr3p1d added a commit that referenced this issue Dec 9, 2022
@intr3p1d
[#9317] Add default parameter for AgentListController
eabe4a8
intr3p1d added a commit to intr3p1d/pinpoint that referenced this issue Dec 15, 2022
@intr3p1d
[pinpoint-apm#9317] change SortByAgentInfo rules name
5a86df5
binDongKim added a commit to binDongKim/pinpoint that referenced this issue Dec 15, 2022
@binDongKim
[pinpoint-apm#9317] Change api for agent-management page
4c35651
binDongKim added a commit that referenced this issue Dec 15, 2022
@binDongKim
[#9317] Change api for agent-management page
71afd7a
intr3p1d added a commit that referenced this issue Dec 15, 2022
@intr3p1d
[#9317] change SortByAgentInfo rules name
4fd0f5b
BillionaireDY pushed a commit to BillionaireDY/pinpoint that referenced this issue Dec 29, 2022
@intr3p1d @BillionaireDY
[pinpoint-apm#9317] Separate AgentListController
bc0f809
BillionaireDY pushed a commit to BillionaireDY/pinpoint that referenced this issue Dec 29, 2022
@intr3p1d @BillionaireDY
[pinpoint-apm#9317] Change api url
ae7ed09
BillionaireDY pushed a commit to BillionaireDY/pinpoint that referenced this issue Dec 29, 2022
@intr3p1d @BillionaireDY
[pinpoint-apm#9317] Change AgentListController api url
bca4ef0
BillionaireDY pushed a commit to BillionaireDY/pinpoint that referenced this issue Dec 29, 2022
@intr3p1d @BillionaireDY
[pinpoint-apm#9317] Separate AgentListController
275de8e
BillionaireDY pushed a commit to BillionaireDY/pinpoint that referenced this issue Dec 29, 2022
@intr3p1d @BillionaireDY
[pinpoint-apm#9317] Change api url
c6fe139
BillionaireDY pushed a commit to BillionaireDY/pinpoint that referenced this issue Dec 29, 2022
@intr3p1d @BillionaireDY
[pinpoint-apm#9317] Change AgentListController api url
51ec016
binDongKim added a commit to binDongKim/pinpoint that referenced this issue Feb 8, 2023
@binDongKim
[pinpoint-apm#9317] Use agent-list api for the list on main page
ea19361
binDongKim added a commit to binDongKim/pinpoint that referenced this issue Feb 9, 2023
@binDongKim
[pinpoint-apm#9317] Use agent-list api for the list on main page
4ecab4b
binDongKim added a commit that referenced this issue Feb 9, 2023
@binDongKim
[#9317] Use agent-list api for the list on main page
d4cc508
This was linked to pull requests Jul 26, 2023
[#9317] change SortByAgentInfo rules name #9515
Merged
[#9317] Add default parameter for AgentListController #9494
Merged
[#9317] Change AgentListController api url #9413
Merged
[#9317] Separate AgentCollectionService #9376
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@intr3p1d intr3p1d

Labels
module:web
Projects
None yet
Milestone
No milestone
1 participant
@intr3p1d