Project

Profile

Help

Task #7935

closed

deprecate an unused get_plugin_storage_path() method in plugin API

Added by dkliban@redhat.com over 3 years ago. Updated about 3 years 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 - CURRENTRELEASEttereshc

Actions
Actions #1

Updated by bmbouter over 3 years ago

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

Actions #2

Updated by daviddavis over 3 years ago

  • Sprint/Milestone set to 3.10.0
  • Sprint set to Sprint 88
Actions #3

Updated by ttereshc about 3 years ago

  • Status changed from NEW to ASSIGNED
  • Assignee set to ttereshc
Actions #4

Updated by dkliban@redhat.com about 3 years ago

  • Assignee changed from ttereshc to dkliban@redhat.com
Actions #5

Updated by rchan about 3 years ago

  • Sprint changed from Sprint 88 to Sprint 89

Added by dkliban@redhat.com about 3 years ago

Revision ebf025d5 | View on GitHub

Deprecate get_plugin_storage_path() method in plugin API

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

Actions #6

Updated by dkliban@redhat.com about 3 years ago

  • Status changed from ASSIGNED to MODIFIED
  • % Done changed from 0 to 100
Actions #7

Updated by pulpbot about 3 years ago

  • Status changed from MODIFIED to CLOSED - CURRENTRELEASE
Actions #8

Updated by ttereshc about 3 years ago

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

Also available in: Atom PDF