)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":7233,"name":"Matthew Oliver","email":"matt@oliver.net.au","username":"mattoliverau"},"change_message_id":"97110d34eeec4fa8205bcd1ac2271b7aea58fa98","unresolved":false,"context_lines":[{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Validate against invalid configs"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"... and display good errors"},{"line_number":10,"context_line":""},{"line_number":11,"context_line":"Change-Id: I451515a7155ff22069acb8ba9cc98d1f9bd7b286"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"dae33548_1ee2212f","line":9,"updated":"2016-02-18 04:42:27.000000000","message":"This commit message could be a little better.. but I know you know that clayg :P\n\nMaybe something like:\n\n  Better hash path config validation\n\n  Instead of expecting an operator to use only printable characters\n  in the HASH_PATH_[PREFIX, SUFFIX], this change validates and nicely\n  errors if it does.","commit_id":"a46dc2d009b79411914f6a0e8410c195bfacc60d"}],"etc/swift.conf-sample":[{"author":{"_account_id":7847,"name":"Alistair Coles","email":"alistairncoles@gmail.com","username":"acoles"},"change_message_id":"5bb38fcd51f016f62751943ff45916f8cc0b4d4b","unresolved":false,"context_lines":[{"line_number":4,"context_line":"# the hashing algorithm when determining data placement in the cluster."},{"line_number":5,"context_line":"# These values should remain secret and MUST NOT change"},{"line_number":6,"context_line":"# once a cluster has been deployed."},{"line_number":7,"context_line":"# Use only printable chars (python -c \"import string; print(string.printable)\")"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"swift_hash_path_suffix \u003d changeme"},{"line_number":10,"context_line":"swift_hash_path_prefix \u003d changeme"}],"source_content_type":"application/octet-stream","patch_set":2,"id":"dae33548_36329ba7","side":"PARENT","line":7,"updated":"2016-02-18 11:26:06.000000000","message":"why remove this info? its a constraint that this patch now enforces so why not document it?","commit_id":"e46d6b17251ab538595b1deb5581360a286b8fd1"}]}