)]}'
{"manifests/key.pp":[{"author":{"_account_id":6796,"name":"Giulio Fidente","email":"gfidente@redhat.com","username":"gfidente"},"change_message_id":"7d284b8d7e83314172dc14477442bcbebe0c32c1","unresolved":false,"context_lines":[{"line_number":122,"context_line":"set -ex"},{"line_number":123,"context_line":"NEW_KEYRING\u003d\\$(mktemp)"},{"line_number":124,"context_line":"ceph-authtool \\$NEW_KEYRING --name \u0027${name}\u0027 --add-key \u0027${secret}\u0027 ${caps}"},{"line_number":125,"context_line":"diff -N \\$NEW_KEYRING ${keyring_path}"},{"line_number":126,"context_line":"RV\u003d$?"},{"line_number":127,"context_line":"rm \\$NEW_KEYRING"},{"line_number":128,"context_line":"exit ${RV}\","}],"source_content_type":"text/x-puppet","patch_set":1,"id":"3a71b18c_1fee04b4","line":125,"updated":"2016-12-05 09:03:06.000000000","message":"I think we still need the grep to figure if there have been changes in the \"new\" version of the keyring?","commit_id":"b9632a6781ecbf616f47537ef914e8dae0d8f622"},{"author":{"_account_id":19564,"name":"Keith Schincke","email":"keith.schincke@gmail.com","username":"colonwq"},"change_message_id":"716b3a307094f7ff03b4a8cee98076675852b4cb","unresolved":false,"context_lines":[{"line_number":122,"context_line":"set -ex"},{"line_number":123,"context_line":"NEW_KEYRING\u003d\\$(mktemp)"},{"line_number":124,"context_line":"ceph-authtool \\$NEW_KEYRING --name \u0027${name}\u0027 --add-key \u0027${secret}\u0027 ${caps}"},{"line_number":125,"context_line":"diff -N \\$NEW_KEYRING ${keyring_path}"},{"line_number":126,"context_line":"RV\u003d$?"},{"line_number":127,"context_line":"rm \\$NEW_KEYRING"},{"line_number":128,"context_line":"exit ${RV}\","}],"source_content_type":"text/x-puppet","patch_set":1,"id":"3a71b18c_ff75c8f9","line":125,"in_reply_to":"3a71b18c_1fee04b4","updated":"2016-12-05 09:06:03.000000000","message":"diff will return 0 if there are no changes and 1 if there are changes.","commit_id":"b9632a6781ecbf616f47537ef914e8dae0d8f622"},{"author":{"_account_id":6796,"name":"Giulio Fidente","email":"gfidente@redhat.com","username":"gfidente"},"change_message_id":"094f161ee9a2a30984de29b46f27c7ad344f32b5","unresolved":false,"context_lines":[{"line_number":122,"context_line":"set -ex"},{"line_number":123,"context_line":"NEW_KEYRING\u003d\\$(mktemp)"},{"line_number":124,"context_line":"ceph-authtool \\$NEW_KEYRING --name \u0027${name}\u0027 --add-key \u0027${secret}\u0027 ${caps}"},{"line_number":125,"context_line":"diff -N \\$NEW_KEYRING ${keyring_path}"},{"line_number":126,"context_line":"RV\u003d$?"},{"line_number":127,"context_line":"rm \\$NEW_KEYRING"},{"line_number":128,"context_line":"exit ${RV}\","}],"source_content_type":"text/x-puppet","patch_set":1,"id":"3a71b18c_9a2302a7","line":125,"in_reply_to":"3a71b18c_ff75c8f9","updated":"2016-12-05 09:14:42.000000000","message":"ack","commit_id":"b9632a6781ecbf616f47537ef914e8dae0d8f622"},{"author":{"_account_id":6796,"name":"Giulio Fidente","email":"gfidente@redhat.com","username":"gfidente"},"change_message_id":"7d284b8d7e83314172dc14477442bcbebe0c32c1","unresolved":false,"context_lines":[{"line_number":151,"context_line":"set -ex"},{"line_number":152,"context_line":"OLD_KEYRING\u003d\\$(mktemp)"},{"line_number":153,"context_line":"ceph ${cluster_option} ${inject_id_option} ${inject_keyring_option} auth get ${name} -o \\$OLD_KEYRING || true"},{"line_number":154,"context_line":"diff -N \\$OLD_KEYRING ${keyring_path} | grep \u0027\u003e\u0027"},{"line_number":155,"context_line":"rm \\$OLD_KEYRING\","},{"line_number":156,"context_line":"      require   \u003d\u003e [ Class[\u0027ceph\u0027], Exec[\"ceph-key-${name}\"], ],"},{"line_number":157,"context_line":"      logoutput \u003d\u003e true,"}],"source_content_type":"text/x-puppet","patch_set":1,"id":"3a71b18c_5f323c60","line":154,"updated":"2016-12-05 09:03:06.000000000","message":"here too","commit_id":"b9632a6781ecbf616f47537ef914e8dae0d8f622"}]}