)]}'
{"specs/pike/approved/deprecate-os-quota-class-sets.rst":[{"author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"change_message_id":"480b4b4659f4111640a67b43c2319712d321ad99","unresolved":false,"context_lines":[{"line_number":24,"context_line":"There is also no way, besides querying the database directly, to know what"},{"line_number":25,"context_line":"quota classes are available."},{"line_number":26,"context_line":""},{"line_number":27,"context_line":"Currently when updating a quota class, it\u0027s a create-or-update operation in"},{"line_number":28,"context_line":"that a new quota_classes record is created in the database if the update fails."},{"line_number":29,"context_line":"However, there is no way in Nova to associate a non-default quota class with"},{"line_number":30,"context_line":"a tenant. Furthermore, when trying to show details on a given quota class, the"}],"source_content_type":"text/x-rst","patch_set":1,"id":"1a6eadb0_15dde3df","line":27,"range":{"start_line":27,"start_character":46,"end_line":27,"end_character":62},"updated":"2016-12-15 15:16:11.000000000","message":"This should be update-or-create.","commit_id":"4f4b77b46b56f8964dabea168d58720a432523d4"},{"author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"change_message_id":"5cb0467f3dfc4e5ec6a681bd46b2ae5920781c03","unresolved":false,"context_lines":[{"line_number":27,"context_line":"Currently when updating a quota class, it\u0027s a create-or-update operation in"},{"line_number":28,"context_line":"that a new quota_classes record is created in the database if the update fails."},{"line_number":29,"context_line":"However, there is no way in Nova to associate a non-default quota class with"},{"line_number":30,"context_line":"a tenant. Furthermore, when trying to show details on a given quota class, the"},{"line_number":31,"context_line":"default quota limits are returned if the class provided does not exist. In"},{"line_number":32,"context_line":"other words, if you run ``nova quota-class-show foo`` and there is no \u0027foo\u0027"},{"line_number":33,"context_line":"quota class in the nova.quota_classes DB table, the API will simply return the"},{"line_number":34,"context_line":"default limits rather than error with a 404."},{"line_number":35,"context_line":""},{"line_number":36,"context_line":"There is also general confusion over the use of quota classes and default"},{"line_number":37,"context_line":"quotas. When calculating limits for a given resource and tenant, the following"}],"source_content_type":"text/x-rst","patch_set":1,"id":"1a6eadb0_9519f38a","line":34,"range":{"start_line":30,"start_character":10,"end_line":34,"end_character":44},"updated":"2016-12-15 15:17:33.000000000","message":"TODO: refer to this bug https://bugs.launchpad.net/nova/+bug/1602396","commit_id":"4f4b77b46b56f8964dabea168d58720a432523d4"},{"author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"change_message_id":"480b4b4659f4111640a67b43c2319712d321ad99","unresolved":false,"context_lines":[{"line_number":84,"context_line":"This eliminates the ability to create custom quota classes (which are unused)"},{"line_number":85,"context_line":"or update default quota limits via the REST API so that any global default"},{"line_number":86,"context_line":"quota changes have to be made via the configuration."},{"line_number":87,"context_line":""},{"line_number":88,"context_line":"Alternatives"},{"line_number":89,"context_line":"------------"},{"line_number":90,"context_line":""}],"source_content_type":"text/x-rst","patch_set":1,"id":"1a6eadb0_15b24394","line":87,"updated":"2016-12-15 15:16:11.000000000","message":"Should probably add something here about making the default quota config option values mutable.","commit_id":"4f4b77b46b56f8964dabea168d58720a432523d4"},{"author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"change_message_id":"480b4b4659f4111640a67b43c2319712d321ad99","unresolved":false,"context_lines":[{"line_number":157,"context_line":"Work Items"},{"line_number":158,"context_line":"----------"},{"line_number":159,"context_line":""},{"line_number":160,"context_line":"* Add the microversion to the `os-quota-class-sets` REST API which deprecates"},{"line_number":161,"context_line":"  the show and update methods."},{"line_number":162,"context_line":"* Deprecate the ``nova quota-class-show`` and ``nova quota-class-update`` CLIs"},{"line_number":163,"context_line":"  in python-novaclient along with the QuotaClassSetManager python API bindings."}],"source_content_type":"text/x-rst","patch_set":1,"id":"1a6eadb0_95649322","line":160,"updated":"2016-12-15 15:16:11.000000000","message":"Should probably add something here about making the default quota config option values mutable.","commit_id":"4f4b77b46b56f8964dabea168d58720a432523d4"},{"author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"change_message_id":"480b4b4659f4111640a67b43c2319712d321ad99","unresolved":false,"context_lines":[{"line_number":166,"context_line":"Dependencies"},{"line_number":167,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"},{"line_number":168,"context_line":""},{"line_number":169,"context_line":"None"},{"line_number":170,"context_line":""},{"line_number":171,"context_line":""},{"line_number":172,"context_line":"Testing"}],"source_content_type":"text/x-rst","patch_set":1,"id":"1a6eadb0_3501ff38","line":169,"updated":"2016-12-15 15:16:11.000000000","message":"We might want to make this dependent on:\n\nhttps://blueprints.launchpad.net/nova/+spec/cells-count-resources-to-check-quota-in-api\n\nBecause then we just care about quota limits in the API node and don\u0027t have to deal with synchronizing default quotas in config on the compute nodes.","commit_id":"4f4b77b46b56f8964dabea168d58720a432523d4"},{"author":{"_account_id":10608,"name":"Matthew Edmonds","email":"edmondsw@us.ibm.com","username":"edmondsw"},"change_message_id":"c5868c57033bd0be50b2193eb251423e39def38f","unresolved":false,"context_lines":[{"line_number":187,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"},{"line_number":188,"context_line":""},{"line_number":189,"context_line":".. [1] https://bugs.launchpad.net/nova/+bug/1602400"},{"line_number":190,"context_line":""},{"line_number":191,"context_line":"There is also background discussion on this topic over the years in the"},{"line_number":192,"context_line":"openstack-dev mailing list, the latest of which is here:"},{"line_number":193,"context_line":""}],"source_content_type":"text/x-rst","patch_set":1,"id":"1a6eadb0_55e3dba3","line":190,"updated":"2016-12-15 15:15:32.000000000","message":"you could also reference https://bugs.launchpad.net/nova/+bug/1602396","commit_id":"4f4b77b46b56f8964dabea168d58720a432523d4"},{"author":{"_account_id":10608,"name":"Matthew Edmonds","email":"edmondsw@us.ibm.com","username":"edmondsw"},"change_message_id":"8130a480f25e7f15e3e3e1f9f14381d7cd4e7d99","unresolved":false,"context_lines":[{"line_number":146,"context_line":"Developer impact"},{"line_number":147,"context_line":"----------------"},{"line_number":148,"context_line":""},{"line_number":149,"context_line":"None, although this is probably something that has infected other services in"},{"line_number":150,"context_line":"OpenStack such as Cinder, and we could also make the same changes there later."},{"line_number":151,"context_line":""},{"line_number":152,"context_line":""}],"source_content_type":"text/x-rst","patch_set":2,"id":"1a6eadb0_62449a1a","line":149,"range":{"start_line":149,"start_character":0,"end_line":149,"end_character":4},"updated":"2016-12-15 16:16:22.000000000","message":"I think there\u0027s an impact here for openstackclient. They have some form of quota class support (see the --class arg for `openstack quota set` and `openstack quota show`)","commit_id":"edef0317a1f193566d3a7ccf2c4ef3314f168e20"},{"author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"change_message_id":"d7b15d731ae5e8637f6bb14afb7bf39ab99a84da","unresolved":false,"context_lines":[{"line_number":146,"context_line":"Developer impact"},{"line_number":147,"context_line":"----------------"},{"line_number":148,"context_line":""},{"line_number":149,"context_line":"None, although this is probably something that has infected other services in"},{"line_number":150,"context_line":"OpenStack such as Cinder, and we could also make the same changes there later."},{"line_number":151,"context_line":""},{"line_number":152,"context_line":""}],"source_content_type":"text/x-rst","patch_set":2,"id":"ba5201f7_ade0332a","line":149,"range":{"start_line":149,"start_character":0,"end_line":149,"end_character":4},"in_reply_to":"1a6eadb0_62449a1a","updated":"2017-01-06 15:22:35.000000000","message":"Yeah, would have to address that also.\n\nHowever, Jay Pipes replied in the ML thread about this and thinks it would be better to only handle the default global quota in the REST API and database and get rid of the config options, that conversation starts here:\n\nhttp://lists.openstack.org/pipermail/openstack-dev/2016-December/109127.html","commit_id":"edef0317a1f193566d3a7ccf2c4ef3314f168e20"}]}