-
Notifications
You must be signed in to change notification settings - Fork 49
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
Upgrade test coverage to PostgreSQL 14+ #776
Comments
@suxxez testing w/ more recent Timescale versions in CircleCI shouldn't be too difficult: also we should probably update our docker compose files to use more recent versions. What could be time consuming though is debugging tests if something breaks. Unfortunately most of our tests are either end-to-end or integration tests which can make it quite difficult at times to pin down the cause of a failure. (Yes, the majority of our tests should've been unit tests, but that's a story for another day.) re: #700, keep in mind that we've never tried running our test suite on Windows w/ WSL. I reckon you'd be better off using a MacOS or Ubuntu machine if you can. That said, even if you don't get around to submitting a PR, it'd be already very helpful to know which versions of Postgres/Timescale/PostGIS you'll test with and what tests fails, if any. Also, if you could please open an issue for each test that fails and attach the stack trace, that'd be great! Thanks sooo much! |
Hey, thanks for the quick reply @c0c0n3 You're right, it was really easy to set up a CircleCI pipeline. I switched out the deprecated I'll test more versions and keep you updated. :) |
Pipeline was successful with
Not sure what went wrong there, but doesn't look like an issue from the PostgreSQL upgrade. |
We should maybe discuss if all tests for PostgreSQL 12-16 are needed or if 12+13 can be removed. |
closed by #777 |
@suxxez let's keep all the PG versions for now, we can drop 12 and 13 at the end of the year. |
Is your feature request related to a problem? Please describe.
We would like to use the pg_read_all_data functionality and in general stay up-to-date with PostgreSQL versions. We have tried using PG16 with QuantumLeap and it worked, but an official test and support would be great.
Describe the solution you'd like
Extend test coverage to PostgreSQL 14+.
Additional context
I have tried to run the tests locally myself, but failed due to the same reasons as in #700. I might spend some more time digging in and then open a PR myself.
The text was updated successfully, but these errors were encountered: