Skip to content

Implement StateManager with gRPC support and periodic container info gathering and change detection in NodeAgentManager to send to StateManager#143

Merged
youngtaekiim merged 3 commits into
eclipse-pullpiri:refactoringfrom
daeyoung-jeong-lge:refactoring
Jul 29, 2025
Merged

Conversation

@daeyoung-jeong-lge
Copy link
Copy Markdown
Contributor

url: #142

feat(state-manager): Implement gRPC communication for ContainerList updates and manage state transitions

  • Add statemanager with asynchronous operation ready.
  • Implement background task in NodeAgentManager to periodically gather container info using inspect().
  • Send current container list to the monitoring server via gRPC.
  • Detect changes in container list and, if changed, send updated list to the state manager via gRPC.
  • Log container list changes and errors during gRPC communication.

feat(state-manager): Implement gRPC communication for ContainerList updates and manage state transitions
@leeeunkoo leeeunkoo added enhancement New feature or request rust Pull requests that update rust code labels Jul 25, 2025
@leeeunkoo leeeunkoo added this to the Refactoring 2nd milestone Jul 25, 2025
@daeyoung-jeong-lge
Copy link
Copy Markdown
Contributor Author

@youngtaekiim Please give me some feedback or merge this.

Comment thread src/common/proto/statemanager.proto
Comment thread src/player/statemanager/src/grpc/receiver.rs Outdated
Comment thread src/player/statemanager/src/main.rs Outdated
@youngtaekiim
Copy link
Copy Markdown
Contributor

complete 1st review, 영작이 너무 오래 걸려서 한글로 진행함

@daeyoung-jeong-lge
Copy link
Copy Markdown
Contributor Author

@youngtaekiim 1st review comments are handled and the code is commited. Please check again.

@youngtaekiim
Copy link
Copy Markdown
Contributor

@youngtaekiim 1st review comments are handled and the code is commited. Please check again.

There is one more comment in src/agent/nodeagent/src/manager.rs

@daeyoung-jeong-lge
Copy link
Copy Markdown
Contributor Author

@youngtaekiim 1st review comments are handled and the code is commited. Please check again.

There is one more comment in src/agent/nodeagent/src/manager.rs

I can't find the comment on src/agent/nodeagent/src/manager.rs. Can you make a comment again?

Comment thread src/agent/nodeagent/src/manager.rs
@youngtaekiim youngtaekiim merged commit f2bcf06 into eclipse-pullpiri:refactoring Jul 29, 2025
5 of 6 checks passed
chaehee-lim pushed a commit to chaehee-lim/pullpiri that referenced this pull request Sep 3, 2025
refactor: Improve code formatting and organization in StateManager files
chaehee-lim pushed a commit to chaehee-lim/pullpiri that referenced this pull request Sep 3, 2025
…ring

Implement StateManager with gRPC support and periodic container info gathering and change detection in NodeAgentManager to send to StateManager
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request rust Pull requests that update rust code

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

[TASK] Implement of statemanager and connect between nodeagent and statemanager.

4 participants