Skip to content

ci: remove rwx from ci#924

Draft
meskill wants to merge 2 commits intomainfrom
push-tmpxoryqppmp
Draft

ci: remove rwx from ci#924
meskill wants to merge 2 commits intomainfrom
push-tmpxoryqppmp

Conversation

@meskill
Copy link
Copy Markdown
Contributor

@meskill meskill commented Apr 22, 2026

No description provided.

@codecov
Copy link
Copy Markdown

codecov Bot commented Apr 22, 2026

Codecov Report

❌ Patch coverage is 23.52941% with 52 lines in your changes missing coverage. Please review.

Files with missing lines Patch % Lines
...nd/replication/logical/publisher/publisher_impl.rs 0.00% 45 Missing ⚠️
...c/backend/replication/logical/subscriber/stream.rs 69.56% 7 Missing ⚠️

📢 Thoughts on this report? Let us know!

@meskill meskill force-pushed the push-tmpxoryqppmp branch from 82bd5de to a17559f Compare April 22, 2026 21:58
@blacksmith-sh

This comment has been minimized.

@meskill meskill changed the title ci: add resharding test to ci ci: remove rwx from ci Apr 23, 2026
@blacksmith-sh

This comment has been minimized.

@meskill meskill force-pushed the push-tmpxoryqppmp branch 7 times, most recently from 2ab35a0 to 9bcf206 Compare April 28, 2026 14:25
@blacksmith-sh

This comment has been minimized.

@meskill meskill force-pushed the push-tmpxoryqppmp branch 8 times, most recently from 45fa833 to 440fadb Compare April 29, 2026 20:02
@blacksmith-sh
Copy link
Copy Markdown
Contributor

blacksmith-sh Bot commented Apr 29, 2026

Found 3 test failures on Blacksmith runners:

Failures

Test View Logs
accounts.full_name: destination still has 4 rows ending in -copy after 180s/
accounts.full_name: destination still has 4 rows ending in -copy after 180s
View Logs
tenants.name: destination still has 2 rows ending in -copy after 180s/
tenants.name: destination still has 2 rows ending in -copy after 180s
View Logs
tenants.name: destination still has 3 rows ending in -copy after 180s/
tenants.name: destination still has 3 rows ending in -copy after 180s
View Logs

Fix in Cursor

@meskill meskill force-pushed the push-tmpxoryqppmp branch from 440fadb to 0bcc4ca Compare May 4, 2026 09:22
@meskill meskill force-pushed the push-tmpxoryqppmp branch from 0bcc4ca to 192eee3 Compare May 4, 2026 10:54
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