Skip to content

[6.18.z] Fix puppet plugin enablement test for OpenVox#21961

Merged
lpramuk merged 1 commit into
SatelliteQE:6.18.zfrom
lpramuk:618_fix_puppetplugin_openvox
Jun 26, 2026
Merged

[6.18.z] Fix puppet plugin enablement test for OpenVox#21961
lpramuk merged 1 commit into
SatelliteQE:6.18.zfrom
lpramuk:618_fix_puppetplugin_openvox

Conversation

@lpramuk

@lpramuk lpramuk commented Jun 24, 2026

Copy link
Copy Markdown
Contributor

Cherrypick of PR: #21651

Problem Statement

With OpenVox puppet plugin enablement test is failing at RPM check:

tests/foreman/destructive/test_puppetplugin.py:97: in test_positive_enable_disable_logic
    assert_puppet_status(target_sat, expected=True)
tests/foreman/destructive/test_puppetplugin.py:42: in assert_puppet_status
    assert (result.status == 0) is expected
E   assert (1 == 0) is True
E    +  where 1 = stdout:\npackage puppetserver is not installed\n\nstderr:\n\nstatus: 1.status

Solution

Allow OpenVox or Perforce puppet server to be installed (they both provide puppetserver)

Related Issues

SAT-44510
SAT-45007
SAT-44871

@lpramuk lpramuk added CherryPick PR needs CherryPick to previous branches AutoMerge_Cherry_Picked The cherrypicked PRs of master PR would be automerged if all checks passing 6.18.z Introduced in or relating directly to Satellite 6.18 labels Jun 24, 2026
@lpramuk lpramuk marked this pull request as draft June 24, 2026 13:36
@lpramuk lpramuk force-pushed the 618_fix_puppetplugin_openvox branch 2 times, most recently from d56e3c3 to 3d1d6c5 Compare June 25, 2026 19:07
@lpramuk lpramuk force-pushed the 618_fix_puppetplugin_openvox branch from 3d1d6c5 to 789427c Compare June 25, 2026 19:13
@lpramuk lpramuk marked this pull request as ready for review June 25, 2026 19:14
@lpramuk

lpramuk commented Jun 25, 2026

Copy link
Copy Markdown
Contributor Author
trigger: test-robottelo
pytest: -v tests/foreman/destructive/test_puppetplugin.py
env:
    ROBOTTELO_server__version__source: nightly
    ROBOTTELO_capsule__version__source: nightly

@lpramuk lpramuk added No-CherryPick PR doesnt need CherryPick to previous branches and removed CherryPick PR needs CherryPick to previous branches labels Jun 25, 2026
@Satellite-QE

Copy link
Copy Markdown
Collaborator

PRT Result

Build Number: 15967
Build Status: SUCCESS
PRT Comment: pytest -v tests/foreman/destructive/test_puppetplugin.py --external-logging
Test Result : ================= 1 passed, 13 warnings in 3959.15s (1:05:59) ==================

@Satellite-QE Satellite-QE added the PRT-Passed Indicates that latest PRT run is passed for the PR label Jun 25, 2026
@lpramuk lpramuk merged commit e53f4cf into SatelliteQE:6.18.z Jun 26, 2026
17 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

6.18.z Introduced in or relating directly to Satellite 6.18 AutoMerge_Cherry_Picked The cherrypicked PRs of master PR would be automerged if all checks passing No-CherryPick PR doesnt need CherryPick to previous branches PRT-Passed Indicates that latest PRT run is passed for the PR

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants