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

pgupgrade: reverting postgresql version to 13 for quay (PROJQUAY-7253) #961

Merged
merged 2 commits into from
Sep 25, 2024

Conversation

deshpandevlab
Copy link
Contributor

@deshpandevlab deshpandevlab commented Sep 23, 2024

This reverts the quay pg upgrade changes in commit 673eff3

Refer #959 (comment) for details.

Testing

  • Tested push-pull
  • Ran e2e test

PG versions queried on quay and clair pods

Quay PG pod:

Found PostgreSQL pod: shudeshp-registry-quay-database-64cb6b96bb-x6vfq
Pod is ready. Executing version query...
                                                    version
---------------------------------------------------------------------------------------------------------------
 PostgreSQL 13.11 on x86_64-redhat-linux-gnu, compiled by gcc (GCC) 11.3.1 20221121 (Red Hat 11.3.1-4), 64-bit

Clair PG pod:

Found PostgreSQL pod: shudeshp-registry-clair-postgres-6b7c7c48cd-rrbld
Pod is ready. Executing version query...
                                                  version
--------------------------------------------------------------------------------------------------------------
PostgreSQL 15.8 on x86_64-redhat-linux-gnu, compiled by gcc (GCC) 11.5.0 20240719 (Red Hat 11.5.0-2), 64-bit
  • validated that the security scan is passed for the tags with this change
Screenshot 2024-09-24 at 10 20 13 PM

@deshpandevlab deshpandevlab self-assigned this Sep 23, 2024
@deshpandevlab deshpandevlab changed the title pgupgrade: reverting postgresqj version to 13 for quay (PROJQUAY-7253) pgupgrade: reverting postgresql version to 13 for quay (PROJQUAY-7253) Sep 23, 2024
@deshpandevlab
Copy link
Contributor Author

/ok-to-test

@deshpandevlab
Copy link
Contributor Author

/retest-required

Copy link

openshift-ci bot commented Sep 24, 2024

@deshpandevlab: The following test failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name Commit Details Required Rerun command
ci/prow/ocp-latest-e2e cab05d3 link true /test ocp-latest-e2e

Full PR test history. Your PR dashboard.

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository. I understand the commands that are listed here.

@deshpandevlab
Copy link
Contributor Author

deshpandevlab commented Sep 25, 2024

The tests are failing for timeouts while getting waiting for quay to get ready. Checking if the test environment is changed. Did not notice any error while deploying locally:
https://shudeshp-registry-quay-quay-enterprise.apps.jonathan-ocp.gcp.quaydev.org/

thanks @bcaton85 for the help with pod logs. The test env needs to be updated to accommodate clair settings

Copy link
Collaborator

@jonathankingfc jonathankingfc left a comment

Choose a reason for hiding this comment

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

LGTM

@jonathankingfc jonathankingfc merged commit c19c3e4 into quay:master Sep 25, 2024
10 of 12 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Development

Successfully merging this pull request may close these issues.

3 participants