[SAT-26559] Incremental Update on a Version contained in a CompositeCV #18687

damoore044 · 2025-06-11T16:18:49Z

*Note: The Version being incrementally updated is not a Version of the Composite, but a version of a filtered CV previously added to the composite.

PRT Case

trigger: test-robottelo
pytest: tests/foreman/api/test_contentview.py::TestContentViewPublishPromote::test_positive_inc_update_ccv_updated_expected_version
Katello:
    katello: 11411

damoore044 · 2025-06-11T16:19:30Z

trigger: test-robottelo
pytest: tests/foreman/api/test_contentview.py::TestContentViewPublishPromote::test_positive_inc_update_cvv_updated_expected_version
Katello:
    katello: 11411

Satellite-QE · 2025-06-11T16:47:43Z

PRT Result

Build Number: 11724
Build Status: UNSTABLE
PRT Comment: pytest tests/foreman/api/test_contentview.py::TestContentViewPublishPromote::test_positive_inc_update_cvv_updated_expected_version --external-logging
Test Result : ================== 1 failed, 7 warnings in 1212.93s (0:20:12) ==================

sambible

A few changes requested, overall the logic looks sounds.

tests/foreman/api/test_contentview.py

damoore044 · 2025-06-13T19:57:06Z

trigger: test-robottelo
pytest: tests/foreman/api/test_contentview.py::TestContentViewPublishPromote::test_positive_inc_update_cvv_updated_expected_version
Katello:
    katello: 11411

Satellite-QE · 2025-06-13T20:36:41Z

PRT Result

Build Number: 11763
Build Status: UNSTABLE
PRT Comment: pytest tests/foreman/api/test_contentview.py::TestContentViewPublishPromote::test_positive_inc_update_cvv_updated_expected_version --external-logging
Test Result : ================== 1 failed, 7 warnings in 1700.88s (0:28:20) ==================

damoore044 · 2025-06-16T14:18:50Z

trigger: test-robottelo
pytest: tests/foreman/api/test_contentview.py::TestContentViewPublishPromote::test_positive_inc_update_cvv_updated_expected_version
Katello:
    katello: 11411

sambible

ACK pending PRT.

Satellite-QE · 2025-06-16T14:50:15Z

PRT Result

Build Number: 11792
Build Status: UNSTABLE
PRT Comment: pytest tests/foreman/api/test_contentview.py::TestContentViewPublishPromote::test_positive_inc_update_cvv_updated_expected_version --external-logging
Test Result : ================== 1 failed, 7 warnings in 1303.53s (0:21:43) ==================

damoore044 · 2025-06-16T14:55:23Z

^ failure to promote the incremental version (v2.1) to Library. It's possible the API call promotes to Library already,
Locally, the CLI Hammer method, I still needed to promote the version after it was created, it was in no environments.
Should be a quick fix (famous last words)

Edit: it's already promoted to Library by api incremental update, removed the explicit promotion call.

damoore044 · 2025-06-16T14:59:48Z

trigger: test-robottelo
pytest: tests/foreman/api/test_contentview.py::TestContentViewPublishPromote::test_positive_inc_update_ccv_updated_expected_version
Katello:
    katello: 11411

Satellite-QE · 2025-06-16T15:30:15Z

PRT Result

Build Number: 11794
Build Status: UNSTABLE
PRT Comment: pytest tests/foreman/api/test_contentview.py::TestContentViewPublishPromote::test_positive_inc_update_ccv_updated_expected_version --external-logging
Test Result : ================== 1 failed, 7 warnings in 1320.96s (0:22:00) ==================

damoore044 · 2025-06-16T16:01:07Z

trigger: test-robottelo
pytest: tests/foreman/api/test_contentview.py::TestContentViewPublishPromote::test_positive_inc_update_ccv_updated_expected_version
Katello:
    katello: 11411

Satellite-QE · 2025-06-16T16:33:44Z

PRT Result

Build Number: 11798
Build Status: UNSTABLE
PRT Comment: pytest tests/foreman/api/test_contentview.py::TestContentViewPublishPromote::test_positive_inc_update_ccv_updated_expected_version --external-logging
Test Result : ================== 1 failed, 7 warnings in 1511.86s (0:25:11) ==================

qcjames53 · 2025-06-18T13:44:05Z

tests/foreman/api/test_contentview.py

+        :id: c8ab2dea-f51a-423e-ae1d-027fe6d85674
+
+        :setup:
+            1. Composite ContentView with 3 regular ContentViews added, auto_publish set to False.


This could be worded a little more clearly. To cover SAT-26559, the composite needs to both have auto_publish set to false and have all its child content views have 'Update to Latest Version'.

I understand this is specified below but I would personally make it clear it needs to happen at the CV level, not the repo level.

Yes I can put it here too to be clearer. I'll also keep it below, to show the order of setup steps

qcjames53 · 2025-06-18T13:46:03Z

tests/foreman/api/test_contentview.py

+        :expectedresults:
+            8. Incrementally Updated Filtered ContentView to v2.1, Composite updated to v1.1.
+            9. CompositeCV contains all the newest CV Versions, except the Unfiltered-CV v3.0 (only expect v2.0).
+            10. After final publish, CompositeCV now contains the Unfiltered-CV v3.0.


Should probably be 1,2,3 not 8,9,10, right?

Sometimes we'll number the expectedresults in order starting with 1 if they are pretty simple.
But if I have specific results expected at specific steps, I'll number result the same as the step that lead to it (from setup).
But, I could also split the :setup: vs :steps: better here, so will have that pushed soon.

damoore044 · 2025-06-18T16:29:01Z

trigger: test-robottelo
pytest: tests/foreman/api/test_contentview.py::TestContentViewPublishPromote::test_positive_inc_update_ccv_updated_expected_version
Katello:
    katello: 11411

Satellite-QE · 2025-06-18T17:00:11Z

PRT Result

Build Number: 11844
Build Status: UNSTABLE
PRT Comment: pytest tests/foreman/api/test_contentview.py::TestContentViewPublishPromote::test_positive_inc_update_ccv_updated_expected_version --external-logging
Test Result : ================== 1 failed, 7 warnings in 1340.41s (0:22:20) ==================

damoore044 · 2025-06-30T12:01:52Z

trigger: test-robottelo
pytest: tests/foreman/api/test_contentview.py::TestContentViewPublishPromote::test_positive_inc_update_ccv_updated_expected_version

Satellite-QE · 2025-06-30T12:26:18Z

PRT Result

Build Number: 11952
Build Status: UNSTABLE
PRT Comment: pytest tests/foreman/api/test_contentview.py::TestContentViewPublishPromote::test_positive_inc_update_ccv_updated_expected_version --external-logging
Test Result : ================== 1 failed, 7 warnings in 1086.51s (0:18:06) ==================

damoore044 self-assigned this Jun 11, 2025

damoore044 requested review from a team as code owners June 11, 2025 16:18

damoore044 added CherryPick PR needs CherryPick to previous branches Stream Introduced in or relating directly to Satellite Stream/Master 6.17.z labels Jun 11, 2025

Satellite-QE added the PRT-Failed Indicates that latest PRT run is failed for the PR label Jun 11, 2025

damoore044 force-pushed the CompositeCV branch 2 times, most recently from bca8dbd to 436174e Compare June 11, 2025 19:16

sambible requested changes Jun 12, 2025

View reviewed changes

tests/foreman/api/test_contentview.py Outdated Show resolved Hide resolved

damoore044 force-pushed the CompositeCV branch from 436174e to d7d3d0b Compare June 13, 2025 19:55

damoore044 force-pushed the CompositeCV branch from d7d3d0b to 5942a2d Compare June 13, 2025 20:25

damoore044 force-pushed the CompositeCV branch from 5942a2d to 0ff68b2 Compare June 16, 2025 14:18

damoore044 force-pushed the CompositeCV branch from 0ff68b2 to 51e66ee Compare June 16, 2025 14:31

sambible approved these changes Jun 16, 2025

View reviewed changes

damoore044 force-pushed the CompositeCV branch from 51e66ee to 0020854 Compare June 16, 2025 14:57

damoore044 force-pushed the CompositeCV branch from 0020854 to 2e56f07 Compare June 16, 2025 15:59

damoore044 force-pushed the CompositeCV branch from 2e56f07 to 8bab6e0 Compare June 16, 2025 16:41

qcjames53 reviewed Jun 18, 2025

View reviewed changes

damoore044 force-pushed the CompositeCV branch from 8bab6e0 to 121f26f Compare June 18, 2025 16:27

damoore044 added 2 commits June 30, 2025 08:00

Inc. Update on a Version used as latest for CompositeCV.

23377d1

repos fixture from another PR

08f4f1f

damoore044 force-pushed the CompositeCV branch from 121f26f to 08f4f1f Compare June 30, 2025 12:01

[SAT-26559] Incremental Update on a Version contained in a CompositeCV #18687

Are you sure you want to change the base?

[SAT-26559] Incremental Update on a Version contained in a CompositeCV #18687

Uh oh!

Conversation

damoore044 commented Jun 11, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

PRT Case

Uh oh!

damoore044 commented Jun 11, 2025

Uh oh!

Satellite-QE commented Jun 11, 2025

Uh oh!

sambible left a comment

Choose a reason for hiding this comment

Uh oh!

Uh oh!

damoore044 commented Jun 13, 2025

Uh oh!

Satellite-QE commented Jun 13, 2025

Uh oh!

damoore044 commented Jun 16, 2025

Uh oh!

sambible left a comment

Choose a reason for hiding this comment

Uh oh!

Satellite-QE commented Jun 16, 2025

Uh oh!

damoore044 commented Jun 16, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

damoore044 commented Jun 16, 2025

Uh oh!

Satellite-QE commented Jun 16, 2025

Uh oh!

damoore044 commented Jun 16, 2025

Uh oh!

Satellite-QE commented Jun 16, 2025

Uh oh!

qcjames53 Jun 18, 2025

Choose a reason for hiding this comment

Uh oh!

damoore044 Jun 18, 2025

Choose a reason for hiding this comment

Uh oh!

qcjames53 Jun 18, 2025

Choose a reason for hiding this comment

Uh oh!

damoore044 Jun 18, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Choose a reason for hiding this comment

Uh oh!

damoore044 commented Jun 18, 2025

Uh oh!

Satellite-QE commented Jun 18, 2025

Uh oh!

damoore044 commented Jun 30, 2025

Uh oh!

Satellite-QE commented Jun 30, 2025

Uh oh!

Uh oh!

damoore044 commented Jun 11, 2025 •

edited

Loading

damoore044 commented Jun 16, 2025 •

edited

Loading

damoore044 Jun 18, 2025 •

edited

Loading