Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix running PG upgrade tests with run_test.py #6829

Merged
merged 1 commit into from
Apr 24, 2023
Merged

Conversation

JelteF
Copy link
Contributor

@JelteF JelteF commented Apr 5, 2023

In #6814 we started using the Python test runner for upgrade tests in
run_test.py, instead of the Perl based one. This had a problem though,
not all tests in minimal_schedule can be run with the Python runner.
This adds a separate minimal schedule for the pg_upgrade tests which
doesn't include the tests that break with the Python runner.

This PR also fixes various other issues that came up while testing
the upgrade tests.

@codecov
Copy link

codecov bot commented Apr 5, 2023

Codecov Report

Merging #6829 (cb6a2df) into main (a6a7271) will decrease coverage by 1.01%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##             main    #6829      +/-   ##
==========================================
- Coverage   93.27%   92.26%   -1.01%     
==========================================
  Files         269      269              
  Lines       57448    57447       -1     
==========================================
- Hits        53584    53003     -581     
- Misses       3864     4444     +580     

@JelteF JelteF requested a review from agedemenli April 6, 2023 06:53
@JelteF JelteF force-pushed the run-test-pg-upgrade-fix branch from 6eccdbd to 1b1818c Compare April 24, 2023 12:29
@JelteF JelteF requested a review from onurctirtir April 24, 2023 12:32
@JelteF JelteF force-pushed the run-test-pg-upgrade-fix branch from 1b1818c to d1a3c2e Compare April 24, 2023 12:59
In #6814 we started using the Python test runner for upgrade tests in
run_test.py, instead of the Perl based one. This had a problem though,
not all tests in minimal_schedule can be run with the Python runner.
This adds a separate minimal schedule for the pg_upgrade tests which
doesn't include the tests that break with the Python runner.
@JelteF JelteF force-pushed the run-test-pg-upgrade-fix branch from d1a3c2e to cb6a2df Compare April 24, 2023 13:41
Copy link
Member

@onurctirtir onurctirtir left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Verified that this helps flaky test detection test in #6866 to pass (once I rebase it), thank you.

@JelteF JelteF merged commit a5f4fec into main Apr 24, 2023
@JelteF JelteF deleted the run-test-pg-upgrade-fix branch April 24, 2023 13: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.

2 participants