)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":9535,"name":"Gorka Eguileor","email":"geguileo@redhat.com","username":"Gorka"},"change_message_id":"0937c5005083ed00ce66149cceab700ef77ff0df","unresolved":false,"context_lines":[{"line_number":10,"context_line":"The problem starts when rtstool, which responsible to create a target for"},{"line_number":11,"context_line":"LVM ISCSI driver configured with LIO helper, detects that this target already"},{"line_number":12,"context_line":"exists and reports back success but actually does nothing. Cinder updates Nova"},{"line_number":13,"context_line":"with newly generated credentials while portal attached access control"},{"line_number":14,"context_line":"lists (acl) still keep old credentials. Any attempt to perform login"},{"line_number":15,"context_line":"from Nova to that portal will fail due to the credentials\u0027 mismatch."},{"line_number":16,"context_line":""}],"source_content_type":"text/x-gerrit-commit-message","patch_set":3,"id":"7a016987_c19b0516","line":13,"updated":"2015-05-29 14:41:14.000000000","message":"Since patch [1] was merged that no longer is true. Now try to reuse existing chap auth username and password and only generate new ones if we don\u0027t have them already. So bug 1409409 is already fixed.\n\nrtstool has a bug of not updating ACL when asked to, so this patch is still needed.\n\n[1]: https://review.openstack.org/#/c/148038/","commit_id":"a98cc01fd8ab1b38d64c64d1ecfb644ad66ba1b6"},{"author":{"_account_id":9535,"name":"Gorka Eguileor","email":"geguileo@redhat.com","username":"Gorka"},"change_message_id":"0937c5005083ed00ce66149cceab700ef77ff0df","unresolved":false,"context_lines":[{"line_number":18,"context_line":"such a mismatch."},{"line_number":19,"context_line":""},{"line_number":20,"context_line":"Change-Id: I030a67a6e2a04c8dda02096b972fcb770e442fd8"},{"line_number":21,"context_line":"Closes-Bug: #1409409"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":3,"id":"7a016987_241427e7","line":21,"updated":"2015-05-29 14:41:14.000000000","message":"That bug\u0027s status should be fixed (I\u0027ll update now) and another one should be open for rtstool.","commit_id":"a98cc01fd8ab1b38d64c64d1ecfb644ad66ba1b6"}],"cinder/cmd/rtstool.py":[{"author":{"_account_id":9535,"name":"Gorka Eguileor","email":"geguileo@redhat.com","username":"Gorka"},"change_message_id":"0937c5005083ed00ce66149cceab700ef77ff0df","unresolved":false,"context_lines":[{"line_number":99,"context_line":"                acl.chap_password \u003d password"},{"line_number":100,"context_line":""},{"line_number":101,"context_line":""},{"line_number":102,"context_line":"def _lookup_target(target_iqn):"},{"line_number":103,"context_line":"    try:"},{"line_number":104,"context_line":"        rtsroot \u003d rtslib.root.RTSRoot()"},{"line_number":105,"context_line":"    except rtslib.utils.RTSLibError:"}],"source_content_type":"text/x-python","patch_set":3,"id":"7a016987_c422f353","line":102,"updated":"2015-05-29 14:41:14.000000000","message":"Nice idea removing unused initiator_iqn","commit_id":"a98cc01fd8ab1b38d64c64d1ecfb644ad66ba1b6"}]}