Project

Profile

Help

Task #7930

Make PulpPluginAppConfig.version required

Added by bmbouter 5 months ago. Updated 3 months 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 - CURRENTRELEASE

<a title="Actions" class="icon-only icon-actions js-contextmenu" href="#">Actions</a>

Associated revisions

Revision 9d5a5b5c View on GitHub
Added by bmbouter 3 months ago

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

History

#1 Updated by mdellweg 5 months ago

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

#2 Updated by bmbouter 5 months 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?

#4 Updated by bmbouter 4 months ago

  • Status changed from NEW to ASSIGNED
  • Assignee set to bmbouter

#5 Updated by bmbouter 4 months ago

  • Description updated (diff)

#6 Updated by pulpbot 3 months ago

  • Status changed from ASSIGNED to POST

#8 Updated by bmbouter 3 months ago

  • Status changed from POST to MODIFIED
  • % Done changed from 0 to 100

#9 Updated by bmbouter 3 months 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

#10 Updated by bmbouter 3 months ago

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

#11 Updated by pulpbot 3 months ago

  • Status changed from MODIFIED to CLOSED - CURRENTRELEASE

Please register to edit this issue

Also available in: Atom PDF