openstack/glance
Code Issues Proposed changes

Correct the deprecation messages of local dir config

Browse Source 
The deprecation messages of node_staging_uri and work_dir
were claiming that local directory is not needed after moving
to multistore. This is not exactly true and gave false
expectations of the current capabilities.

For now only the way to configure those needed folder and
how they are internally accessed is changing.

This change corrects the deprecation message to reflect
current state.

Change-Id: I39c170903c64181841a22c7b9bdaa3b5a1346caa
Closes-Bug: #1843891
This commit is contained in:
Erno Kuvaja 2019-09-13 13:55:45 +01:00
parent 99f3235acf
commit b639ed080d
1 changed files with 10 additions and 7 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

17
glance/common/config.py
View File

@ -133,9 +133,10 @@ Related Options:
deprecated_for_removal=True,
deprecated_since="Train",
deprecated_reason=_("""
With the introduction of the Glance multistore feature, it is
no longer necessary to reserve a local directory for the tasks
workspace. Instead, a more flexible filesystem store can be used.
With the introduction of the Glance multistore feature, the way of
configuring this option has changed. Please refer to
[os_glance_tasks_store]/filesystem_store_datadir for multistore
configuration.
This option is scheduled to be removed early in the 'U' development
cycle.
@ -172,6 +173,7 @@ Possible values:
Related Options:
* enabled_backends
* [os_glance_tasks_store]/filesystem_store_datadir
""")),
]
@ -676,10 +678,10 @@ Relation options:
deprecated_for_removal=True,
deprecated_since="Train",
deprecated_reason=_("""
With the introduction of the Glance multistore feature, it is
no longer necessary to reserve a local directory for the temporary
image staging area. Instead, a more flexible filesystem store can
be used.
With the introduction of the Glance multistore feature, the way of
configuring this option has changed. Please refer to
[os_glance_staging_store]/filesystem_store_datadir for multistore
configuration.
This option is scheduled to be removed early in the 'U' development
cycle.
@ -713,6 +715,7 @@ Possible values:
Related options:
* [task]/work_dir
* enabled_backends
* [os_glance_staging_store]/filesystem_store_datadir
""")),
cfg.ListOpt('enabled_import_methods',