)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"change_message_id":"0c07b5523b04b761aa263440a48cdb1d4ebf96b2","unresolved":false,"context_lines":[{"line_number":5,"context_line":"CommitDate: 2018-12-12 12:57:04 -0800"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"WIP: Synchronize all lvm operations"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Change-Id: Idb4053685486f2163e7cb135cd579eba5d78f369"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"3f79a3b5_d773c11a","line":8,"updated":"2018-12-12 21:00:00.000000000","message":"Related to bug 1739482 and bug 1763712 which might be the same underlying issue.","commit_id":"af25e654689e069cf602ef8dbd0a387d416c43b9"},{"author":{"_account_id":4393,"name":"Dan Smith","email":"dms@danplanet.com","username":"danms"},"change_message_id":"d384be0151d1cebb4267ec723e84662fceff36fc","unresolved":false,"context_lines":[{"line_number":5,"context_line":"CommitDate: 2018-12-12 12:57:04 -0800"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"WIP: Synchronize all lvm operations"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Change-Id: Idb4053685486f2163e7cb135cd579eba5d78f369"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"3f79a3b5_97b22995","line":8,"in_reply_to":"3f79a3b5_d773c11a","updated":"2018-12-12 21:07:34.000000000","message":"Yep, I figure we\u0027ll recheck this a couple times to make sure it\u0027s good and then I\u0027ll write a test and proper commit message if the cinder folk want to float it in the gate.","commit_id":"af25e654689e069cf602ef8dbd0a387d416c43b9"}],"cinder/brick/local_dev/lvm.py":[{"author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"change_message_id":"25684c005d59d72096171abd1b7a7733a3ea9f99","unresolved":false,"context_lines":[{"line_number":41,"context_line":"    LVM_CMD_PREFIX \u003d [\u0027env\u0027, \u0027LC_ALL\u003dC\u0027]"},{"line_number":42,"context_line":"    _supports_pvs_ignoreskippedcluster \u003d None"},{"line_number":43,"context_line":""},{"line_number":44,"context_line":"    @lockutils.synchronized(\u0027cinder-lvm-exec\u0027, external\u003dTrue)"},{"line_number":45,"context_line":"    def _execute(self, *args, **kwargs):"},{"line_number":46,"context_line":"        return super(LVM, self)._execute(*args, **kwargs)"},{"line_number":47,"context_line":""}],"source_content_type":"text/x-python","patch_set":1,"id":"3f79a3b5_12e627dd","line":44,"updated":"2018-12-12 21:49:48.000000000","message":"Add a comment here as well (in the method, whatever) about why we need this (because lvm locks by default which can block and timeout operations).","commit_id":"af25e654689e069cf602ef8dbd0a387d416c43b9"}]}