Bug 2005392 - If the migration plan is empty, all repositories get migrated.
Summary: If the migration plan is empty, all repositories get migrated.
Keywords:
Status: CLOSED ERRATA
Alias: None
Product: Red Hat Satellite
Classification: Red Hat
Component: Repositories
Version: 6.9.0
Hardware: Unspecified
OS: Unspecified
unspecified
high
Target Milestone: 6.9.9
Assignee: Justin Sherrill
QA Contact: Akhil Jha
URL:
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2021-09-17 14:19 UTC by Ian Ballou
Modified: 2022-04-20 20:35 UTC (History)
3 users (show)

Fixed In Version: tfm-rubygem-katello-3.18.1.51-1
Doc Type: If docs needed, set a value
Doc Text:
Clone Of:
Environment:
Last Closed: 2022-04-20 20:34:52 UTC
Target Upstream Version:
Embargoed:


Attachments (Terms of Use)


Links
System ID Private Priority Status Summary Last Updated
Foreman Issue Tracker 34550 0 Normal New If the migration plan is empty, all repositories get migrated. 2022-03-02 19:46:10 UTC
Red Hat Product Errata RHSA-2022:1478 0 None None None 2022-04-20 20:35:14 UTC

Description Ian Ballou 2021-09-17 14:19:52 UTC
Description of problem:
With the BZ listed below, we are now filtering out repositories for the migration if they're being created or if they're related to a publishing content view. This means people can now migrate safely while doing these actions.  However, in the very rare case that all repositories are excluded from the migration plan, this will actually tell Pulp to migrate all repositories.

Version-Release number of selected component (if applicable):

Satellite 6.9.0

How reproducible:
Always, under the circumstances above.

Steps to Reproduce:
1. Have a Satellite with one repository and put that repo in a content view
2. Publish the content view and immediately start the Pulp 2 -> 3 migration
3. If the timing is right, in the logs, you will see an empty migration plan.  However, the CV's repositories will receive version_hrefs if there is no error during the migration.

Actual results:
The CV's repositories are migrated (have version_hrefs).

Expected results:
The CV's repositories are not migrated (nil version_hrefs).

Additional info:
Other related BZ that found this issue: https://bugzilla.redhat.com/show_bug.cgi?id=1973245

Comment 2 Justin Sherrill 2022-03-02 19:46:09 UTC
Created redmine issue https://projects.theforeman.org/issues/34550 from this bug

Comment 3 Bryan Kearney 2022-03-02 20:04:44 UTC
Upstream bug assigned to jsherril

Comment 4 Bryan Kearney 2022-03-02 20:04:46 UTC
Upstream bug assigned to jsherril

Comment 5 Brad Buckingham 2022-03-08 11:16:19 UTC
Moving to POST as upstream PR is merged.

Comment 7 Akhil Jha 2022-03-28 08:55:29 UTC
Verified.
Satellite 6.9.9-1.0

Synced large repos(rhscl, rhel7, extra, optional, baseos, appstream) and put that in CV.
Just after `publishing`, ran `satellite-maintain content prepare`. Migration fails.

Comment 12 errata-xmlrpc 2022-04-20 20:34:52 UTC
Since the problem described in this bug report should be
resolved in a recent advisory, it has been closed with a
resolution of ERRATA.

For information on the advisory (Important: Satellite 6.9.9 Async Bug Fix Update), and where to find the updated
files, follow the link below.

If the solution does not work for you, open a new bug report.

https://access.redhat.com/errata/RHSA-2022:1478


Note You need to log in before you can comment on or make changes to this bug.