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.

Sign up for GitHub

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

Jump to bottom

Assert installer status when enabling OpenSCAP #14755

Merged
merged 1 commit into from
Apr 12, 2024
Merged

Assert installer status when enabling OpenSCAP #14755

merged 1 commit into from
Apr 12, 2024

Conversation

lpramuk
Copy link
Contributor

@lpramuk lpramuk commented Apr 12, 2024

Problem Statement

#14693 missed to address changes in installer status check

Solution

Add installer status check in a form of an assertion.

Related Issues

@lpramuk lpramuk requested a review from a team as a code owner April 12, 2024 08:43
@lpramuk lpramuk self-assigned this Apr 12, 2024
@lpramuk lpramuk added CherryPick PR needs CherryPick to previous branches 6.12.z Introduced in or relating directly to Satellite 6.12 AutoMerge_Cherry_Picked The cherrypicked PRs of master PR would be automerged if all checks passing 6.13.z Introduced in or relating directly to Satellite 6.13 6.14.z Introduced in or relating directly to Satellite 6.14 6.15.z Introduced in or relating directly to Satellite 6.15 labels Apr 12, 2024
@lpramuk
Copy link
Contributor Author

lpramuk commented Apr 12, 2024

trigger: test-robottelo
pytest: tests/foreman/api/test_capsule.py::test_positive_update_capsule

@Satellite-QE
Copy link
Collaborator

PRT Result

Build Number: 6476
Build Status: SUCCESS
PRT Comment: pytest tests/foreman/api/test_capsule.py::test_positive_update_capsule --external-logging
Test Result : ================= 1 passed, 13 warnings in 1542.28s (0:25:42) ==================

@Satellite-QE Satellite-QE added the PRT-Passed Indicates that latest PRT run is passed for the PR label Apr 12, 2024
@Gauravtalreja1 Gauravtalreja1 added the Easy Fix :) Easiest Fix to review and quick merge request. label Apr 12, 2024
@Gauravtalreja1 Gauravtalreja1 enabled auto-merge (squash) April 12, 2024 09:24
@Gauravtalreja1 Gauravtalreja1 merged commit f51bc59 into SatelliteQE:master Apr 12, 2024
15 of 16 checks passed
github-actions bot pushed a commit that referenced this pull request Apr 12, 2024
@lpramuk @web-flow
Assert installer status when enabling OpenSCAP (#14755)
6c69d4c 
(cherry picked from commit f51bc59)
github-actions bot pushed a commit that referenced this pull request Apr 12, 2024
@lpramuk @web-flow
Assert installer status when enabling OpenSCAP (#14755)
7997fe4 
(cherry picked from commit f51bc59)
This was referenced Apr 12, 2024
Closed
Merged
Merged
github-actions bot pushed a commit that referenced this pull request Apr 12, 2024
@lpramuk @web-flow
Assert installer status when enabling OpenSCAP (#14755)
2dd1432 
(cherry picked from commit f51bc59)
@Satellite-QE Satellite-QE mentioned this pull request Apr 12, 2024
Merged
@lpramuk lpramuk removed the 6.12.z Introduced in or relating directly to Satellite 6.12 label Apr 12, 2024
jyejare pushed a commit to jyejare/robottelo that referenced this pull request Oct 19, 2024
@lpramuk @jyejare
Assert installer status when enabling OpenSCAP (SatelliteQE#14755)
c7ad154
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@Gauravtalreja1 Gauravtalreja1 Gauravtalreja1 approved these changes

@pondrejk pondrejk Awaiting requested review from pondrejk

@ogajduse ogajduse Awaiting requested review from ogajduse

@Griffin-Sullivan Griffin-Sullivan Awaiting requested review from Griffin-Sullivan

Assignees

@lpramuk lpramuk

Labels
6.13.z Introduced in or relating directly to Satellite 6.13 6.14.z Introduced in or relating directly to Satellite 6.14 6.15.z Introduced in or relating directly to Satellite 6.15 AutoMerge_Cherry_Picked The cherrypicked PRs of master PR would be automerged if all checks passing CherryPick PR needs CherryPick to previous branches Easy Fix :) Easiest Fix to review and quick merge request. PRT-Passed Indicates that latest PRT run is passed for the PR
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@lpramuk @Satellite-QE @Gauravtalreja1