Skip to content

[release-19.0] DML test fix for duplicate column value#18132

Merged
dbussink merged 2 commits intovitessio:release-19.0from
planetscale:backport-17980-release-19.0
Apr 8, 2025
Merged

[release-19.0] DML test fix for duplicate column value#18132
dbussink merged 2 commits intovitessio:release-19.0from
planetscale:backport-17980-release-19.0

Conversation

@harshit-gangal
Copy link
Copy Markdown
Member

Description

This is a backport of #17980

…vitessio#17980) (vitessio#17987)

Signed-off-by: Harshit Gangal <harshit@planetscale.com>
Co-authored-by: vitess-bot[bot] <108069721+vitess-bot[bot]@users.noreply.github.com>
@vitess-bot vitess-bot bot added NeedsBackportReason If backport labels have been applied to a PR, a justification is required NeedsDescriptionUpdate The description is not clear or comprehensive enough, and needs work NeedsIssue A linked issue is missing for this Pull Request NeedsWebsiteDocsUpdate What it says labels Apr 8, 2025
@harshit-gangal harshit-gangal added Type: Bug Component: Query Serving Type: Testing Backport This is a backport and removed NeedsDescriptionUpdate The description is not clear or comprehensive enough, and needs work NeedsWebsiteDocsUpdate What it says NeedsIssue A linked issue is missing for this Pull Request NeedsBackportReason If backport labels have been applied to a PR, a justification is required labels Apr 8, 2025
@harshit-gangal harshit-gangal added this to the v20.0.7 milestone Apr 8, 2025
@harshit-gangal
Copy link
Copy Markdown
Member Author

This PR is needed to make CI pass on release-20.0

@vitessio vitessio deleted a comment from vitess-bot bot Apr 8, 2025
@harshit-gangal harshit-gangal removed this from the v20.0.7 milestone Apr 8, 2025
@codecov
Copy link
Copy Markdown

codecov bot commented Apr 8, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 65.20%. Comparing base (8ef0cae) to head (8beb893).
Report is 1 commits behind head on release-19.0.

Additional details and impacted files
@@              Coverage Diff              @@
##           release-19.0   #18132   +/-   ##
=============================================
  Coverage         65.20%   65.20%           
=============================================
  Files              1561     1561           
  Lines            238836   238836           
=============================================
+ Hits             155729   155730    +1     
+ Misses            83107    83106    -1     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

Signed-off-by: Harshit Gangal <harshit@planetscale.com>
@dbussink dbussink merged commit 20959b2 into vitessio:release-19.0 Apr 8, 2025
103 checks passed
@dbussink dbussink deleted the backport-17980-release-19.0 branch April 8, 2025 08:16
tanjinx pushed a commit to slackhq/vitess that referenced this pull request Jan 26, 2026
tanjinx added a commit to slackhq/vitess that referenced this pull request Jan 26, 2026
vitessio#18132) (#780)

* [release-19.0] DML test fix for duplicate column value (vitessio#18132)

* fix: enable Percona tools repository before ps80 setup

Percona repository structure now requires the tools repository to be
enabled before setting up ps80. This fixes the CI error:
"Selected product uses 'ps-80 tools' repositories. But the 'tools'
repository is disabled"

Co-Authored-By: Claude <svc-devxp-claude@slack-corp.com>

---------

Co-authored-by: Harshit Gangal <harshit@planetscale.com>
Co-authored-by: Claude <svc-devxp-claude@slack-corp.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants