)]}'
{"tempest/api_schema/compute/servers.py":[{"author":{"_account_id":5292,"name":"Christopher Yeoh","email":"cbkyeoh@gmail.com","username":"cyeoh-0"},"change_message_id":"e51a8b00b543b35dae256e2c97dc44ed1038c60c","unresolved":false,"context_lines":[{"line_number":62,"context_line":"                    \u0027properties\u0027: {"},{"line_number":63,"context_line":"                        # NOTE: Now the type of \u0027id\u0027 is uuid, but here allows"},{"line_number":64,"context_line":"                        # \u0027integer\u0027 also because old OpenStack uses \u0027integer\u0027"},{"line_number":65,"context_line":"                        # as a server id."},{"line_number":66,"context_line":"                        \u0027id\u0027: {\u0027type\u0027: [\u0027integer\u0027, \u0027string\u0027]},"},{"line_number":67,"context_line":"                        \u0027links\u0027: parameter_types.links,"},{"line_number":68,"context_line":"                        \u0027name\u0027: {\u0027type\u0027: \u0027string\u0027}"}],"source_content_type":"text/x-python","patch_set":2,"id":"AAAAXX%2F%2FyIM%3D","line":65,"updated":"2014-04-27 19:16:25.000000000","message":"How old a version of Openstack still returns an integer instead a UUID for the id? It doesn\u0027t seem to be a valid response out of the db anymore?","commit_id":"58a6caa350911632aa49ebd6afb614e13e84ce7a"},{"author":{"_account_id":6167,"name":"Ken\u0027ichi Ohmichi","email":"ken1ohmichi@gmail.com","username":"oomichi"},"change_message_id":"1750144d8cb8b2d775bc282346d2beee81d05802","unresolved":false,"context_lines":[{"line_number":62,"context_line":"                    \u0027properties\u0027: {"},{"line_number":63,"context_line":"                        # NOTE: Now the type of \u0027id\u0027 is uuid, but here allows"},{"line_number":64,"context_line":"                        # \u0027integer\u0027 also because old OpenStack uses \u0027integer\u0027"},{"line_number":65,"context_line":"                        # as a server id."},{"line_number":66,"context_line":"                        \u0027id\u0027: {\u0027type\u0027: [\u0027integer\u0027, \u0027string\u0027]},"},{"line_number":67,"context_line":"                        \u0027links\u0027: parameter_types.links,"},{"line_number":68,"context_line":"                        \u0027name\u0027: {\u0027type\u0027: \u0027string\u0027}"}],"source_content_type":"text/x-python","patch_set":2,"id":"9ad9bd40_32c26834","line":65,"in_reply_to":"AAAAXX%2F%2FyIM%3D","updated":"2014-05-07 10:36:30.000000000","message":"Sorry for my misleading. A server id included in a response body is always uuid(string), not integer.\n\nIf checking out 2012.1 of nova, we can see nova/db/sqlalchemy/migrate_repo/versions/025_add_uuid_to_instances.py:\n\n 22 def upgrade(migrate_engine):\n 23     meta \u003d MetaData()\n 24     meta.bind \u003d migrate_engine\n 25\n 26     instances \u003d Table(\u0027instances\u0027, meta, autoload\u003dTrue)\n 27\n 28     uuid_column \u003d Column(\"uuid\", String(36))\n 29     instances.create_column(uuid_column)\n 30\n 31     rows \u003d migrate_engine.execute(instances.select())\n 32     for row in rows:\n 33         instance_uuid \u003d str(utils.gen_uuid())\n 34         migrate_engine.execute(instances.update()\\\n 35                 .where(instances.c.id \u003d\u003d row[0])\\\n 36                 .values(uuid\u003dinstance_uuid))\n\nWhen adding \"uuid\" column to instances table, \"uuid\" is generated with line33. and a server id of response body is gotten from this \"uuid\", so a server id is always uuid.","commit_id":"58a6caa350911632aa49ebd6afb614e13e84ce7a"}]}
