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

Test template sync using proxy #16879

Open
wants to merge 8 commits into
base: master
Choose a base branch
from

Conversation

lhellebr
Copy link
Contributor

@lhellebr lhellebr commented Nov 6, 2024

Tests for of SAT-27349.

UI part requires SatelliteQE/airgun#1621

Draft because it contains a workaround for discovered regressions and because the feature hasn't been merged yet.

@lhellebr lhellebr changed the title Template sync proxy UI Template sync proxy Nov 6, 2024
@lhellebr lhellebr changed the title Template sync proxy Test template sync using proxy Nov 6, 2024
@lhellebr lhellebr added the No-CherryPick PR doesnt need CherryPick to previous branches label Nov 6, 2024
@lhellebr
Copy link
Contributor Author

trigger: test-robottelo
pytest: tests/foreman/api/test_templatesync.py::TestTemplateSyncTestCase::test_positive_export_all_templates_to_repo tests/foreman/cli/test_templatesync.py::TestTemplateSyncTestCase::test_positive_import_dir_filtered tests/foreman/ui/test_templatesync.py::test_positive_import_templates
airgun: 1621

@lhellebr lhellebr marked this pull request as ready for review November 11, 2024 16:08
@lhellebr lhellebr requested review from a team as code owners November 11, 2024 16:08
@Satellite-QE
Copy link
Collaborator

PRT Result

Build Number: 9325
Build Status: UNSTABLE
PRT Comment: pytest tests/foreman/api/test_templatesync.py::TestTemplateSyncTestCase::test_positive_export_all_templates_to_repo tests/foreman/cli/test_templatesync.py::TestTemplateSyncTestCase::test_positive_import_dir_filtered tests/foreman/ui/test_templatesync.py::test_positive_import_templates --external-logging
Test Result : ====== 8 failed, 4 passed, 16 skipped, 180 warnings in 2224.96s (0:37:04) ======

@Satellite-QE Satellite-QE added the PRT-Failed Indicates that latest PRT run is failed for the PR label Nov 11, 2024
@lhellebr
Copy link
Contributor Author

lhellebr commented Nov 11, 2024

PRT results as expected. Tests where proxy is used fail. Tests where it is not used pass. I expect everything to start passing once PR theforeman/foreman_templates#191 is merged in Foreman.

Note: SSH tests are currently being skipped due to SAT-28933.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Do Not Merge No-CherryPick PR doesnt need CherryPick to previous branches PRT-Failed Indicates that latest PRT run is failed for the PR
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants