)]}'
{"neutron/extensions/flavors.py":[{"author":{"_account_id":11343,"name":"Gal Sagie","email":"gal.sagie@gmail.com","username":"GalSagie"},"change_message_id":"3ada7033e83c3ccaa4fe1321620077c9b567a4f5","unresolved":false,"context_lines":[{"line_number":87,"context_line":"                              \u0027is_visible\u0027: True},"},{"line_number":88,"context_line":"                       \u0027tenant_id\u0027: {\u0027allow_post\u0027: True, \u0027allow_put\u0027: False,"},{"line_number":89,"context_line":"                                     \u0027required_by_policy\u0027: True,"},{"line_number":90,"context_line":"                                     \u0027validate\u0027: {\u0027type:string\u0027:"},{"line_number":91,"context_line":"                                                  attr.TENANT_ID_MAX_LEN},"},{"line_number":92,"context_line":"                                     \u0027is_visible\u0027: True}}"},{"line_number":93,"context_line":"    }"}],"source_content_type":"text/x-python","patch_set":2,"id":"da20952f_9444671b","line":90,"updated":"2015-08-29 17:00:40.000000000","message":"Just wondering, i see it all over not just here so not related to this patch, but why tenant_id is not a uuid but a string ?(although its generating a UUID for it in the DB)","commit_id":"fc9cf56a12d687ead0767af36bdfc3fe44a3470a"},{"author":{"_account_id":12999,"name":"James Arendt","email":"james.arendt@hpe.com","username":"jwarendt"},"change_message_id":"c807040aebc2b2c4da7e87229d2d146c2435bbc8","unresolved":false,"context_lines":[{"line_number":87,"context_line":"                              \u0027is_visible\u0027: True},"},{"line_number":88,"context_line":"                       \u0027tenant_id\u0027: {\u0027allow_post\u0027: True, \u0027allow_put\u0027: False,"},{"line_number":89,"context_line":"                                     \u0027required_by_policy\u0027: True,"},{"line_number":90,"context_line":"                                     \u0027validate\u0027: {\u0027type:string\u0027:"},{"line_number":91,"context_line":"                                                  attr.TENANT_ID_MAX_LEN},"},{"line_number":92,"context_line":"                                     \u0027is_visible\u0027: True}}"},{"line_number":93,"context_line":"    }"}],"source_content_type":"text/x-python","patch_set":2,"id":"da20952f_a880f58b","line":90,"in_reply_to":"da20952f_9444671b","updated":"2015-08-31 18:28:18.000000000","message":"Agreed not related to this patch; The base V2 api attributes for Neutron use a type:string with TENANT_ID_MAX_LEN, and all of the extension interfaces follow suite including this one.  Changing to a stronger UUID data structure is a broader issue that should be raised at the base neutron level,  \n\nOf note here is that the tenant_id is not being stored in the database by flavors nor generated here, simply is used to look up capabilities of the creator as part of base.py.  But for base objects that do store tenant, it is simply a free form string (not a UUID) in the database.  See db/models_base.py.","commit_id":"fc9cf56a12d687ead0767af36bdfc3fe44a3470a"},{"author":{"_account_id":7787,"name":"Kevin Benton","email":"kevin@benton.pub","username":"blak111"},"change_message_id":"26d78776b3c02f2a055821188699b647d8256d23","unresolved":false,"context_lines":[{"line_number":87,"context_line":"                              \u0027is_visible\u0027: True},"},{"line_number":88,"context_line":"                       \u0027tenant_id\u0027: {\u0027allow_post\u0027: True, \u0027allow_put\u0027: False,"},{"line_number":89,"context_line":"                                     \u0027required_by_policy\u0027: True,"},{"line_number":90,"context_line":"                                     \u0027validate\u0027: {\u0027type:string\u0027:"},{"line_number":91,"context_line":"                                                  attr.TENANT_ID_MAX_LEN},"},{"line_number":92,"context_line":"                                     \u0027is_visible\u0027: True}}"},{"line_number":93,"context_line":"    }"}],"source_content_type":"text/x-python","patch_set":2,"id":"da20952f_7f548cc9","line":90,"in_reply_to":"da20952f_a880f58b","updated":"2015-09-01 23:34:15.000000000","message":"Tenant_id only happens to be a UUID when integrated with keystone. There are no such constraints in general for Neutron.","commit_id":"fc9cf56a12d687ead0767af36bdfc3fe44a3470a"}]}