Project

Profile

Help

Actions
Send by e-mail Copy link

Issue #4951

closed

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

Added by quba42 over 5 years ago. Updated over 5 years ago.

Status:
CLOSED - CURRENTRELEASE
Priority:
Urgent
Assignee:
quba42
Sprint/Milestone:
Pulp - 2.20.0
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:
Quarter:

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 - CURRENTRELEASEquba42Actions
Actions
Copy link
 #1

Updated by quba42 over 5 years ago

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

Added by quba42 over 5 years ago

Revision 4492d2fd | View on GitHub

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

Added by quba42 over 5 years ago

Revision 4492d2fd | View on GitHub

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

Actions
Copy link
 #3

Updated by mdellweg over 5 years ago

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

Added by quba42 over 5 years ago

Revision 37b8f9f3 | View on GitHub

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

Actions
Copy link
 #4

Updated by quba42 over 5 years ago

  • Status changed from NEW to MODIFIED
Actions
Copy link
 #5

Updated by mdellweg over 5 years ago

  • Tags Pulp 2 added
Actions
Copy link
 #6

Updated by ttereshc over 5 years ago

  • Platform Release set to 2.20.0
Actions
Copy link
 #7

Updated by ttereshc over 5 years ago

  • Sprint/Milestone set to 2.20.0
Actions
Copy link
 #8

Updated by ttereshc over 5 years ago

  • Status changed from MODIFIED to 5
Actions
Copy link
 #9

Updated by ttereshc over 5 years ago

  • Status changed from 5 to CLOSED - CURRENTRELEASE
Actions
Send by e-mail Copy link

Also available in: Atom PDF