Skip to content

fix gRPC timing issue - #156#157

Merged
leeeunkoo merged 1 commit into
eclipse-pullpiri:mainfrom
youngtaekiim:main
Aug 6, 2025
Merged

fix gRPC timing issue - #156#157
leeeunkoo merged 1 commit into
eclipse-pullpiri:mainfrom
youngtaekiim:main

Conversation

@youngtaekiim
Copy link
Copy Markdown
Contributor

nodeagent

remove unwrap() in gRPC sender and do not send any container list if gRPC server is not available.

[ec2-user@ip-172-31-4-38 src]$ sudo podman logs -f piccolo-agent-nodeagent
Starting NodeAgent on host: HPC
NodeAgentManager init
NodeAgentManager successfully initialized
NodeAgent listening on 0.0.0.0:47004

thread 'tokio-runtime-worker' panicked at nodeagent/src/grpc/sender.rs:38:18:
called Result::unwrap() on an Err value: tonic::transport::Error(Transport, ConnectError(ConnectError("tcp connect error", Os { code: 111, kind: ConnectionRefused, message: "Connection refused" })))
note: run with RUST_BACKTRACE=1 environment variable to display a backtrace
NodeAgentManager stopped

filtergateway

If filtergateway cannot get any scenario info from etcd, return empty scenario instead of Err()

[ec2-user@ip-172-31-4-38 yaml]$ sudo podman logs -f piccolo-player-filtergateway
FilterGatewayManager init
Piccolod gateway listening on 0.0.0.0:47002
Failed to initialize FilterGatewayManager: GRpcStatus(Status { code: Unavailable, message: "tcp connect error: Connection refused (os error 111)", source: Some(tonic::transport::Error(Transport, ConnectError(ConnectError("tcp connect error", Os { code: 111, kind: ConnectionRefused, message: "Connection refused" })))) })

@youngtaekiim youngtaekiim linked an issue Aug 6, 2025 that may be closed by this pull request
@leeeunkoo leeeunkoo merged commit bd302bb into eclipse-pullpiri:main Aug 6, 2025
5 of 6 checks passed
@leeeunkoo leeeunkoo added this to the Refactoring 2nd milestone Aug 6, 2025
chaehee-lim pushed a commit to chaehee-lim/pullpiri that referenced this pull request Sep 3, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: No status

Development

Successfully merging this pull request may close these issues.

[issue]nodeagent, filtergateway connection error

2 participants