Project

Profile

Help

Task #7935

deprecate an unused get_plugin_storage_path() method in plugin API

Added by dkliban@redhat.com 8 months ago. Updated 6 months ago.

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

100%

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

Description

A developer in #pulp-dev noticed that the get_plugin_storage_path() has a storage path hard coded instead of using a setting. Upon further investigation, I discovered that this method is not used anywhere. We should deprecate this method in the next release and then remove it in the following.

[0] https://github.com/pulp/pulpcore/blob/7ce37dfe60e2561c0988361c1a27f4b9bf9431c6/pulpcore/plugin/storage.py#L11


Related issues

Related to Pulp - Task #8343: Remove get_plugin_storage_path() from the plugin apiCLOSED - CURRENTRELEASE

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

Associated revisions

Revision ebf025d5 View on GitHub
Added by dkliban@redhat.com 6 months ago

Deprecate get_plugin_storage_path() method in plugin API

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

History

#1 Updated by bmbouter 8 months ago

+1 to this, I also don't see it used and it is very old.

#2 Updated by daviddavis 7 months ago

  • Sprint/Milestone set to 3.10.0
  • Sprint set to Sprint 88

#3 Updated by ttereshc 6 months ago

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

#4 Updated by dkliban@redhat.com 6 months ago

  • Assignee changed from ttereshc to dkliban@redhat.com

#5 Updated by rchan 6 months ago

  • Sprint changed from Sprint 88 to Sprint 89

#6 Updated by dkliban@redhat.com 6 months ago

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

#7 Updated by pulpbot 6 months ago

  • Status changed from MODIFIED to CLOSED - CURRENTRELEASE

#8 Updated by ttereshc 5 months ago

  • Related to Task #8343: Remove get_plugin_storage_path() from the plugin api added

Please register to edit this issue

Also available in: Atom PDF