Skip to content

(CONT-347) - correct centos/alma/rocky 8 hanging tests #1446

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

Merged
merged 1 commit into from
Jun 19, 2023

Conversation

jordanbreen28
Copy link
Contributor

@jordanbreen28 jordanbreen28 commented Jun 13, 2023

This PR fixes the hanging tests found on el8 based litmus images like centos 8, alma 8 and rocky 8.

This issue was caused by Postgres smart shutdown functionality that would not shut down PGSQL if there was an active client connection.

This PR utilises pg_ctl to allow rspec to terminate those connections before executing the next set of tests.

@jordanbreen28 jordanbreen28 marked this pull request as draft June 13, 2023 10:57
@jordanbreen28 jordanbreen28 force-pushed the CONT-347-address_cancelled_ci_runs branch 21 times, most recently from 6a3d884 to 2ecf852 Compare June 14, 2023 13:29
@jordanbreen28 jordanbreen28 force-pushed the CONT-347-address_cancelled_ci_runs branch 11 times, most recently from 5f18535 to ee5bf23 Compare June 19, 2023 15:01
@jordanbreen28 jordanbreen28 reopened this Jun 19, 2023
@jordanbreen28 jordanbreen28 force-pushed the CONT-347-address_cancelled_ci_runs branch 6 times, most recently from 3486054 to 72e79a1 Compare June 19, 2023 15:41
@jordanbreen28 jordanbreen28 changed the title (WIP) (CONT-347) - correct centos/alma/rocky 8 hanging tests (CONT-347) - correct centos/alma/rocky 8 hanging tests Jun 19, 2023
@jordanbreen28 jordanbreen28 force-pushed the CONT-347-address_cancelled_ci_runs branch from 72e79a1 to edd4a5e Compare June 19, 2023 15:59
@jordanbreen28 jordanbreen28 marked this pull request as ready for review June 19, 2023 16:06
@jordanbreen28 jordanbreen28 force-pushed the CONT-347-address_cancelled_ci_runs branch 2 times, most recently from 38b85d1 to cd3f3f4 Compare June 19, 2023 16:29
@jordanbreen28 jordanbreen28 force-pushed the CONT-347-address_cancelled_ci_runs branch from cd3f3f4 to 2e294c7 Compare June 19, 2023 16:51
@chelnak chelnak merged commit 9c0dae1 into main Jun 19, 2023
@chelnak chelnak deleted the CONT-347-address_cancelled_ci_runs branch June 19, 2023 21:43
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