)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":7806,"name":"Sushil Kumar","email":"skm.net@gmail.com","username":"sushilkm"},"change_message_id":"51d6266ac1a15b02bc2c0224e20f22b059b978d0","unresolved":false,"context_lines":[{"line_number":13,"context_line":"The datastore already supported by kick-start (mysql, redis, percona) have been"},{"line_number":14,"context_line":"given default packages. Packages for supported datastores can be overwritten by"},{"line_number":15,"context_line":"setting the $PACKAGES variable before running cmd_set_datastore()."},{"line_number":16,"context_line":""},{"line_number":17,"context_line":"Change-Id: I72a715ea460e74e870a3385b753da9d8ad1411aa"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":4,"id":"AAAATn%2F9SEM%3D","line":16,"updated":"2013-12-13 03:10:56.000000000","message":"Please follow common rules on git commit descriptions.\nYou may refer:\nhttps://wiki.openstack.org/wiki/Gerrit_Workflow#Committing_Changes","commit_id":"79fe6d407be922f1f79c563a3ee9dda3baa5a7c5"},{"author":{"_account_id":739,"name":"Michael Basnight","email":"mbasnight@gmail.com","username":"hubcap"},"change_message_id":"38ba2369873e25deffc60ba182e18e548d866e63","unresolved":false,"context_lines":[{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Add support for kick-starting new datastores"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"The datastore type and version config settings are now parameterized to allow any"},{"line_number":10,"context_line":"datastore. The only missing information for new datastores is the list of packages"},{"line_number":11,"context_line":"to install on the instance."},{"line_number":12,"context_line":""}],"source_content_type":"text/x-gerrit-commit-message","patch_set":6,"id":"AAAATn%2F857Q%3D","line":9,"updated":"2013-12-18 23:39:19.000000000","message":"lines are too long at 80 characters.","commit_id":"a106b41655d6bdc8d077bed863854b893fe458b5"},{"author":{"_account_id":739,"name":"Michael Basnight","email":"mbasnight@gmail.com","username":"hubcap"},"change_message_id":"38ba2369873e25deffc60ba182e18e548d866e63","unresolved":false,"context_lines":[{"line_number":15,"context_line":"setting the $PACKAGES variable before running cmd_set_datastore()."},{"line_number":16,"context_line":""},{"line_number":17,"context_line":"Change-Id: I72a715ea460e74e870a3385b753da9d8ad1411aa"},{"line_number":18,"context_line":"Co-Authored-By: Conrad Weidenkeller \u003cconrad@weidenkeller.com\u003e"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":6,"id":"AAAATn%2F857I%3D","line":18,"updated":"2013-12-18 23:39:19.000000000","message":"I think the co-author would be kevin since you took it over (i know it says hes the author, but the commit of the code will be you)","commit_id":"a106b41655d6bdc8d077bed863854b893fe458b5"},{"author":{"_account_id":7806,"name":"Sushil Kumar","email":"skm.net@gmail.com","username":"sushilkm"},"change_message_id":"1944fbd5a001bb25235f75e9a5cb2415bc281a45","unresolved":false,"context_lines":[{"line_number":17,"context_line":"setting the $PACKAGES variable before running cmd_set_datastore()."},{"line_number":18,"context_line":""},{"line_number":19,"context_line":"Change-Id: I72a715ea460e74e870a3385b753da9d8ad1411aa"},{"line_number":20,"context_line":"co-authored-by: Kevin Conway \u003ckevinjacobconway@gmail.com\u003e"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":8,"id":"AAAAUH%2F%2FwYs%3D","line":20,"updated":"2014-01-08 20:24:18.000000000","message":"no blueprints/bugs","commit_id":"19f257086abe992ffec5aea280de6f6e00f1416d"}],"scripts/redstack":[{"author":{"_account_id":7806,"name":"Sushil Kumar","email":"skm.net@gmail.com","username":"sushilkm"},"change_message_id":"146e0a9afbae7e39ddb0adc5f6bf30a215603f63","unresolved":false,"context_lines":[{"line_number":374,"context_line":"    if [ \"$SERVICE_TYPE\" \u003d\u003d \"mysql\" ]; then"},{"line_number":375,"context_line":"        PACKAGES\u003d${PACKAGES:-\"mysql-server-5.5\"}"},{"line_number":376,"context_line":"    elif [ \"$SERVICE_TYPE\" \u003d\u003d \"percona\" ]; then"},{"line_number":377,"context_line":"        PACKAGES\u003d${PACKAGE_NAME:-\"percona-server-server-5.5\"}"},{"line_number":378,"context_line":"    elif [ \"$SERVICE_TYPE\" \u003d\u003d \"redis\" ]; then"},{"line_number":379,"context_line":"        PACKAGES\u003d${PACKAGE_NAME:-\"redis-server\"}"},{"line_number":380,"context_line":"    else"}],"source_content_type":"application/octet-stream","patch_set":3,"id":"AAAATn%2F9Ugs%3D","line":377,"updated":"2013-12-12 17:29:29.000000000","message":"why do we have different variables used for similar task\nPACKAGES with mysql and PACKAGE_NAME with percon and redis?","commit_id":"ee6338753b599299efb12ab5657f458922f5ff6f"},{"author":{"_account_id":8214,"name":"Auston McReynolds","email":"amcreynolds@ebaysf.com","username":"amcreynolds"},"change_message_id":"6ab74cd149b7c28970609b3920582ee8e6fff531","unresolved":false,"context_lines":[{"line_number":367,"context_line":""},{"line_number":368,"context_line":"function cmd_set_datastore() {"},{"line_number":369,"context_line":"    IMAGEID\u003d$1"},{"line_number":370,"context_line":"    SERVICE_TYPE\u003d$2"},{"line_number":371,"context_line":"    # rd_manage datastore_update \u003cdatastore_name\u003e \u003cdatastore_manager\u003e \u003cdefault_version\u003e"},{"line_number":372,"context_line":"    rd_manage datastore_update \"Test_$SERVICE_TYPE\" \"$SERVICE_TYPE\" \"\""},{"line_number":373,"context_line":"    PACKAGES\u003d${PACKAGES:-\"\"}"}],"source_content_type":"application/octet-stream","patch_set":4,"id":"AAAATn%2F9TG0%3D","line":370,"updated":"2013-12-12 22:03:50.000000000","message":"(1) is there a reason we\u0027re using somewhat nonsensical names for the datastores and versions? naming it \"Test_mysql\" seems a bit unnecessary; why not name it according to a scheme that leads the user in the right direction for a production deployment? a user new to openstack will find having a datastore called \"Test_mysql\" with two versions: \"mysql_test_version\" \u0026 \"mysql_inactive_version\" fairly confusing. nova doesn\u0027t prefix it\u0027s pre-baked flavors with \"Test_\", so i\u0027m not sure why we should.\n\n(2) can we rename SERVICE_TYPE to DATASTORE_TYPE? SERVICE_TYPE used to directly map to a field aptly named service_type, but now that datastores has merged, I believe the nomenclature should align accordingly.","commit_id":"79fe6d407be922f1f79c563a3ee9dda3baa5a7c5"},{"author":{"_account_id":8214,"name":"Auston McReynolds","email":"amcreynolds@ebaysf.com","username":"amcreynolds"},"change_message_id":"beadef6080afeec2a1f0bee5006341e98aa7c572","unresolved":false,"context_lines":[{"line_number":367,"context_line":""},{"line_number":368,"context_line":"function cmd_set_datastore() {"},{"line_number":369,"context_line":"    IMAGEID\u003d$1"},{"line_number":370,"context_line":"    SERVICE_TYPE\u003d$2"},{"line_number":371,"context_line":"    # rd_manage datastore_update \u003cdatastore_name\u003e \u003cdatastore_manager\u003e \u003cdefault_version\u003e"},{"line_number":372,"context_line":"    rd_manage datastore_update \"Test_$SERVICE_TYPE\" \"$SERVICE_TYPE\" \"\""},{"line_number":373,"context_line":"    PACKAGES\u003d${PACKAGES:-\"\"}"}],"source_content_type":"application/octet-stream","patch_set":4,"id":"AAAATn%2F9Nt4%3D","line":370,"in_reply_to":"AAAATn%2F9OpE%3D","updated":"2013-12-13 23:01:27.000000000","message":"considering the conversation that just transpired in irc, please move forward with the style of (name\u003dmysql-5.5, version\u003d5.5.XX), considering the (name\u003dmysql, version\u003d5.5) will likely not be supported.","commit_id":"79fe6d407be922f1f79c563a3ee9dda3baa5a7c5"},{"author":{"_account_id":8214,"name":"Auston McReynolds","email":"amcreynolds@ebaysf.com","username":"amcreynolds"},"change_message_id":"9b427472b45f4ca4a54f8ded9965be8e4ff19ad4","unresolved":false,"context_lines":[{"line_number":367,"context_line":""},{"line_number":368,"context_line":"function cmd_set_datastore() {"},{"line_number":369,"context_line":"    IMAGEID\u003d$1"},{"line_number":370,"context_line":"    SERVICE_TYPE\u003d$2"},{"line_number":371,"context_line":"    # rd_manage datastore_update \u003cdatastore_name\u003e \u003cdatastore_manager\u003e \u003cdefault_version\u003e"},{"line_number":372,"context_line":"    rd_manage datastore_update \"Test_$SERVICE_TYPE\" \"$SERVICE_TYPE\" \"\""},{"line_number":373,"context_line":"    PACKAGES\u003d${PACKAGES:-\"\"}"}],"source_content_type":"application/octet-stream","patch_set":4,"id":"AAAATn%2F9OpE%3D","line":370,"in_reply_to":"AAAATn%2F9QKs%3D","updated":"2013-12-13 18:54:56.000000000","message":"as for (1), i\u0027d go with the style of (name\u003dmysql, version\u003d5.5) or (name\u003dmysql-5.5, version\u003d5.5.XX). rinse and repeat for other datastores and their appropriate versions.","commit_id":"79fe6d407be922f1f79c563a3ee9dda3baa5a7c5"},{"author":{"_account_id":8415,"name":"Denis M.","email":"lildee1991@gmail.com","username":"dmakogon"},"change_message_id":"f9850bc00852332b698047c3c39f78f7bf7a0542","unresolved":false,"context_lines":[{"line_number":367,"context_line":""},{"line_number":368,"context_line":"function cmd_set_datastore() {"},{"line_number":369,"context_line":"    IMAGEID\u003d$1"},{"line_number":370,"context_line":"    SERVICE_TYPE\u003d$2"},{"line_number":371,"context_line":"    # rd_manage datastore_update \u003cdatastore_name\u003e \u003cdatastore_manager\u003e \u003cdefault_version\u003e"},{"line_number":372,"context_line":"    rd_manage datastore_update \"Test_$SERVICE_TYPE\" \"$SERVICE_TYPE\" \"\""},{"line_number":373,"context_line":"    PACKAGES\u003d${PACKAGES:-\"\"}"}],"source_content_type":"application/octet-stream","patch_set":4,"id":"AAAATn%2F87ko%3D","line":370,"in_reply_to":"AAAATn%2F9QKs%3D","updated":"2013-12-18 16:53:07.000000000","message":"this variables are also have same names in trove tests, please take a look at them","commit_id":"79fe6d407be922f1f79c563a3ee9dda3baa5a7c5"},{"author":{"_account_id":7850,"name":"Kevin Conway","email":"kevinjacobconway@gmail.com","username":"kevinconway"},"change_message_id":"ce726f39c13fa2782a25904c7ec6b86a4ca571ad","unresolved":false,"context_lines":[{"line_number":367,"context_line":""},{"line_number":368,"context_line":"function cmd_set_datastore() {"},{"line_number":369,"context_line":"    IMAGEID\u003d$1"},{"line_number":370,"context_line":"    SERVICE_TYPE\u003d$2"},{"line_number":371,"context_line":"    # rd_manage datastore_update \u003cdatastore_name\u003e \u003cdatastore_manager\u003e \u003cdefault_version\u003e"},{"line_number":372,"context_line":"    rd_manage datastore_update \"Test_$SERVICE_TYPE\" \"$SERVICE_TYPE\" \"\""},{"line_number":373,"context_line":"    PACKAGES\u003d${PACKAGES:-\"\"}"}],"source_content_type":"application/octet-stream","patch_set":4,"id":"AAAATn%2F9QKs%3D","line":370,"in_reply_to":"AAAATn%2F9TG0%3D","updated":"2013-12-13 13:47:31.000000000","message":"1) I\u0027m not really sure why these names were chosen. I can remove the \"Test_\" for the datastore names and remove the lines that add null datastores. Does anyone have a preference on what should go in datastore version? I\u0027m considering just removing the \"_test_version\" and leaving both the name and version the same for kick-start.\n\n2) Yes, shame on me for using legacy variable names.","commit_id":"79fe6d407be922f1f79c563a3ee9dda3baa5a7c5"},{"author":{"_account_id":5293,"name":"Nikhil","email":"SlickNik@gmail.com","username":"slicknik"},"change_message_id":"e2655b631011b7cf0885bd3d4923cf5e9d58ec71","unresolved":false,"context_lines":[{"line_number":387,"context_line":"    rd_manage datastore_update \"Test_$SERVICE_TYPE\" \"$SERVICE_TYPE\" \"$SERVICE_TYPE\"_test_version"},{"line_number":388,"context_line":"    rd_manage datastore_update Test_Datastore_1 manager1 \"\""},{"line_number":389,"context_line":"    cmd_stop"},{"line_number":390,"context_line":"    iniset $TROVE_CONF_DIR/trove.conf DEFAULT default_datastore \"Test_$SERVICE_TYPE\""},{"line_number":391,"context_line":"    sleep 1.5"},{"line_number":392,"context_line":"    cmd_start"},{"line_number":393,"context_line":"}"}],"source_content_type":"application/octet-stream","patch_set":4,"id":"AAAATn%2F9SmY%3D","line":390,"updated":"2013-12-13 00:03:27.000000000","message":"It looks like the current fix always updates the default datastore to what\u0027s passed in. This means that if we call this function multiple times, the last service type is _always_ set as the default service type. This is somewhat inflexible, so you might want to pass in an option/flag to optionally set the datastore_type as default.","commit_id":"79fe6d407be922f1f79c563a3ee9dda3baa5a7c5"},{"author":{"_account_id":8214,"name":"Auston McReynolds","email":"amcreynolds@ebaysf.com","username":"amcreynolds"},"change_message_id":"9b427472b45f4ca4a54f8ded9965be8e4ff19ad4","unresolved":false,"context_lines":[{"line_number":387,"context_line":"    rd_manage datastore_update \"Test_$SERVICE_TYPE\" \"$SERVICE_TYPE\" \"$SERVICE_TYPE\"_test_version"},{"line_number":388,"context_line":"    rd_manage datastore_update Test_Datastore_1 manager1 \"\""},{"line_number":389,"context_line":"    cmd_stop"},{"line_number":390,"context_line":"    iniset $TROVE_CONF_DIR/trove.conf DEFAULT default_datastore \"Test_$SERVICE_TYPE\""},{"line_number":391,"context_line":"    sleep 1.5"},{"line_number":392,"context_line":"    cmd_start"},{"line_number":393,"context_line":"}"}],"source_content_type":"application/octet-stream","patch_set":4,"id":"AAAATn%2F9Ops%3D","line":390,"in_reply_to":"AAAATn%2F9QKg%3D","updated":"2013-12-13 18:54:56.000000000","message":"without setting it, the traditional devstack+redstack-kickstart combo won\u0027t work from a ui perspective (because the horizon trove dashboard hasn\u0027t been updated to support selecting datastores, meaning it relies on the default_datastore being set). by having a flag, as slicknik suggested, the user can signify that the iniset should not evaluate for additional runs.","commit_id":"79fe6d407be922f1f79c563a3ee9dda3baa5a7c5"},{"author":{"_account_id":7850,"name":"Kevin Conway","email":"kevinjacobconway@gmail.com","username":"kevinconway"},"change_message_id":"ce726f39c13fa2782a25904c7ec6b86a4ca571ad","unresolved":false,"context_lines":[{"line_number":387,"context_line":"    rd_manage datastore_update \"Test_$SERVICE_TYPE\" \"$SERVICE_TYPE\" \"$SERVICE_TYPE\"_test_version"},{"line_number":388,"context_line":"    rd_manage datastore_update Test_Datastore_1 manager1 \"\""},{"line_number":389,"context_line":"    cmd_stop"},{"line_number":390,"context_line":"    iniset $TROVE_CONF_DIR/trove.conf DEFAULT default_datastore \"Test_$SERVICE_TYPE\""},{"line_number":391,"context_line":"    sleep 1.5"},{"line_number":392,"context_line":"    cmd_start"},{"line_number":393,"context_line":"}"}],"source_content_type":"application/octet-stream","patch_set":4,"id":"AAAATn%2F9QKg%3D","line":390,"in_reply_to":"AAAATn%2F9SmY%3D","updated":"2013-12-13 13:47:31.000000000","message":"What do you think about Sushil\u0027s comment of simply not setting the default at all?","commit_id":"79fe6d407be922f1f79c563a3ee9dda3baa5a7c5"},{"author":{"_account_id":7806,"name":"Sushil Kumar","email":"skm.net@gmail.com","username":"sushilkm"},"change_message_id":"e0ab3483a87299a2b9e5ed7c3cff0b4d9dffd878","unresolved":false,"context_lines":[{"line_number":388,"context_line":"    rd_manage datastore_update Test_Datastore_1 manager1 \"\""},{"line_number":389,"context_line":"    cmd_stop"},{"line_number":390,"context_line":"    iniset $TROVE_CONF_DIR/trove.conf DEFAULT default_datastore \"Test_$SERVICE_TYPE\""},{"line_number":391,"context_line":"    sleep 1.5"},{"line_number":392,"context_line":"    cmd_start"},{"line_number":393,"context_line":"}"},{"line_number":394,"context_line":""}],"source_content_type":"application/octet-stream","patch_set":4,"id":"AAAATn%2F9SEw%3D","line":391,"updated":"2013-12-13 03:08:51.000000000","message":"Following SlickNik\u0027s comment, do we really need this paramter with multiple datastores.","commit_id":"79fe6d407be922f1f79c563a3ee9dda3baa5a7c5"},{"author":{"_account_id":7806,"name":"Sushil Kumar","email":"skm.net@gmail.com","username":"sushilkm"},"change_message_id":"cf30cce7261d63bf032fc9bafe2b0fb7f7da6d3f","unresolved":false,"context_lines":[{"line_number":388,"context_line":"    rd_manage datastore_update Test_Datastore_1 manager1 \"\""},{"line_number":389,"context_line":"    cmd_stop"},{"line_number":390,"context_line":"    iniset $TROVE_CONF_DIR/trove.conf DEFAULT default_datastore \"Test_$SERVICE_TYPE\""},{"line_number":391,"context_line":"    sleep 1.5"},{"line_number":392,"context_line":"    cmd_start"},{"line_number":393,"context_line":"}"},{"line_number":394,"context_line":""}],"source_content_type":"application/octet-stream","patch_set":4,"id":"AAAATn%2F9RBg%3D","line":391,"in_reply_to":"AAAATn%2F9SEw%3D","updated":"2013-12-13 10:16:47.000000000","message":"can we think of getting this iniset out of this method.\n(jst a thought).","commit_id":"79fe6d407be922f1f79c563a3ee9dda3baa5a7c5"},{"author":{"_account_id":739,"name":"Michael Basnight","email":"mbasnight@gmail.com","username":"hubcap"},"change_message_id":"ebba71913614fe8a7c44525ac7b0dc9909f0b393","unresolved":false,"context_lines":[{"line_number":390,"context_line":"    rd_manage datastore_update \"$DATASTORE_TYPE\" \"$DATASTORE_TYPE\" \"$DATASTORE_TYPE$VERSION\""},{"line_number":391,"context_line":"    rd_manage datastore_update Test_Datastore_1 manager1 \"\""},{"line_number":392,"context_line":"    cmd_stop"},{"line_number":393,"context_line":"    iniset $TROVE_CONF_DIR/trove.conf DEFAULT default_datastore \"$DATASTORE_TYPE\""},{"line_number":394,"context_line":"    sleep 1.5"},{"line_number":395,"context_line":"    cmd_start"},{"line_number":396,"context_line":"}"}],"source_content_type":"application/octet-stream","patch_set":6,"id":"AAAATn%2F812Y%3D","line":393,"updated":"2013-12-19 16:58:56.000000000","message":"I think this is failing because the int-tests are relying on the string Test_Mysql. Fix this in the tests to be the string mysql rather than Test_Mysql.","commit_id":"a106b41655d6bdc8d077bed863854b893fe458b5"},{"author":{"_account_id":8415,"name":"Denis M.","email":"lildee1991@gmail.com","username":"dmakogon"},"change_message_id":"98ac9b49a4f807869dad6a41a7c966d6c3993464","unresolved":false,"context_lines":[{"line_number":390,"context_line":"    rd_manage datastore_update \"$DATASTORE_TYPE\" \"$DATASTORE_TYPE\" \"$DATASTORE_TYPE$VERSION\""},{"line_number":391,"context_line":"    rd_manage datastore_update Test_Datastore_1 manager1 \"\""},{"line_number":392,"context_line":"    cmd_stop"},{"line_number":393,"context_line":"    iniset $TROVE_CONF_DIR/trove.conf DEFAULT default_datastore \"$DATASTORE_TYPE\""},{"line_number":394,"context_line":"    sleep 1.5"},{"line_number":395,"context_line":"    cmd_start"},{"line_number":396,"context_line":"}"}],"source_content_type":"application/octet-stream","patch_set":6,"id":"AAAATn%2F811g%3D","line":393,"in_reply_to":"AAAATn%2F812Y%3D","updated":"2013-12-19 17:00:37.000000000","message":"Agreed, just looked over reports and saw it.","commit_id":"a106b41655d6bdc8d077bed863854b893fe458b5"},{"author":{"_account_id":8415,"name":"Denis M.","email":"lildee1991@gmail.com","username":"dmakogon"},"change_message_id":"6672ae16010ddafc2e772db5f02676497a830e33","unresolved":false,"context_lines":[{"line_number":365,"context_line":"    echo `$@ | grep \u0027 id \u0027 | get_field 2`"},{"line_number":366,"context_line":"}"},{"line_number":367,"context_line":""},{"line_number":368,"context_line":"function cmd_set_datastore() {"},{"line_number":369,"context_line":"    IMAGEID\u003d$1"},{"line_number":370,"context_line":"    DATASTORE_TYPE\u003d$2"},{"line_number":371,"context_line":"    # rd_manage datastore_update \u003cdatastore_name\u003e \u003cdatastore_manager\u003e \u003cdefault_version\u003e"}],"source_content_type":"application/octet-stream","patch_set":8,"id":"AAAAT3%2F%2F4yg%3D","line":368,"updated":"2013-12-27 23:51:52.000000000","message":"I agreed that we all need this change. bug could you please explain why do we have to change name of datastore name/manager/version ? Why does previous names were bad ?","commit_id":"19f257086abe992ffec5aea280de6f6e00f1416d"},{"author":{"_account_id":4240,"name":"Craig Vyvial","email":"cp16net@gmail.com","username":"cp16net"},"change_message_id":"a9bf99d7bafe16684cddd407043a1df42b77c1dd","unresolved":false,"context_lines":[{"line_number":365,"context_line":"    echo `$@ | grep \u0027 id \u0027 | get_field 2`"},{"line_number":366,"context_line":"}"},{"line_number":367,"context_line":""},{"line_number":368,"context_line":"function cmd_set_datastore() {"},{"line_number":369,"context_line":"    IMAGEID\u003d$1"},{"line_number":370,"context_line":"    DATASTORE_TYPE\u003d$2"},{"line_number":371,"context_line":"    # rd_manage datastore_update \u003cdatastore_name\u003e \u003cdatastore_manager\u003e \u003cdefault_version\u003e"}],"source_content_type":"application/octet-stream","patch_set":8,"id":"AAAAT3%2F%2Fu1M%3D","line":368,"in_reply_to":"AAAAT3%2F%2F4yg%3D","updated":"2014-01-02 18:39:45.000000000","message":"the names that were used do not correspond with an actual setup. these names are good examples for providers to use in their deployment.","commit_id":"19f257086abe992ffec5aea280de6f6e00f1416d"},{"author":{"_account_id":7806,"name":"Sushil Kumar","email":"skm.net@gmail.com","username":"sushilkm"},"change_message_id":"1944fbd5a001bb25235f75e9a5cb2415bc281a45","unresolved":false,"context_lines":[{"line_number":365,"context_line":"    echo `$@ | grep \u0027 id \u0027 | get_field 2`"},{"line_number":366,"context_line":"}"},{"line_number":367,"context_line":""},{"line_number":368,"context_line":"function cmd_set_datastore() {"},{"line_number":369,"context_line":"    IMAGEID\u003d$1"},{"line_number":370,"context_line":"    DATASTORE_TYPE\u003d$2"},{"line_number":371,"context_line":"    # rd_manage datastore_update \u003cdatastore_name\u003e \u003cdatastore_manager\u003e \u003cdefault_version\u003e"}],"source_content_type":"application/octet-stream","patch_set":8,"id":"AAAAUH%2F%2FwYU%3D","line":368,"in_reply_to":"AAAAT3%2F%2Fu1M%3D","updated":"2014-01-08 20:24:18.000000000","message":"Agree to new names.","commit_id":"19f257086abe992ffec5aea280de6f6e00f1416d"},{"author":{"_account_id":739,"name":"Michael Basnight","email":"mbasnight@gmail.com","username":"hubcap"},"change_message_id":"e92d3008463e3820e4f2a5d849eff371a0f07527","unresolved":false,"context_lines":[{"line_number":365,"context_line":"    echo `$@ | grep \u0027 id \u0027 | get_field 2`"},{"line_number":366,"context_line":"}"},{"line_number":367,"context_line":""},{"line_number":368,"context_line":"function cmd_set_datastore() {"},{"line_number":369,"context_line":"    IMAGEID\u003d$1"},{"line_number":370,"context_line":"    DATASTORE_TYPE\u003d$2"},{"line_number":371,"context_line":"    # rd_manage datastore_update \u003cdatastore_name\u003e \u003cdatastore_manager\u003e \u003cdefault_version\u003e"}],"source_content_type":"application/octet-stream","patch_set":8,"id":"AAAAUH%2F%2FxS4%3D","line":368,"in_reply_to":"AAAAT3%2F%2Fu1M%3D","updated":"2014-01-08 17:04:20.000000000","message":"correct. this is good.","commit_id":"19f257086abe992ffec5aea280de6f6e00f1416d"}]}