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

PIT markers for Interoperability Testing: CapsuleContent #15892

Merged
merged 1 commit into from
Aug 15, 2024
Merged

PIT markers for Interoperability Testing: CapsuleContent #15892

merged 1 commit into from
Aug 15, 2024

Conversation

damoore044
Copy link
Contributor

Proposing PIT markers added to these CapsuleContent tests

@damoore044 damoore044 added CherryPick PR needs CherryPick to previous branches Stream Introduced in or relating directly to Satellite Stream/Master 6.15.z Introduced in or relating directly to Satellite 6.15 labels Aug 8, 2024
@damoore044 damoore044 requested a review from a team August 8, 2024 12:54
@damoore044 damoore044 requested a review from a team as a code owner August 8, 2024 12:54
@damoore044
Copy link
Contributor Author

trigger: test-robottelo
pytest: tests/foreman/api/test_capsulecontent.py -k 'test_positive_sync_updated_repo or test_positive_capsule_sync and not test_positive_capsule_sync_ or test_positive_sync_container_repo_end_to_end'

@Satellite-QE
Copy link
Collaborator

PRT Result

Build Number: 8032
Build Status: UNSTABLE
PRT Comment: pytest tests/foreman/api/test_capsulecontent.py -k test_positive_sync_updated_repo or test_positive_capsule_sync and not test_positive_capsule_sync_ or test_positive_sync_container_repo_end_to_end --external-logging
Test Result : ========== 21 deselected, 29 warnings, 3 errors in 200.21s (0:03:20) ===========

@Satellite-QE Satellite-QE added the PRT-Failed Indicates that latest PRT run is failed for the PR label Aug 8, 2024
@damoore044
Copy link
Contributor Author

^^ BrokerError for 3/3 API cases, rebuilding

@Satellite-QE
Copy link
Collaborator

PRT Result

Build Number: 8036
Build Status: UNSTABLE
PRT Comment: pytest tests/foreman/api/test_capsulecontent.py -k test_positive_sync_updated_repo or test_positive_capsule_sync and not test_positive_capsule_sync_ or test_positive_sync_container_repo_end_to_end --external-logging
Test Result : ========== 21 deselected, 29 warnings, 3 errors in 194.68s (0:03:14) ===========

vijaysawant
vijaysawant reviewed Aug 8, 2024
View reviewed changes
Copy link
Contributor

@vijaysawant vijaysawant left a comment

Choose a reason for hiding this comment

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

Right now we don't mark pit_server for the test which uses *_capsule_configured fixture, right @ogajduse ?

@ogajduse
Copy link
Member

ogajduse commented Aug 8, 2024

Right now we don't mark pit_server for the test which uses *_capsule_configured fixture, right @ogajduse ?

That is correct. Good catch.

@damoore044 please see: #15756 (review)

ColeHiggins2
ColeHiggins2 approved these changes Aug 8, 2024
View reviewed changes
Copy link
Member

@ColeHiggins2 ColeHiggins2 left a comment

Choose a reason for hiding this comment

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

ack pending prt

vsedmik
vsedmik requested changes Aug 9, 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.

@pytest.mark.pit_server is not applicable as per previous offline communication, please remove.

@damoore044 damoore044 force-pushed the interop-tags-capscont branch 2 times, most recently from d64f32f to a2b4c07 Compare August 13, 2024 18:13
@damoore044
Copy link
Contributor Author

trigger: test-robottelo
pytest: tests/foreman/api/test_capsulecontent.py -k 'test_positive_sync_updated_repo or test_positive_capsule_sync and not test_positive_capsule_sync_ or test_positive_sync_container_repo_end_to_end'

@Satellite-QE
Copy link
Collaborator

PRT Result

Build Number: 8099
Build Status: UNSTABLE
PRT Comment: pytest tests/foreman/api/test_capsulecontent.py -k test_positive_sync_updated_repo or test_positive_capsule_sync and not test_positive_capsule_sync_ or test_positive_sync_container_repo_end_to_end --external-logging
Test Result : ==== 1 failed, 2 passed, 21 deselected, 932 warnings in 9930.60s (2:45:30) =====

@damoore044
Copy link
Contributor Author

The one failing test in above PRT is expected for now due to BZ,
to be resolved in stream soon: SAT-25813

@damoore044
Copy link
Contributor Author

trigger: test-robottelo
pytest: tests/foreman/cli/test_capsulecontent.py -k 'test_positive_update_counts or test_positive_content_counts_for_mixed_cv'

@damoore044 damoore044 requested a review from vsedmik August 14, 2024 14:50
@Satellite-QE
Copy link
Collaborator

PRT Result

Build Number: 8108
Build Status: SUCCESS
PRT Comment: pytest tests/foreman/cli/test_capsulecontent.py -k test_positive_update_counts or test_positive_content_counts_for_mixed_cv --external-logging
Test Result : ========== 2 passed, 25 deselected, 92 warnings in 1705.51s (0:28:25) ==========

@Satellite-QE Satellite-QE added PRT-Passed Indicates that latest PRT run is passed for the PR and removed PRT-Failed Indicates that latest PRT run is failed for the PR labels Aug 14, 2024
@damoore044
Copy link
Contributor Author

trigger: test-robottelo
pytest: tests/foreman/ui/test_capsulecontent.py::test_positive_content_counts_for_mixed_cv

@Satellite-QE
Copy link
Collaborator

PRT Result

Build Number: 8110
Build Status: SUCCESS
PRT Comment: pytest tests/foreman/ui/test_capsulecontent.py::test_positive_content_counts_for_mixed_cv --external-logging
Test Result : ================= 1 passed, 14 warnings in 1946.55s (0:32:26) ==================

@damoore044 damoore044 requested a review from vijaysawant August 14, 2024 17:50
@damoore044 damoore044 force-pushed the interop-tags-capscont branch from a2b4c07 to 650bd55 Compare August 15, 2024 12:46
@Satellite-QE Satellite-QE removed the PRT-Passed Indicates that latest PRT run is passed for the PR label Aug 15, 2024
@vsedmik vsedmik merged commit 93bd5ee into SatelliteQE:master Aug 15, 2024
9 checks passed
@Satellite-QE Satellite-QE mentioned this pull request Aug 15, 2024
Closed
damoore044 added a commit to damoore044/robottelo that referenced this pull request Aug 15, 2024
@damoore044
PIT markers for Interoperability Testing: CapsuleContent (SatelliteQE…
238091b 
…#15892)

PIT markers for interop, CapsuleContent
@damoore044 damoore044 mentioned this pull request Aug 15, 2024
Merged
vsedmik pushed a commit that referenced this pull request Aug 16, 2024
@damoore044
[6.15.z] PIT markers for Interoperability Testing: CapsuleContent (#1…
1b8743e 
…5892) (#15943)

PIT markers for Interoperability Testing: CapsuleContent (#15892)

PIT markers for interop, CapsuleContent
jyejare pushed a commit to jyejare/robottelo that referenced this pull request Oct 19, 2024
@damoore044 @jyejare
PIT markers for Interoperability Testing: CapsuleContent (SatelliteQE…
c530f38 
…#15892)

PIT markers for interop, CapsuleContent
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@pondrejk pondrejk pondrejk approved these changes

@ColeHiggins2 ColeHiggins2 ColeHiggins2 approved these changes

@vsedmik vsedmik vsedmik approved these changes

@vijaysawant vijaysawant Awaiting requested review from vijaysawant

Assignees

@damoore044 damoore044

Labels
6.15.z Introduced in or relating directly to Satellite 6.15 CherryPick PR needs CherryPick to previous branches 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.
7 participants
@damoore044 @Satellite-QE @ogajduse @pondrejk @ColeHiggins2 @vijaysawant @vsedmik