)]}'
{"playbooks/roles/os_neutron/templates/plugins/plumgrid/pgrc":[{"author":{"_account_id":7353,"name":"Kevin Carter","email":"kevin@cloudnull.com","username":"cloudnull"},"change_message_id":"0616f5f307f5eaff77df4ce204be6963579fc8b8","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"ba63e191_57cb931e","updated":"2016-01-19 18:24:28.000000000","message":"Being that this is a bash file with variable expansion it may be a good idea to quote everything after the \"\u003d\" sign.\n\nIE: \n\nexport os_auth_url\u003d\u0027{{ keystone_service_adminuri }}/v3\u0027","commit_id":"6f8969effb35317c58321a9f682a9b11ca3de738"},{"author":{"_account_id":15526,"name":"Javeria Khan","email":"javeria.ak@gmail.com","username":"JaveriaK"},"change_message_id":"b67ee48ba18a2a935541c9cb6cf71e2ffb9a80f0","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"ba63e191_77301702","in_reply_to":"ba63e191_57cb931e","updated":"2016-01-19 18:28:04.000000000","message":"I didn\u0027t have any issues with the variable replacements, but sure. Pushed a patch.","commit_id":"6f8969effb35317c58321a9f682a9b11ca3de738"},{"author":{"_account_id":7353,"name":"Kevin Carter","email":"kevin@cloudnull.com","username":"cloudnull"},"change_message_id":"dc9b340ba079034bf5075ad3bf21e80a1e6f58ae","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"ba63e191_cbfa7ebd","in_reply_to":"ba63e191_77301702","updated":"2016-01-19 19:37:20.000000000","message":"More of just a guard in terms of someone adding a \";\" or \" \" to a variable causing the bash script to escape and run something unexpected.","commit_id":"6f8969effb35317c58321a9f682a9b11ca3de738"}]}