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

[RFE] Updating HostCollections and HostContent to be SCA Only #15398

Merged
merged 4 commits into from
Jun 26, 2024
Merged

[RFE] Updating HostCollections and HostContent to be SCA Only #15398

merged 4 commits into from
Jun 26, 2024

Conversation

ColeHiggins2
Copy link
Member

Problem Statement

Updating Host-Content and Host Collections to be sca only.

Related Issues

SatelliteQE/airgun#1425

@ColeHiggins2 ColeHiggins2 added RFE Issues that are requesting an enhancement to robottelo No-CherryPick PR doesnt need CherryPick to previous branches Stream Introduced in or relating directly to Satellite Stream/Master labels Jun 12, 2024
@ColeHiggins2 ColeHiggins2 self-assigned this Jun 12, 2024
@ColeHiggins2 ColeHiggins2 requested a review from a team as a code owner June 12, 2024 19:48
@ColeHiggins2
Copy link
Member Author

trigger: test-robottelo
pytest: tests/foreman/ui/test_contenthost.py -k test_positive_end_to_end
airgun: 1425

@Satellite-QE
Copy link
Collaborator

PRT Result

Build Number: 7377
Build Status: UNSTABLE
PRT Comment: pytest tests/foreman/ui/test_contenthost.py -k test_positive_end_to_end --external-logging
Test Result : ==== 1 failed, 1 passed, 21 deselected, 142 warnings in 1588.99s (0:26:28) =====

@Satellite-QE Satellite-QE added the PRT-Failed Indicates that latest PRT run is failed for the PR label Jun 12, 2024
vsedmik
vsedmik approved these changes Jun 13, 2024
View reviewed changes
Copy link
Contributor

@vsedmik vsedmik left a comment

Choose a reason for hiding this comment

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

Codewisely looks good to me, just one minor thing.
PRT needs a fix.

tests/foreman/ui/test_hostcollection.py Outdated Show resolved Hide resolved
@ColeHiggins2 ColeHiggins2 force-pushed the update-host-tests-sca-only branch from 03b71b7 to 32de7d1 Compare June 20, 2024 17:39
@ColeHiggins2 ColeHiggins2 force-pushed the update-host-tests-sca-only branch from 32de7d1 to 0cc74f1 Compare June 20, 2024 17:42
@ColeHiggins2 ColeHiggins2 requested a review from vsedmik June 20, 2024 17:54
@damoore044
Copy link
Contributor

trigger: test-robottelo
pytest: tests/foreman/ui/test_contenthost.py::test_positive_end_to_end

@damoore044
Copy link
Contributor

trigger: test-robottelo
pytest: tests/foreman/ui/test_contenthost.py::test_positive_end_to_end[rhel7_contenthost0-module_repos_collection_with_manifest0]

@damoore044
Copy link
Contributor

trigger: test-robottelo
pytest: tests/foreman/ui/test_contenthost.py -k 'test_positive_end_to_end and not test_positive_end_to_end_bulk_update'

@damoore044
Copy link
Contributor

I just realized :BlockedBy: will prevent collection in PRT, disregard those comments lol

@vsedmik vsedmik merged commit 468a0f9 into SatelliteQE:master Jun 26, 2024
8 of 9 checks passed
jyejare pushed a commit to jyejare/robottelo that referenced this pull request Oct 19, 2024
@ColeHiggins2 @jyejare
[RFE] Updating HostCollections and HostContent to be SCA Only (Satell…
647ef13 
…iteQE#15398)

* Updating Host tests to be sca only

* Addressing comments and adding to docstring

* adding merge issue

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

@vsedmik vsedmik vsedmik approved these changes

Assignees

@ColeHiggins2 ColeHiggins2

Labels
No-CherryPick PR doesnt need CherryPick to previous branches PRT-Failed Indicates that latest PRT run is failed for the PR RFE Issues that are requesting an enhancement to robottelo Stream Introduced in or relating directly to Satellite Stream/Master
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@ColeHiggins2 @Satellite-QE @damoore044 @vsedmik