)]}'
{"lib/cinder_plugins/nfs":[{"author":{"_account_id":7369,"name":"Sergey Skripnick","email":"sskripnick@mirantis.com","username":"redixin"},"change_message_id":"5c6bbd0219e5d5a895013c04aec9938b23d7bbcf","unresolved":false,"context_lines":[{"line_number":30,"context_line":"function configure_cinder_driver {"},{"line_number":31,"context_line":"    iniset $CINDER_CONF DEFAULT volume_driver \"cinder.volume.drivers.nfs.NfsDriver\""},{"line_number":32,"context_line":"    iniset $CINDER_CONF DEFAULT nfs_shares_config \"$CINDER_CONF_DIR/nfs_shares.conf\""},{"line_number":33,"context_line":"    echo \"$CINDER_NFS_SERVERPATH\" | sudo tee \"$CINDER_CONF_DIR/nfs_shares.conf\""},{"line_number":34,"context_line":"    sudo chmod 660 $CINDER_CONF_DIR/nfs_shares.conf"},{"line_number":35,"context_line":"    sudo chown $STACK_USER:$STACK_USER $CINDER_CONF_DIR/nfs_shares.conf"},{"line_number":36,"context_line":"}"}],"source_content_type":"application/octet-stream","patch_set":2,"id":"AAAAWH%2F9k5U%3D","line":33,"updated":"2014-04-03 15:29:51.000000000","message":"@Thang\n\nYou can just remove sudo here, and remove line 35","commit_id":"dc50665d106a956578ca5555457649b48d7f45c2"},{"author":{"_account_id":8247,"name":"Thang Pham","email":"thang.g.pham@gmail.com","username":"thang.pham"},"change_message_id":"5a520eaec21cd2f216e590cea0140c808a3fc5e4","unresolved":false,"context_lines":[{"line_number":30,"context_line":"function configure_cinder_driver {"},{"line_number":31,"context_line":"    iniset $CINDER_CONF DEFAULT volume_driver \"cinder.volume.drivers.nfs.NfsDriver\""},{"line_number":32,"context_line":"    iniset $CINDER_CONF DEFAULT nfs_shares_config \"$CINDER_CONF_DIR/nfs_shares.conf\""},{"line_number":33,"context_line":"    echo \"$CINDER_NFS_SERVERPATH\" | sudo tee \"$CINDER_CONF_DIR/nfs_shares.conf\""},{"line_number":34,"context_line":"    sudo chmod 660 $CINDER_CONF_DIR/nfs_shares.conf"},{"line_number":35,"context_line":"    sudo chown $STACK_USER:$STACK_USER $CINDER_CONF_DIR/nfs_shares.conf"},{"line_number":36,"context_line":"}"}],"source_content_type":"application/octet-stream","patch_set":2,"id":"AAAAWH%2F9b%2FA%3D","line":33,"in_reply_to":"AAAAWH%2F9k5U%3D","updated":"2014-04-04 14:42:29.000000000","message":"Done","commit_id":"dc50665d106a956578ca5555457649b48d7f45c2"},{"author":{"_account_id":7369,"name":"Sergey Skripnick","email":"sskripnick@mirantis.com","username":"redixin"},"change_message_id":"5c6bbd0219e5d5a895013c04aec9938b23d7bbcf","unresolved":false,"context_lines":[{"line_number":31,"context_line":"    iniset $CINDER_CONF DEFAULT volume_driver \"cinder.volume.drivers.nfs.NfsDriver\""},{"line_number":32,"context_line":"    iniset $CINDER_CONF DEFAULT nfs_shares_config \"$CINDER_CONF_DIR/nfs_shares.conf\""},{"line_number":33,"context_line":"    echo \"$CINDER_NFS_SERVERPATH\" | sudo tee \"$CINDER_CONF_DIR/nfs_shares.conf\""},{"line_number":34,"context_line":"    sudo chmod 660 $CINDER_CONF_DIR/nfs_shares.conf"},{"line_number":35,"context_line":"    sudo chown $STACK_USER:$STACK_USER $CINDER_CONF_DIR/nfs_shares.conf"},{"line_number":36,"context_line":"}"},{"line_number":37,"context_line":""}],"source_content_type":"application/octet-stream","patch_set":2,"id":"AAAAWH%2F9k5A%3D","line":34,"updated":"2014-04-03 15:29:51.000000000","message":"Not sure if this file contains any sensitive data. IMO this line is not necessary.","commit_id":"dc50665d106a956578ca5555457649b48d7f45c2"},{"author":{"_account_id":8247,"name":"Thang Pham","email":"thang.g.pham@gmail.com","username":"thang.pham"},"change_message_id":"5a520eaec21cd2f216e590cea0140c808a3fc5e4","unresolved":false,"context_lines":[{"line_number":31,"context_line":"    iniset $CINDER_CONF DEFAULT volume_driver \"cinder.volume.drivers.nfs.NfsDriver\""},{"line_number":32,"context_line":"    iniset $CINDER_CONF DEFAULT nfs_shares_config \"$CINDER_CONF_DIR/nfs_shares.conf\""},{"line_number":33,"context_line":"    echo \"$CINDER_NFS_SERVERPATH\" | sudo tee \"$CINDER_CONF_DIR/nfs_shares.conf\""},{"line_number":34,"context_line":"    sudo chmod 660 $CINDER_CONF_DIR/nfs_shares.conf"},{"line_number":35,"context_line":"    sudo chown $STACK_USER:$STACK_USER $CINDER_CONF_DIR/nfs_shares.conf"},{"line_number":36,"context_line":"}"},{"line_number":37,"context_line":""}],"source_content_type":"application/octet-stream","patch_set":2,"id":"AAAAWH%2F9b7A%3D","line":34,"in_reply_to":"AAAAWH%2F9k5A%3D","updated":"2014-04-04 14:42:29.000000000","message":"I took your suggestion and removed the sudo and line 35. It is a simpler solution and sets the file ownership to $stack_user, allowing it to be read by c-vol.","commit_id":"dc50665d106a956578ca5555457649b48d7f45c2"},{"author":{"_account_id":2813,"name":"Mehdi Abaakouk","email":"sileht@sileht.net","username":"sileht"},"change_message_id":"e01e8a7205b5f32d5de38562e6fd40e9a0f377bb","unresolved":false,"context_lines":[{"line_number":31,"context_line":"    iniset $CINDER_CONF DEFAULT volume_driver \"cinder.volume.drivers.nfs.NfsDriver\""},{"line_number":32,"context_line":"    iniset $CINDER_CONF DEFAULT nfs_shares_config \"$CINDER_CONF_DIR/nfs_shares.conf\""},{"line_number":33,"context_line":"    echo \"$CINDER_NFS_SERVERPATH\" | tee \"$CINDER_CONF_DIR/nfs_shares.conf\""},{"line_number":34,"context_line":"    sudo chmod 660 $CINDER_CONF_DIR/nfs_shares.conf"},{"line_number":35,"context_line":"}"},{"line_number":36,"context_line":""},{"line_number":37,"context_line":"# Restore xtrace"}],"source_content_type":"application/octet-stream","patch_set":3,"id":"AAAAWn%2F%2F8Dc%3D","line":34,"updated":"2014-04-08 11:43:37.000000000","message":"This sudo is no more needed.","commit_id":"06f3bf3bad38c2abac186b78e39f3aed617c25e4"},{"author":{"_account_id":8247,"name":"Thang Pham","email":"thang.g.pham@gmail.com","username":"thang.pham"},"change_message_id":"4bbb703723f1954d320d4a8e33f90a12832e2e94","unresolved":false,"context_lines":[{"line_number":31,"context_line":"    iniset $CINDER_CONF DEFAULT volume_driver \"cinder.volume.drivers.nfs.NfsDriver\""},{"line_number":32,"context_line":"    iniset $CINDER_CONF DEFAULT nfs_shares_config \"$CINDER_CONF_DIR/nfs_shares.conf\""},{"line_number":33,"context_line":"    echo \"$CINDER_NFS_SERVERPATH\" | tee \"$CINDER_CONF_DIR/nfs_shares.conf\""},{"line_number":34,"context_line":"    sudo chmod 660 $CINDER_CONF_DIR/nfs_shares.conf"},{"line_number":35,"context_line":"}"},{"line_number":36,"context_line":""},{"line_number":37,"context_line":"# Restore xtrace"}],"source_content_type":"application/octet-stream","patch_set":3,"id":"AAAAWn%2F%2F6tU%3D","line":34,"in_reply_to":"AAAAWn%2F%2F8Dc%3D","updated":"2014-04-08 14:14:48.000000000","message":"Done.  The whole sudo line is not needed because nfs_shares.conf will be created with 644 permissions and owned by $stack_user:$stack_user.","commit_id":"06f3bf3bad38c2abac186b78e39f3aed617c25e4"}]}