Project

Profile

Help

Task #7930

closed

Make PulpPluginAppConfig.version required

Added by bmbouter almost 2 years ago. Updated over 1 year 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 almost 2 years ago

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

Actions #2

Updated by bmbouter almost 2 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 over 1 year ago

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

Updated by bmbouter over 1 year ago

  • Description updated (diff)
Actions #6

Updated by pulpbot over 1 year ago

  • Status changed from ASSIGNED to POST

Added by bmbouter over 1 year ago

Revision 9d5a5b5c

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 over 1 year ago

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

Updated by bmbouter over 1 year 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 over 1 year 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 over 1 year ago

  • Status changed from MODIFIED to CLOSED - CURRENTRELEASE

Also available in: Atom PDF