Browse Source

Merge "Deprecate store_capabilities_update_min_interval"

Zuul 10 months ago
parent
commit
8cdccfe68a

+ 47
- 0
specs/rocky/approved/glance_store/deprecate-store_capabilities_update_min_interval.rst View File

@@ -0,0 +1,47 @@
1
+..
2
+ This work is licensed under a Creative Commons Attribution 3.0 Unported
3
+ License.
4
+
5
+ http://creativecommons.org/licenses/by/3.0/legalcode
6
+
7
+===========================================================
8
+Spec Lite: Deprecate store_capabilities_update_min_interval
9
+===========================================================
10
+
11
+:project: glance_store
12
+
13
+:problem: The configuration option ``store_capabilities_update_min_interval``
14
+          is confusing because no existing stores implement the
15
+          StoreCapability.update_capabilities() method.  This has come up in
16
+          the context of nfs being used for the filesystem backend.  If nfs
17
+          is not ready for writing when the glance api starts, glance will
18
+          mark the filesystem as not writeable.  Operators have tried to get
19
+          around this problem by setting a non-zero positive value for this
20
+          option only to find that it doesn't work.
21
+
22
+:solution: Use oslo.config to mark the option as 'deprecated' with an
23
+           appropriate note.  Option will be deprecated in Rocky for removal
24
+           in 'S'.
25
+
26
+:alternatives: An alternative would be to rewrite the option help text to make
27
+               it clear that there is no current store for which the option is
28
+               actionable, but that a framework is in place through which
29
+               dynamic capability determination could be implemented.
30
+               Currently a debug level message to this effect is logged on
31
+               store startup although it is not obvious that the message is
32
+               related to the ineffectiveness of setting the
33
+               ``store_capabilities_update_min_interval`` option.  The message
34
+               is: "Store %s doesn't support updating dynamic storage
35
+               capabilities. Please overwrite 'update_capabilities' method of
36
+               the store to implement updating logics if needed."  (This
37
+               message is logged independently of setting the option.)
38
+
39
+               The advantage to this approach is that the framework would be
40
+               available to someone who wanted to implement dynamic updates
41
+               for a store, and the option would not have to be re-introduced.
42
+
43
+:impacts: None
44
+
45
+:timeline: Rocky milestone 2
46
+
47
+:assignee: rosmaita

+ 5
- 4
specs/rocky/approved/index.rst View File

@@ -17,10 +17,11 @@ Rocky approved specs for Glance:
17 17
 
18 18
 Rocky approved specs for glance_store:
19 19
 
20
-   There are no specs approved for glance_store for Rocky.
21
-   Please consult the :doc:`../../untargeted/index` if
22
-   you'd like to work on glance_store during the Rocky
23
-   development cycle.
20
+.. toctree::
21
+   :glob:
22
+   :maxdepth: 1
23
+
24
+   glance_store/*
24 25
 
25 26
 Rocky approved specs for python-glanceclient:
26 27
 

Loading…
Cancel
Save