Bug 2005392

Summary:	If the migration plan is empty, all repositories get migrated.
Product:	Red Hat Satellite	Reporter:	Ian Ballou <iballou>
Component:	Repositories	Assignee:	Justin Sherrill <jsherril>
Status:	CLOSED ERRATA	QA Contact:	Akhil Jha <akjha>
Severity:	high	Docs Contact:
Priority:	unspecified
Version:	6.9.0	CC:	gtalreja, osousa, vijsingh
Target Milestone:	6.9.9	Keywords:	Triaged
Target Release:	Unused
Hardware:	Unspecified
OS:	Unspecified
Whiteboard:
Fixed In Version:	tfm-rubygem-katello-3.18.1.51-1	Doc Type:	If docs needed, set a value
Doc Text:		Story Points:	---
Clone Of:		Environment:
Last Closed:	2022-04-20 20:34:52 UTC	Type:	Bug
Regression:	---	Mount Type:	---
Documentation:	---	CRM:
Verified Versions:		Category:	---
oVirt Team:	---	RHEL 7.3 requirements from Atomic Host:
Cloudforms Team:	---	Target Upstream Version:
Embargoed:

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