)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":7040,"name":"GuoHui LIu","email":"liuguohui@gmail.com","username":"guohliu"},"change_message_id":"6cbdb86d3a8c0f6202096ac5520d92ab06ec472b","unresolved":false,"context_lines":[{"line_number":7,"context_line":"Remove uuidutils.generate_uuid() from nova"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"uuidutils.generate_uuid() is going to be removed from oslo, so we"},{"line_number":10,"context_line":"need to remove its use in nova"},{"line_number":11,"context_line":""},{"line_number":12,"context_line":"Change-Id: I4506696d50669f16810d4c7c2cd9d21b78bf1f4d"},{"line_number":13,"context_line":"Closes-Bug: #1253497"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"AAAATn%2F%2BuK0%3D","line":10,"updated":"2013-11-21 03:50:27.000000000","message":"Guang Ya, could you please write a little bit more on this? like using the new approach to replace the old one?","commit_id":"28dd9b0539f52f33710d48adb4582564c60393b4"},{"author":{"_account_id":7494,"name":"Guangya Liu","email":"gyliu513@gmail.com","username":"JayLau"},"change_message_id":"2159447577845a20e9ca801c29919d27c968b326","unresolved":false,"context_lines":[{"line_number":7,"context_line":"Remove uuidutils.generate_uuid() from nova"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"uuidutils.generate_uuid() is going to be removed from oslo, so we"},{"line_number":10,"context_line":"need to remove its use in nova"},{"line_number":11,"context_line":""},{"line_number":12,"context_line":"Change-Id: I4506696d50669f16810d4c7c2cd9d21b78bf1f4d"},{"line_number":13,"context_line":"Closes-Bug: #1253497"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"AAAATn%2F%2BuHs%3D","line":10,"in_reply_to":"AAAATn%2F%2BuK0%3D","updated":"2013-11-21 04:12:19.000000000","message":"Done","commit_id":"28dd9b0539f52f33710d48adb4582564c60393b4"}],"nova/tests/virt/libvirt/test_libvirt.py":[{"author":{"_account_id":1779,"name":"Daniel Berrange","email":"berrange@redhat.com","username":"berrange"},"change_message_id":"2b85393310b8a44220e2a75cc74d7cddd2c95eaa","unresolved":false,"context_lines":[{"line_number":26,"context_line":"import re"},{"line_number":27,"context_line":"import shutil"},{"line_number":28,"context_line":"import tempfile"},{"line_number":29,"context_line":"import uuid as stdlib_uuid"},{"line_number":30,"context_line":""},{"line_number":31,"context_line":"from eventlet import greenthread"},{"line_number":32,"context_line":"from lxml import etree"}],"source_content_type":"text/x-python","patch_set":6,"id":"AAAATn%2F%2BgOw%3D","line":29,"updated":"2013-11-25 12:11:13.000000000","message":"Why are you renaming this to \u0027stdlib_uuid\u0027 ?  I don\u0027t see a clash with the regular \u0027\u0027uuid\u0027 name here.","commit_id":"c6fe8bcbeb5d60bc72d6573362ec4ee1ace458d2"},{"author":{"_account_id":7494,"name":"Guangya Liu","email":"gyliu513@gmail.com","username":"JayLau"},"change_message_id":"4bd7436e9cf519acad6c4cb928b4183da9ed28a2","unresolved":false,"context_lines":[{"line_number":26,"context_line":"import re"},{"line_number":27,"context_line":"import shutil"},{"line_number":28,"context_line":"import tempfile"},{"line_number":29,"context_line":"import uuid as stdlib_uuid"},{"line_number":30,"context_line":""},{"line_number":31,"context_line":"from eventlet import greenthread"},{"line_number":32,"context_line":"from lxml import etree"}],"source_content_type":"text/x-python","patch_set":6,"id":"AAAATn%2F%2Bf7M%3D","line":29,"in_reply_to":"AAAATn%2F%2Bf9Q%3D","updated":"2013-11-25 13:21:30.000000000","message":"OK, I will follow up later. ;-)","commit_id":"c6fe8bcbeb5d60bc72d6573362ec4ee1ace458d2"},{"author":{"_account_id":1779,"name":"Daniel Berrange","email":"berrange@redhat.com","username":"berrange"},"change_message_id":"f56a8865c3c39f3a8c2c4a40eddd43cdb56ff745","unresolved":false,"context_lines":[{"line_number":26,"context_line":"import re"},{"line_number":27,"context_line":"import shutil"},{"line_number":28,"context_line":"import tempfile"},{"line_number":29,"context_line":"import uuid as stdlib_uuid"},{"line_number":30,"context_line":""},{"line_number":31,"context_line":"from eventlet import greenthread"},{"line_number":32,"context_line":"from lxml import etree"}],"source_content_type":"text/x-python","patch_set":6,"id":"AAAATn%2F%2Bf9Q%3D","line":29,"in_reply_to":"AAAATn%2F%2Bf9w%3D","updated":"2013-11-25 13:13:39.000000000","message":"Yes, it is much simpler \u0026 clearer to just rename the variables and use a standard import name","commit_id":"c6fe8bcbeb5d60bc72d6573362ec4ee1ace458d2"},{"author":{"_account_id":7494,"name":"Guangya Liu","email":"gyliu513@gmail.com","username":"JayLau"},"change_message_id":"0b2b0e32740ee9fce6cbfa80fc7014aea1b6d03a","unresolved":false,"context_lines":[{"line_number":26,"context_line":"import re"},{"line_number":27,"context_line":"import shutil"},{"line_number":28,"context_line":"import tempfile"},{"line_number":29,"context_line":"import uuid as stdlib_uuid"},{"line_number":30,"context_line":""},{"line_number":31,"context_line":"from eventlet import greenthread"},{"line_number":32,"context_line":"from lxml import etree"}],"source_content_type":"text/x-python","patch_set":6,"id":"AAAATn%2F%2Bf9w%3D","line":29,"in_reply_to":"AAAATn%2F%2BgOw%3D","updated":"2013-11-25 13:11:31.000000000","message":"Thanks Daniel. Please take a look at line 247 and 287, those two lines shadow uuid.\n\nIs it OK that I change the variable name \"uuid\" to \"uuid_str\" or what else? Once the variable name changed, then I can use \"import uuid\" directly.","commit_id":"c6fe8bcbeb5d60bc72d6573362ec4ee1ace458d2"}]}
