Skip to content

Container PyTest suite for Postgresql-container#640

Open
phracek wants to merge 12 commits intosclorg:masterfrom
phracek:container_pytests
Open

Container PyTest suite for Postgresql-container#640
phracek wants to merge 12 commits intosclorg:masterfrom
phracek:container_pytests

Conversation

@phracek
Copy link
Member

@phracek phracek commented Jan 6, 2026 •
edited by github-actions bot
Loading

This pull request adds PyTest suite migrated from run_pytest scripts.

run_container_creation_tests -> test_container_configuration.py
run_general_tests -> test_container_general.py
run_change_password_test -> test_container_password.py
run_replication_test -> test_container_replication.py
run_s2i_test -> test_container_basics.py
run_test_cfg_hook -> test_container_configuration.py
run_s2i_bake_data_test -> test_container_ssl.py
run_s2i_enable_ssl_test -> test_container_ssl.py
run_pgaudit_test -> test_container_extensions.py
run_pgvector_test -> test_container_extensions.py
run_env_extension_load_test -> test_container_extensions.py
run_logging_test -> test_container_logging.py

All tests are migrated except run_migration_test and run_upgrade_test.

@github-actions
Copy link

github-actions bot commented Jan 6, 2026

Pull Request validation

Failed

🔴 Review - Missing review from a member (1 required)

Success

🟢 CI - All checks have passed

@phracek
Copy link
Member Author

phracek commented Jan 6, 2026

Let's try first round
[test-pytest]

@github-actions
Copy link

github-actions bot commented Jan 6, 2026 •
edited
Loading

Testing Farm results

namecomposearchstatusstarted (UTC)timelogs
Fedora - PyTest - 16Fedora-latestx86_64✅ passed24.03.2026 08:04:499min 7stest pipeline
Fedora - PyTest - 15Fedora-latestx86_64❌ error23.03.2026 16:40:0410min 22stest pipeline
CentOS Stream 9 - PyTest - 13CentOS-Stream-9x86_64✅ passed24.03.2026 08:04:4912min 42stest pipeline
CentOS Stream 10 - PyTest - 16CentOS-Stream-10x86_64✅ passed24.03.2026 08:04:4914min 11stest pipeline
CentOS Stream 9 - PyTest - 15CentOS-Stream-9x86_64✅ passed24.03.2026 08:04:4914min 16stest pipeline
CentOS Stream 9 - PyTest - 16CentOS-Stream-9x86_64✅ passed24.03.2026 08:17:0714min 16stest pipeline
RHEL10 - Unsubscribed host - PyTest - 16RHEL-10-Nightlyx86_64✅ passed24.03.2026 08:04:5125min 16stest pipeline
RHEL8 - PyTest - 12RHEL-8.10.0-Nightlyx86_64✅ passed24.03.2026 08:04:4923min 5stest pipeline
RHEL8 - PyTest - 16RHEL-8.10.0-Nightlyx86_64✅ passed24.03.2026 08:15:1416min 55stest pipeline
RHEL9 - Unsubscribed host - PyTest - 13RHEL-9.6.0-Nightlyx86_64✅ passed24.03.2026 08:04:4926min 54stest pipeline
RHEL9 - Unsubscribed host - PyTest - 16RHEL-9.6.0-Nightlyx86_64✅ passed24.03.2026 08:04:4928min 36stest pipeline
RHEL9 - Unsubscribed host - PyTest - 15RHEL-9.6.0-Nightlyx86_64✅ passed23.03.2026 16:40:0521min 56stest pipeline
RHEL9 - PyTest - 15RHEL-9.6.0-Nightlyx86_64❌ error23.03.2026 16:40:0324min 1stest pipeline
RHEL8 - PyTest - 15RHEL-8.10.0-Nightlyx86_64✅ passed24.03.2026 08:04:5023min 46stest pipeline
RHEL9 - PyTest - 13RHEL-9.6.0-Nightlyx86_64✅ passed24.03.2026 08:04:4930min 40stest pipeline
RHEL9 - PyTest - 16RHEL-9.6.0-Nightlyx86_64✅ passed24.03.2026 08:04:4830min 11stest pipeline
RHEL10 - PyTest - 16RHEL-10-Nightlyx86_64✅ passed24.03.2026 08:04:4925min 16stest pipeline
RHEL8 - PyTest - 13RHEL-8.10.0-Nightlyx86_64✅ passed24.03.2026 08:19:0616min 58stest pipeline
Fedora - PyTest - 18Fedora-latestx86_64✅ passed24.03.2026 08:04:4910min 13stest pipeline
CentOS Stream 9 - PyTest - 18CentOS-Stream-9x86_64✅ passed24.03.2026 08:15:1213min 56stest pipeline
RHEL9 - Unsubscribed host - PyTest - 18RHEL-9.6.0-Nightlyx86_64✅ passed24.03.2026 08:04:4928min 18stest pipeline
RHEL10 - PyTest - 18RHEL-10-Nightlyx86_64✅ passed24.03.2026 08:19:1119min 37stest pipeline
RHEL10 - Unsubscribed host - PyTest - 18RHEL-10-Nightlyx86_64✅ passed24.03.2026 08:04:5024min 59stest pipeline
RHEL9 - PyTest - 18RHEL-9.6.0-Nightlyx86_64✅ passed24.03.2026 08:04:4930min 34stest pipeline
CentOS Stream 10 - PyTest - 18CentOS-Stream-10x86_64✅ passed24.03.2026 08:04:5113min 43stest pipeline

@phracek phracek force-pushed the container_pytests branch from d7126c2 to a7a86fd Compare January 7, 2026 13:52
@phracek
Copy link
Member Author

phracek commented Jan 7, 2026

Let's try next round
[test-pytest]

pkhartsk
pkhartsk requested changes Jan 8, 2026
View reviewed changes
pkhartsk
pkhartsk requested changes Jan 14, 2026
View reviewed changes
Copy link
Contributor

@pkhartsk pkhartsk left a comment

Choose a reason for hiding this comment

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

Found some things and included proposed fixes for both

@phracek
Copy link
Member Author

phracek commented Mar 18, 2026

I will rewrite it so that each commit will represent each test suite and ocp_renaming will be a part of another PR.

@phracek phracek mentioned this pull request Mar 19, 2026
Merged
phracek added 9 commits March 23, 2026 15:33
@phracek
Update conftest.py for using in container PyTest
55315b1 
Signed-off-by: Petr "Stone" Hracek <phracek@redhat.com>
@phracek
Add test_container_basics.py testing suite
2c1067d 
run_s2i_test -> test_container_basics.py

Signed-off-by: Petr "Stone" Hracek <phracek@redhat.com>
@phracek
Add testing suite test_container_configuration.py
a1894bc 
run_container_creation_tests -> test_container_configuration.py
run_test_cfg_hook -> test_container_configuration.py

Signed-off-by: Petr "Stone" Hracek <phracek@redhat.com>
@phracek
Add testing suite for extensions test_container_extensions.py
c3bdf8b 
run_pgaudit_test -> test_container_extensions.py
run_pgvector_test -> test_container_extensions.py
run_env_extension_load_test -> test_container_extensions.py

Signed-off-by: Petr "Stone" Hracek <phracek@redhat.com>
@phracek
Add general tests to test_container_generall.py
6da50e5 
run_general_tests -> test_container_general.py

Signed-off-by: Petr "Stone" Hracek <phracek@redhat.com>
@phracek
Add logging test for Postgresql
169bf2d 
run_logging_test -> test_container_logging.py

Signed-off-by: Petr "Stone" Hracek <phracek@redhat.com>
@phracek
Add password test suite
288311d 
run_change_password_test -> test_container_password

Signed-off-by: Petr "Stone" Hracek <phracek@redhat.com>
@phracek
Add test suite for replication
59ad165 
run_replication_test -> test_container_replication.py

Signed-off-by: Petr "Stone" Hracek <phracek@redhat.com>
@phracek
Add test suite for SSL functionality
7eedd72 
run_s2i_bake_data_test -> test_container_ssl.py
run_s2i_enable_ssl_test -> test_container_ssl.py

Signed-off-by: Petr "Stone" Hracek <phracek@redhat.com>
@phracek phracek force-pushed the container_pytests branch from 8f8b048 to 7eedd72 Compare March 23, 2026 14:40
@phracek
Copy link
Member Author

phracek commented Mar 23, 2026

[test-pytest]

@phracek
Fix calling PyTest suite with propoer Python interpreter
cc0693f 
Signed-off-by: Petr "Stone" Hracek <phracek@redhat.com>
@phracek phracek requested a review from frenzymadness March 23, 2026 14:50
@phracek
Copy link
Member Author

phracek commented Mar 23, 2026

[test-pytest]

phracek added 2 commits March 24, 2026 08:38
@phracek
Do not call db output check twice
10e8bbb 
Signed-off-by: Petr "Stone" Hracek <phracek@redhat.com>
@phracek
Let's check_db_output and do not duplicate code
f25566b 
Signed-off-by: Petr "Stone" Hracek <phracek@redhat.com>
@phracek
Copy link
Member Author

phracek commented Mar 24, 2026

[test-pytest]

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@pkhartsk pkhartsk pkhartsk requested changes

@frenzymadness frenzymadness Awaiting requested review from frenzymadness

Requested changes must be addressed to merge this pull request.

Assignees

No one assigned

Labels

pr/missing-review ready for review

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@phracek @pkhartsk