Skip to content

Classify CH access denied#4528

Merged
ilidemi merged 2 commits into
mainfrom
classify-ch-permissions
Jul 2, 2026
Merged

Classify CH access denied#4528
ilidemi merged 2 commits into
mainfrom
classify-ch-permissions

Conversation

@ilidemi

@ilidemi ilidemi commented Jul 1, 2026

Copy link
Copy Markdown
Contributor

No description provided.

@ilidemi
ilidemi requested a review from a team as a code owner July 1, 2026 23:38
@claude

claude Bot commented Jul 1, 2026

Copy link
Copy Markdown

Code review

No issues found. Checked for bugs and CLAUDE.md compliance.

@codecov

codecov Bot commented Jul 1, 2026

Copy link
Copy Markdown

❌ 2 Tests Failed:

Tests completed Failed Passed Skipped
2641 2 2639 330
View the top 2 failed test(s) by shortest run time 
github.com/PeerDB-io/peerdb/flow/e2e::TestPeerFlowE2ETestSuiteMySQL_CH
Stack Traces | 0.02s run time 
=== RUN   TestPeerFlowE2ETestSuiteMySQL_CH
=== PAUSE TestPeerFlowE2ETestSuiteMySQL_CH
=== CONT  TestPeerFlowE2ETestSuiteMySQL_CH
--- FAIL: TestPeerFlowE2ETestSuiteMySQL_CH (0.02s)

github.com/PeerDB-io/peerdb/flow/e2e::TestPeerFlowE2ETestSuiteMySQL_CH/Test_MySQL_BinlogIncident
Stack Traces | 34.9s run time 
=== RUN   TestPeerFlowE2ETestSuiteMySQL_CH/Test_MySQL_BinlogIncident
=== PAUSE TestPeerFlowE2ETestSuiteMySQL_CH/Test_MySQL_BinlogIncident
=== CONT  TestPeerFlowE2ETestSuiteMySQL_CH/Test_MySQL_BinlogIncident
2026/07/01 23:56:57 INFO Received AWS credentials from peer for connector: ci x-peerdb-additional-metadata={Operation:FLOW_OPERATION_UNKNOWN}
2026/07/01 23:56:57 INFO Received AWS credentials from peer for connector: clickhouse x-peerdb-additional-metadata={Operation:FLOW_OPERATION_UNKNOWN}
    clickhouse_mysql_test.go:1790: 
        	Error Trace:	.../flow/e2e/clickhouse_mysql_test.go:1790
        	Error:      	Received unexpected error:
        	            	MySQL execute error: readInitialHandshake: io.ReadFull(header) failed. err EOF: connection was bad
        	Test:       	TestPeerFlowE2ETestSuiteMySQL_CH/Test_MySQL_BinlogIncident
2026/07/01 23:57:26 INFO fetched schema x-peerdb-additional-metadata={Operation:FLOW_OPERATION_UNKNOWN} table=e2e_test_mych_un4kcpga.test_nullengine
2026/07/01 23:57:26 INFO fetched schema x-peerdb-additional-metadata={Operation:FLOW_OPERATION_UNKNOWN} table=e2e_test_mych_b5endtuc.test_nullable_column
--- FAIL: TestPeerFlowE2ETestSuiteMySQL_CH/Test_MySQL_BinlogIncident (34.88s)

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

github-actions Bot commented Jul 2, 2026

Copy link
Copy Markdown
Contributor

🔄 Flaky Test Detected

Analysis: The sole failure, Test_MySQL_BinlogIncident, is a transient MySQL handshake connection error ("io.ReadFull(header) failed. err EOF: connection was bad") unrelated to the PR's ClickHouse permission-classification changes, indicating flakiness rather than a real bug.
Confidence: 0.85

✅ Automatically retrying the workflow

View workflow run

@ilidemi
ilidemi merged commit 8306ce3 into main Jul 2, 2026
16 checks passed
@ilidemi
ilidemi deleted the classify-ch-permissions branch July 2, 2026 00:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@jgao54 jgao54 jgao54 approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@ilidemi @jgao54