)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":2218,"name":"Adam Young","email":"adam@younglogic.com","username":"ayoung"},"change_message_id":"5302a9e5204a5b62604b597bcc75f414ec41bfb9","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"3ff4193d_d96a2ef3","updated":"2021-12-23 15:37:36.000000000","message":"Please add this link to the commit message instead, as it is the real summary.\n\nhttps://www.python.org/dev/peps/pep-0506/\n\nThanks for catching and implementing.\n\nAre there any other places in OpenStack code where os.urandom is used?","commit_id":"de69880650fd0c5bc8246d59f3041a966dc2af6c"},{"author":{"_account_id":16198,"name":"Ilya Popov","email":"hebulrih@gmail.com","username":"IPO"},"change_message_id":"6bc2354e53a1a9dd45a6e7e347ac177b2cc7e241","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":1,"id":"4d930e57_c77e2794","updated":"2021-12-24 08:56:26.000000000","message":"There are several change ids in commit message. Should be only one.\nCould you please remove Change-Id: If0877968ba257fa913b03d512633f2d62e5d22de\nfrom the commit message\n","commit_id":"de69880650fd0c5bc8246d59f3041a966dc2af6c"},{"author":{"_account_id":16198,"name":"Ilya Popov","email":"hebulrih@gmail.com","username":"IPO"},"change_message_id":"d67deaa747ce087f8b5f40de88a9c34dae80696a","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"d5523a78_eff3803d","in_reply_to":"3ff4193d_d96a2ef3","updated":"2021-12-24 07:30:38.000000000","message":"At least in nova there are 2-3 places. But in general I believe it is good change","commit_id":"de69880650fd0c5bc8246d59f3041a966dc2af6c"},{"author":{"_account_id":34392,"name":"bieji","display_name":"yourijing","email":"linjiangbieji@qq.com","username":"linjiang"},"change_message_id":"236ddbe1869526a3f196899b50f6b8f340f5b3e9","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"3a479911_ece83936","in_reply_to":"3ff4193d_d96a2ef3","updated":"2021-12-27 15:40:48.000000000","message":"OK,I will add this link to the commit message instead. I only focus on keystone at present,That\u0027s all I found in keystone","commit_id":"de69880650fd0c5bc8246d59f3041a966dc2af6c"},{"author":{"_account_id":34392,"name":"bieji","display_name":"yourijing","email":"linjiangbieji@qq.com","username":"linjiang"},"change_message_id":"236ddbe1869526a3f196899b50f6b8f340f5b3e9","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"c06a96de_24e772bd","in_reply_to":"4d930e57_c77e2794","updated":"2021-12-27 15:40:48.000000000","message":"OK","commit_id":"de69880650fd0c5bc8246d59f3041a966dc2af6c"},{"author":{"_account_id":16198,"name":"Ilya Popov","email":"hebulrih@gmail.com","username":"IPO"},"change_message_id":"3a490a2dd2fe32b2fe1e33206da318156581cb56","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":2,"id":"6cde0d91_64f89dd0","updated":"2021-12-24 10:53:20.000000000","message":"Thank you for quick fix about change id.\n\nI propose you perform at least the following additional changes into commit message:\n\n- 1-st raw should be reduced to not more than 50 characters. It should contain summary of fix.\n- After fist raw please create one empty raw\n- Next raws should been limited by 72 characters length\n\nPlease have a look to documentation about commit messages for more info for future commits:\nhttps://wiki.openstack.org/wiki/GitCommitMessages\n\nThanks a lot","commit_id":"f771f14cd60931348af43192804092488180fffb"},{"author":{"_account_id":34392,"name":"bieji","display_name":"yourijing","email":"linjiangbieji@qq.com","username":"linjiang"},"change_message_id":"236ddbe1869526a3f196899b50f6b8f340f5b3e9","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"e3326ecf_60e968a7","in_reply_to":"6cde0d91_64f89dd0","updated":"2021-12-27 15:40:48.000000000","message":"OK","commit_id":"f771f14cd60931348af43192804092488180fffb"},{"author":{"_account_id":34392,"name":"bieji","display_name":"yourijing","email":"linjiangbieji@qq.com","username":"linjiang"},"change_message_id":"172844aefa6cb8c9a57b44152f13cccd6098a248","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"13399fde_c7a7f00e","updated":"2021-12-27 15:42:13.000000000","message":"thank you for you review","commit_id":"f5787df9d49140e31a2f807bc10d55754a3ca297"},{"author":{"_account_id":7414,"name":"David Wilde","email":"dwilde@redhat.com","username":"d34dh0r53"},"change_message_id":"6ee3ef8c4349a7d1a8670e68d3909c85e8b1f1ab","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":5,"id":"8f723bf5_adf92c2c","updated":"2022-01-21 16:48:57.000000000","message":"Looks good to me","commit_id":"0b64050e6b3daeed0aee4496d6cba2c31eeb7d83"},{"author":{"_account_id":7973,"name":"Douglas Mendizábal","email":"dmendiza@redhat.com","username":"dougmendizabal"},"change_message_id":"33204aaf0cd48c3e1957869bc3a723ee1e571310","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":5,"id":"ff3dedf8_e895737d","updated":"2022-01-21 16:49:10.000000000","message":"Per docs [1] these are cryptographically strong functions, so this LGTM.\n\n[1] https://docs.python.org/3/library/secrets.html#secrets.token_bytes","commit_id":"0b64050e6b3daeed0aee4496d6cba2c31eeb7d83"}]}