Project

Profile

Help

Task #7930

closed

Make PulpPluginAppConfig.version required

Added by bmbouter over 3 years ago. Updated about 3 years ago.

Status:
CLOSED - CURRENTRELEASE
Priority:
Normal
Assignee:
Category:
-
Sprint/Milestone:
Start date:
Due date:
% Done:

100%

Estimated time:
Platform Release:
Groomed:
No
Sprint Candidate:
No
Tags:
Sprint:
Quarter:

Description

With 3.9 we introduced warnings in PulpPluginAppConfig.__init__ encouraging plugin writers to add a version attribute to their PulpPluginAppConfig subclasses.

With 3.10 we should remove the warnings and Pulp should refuse to start if any of its plugins do not set version in this way.


Related issues

Related to Pulp - Task #8198: Make the status API and openAPI schema version listing report on django installed plugins not python packagesCLOSED - CURRENTRELEASEbmbouter

Actions
Actions #1

Updated by mdellweg over 3 years ago

Additionally to the status api, the version listing in the api doc should do the same.

Actions #2

Updated by bmbouter over 3 years ago

  • Subject changed from Make PulpPluginAppConfig.version required and used by the status API to Make PulpPluginAppConfig.version required and used by the status API and API schema version listing
  • Description updated (diff)

Great idea, but I'm not sure where in the code that happens. Do you know?

Actions #4

Updated by bmbouter about 3 years ago

  • Status changed from NEW to ASSIGNED
  • Assignee set to bmbouter
Actions #5

Updated by bmbouter about 3 years ago

  • Description updated (diff)
Actions #6

Updated by pulpbot about 3 years ago

  • Status changed from ASSIGNED to POST

Added by bmbouter about 3 years ago

Revision 9d5a5b5c | View on GitHub

Requires Pulp plugins to provide version

It adds enforcement that the version to be set on the PulpPluginAppConfig subclass. If an installed plugin does not specify its version Pulp will refuse to start.

closes #7930

Actions #8

Updated by bmbouter about 3 years ago

  • Status changed from POST to MODIFIED
  • % Done changed from 0 to 100
Actions #9

Updated by bmbouter about 3 years ago

  • Subject changed from Make PulpPluginAppConfig.version required and used by the status API and API schema version listing to Make PulpPluginAppConfig.version required
  • Description updated (diff)

I moved the status API and openAPI schema parts to this 3.11 issue: https://pulp.plan.io/issues/8198

Actions #10

Updated by bmbouter about 3 years ago

  • Related to Task #8198: Make the status API and openAPI schema version listing report on django installed plugins not python packages added
Actions #11

Updated by pulpbot about 3 years ago

  • Status changed from MODIFIED to CLOSED - CURRENTRELEASE

Also available in: Atom PDF