Skip to content

Commit ee7a59c

Browse files
committed
increase log verbosity to check CI errors
1 parent f37668e commit ee7a59c

3 files changed

Lines changed: 10 additions & 4 deletions

File tree

.github/workflows/e2e.yaml

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -96,6 +96,8 @@ jobs:
9696
docker tag gcr.io/k8s-staging-kas-network-proxy/proxy-agent:master gcr.io/k8s-staging-kas-network-proxy/proxy-agent-amd64:master
9797
- name: Run e2e tests
9898
run: make test-e2e-ci
99+
100+
99101
e2e:
100102
name: e2e
101103
runs-on: ubuntu-24.04
@@ -217,6 +219,8 @@ jobs:
217219
/usr/local/bin/kubectl wait --timeout=1m --for=condition=ready pods test
218220
/usr/local/bin/kubectl get pods -A -owide
219221
/usr/local/bin/kubectl wait --timeout=1m --for=condition=ready pods --namespace=kube-system -l k8s-app=konnectivity-agent
222+
/usr/local/bin/kubectl get pods -A -owide
223+
sleep 10
220224
/usr/local/bin/kubectl logs test
221225
222226

pkg/agent/client.go

Lines changed: 5 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -568,6 +568,7 @@ func (a *Client) Serve() {
568568
}
569569
}
570570

571+
// TODO: e2e test failure: No "Received data from agent" in server, only ACKS are received
571572
func (a *Client) remoteToProxy(connID int64, eConn *endpointConn) {
572573
defer func() {
573574
if panicInfo := recover(); panicInfo != nil {
@@ -585,7 +586,7 @@ func (a *Client) remoteToProxy(connID int64, eConn *endpointConn) {
585586

586587
for {
587588
n, err := eConn.conn.Read(buf[:])
588-
klog.V(5).InfoS("received data from remote", "bytes", n, "connectionID", connID)
589+
klog.V(4).InfoS("received data from remote", "bytes", n, "connectionID", connID)
589590

590591
if err == io.EOF {
591592
klog.V(2).InfoS("remote connection EOF", "connectionID", connID)
@@ -615,6 +616,7 @@ func (a *Client) remoteToProxy(connID int64, eConn *endpointConn) {
615616
if err := a.Send(resp); err != nil {
616617
klog.ErrorS(err, "could not send DATA", "connectionID", connID)
617618
}
619+
klog.V(4).InfoS("send data to server successfully", "bytes", n, "connectionID", connID)
618620
}
619621
}
620622

@@ -654,11 +656,11 @@ func (a *Client) proxyToRemote(connID int64, eConn *endpointConn) {
654656
resp.Payload = &client.Packet_DataAck{DataAck: &client.DataAck{
655657
ConnectID: connID,
656658
}}
657-
klog.V(4).InfoS("DANIEL: send DATA_ACK", "connectionID", connID)
659+
klog.V(4).InfoS("send DATA_ACK", "connectionID", connID)
658660
if err := a.Send(resp); err != nil {
659661
klog.ErrorS(err, "stream send ack failure")
660662
}
661-
klog.V(4).InfoS("DANIEL: send DATA_ACK SUCCESSFULLY", "connectionID", connID)
663+
klog.V(4).InfoS("send DATA_ACK SUCCESSFULLY", "connectionID", connID)
662664

663665
pos := 0
664666
for {

pkg/server/server.go

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -982,7 +982,7 @@ func (s *ProxyServer) serveRecvBackend(backend *Backend, agentID string, recvCh
982982

983983
case client.PacketType_DATA:
984984
resp := pkt.GetData()
985-
klog.V(5).InfoS("Received data from agent", "bytes", len(resp.Data), "agentID", agentID, "connectionID", resp.ConnectID)
985+
klog.V(4).InfoS("Received data from agent", "bytes", len(resp.Data), "agentID", agentID, "connectionID", resp.ConnectID)
986986
if resp.ConnectID == 0 {
987987
klog.ErrorS(nil, "Received packet missing ConnectID from agent", "packetType", "DATA")
988988
continue

0 commit comments

Comments
 (0)