Skip to content

fix(sender): update StateManagerConnectionClient connection to use master IP from config (url : #330)#338

Merged
youngtaekiim merged 1 commit intoeclipse-pullpiri:mainfrom
daeyoung-jeong-lge:main
Oct 14, 2025
Merged

fix(sender): update StateManagerConnectionClient connection to use master IP from config (url : #330)#338
youngtaekiim merged 1 commit intoeclipse-pullpiri:mainfrom
daeyoung-jeong-lge:main

Conversation

@daeyoung-jeong-lge
Copy link
Contributor

related to #330

There're "Failed to connect" errors when sending changed container list from NodeAgent to StateManager. Because NodeAgent lies on each Node, NodeAgent needs some other approach to connect to StateManager.

@daeyoung-jeong-lge
Copy link
Contributor Author

@leeeunkoo @youngtaekiim
This is a patch for the communication error between NodeAgent and StateManager that occurred during the demo test with Cloud. Please review and merge this. Thanks.
Cloud와의 데모 시험 중 발생한 NodeAgent와 StateManager간 통신 오류에 대한 패치입니다. 검토 후 머지 부탁드립니다.

@youngtaekiim
Copy link
Contributor

수정한 부분은 명확합니다. merge 하도록 하겠습니다.

그러나 이 commit에는 포함되지 않은 config.rs 는 궁금한 점이 있습니다.

#[derive(Debug, Deserialize, Clone, Default, PartialEq)]
pub struct Config {
    pub nodeagent: NodeAgentConfig,
}

config struct 가 추후 확장될 여지가 있어서 두 번 포장을 하였지 아니면 nodeagent가 아닌 다른 module 에서도 config 쓰는 곳이 있어서 동일한 형식으로 했는지...

@youngtaekiim youngtaekiim merged commit c220646 into eclipse-pullpiri:main Oct 14, 2025
6 checks passed
@daeyoung-jeong-lge
Copy link
Contributor Author

수정한 부분은 명확합니다. merge 하도록 하겠습니다.

그러나 이 commit에는 포함되지 않은 config.rs 는 궁금한 점이 있습니다.

#[derive(Debug, Deserialize, Clone, Default, PartialEq)]
pub struct Config {
    pub nodeagent: NodeAgentConfig,
}

config struct 가 추후 확장될 여지가 있어서 두 번 포장을 하였지 아니면 nodeagent가 아닌 다른 module 에서도 config 쓰는 곳이 있어서 동일한 형식으로 했는지...

아래 commit에서 추가된 사항인데 아마 은구 책임님께서 copilot agent를 이용해서 추가한 코드로 보입니다.
84b0034

@leeeunkoo leeeunkoo linked an issue Oct 14, 2025 that may be closed by this pull request
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.

[BUG] On the dashboard screen, the ContainerInfo keeps changing

2 participants