)]}'
{"nova/tests/hyperv/hypervutils.py":[{"author":{"_account_id":4393,"name":"Dan Smith","email":"dms@danplanet.com","username":"danms"},"change_message_id":"528990f583aef2157f1e97546e05fae78e7dcb53","unresolved":false,"context_lines":[{"line_number":39,"context_line":"        self.__conn_wmi \u003d None"},{"line_number":40,"context_line":"        self.__conn_storage \u003d None"},{"line_number":41,"context_line":"        self._volumeutils \u003d volumeutilsV2.VolumeUtilsV2("},{"line_number":42,"context_line":"        self._conn_storage, self._conn_wmi)"},{"line_number":43,"context_line":""},{"line_number":44,"context_line":"    @property"},{"line_number":45,"context_line":"    def _conn(self):"}],"source_content_type":"text/x-python","patch_set":2,"id":"AAAAMH%2F%2FxQE%3D","line":42,"updated":"2012-11-26 17:21:07.000000000","message":"I\u0027m surprised this passes PEP8, but can you add some whitespace in front of this line (at least four more spaces) so that it\u0027s clear that it\u0027s not a standalone statement?\n\nActually, ideally I think the first argument should be on the same line as above, and split at the comma, aligning the first chars of the two arguments...","commit_id":"d0da19b1b59376803b8a4752bfd5c178e411e76a"}],"nova/virt/hyperv/driver.py":[{"author":{"_account_id":4393,"name":"Dan Smith","email":"dms@danplanet.com","username":"danms"},"change_message_id":"528990f583aef2157f1e97546e05fae78e7dcb53","unresolved":false,"context_lines":[{"line_number":62,"context_line":"\"\"\""},{"line_number":63,"context_line":""},{"line_number":64,"context_line":"from nova import config"},{"line_number":65,"context_line":"from nova.openstack.common import cfg"},{"line_number":66,"context_line":"from nova.openstack.common import log as logging"},{"line_number":67,"context_line":"from nova.virt import driver"},{"line_number":68,"context_line":"from nova.virt.hyperv import hostops"}],"source_content_type":"text/x-python","patch_set":2,"id":"AAAAMH%2F%2FxPw%3D","line":65,"updated":"2012-11-26 17:21:07.000000000","message":"Are these used?","commit_id":"d0da19b1b59376803b8a4752bfd5c178e411e76a"}],"nova/virt/hyperv/volumeutilsV2.py":[{"author":{"_account_id":4393,"name":"Dan Smith","email":"dms@danplanet.com","username":"danms"},"change_message_id":"528990f583aef2157f1e97546e05fae78e7dcb53","unresolved":false,"context_lines":[{"line_number":50,"context_line":"            target.Connect(NodeAddress\u003dtarget_iqn,"},{"line_number":51,"context_line":"                           IsPersistent\u003dTrue)"},{"line_number":52,"context_line":"            #Waiting the disk to be mounted. Research this"},{"line_number":53,"context_line":"            time.sleep(CONF.hyperv_wait_between_attach_retry)"},{"line_number":54,"context_line":""},{"line_number":55,"context_line":"        def logout_storage_target(self, target_iqn):"},{"line_number":56,"context_line":"            \"\"\" Logs out storage target through its session id \"\"\""}],"source_content_type":"text/x-python","patch_set":2,"id":"AAAAMH%2F%2FxPY%3D","line":53,"updated":"2012-11-26 17:21:07.000000000","message":"Apparently this is already in the existing driver, but it sure is uncool to sleep like this...","commit_id":"d0da19b1b59376803b8a4752bfd5c178e411e76a"}]}