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

[6.14.z & 6.13.z] add test steps, refresh uploaded manifest file #14208

Merged
merged 1 commit into from
Feb 28, 2024
Merged

[6.14.z & 6.13.z] add test steps, refresh uploaded manifest file #14208

merged 1 commit into from
Feb 28, 2024

Conversation

vijaysawant
Copy link
Contributor

Problem Statement

As per comment on original PR #14195 (comment) from @shweta83 we are suppose to use sca-disable org for 6.14.z and lower version.

Solution

I talked with Phoenix team and we decided to use function scope fixture instead of module scope just to avoid any future failure with module scope org

CC: @vsedmik @synkd

Related Issues

NA

@vijaysawant vijaysawant 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.13.z Introduced in or relating directly to Satellite 6.13 6.14.z Introduced in or relating directly to Satellite 6.14 labels Feb 28, 2024
@vijaysawant
Copy link
Contributor Author

trigger: test-robottelo
pytest: tests/foreman/api/test_repository.py -k test_missing_content_id

@vijaysawant
Copy link
Contributor Author

vijaysawant commented Feb 28, 2024 •
edited
Loading 

trigger: test-robottelo
pytest: tests/foreman/api/test_repository.py -k test_missing_content_id
env:
    ROBOTTELO_server__deploy_arguments__deploy_sat_version: '6.14'

@vijaysawant vijaysawant mentioned this pull request Feb 28, 2024
Merged
@vijaysawant vijaysawant requested a review from shweta83 February 28, 2024 13:13
@vijaysawant vijaysawant marked this pull request as ready for review February 28, 2024 13:14
@Satellite-QE
Copy link
Collaborator

PRT Result

Build Number: 5882
Build Status: SUCCESS
PRT Comment: pytest tests/foreman/api/test_repository.py -k test_missing_content_id --external-logging
Test Result : =========== 1 passed, 168 deselected, 1 warning in 733.59s (0:12:13) ===========

@Satellite-QE Satellite-QE added the PRT-Passed Indicates that latest PRT run is passed for the PR label Feb 28, 2024
@Satellite-QE
Copy link
Collaborator

PRT Result

Build Number: 5883
Build Status: SUCCESS
PRT Comment: pytest tests/foreman/api/test_repository.py -k test_missing_content_id --external-logging
Test Result : =========== 1 passed, 168 deselected, 1 warning in 774.91s (0:12:54) ===========

@vsedmik vsedmik removed the 6.14.z Introduced in or relating directly to Satellite 6.14 label Feb 28, 2024
@sambible sambible merged commit fed5206 into SatelliteQE:6.14.z Feb 28, 2024
30 checks passed
github-actions bot pushed a commit that referenced this pull request Feb 28, 2024
@vijaysawant @web-flow
[6.14.z & 6.13.z] add test steps, refresh uploaded manifest file (#14208
845c55d 
)

(cherry picked from commit fed5206)
@Satellite-QE Satellite-QE mentioned this pull request Feb 28, 2024
Merged
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

@sambible sambible sambible approved these changes

@shweta83 shweta83 Awaiting requested review from shweta83

@ogajduse ogajduse Awaiting requested review from ogajduse

Assignees

@vijaysawant vijaysawant

Labels
6.13.z Introduced in or relating directly to Satellite 6.13 AutoMerge_Cherry_Picked The cherrypicked PRs of master PR would be automerged if all checks passing CherryPick PR needs CherryPick to previous branches 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.
4 participants
@vijaysawant @Satellite-QE @sambible @vsedmik