Project

Profile

Help

Issue #4951

Existence of /usr/lib/pulp/plugins/types/deb.json will prevent migration 0004 from working.

Added by quba42 about 1 year ago. Updated about 1 year ago.

Status:
CLOSED - CURRENTRELEASE
Priority:
Urgent
Assignee:
Sprint/Milestone:
Start date:
Due date:
Estimated time:
Severity:
4. Urgent
Version - Debian:
Platform Release:
2.20.0
Target Release - Debian:
OS:
Triaged:
Yes
Groomed:
No
Sprint Candidate:
No
Tags:
Pulp 2
Sprint:

Description

/usr/lib/pulp/plugins/types/deb.json is apparently a legacy way for pulp plugins to declare their unit keys.

The existence of this file causes a different mechanism for index creation to be used during db migrations.

This mechanism is incompatible with pulp_deb migration 0004.

Both testing and discussions with other developers suggest that this file is no longer needed and can simply be removed. (I am not aware of any plugins besides pulp_deb still packaging this file).

To stop this file from being packaged will require both a change in pulp_deb and in pulp-packaging repositories.

I will add PRs referencing this issue against both repositories soon.


Related issues

Related to Debian Support - Issue #4138: Handling publish paths correctlyCLOSED - CURRENTRELEASE<a title="Actions" class="icon-only icon-actions js-contextmenu" href="#">Actions</a>

Associated revisions

Revision 4492d2fd View on GitHub
Added by quba42 about 1 year ago

Remove /usr/lib/pulp/plugins/types/deb.json from pulp-deb

This file is a legacy way to provide plugin type definitions and is no longer needed. The presence of this file will break reindex migrations.

WARNING: Also requires a change in pulp_deb: https://github.com/pulp/pulp_deb/pull/93

fixes #4951 https://pulp.plan.io/issues/4951

Revision 4492d2fd View on GitHub
Added by quba42 about 1 year ago

Remove /usr/lib/pulp/plugins/types/deb.json from pulp-deb

This file is a legacy way to provide plugin type definitions and is no longer needed. The presence of this file will break reindex migrations.

WARNING: Also requires a change in pulp_deb: https://github.com/pulp/pulp_deb/pull/93

fixes #4951 https://pulp.plan.io/issues/4951

Revision 37b8f9f3 View on GitHub
Added by quba42 about 1 year ago

Remove /usr/lib/pulp/plugins/types/deb.json from setup.py

This legacy file will prevent migration 0004 from working.

WARNING: This will also require removing the file from the .spec file in pulp-packaging repository!

fixes #4951 https://pulp.plan.io/issues/4951

History

#1 Updated by quba42 about 1 year ago

  • Related to Issue #4138: Handling publish paths correctly added

#3 Updated by mdellweg about 1 year ago

  • Priority changed from Normal to Urgent
  • Severity changed from 2. Medium to 4. Urgent
  • Triaged changed from No to Yes

#4 Updated by quba42 about 1 year ago

  • Status changed from NEW to MODIFIED

#5 Updated by mdellweg about 1 year ago

  • Tags Pulp 2 added

#6 Updated by ttereshc about 1 year ago

  • Platform Release set to 2.20.0

#7 Updated by ttereshc about 1 year ago

  • Sprint/Milestone set to 2.20.0

#8 Updated by ttereshc about 1 year ago

  • Status changed from MODIFIED to 5

#9 Updated by ttereshc about 1 year ago

  • Status changed from 5 to CLOSED - CURRENTRELEASE

Please register to edit this issue

Also available in: Atom PDF