)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":5293,"name":"Nikhil","email":"SlickNik@gmail.com","username":"slicknik"},"change_message_id":"63c4241dbb224942e9057b1863decf876f32c41a","unresolved":false,"context_lines":[{"line_number":15,"context_line":"Install the native Python client for Redis."},{"line_number":16,"context_line":""},{"line_number":17,"context_line":"Also removed an unnecessary check for trove directories in redstack"},{"line_number":18,"context_line":"- devstack checks them out if they don\u0027t exist."},{"line_number":19,"context_line":""},{"line_number":20,"context_line":"Change-Id: I436fd63005a1ddee27755277ca7d4680747f400c"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":4,"id":"fa32b979_e3fada42","line":18,"updated":"2015-06-17 08:28:47.000000000","message":"This has nothing to do with upgrading the redis elements to 3.0 and applying the recommended tweaks for redis. Please separate this out into a different patchset to address a cleaner separation of concern. Thanks!","commit_id":"8cbdcb1300d97fff7a52540d3095598694814d0e"}],"scripts/files/elements/ubuntu-redis/install.d/10-redis":[{"author":{"_account_id":7806,"name":"Sushil Kumar","email":"skm.net@gmail.com","username":"sushilkm"},"change_message_id":"652227b48d6dcc615b29bda6d43f08a42236319f","unresolved":false,"context_lines":[{"line_number":28,"context_line":""},{"line_number":29,"context_line":"_EOF_"},{"line_number":30,"context_line":""},{"line_number":31,"context_line":"sudo add-apt-repository -y ppa:chris-lea/redis-server"},{"line_number":32,"context_line":"sudo apt-get -y update"},{"line_number":33,"context_line":"apt-get install -y redis-server"},{"line_number":34,"context_line":""}],"source_content_type":"application/x-shellscript","patch_set":1,"id":"3afb71cf_eccb54a2","line":31,"updated":"2015-06-11 06:06:04.000000000","message":"No need to use sudo, the script is already running as root","commit_id":"48ca7ab39d9a3d1ee6c1b7c925ee571773c6ccf7"},{"author":{"_account_id":14576,"name":"Petr Malik","email":"pmalik@tesora.com","username":"pmalik"},"change_message_id":"f622972fd88fad93d07017982c9bd955ef9bb947","unresolved":false,"context_lines":[{"line_number":28,"context_line":""},{"line_number":29,"context_line":"_EOF_"},{"line_number":30,"context_line":""},{"line_number":31,"context_line":"sudo add-apt-repository -y ppa:chris-lea/redis-server"},{"line_number":32,"context_line":"sudo apt-get -y update"},{"line_number":33,"context_line":"apt-get install -y redis-server"},{"line_number":34,"context_line":""}],"source_content_type":"application/x-shellscript","patch_set":1,"id":"3afb71cf_47a47a49","line":31,"in_reply_to":"3afb71cf_eccb54a2","updated":"2015-06-11 10:21:43.000000000","message":"Thx, good catch.","commit_id":"48ca7ab39d9a3d1ee6c1b7c925ee571773c6ccf7"},{"author":{"_account_id":10440,"name":"Alex Tomic","email":"atomic777@gmail.com","username":"atomic77"},"change_message_id":"7b96ce3aa8ee11c72ac4bb5dccdfeda28d7cd831","unresolved":false,"context_lines":[{"line_number":28,"context_line":""},{"line_number":29,"context_line":"_EOF_"},{"line_number":30,"context_line":""},{"line_number":31,"context_line":"add-apt-repository -y ppa:chris-lea/redis-server"},{"line_number":32,"context_line":"apt-get -y update"},{"line_number":33,"context_line":"apt-get install -y redis-server"},{"line_number":34,"context_line":""}],"source_content_type":"application/x-shellscript","patch_set":4,"id":"1af86dd1_0369ce8b","line":31,"updated":"2015-06-11 21:16:14.000000000","message":"I get nervous when I see some random ppa repo but it seems like there\u0027s no other option for 3.0. Maybe we should build from source :)","commit_id":"8cbdcb1300d97fff7a52540d3095598694814d0e"},{"author":{"_account_id":5293,"name":"Nikhil","email":"SlickNik@gmail.com","username":"slicknik"},"change_message_id":"63c4241dbb224942e9057b1863decf876f32c41a","unresolved":false,"context_lines":[{"line_number":28,"context_line":""},{"line_number":29,"context_line":"_EOF_"},{"line_number":30,"context_line":""},{"line_number":31,"context_line":"add-apt-repository -y ppa:chris-lea/redis-server"},{"line_number":32,"context_line":"apt-get -y update"},{"line_number":33,"context_line":"apt-get install -y redis-server"},{"line_number":34,"context_line":""}],"source_content_type":"application/x-shellscript","patch_set":4,"id":"fa32b979_a68aa019","line":31,"in_reply_to":"1af86dd1_0369ce8b","updated":"2015-06-17 08:28:47.000000000","message":"I am a bit nervous about this too -- but the conversation at https://github.com/antirez/redis/issues/1732 gives me a bit more confidence, and I think I\u0027m okay going down this path until antirez / chris find a way to make this official.","commit_id":"8cbdcb1300d97fff7a52540d3095598694814d0e"},{"author":{"_account_id":13355,"name":"Edmond Kotowski","email":"ekotowski@gmail.com","username":"ekotowski"},"change_message_id":"ad63fde6442c7b46af01ce7550c56ce8c35db741","unresolved":false,"context_lines":[{"line_number":32,"context_line":""},{"line_number":33,"context_line":"_EOF_"},{"line_number":34,"context_line":""},{"line_number":35,"context_line":"add-apt-repository -y ppa:chris-lea/redis-server"},{"line_number":36,"context_line":"apt-get -y update"},{"line_number":37,"context_line":"apt-get install -y redis-server"},{"line_number":38,"context_line":""}],"source_content_type":"application/x-shellscript","patch_set":6,"id":"fa32b979_c14860cf","line":35,"updated":"2015-06-19 20:47:05.000000000","message":"Seems odd that we are just using chris-lea repository vs. an official redis repo.","commit_id":"e007a74f3a3fad87f8f5de66c45680a9da0c89cf"},{"author":{"_account_id":5293,"name":"Nikhil","email":"SlickNik@gmail.com","username":"slicknik"},"change_message_id":"53441728431615374148e045a71727de5489dfb7","unresolved":false,"context_lines":[{"line_number":32,"context_line":""},{"line_number":33,"context_line":"_EOF_"},{"line_number":34,"context_line":""},{"line_number":35,"context_line":"add-apt-repository -y ppa:chris-lea/redis-server"},{"line_number":36,"context_line":"apt-get -y update"},{"line_number":37,"context_line":"apt-get install -y redis-server"},{"line_number":38,"context_line":""}],"source_content_type":"application/x-shellscript","patch_set":6,"id":"fa32b979_38793d30","line":35,"in_reply_to":"fa32b979_c14860cf","updated":"2015-06-19 23:47:37.000000000","message":"See my comments from the earlier patchset. There isn\u0027t an official repo, and chris-lea is working with antirez to get his packages published (https://github.com/antirez/redis/issues/1732)","commit_id":"e007a74f3a3fad87f8f5de66c45680a9da0c89cf"}],"scripts/redstack":[{"author":{"_account_id":10215,"name":"Peter Stachowski","email":"peter@tesora.com","username":"peterstac"},"change_message_id":"bd4c68ef38518a2b50b49938729241b4d502f31f","unresolved":false,"context_lines":[{"line_number":466,"context_line":"    elif [ \"$DATASTORE_TYPE\" \u003d\u003d \"mongodb\" ]; then"},{"line_number":467,"context_line":"        PACKAGES\u003d${PACKAGES:-\"mongodb\"}"},{"line_number":468,"context_line":"        VERSION\u003d\"2.0.4\""},{"line_number":469,"context_line":"    elif [ \"$DATASTORE_TYPE\" \u003d\u003d \"redis\" ]; then"},{"line_number":470,"context_line":"        PACKAGES\u003d${PACKAGES:-\"redis-server\"}"},{"line_number":471,"context_line":"        VERSION\u003d\"2.8\""},{"line_number":472,"context_line":"    elif [ \"$DATASTORE_TYPE\" \u003d\u003d \"cassandra\" ]; then"},{"line_number":473,"context_line":"        PACKAGES\u003d${PACKAGES:-\"cassandra\"}"},{"line_number":474,"context_line":"        VERSION\u003d\"2.1.0\""}],"source_content_type":"application/octet-stream","patch_set":3,"id":"3afb71cf_7a85b6ab","line":471,"range":{"start_line":469,"start_character":1,"end_line":471,"end_character":21},"updated":"2015-06-11 18:16:17.000000000","message":"You should update this as well","commit_id":"cbf52a2f37ee1e8c173502899d1ab38429655d61"},{"author":{"_account_id":5293,"name":"Nikhil","email":"SlickNik@gmail.com","username":"slicknik"},"change_message_id":"63c4241dbb224942e9057b1863decf876f32c41a","unresolved":false,"context_lines":[{"line_number":339,"context_line":"    USER_OPTS_TAG_END\u003d\"$MARKER_TOKEN End Of User Specified Options $MARKER_TOKEN\""},{"line_number":340,"context_line":"    ADD_OPTS_TAG\u003d\"$MARKER_TOKEN Additional Options $MARKER_TOKEN\""},{"line_number":341,"context_line":"    ADD_OPTS_TAG_END\u003d\"$MARKER_TOKEN End Of Additional Options $MARKER_TOKEN\""},{"line_number":342,"context_line":"    set_trove_plugin_vars false"},{"line_number":343,"context_line":""},{"line_number":344,"context_line":"    pushd \"$PATH_DEVSTACK_SRC\""},{"line_number":345,"context_line":"    DEVSTACK_LOCAL_CONF\u003d$LOCAL_CONF"}],"source_content_type":"application/octet-stream","patch_set":4,"id":"fa32b979_e6e50838","line":342,"updated":"2015-06-17 08:28:47.000000000","message":"This has nothing to do with upgrading the redis elements to 3.0 and applying the recommended tweaks for redis. Please separate this out into a different patchset to address a cleaner separation of concern. Thanks!","commit_id":"8cbdcb1300d97fff7a52540d3095598694814d0e"}]}