Skip to content

simplfy ssh tunnel logic#4319

Draft
jgao54 wants to merge 1 commit into
mainfrom
ssh-client-refactor
Draft

simplfy ssh tunnel logic#4319
jgao54 wants to merge 1 commit into
mainfrom
ssh-client-refactor

Conversation

@jgao54

@jgao54 jgao54 commented May 18, 2026

Copy link
Copy Markdown
Contributor

No description provided.

@codecov

codecov Bot commented May 18, 2026

Copy link
Copy Markdown

❌ 1 Tests Failed:

Tests completed Failed Passed Skipped
2241 1 2240 203
View the top 1 failed test(s) by shortest run time
::TestMain
Stack Traces | 0s run time
FAIL	github..../flow/connectors/utils [build failed]

To view more test analytics, go to the Test Analytics Dashboard
📋 Got 3 mins? Take this short survey to help us improve Test Analytics.

@github-actions

Copy link
Copy Markdown
Contributor

❌ Test Failure

Analysis: Real bug: ssh_test.go fails to compile because it references the exported field tunnel.Err, but the SSHTunnel struct only has the unexported field err.
Confidence: 0.98

⚠️ This appears to be a real bug - manual intervention needed

View workflow run

@jgao54
jgao54 force-pushed the ssh-client-refactor branch from d30a390 to bde5d53 Compare May 18, 2026 21:32
@github-actions

Copy link
Copy Markdown
Contributor

🔄 Flaky Test Detected

Analysis: The e2e suite hit its 20-minute panic timeout on one matrix variant (mysql-pos) while the other two variants on the same PR passed; the cascade of STATUS_SETUP/STATUS_SNAPSHOT "timeout" failures is the standard signature of in-flight tests being killed by the suite-level deadline rather than a real bug.
Confidence: 0.8

✅ Automatically retrying the workflow

View workflow run

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.

1 participant