)]}'
{"scripts/conf/vertica.conf":[{"author":{"_account_id":10215,"name":"Peter Stachowski","email":"peter@tesora.com","username":"peterstac"},"change_message_id":"90959818fffef13cff90ab350acaee820a3fe2a2","unresolved":false,"context_lines":[{"line_number":1,"context_line":"    \"instance_flavor_name\": \"test.small\","},{"line_number":2,"context_line":"    \"instance_bigger_flavor_name\": \"test.medium.resize\","},{"line_number":3,"context_line":"    \"instance_eph_flavor_name\": \"test.eph.small\","},{"line_number":4,"context_line":"    \"instance_bigger_eph_flavor_name\": \"test.eph.medium.resize\","},{"line_number":5,"context_line":"    \"trove_volume_support\": true,"}],"source_content_type":"text/plain","patch_set":19,"id":"dae33548_89712fe3","line":2,"range":{"start_line":2,"start_character":41,"end_line":2,"end_character":54},"updated":"2016-02-16 19:52:14.000000000","message":"Why are we going from a test.small to test.medium.resize ?  Shouldn\u0027t test.small.resize be sufficient?","commit_id":"b449f1fb409ae95e1a7c267ea5dfa2b6bc0bb7ee"}],"scripts/localrc.rc":[{"author":{"_account_id":14176,"name":"Nirav Shah","email":"nshah@tesora.com","username":"nshah"},"change_message_id":"31340989a5479d203bb6e5f77c8326a2bae0d334","unresolved":false,"context_lines":[{"line_number":69,"context_line":"SERVICE_DIR\u003d$SERVICE_DIR"},{"line_number":70,"context_line":"VOLUME_GROUP\u003d${VOLUME_GROUP}"},{"line_number":71,"context_line":"VOLUME_BACKING_FILE\u003d${VOLUME_BACKING_FILE}"},{"line_number":72,"context_line":"VOLUME_BACKING_FILE_SIZE\u003d$(VOLUME_BACKING_FILE_SIZE)"},{"line_number":73,"context_line":"# The lock_path is by default /opt/stack/nova; if this path is a shared"},{"line_number":74,"context_line":"# folder in VirtualBox things seem to break. We fix it by setting EXTRA_OPS"},{"line_number":75,"context_line":"# to force lock_path to /tmp."}],"source_content_type":"application/octet-stream","patch_set":5,"id":"5a5ae5dd_96a21f4a","line":72,"updated":"2016-02-05 15:23:05.000000000","message":"Need to change this to {}, instead of ().\n\nYou will otherwise see this during install:\n\nredstack: line 448: VOLUME_BACKING_FILE_SIZE: command not found","commit_id":"8a02b59b9d6695eb01793e2777f28ef6e0497d90"}],"scripts/redstack":[{"author":{"_account_id":10215,"name":"Peter Stachowski","email":"peter@tesora.com","username":"peterstac"},"change_message_id":"920e144cca8c1ea5f4c1c5d6c9fa7b288c29690a","unresolved":false,"context_lines":[{"line_number":686,"context_line":"        sed -i \"/%shared_network_subnet%/d\" $TEST_CONF"},{"line_number":687,"context_line":"    fi"},{"line_number":688,"context_line":""},{"line_number":689,"context_line":"    # Increase default quota."},{"line_number":690,"context_line":"    iniset $TROVE_CONF DEFAULT max_accepted_volume_size 10"},{"line_number":691,"context_line":"    iniset $TROVE_CONF DEFAULT max_instances_per_user 10"},{"line_number":692,"context_line":"    iniset $TROVE_CONF DEFAULT max_volumes_per_user 10"},{"line_number":693,"context_line":""},{"line_number":694,"context_line":"    # Test instance properties."},{"line_number":695,"context_line":"    TROVE_VOLUME_SUPPORT\u003d\"true\""}],"source_content_type":"application/octet-stream","patch_set":5,"id":"7a5de9d1_5c2fa13e","line":692,"range":{"start_line":689,"start_character":0,"end_line":692,"end_character":54},"updated":"2016-02-01 20:35:05.000000000","message":"This is already in trove/devstack/plugin.sh (so can be removed from here).  Although it would probably be a good idea to reference the variables in redstack.rc to control the values.  I.E.\n\n  # Set up quota values\n  TROVE_MAX_ACCEPTED_VOLUME_SIZE\u003d${TROVE_MAX_ACCEPTED_VOLUME_SIZE:-10}\n  TROVE_MAX_INSTANCES_PER_USER\u003d${TROVE_MAX_INSTANCES_PER_USER:-10}\n  TROVE_MAX_VOLUMES_PER_USER\u003d${TROVE_MAX_VOLUMES_PER_USER:-10}\n\nSee: https://review.openstack.org/274872","commit_id":"8a02b59b9d6695eb01793e2777f28ef6e0497d90"},{"author":{"_account_id":4240,"name":"Craig Vyvial","email":"cp16net@gmail.com","username":"cp16net"},"change_message_id":"8062c4076de895b78f9d1893b8b4009c7c992504","unresolved":false,"context_lines":[{"line_number":689,"context_line":"    # Increase default quota."},{"line_number":690,"context_line":"    iniset $TROVE_CONF DEFAULT max_accepted_volume_size 10"},{"line_number":691,"context_line":"    iniset $TROVE_CONF DEFAULT max_instances_per_user 10"},{"line_number":692,"context_line":"    iniset $TROVE_CONF DEFAULT max_volumes_per_user 10"},{"line_number":693,"context_line":""},{"line_number":694,"context_line":"    # Test instance properties."},{"line_number":695,"context_line":"    TROVE_VOLUME_SUPPORT\u003d\"true\""}],"source_content_type":"application/octet-stream","patch_set":5,"id":"7a5de9d1_8793ef4e","line":692,"updated":"2016-01-29 08:17:46.000000000","message":"i was adding something similar[1] but thinking about it defaulting all these fields including the VOLUME_BACKING_SIZE would be better than the change i was making. \n\n[1] https://review.openstack.org/#/c/270668/2/scripts/local.conf.d/use_large_volume.rc","commit_id":"8a02b59b9d6695eb01793e2777f28ef6e0497d90"},{"author":{"_account_id":4240,"name":"Craig Vyvial","email":"cp16net@gmail.com","username":"cp16net"},"change_message_id":"daf7dea377b8074179c23647e6ca08b88d898d69","unresolved":false,"context_lines":[{"line_number":723,"context_line":"        # MongoDB require replicas require 5GB data volume."},{"line_number":724,"context_line":"        # Increase the Cinder backing file size to accommodate at least 8 such volumes."},{"line_number":725,"context_line":"        TROVE_VOLUME_SIZE\u003d\"5\""},{"line_number":726,"context_line":"        export VOLUME_BACKING_FILE_SIZE\u003d\"50G\""},{"line_number":727,"context_line":"    elif [[ \"$DATASTORE_TYPE\" \u003d \"couchbase\" || \"$DATASTORE_TYPE\" \u003d \"couchdb\" || \"$DATASTORE_TYPE\" \u003d \"redis\" ]]; then"},{"line_number":728,"context_line":"        INSTANCE_FLAVOR\u003d\"test.tiny\""},{"line_number":729,"context_line":"        BIGGER_INSTANCE_FLAVOR\u003d\"test.tiny.resize\""}],"source_content_type":"application/octet-stream","patch_set":5,"id":"7a5de9d1_67759396","line":726,"updated":"2016-01-29 08:09:44.000000000","message":"Why do this if its defaulted higher?","commit_id":"8a02b59b9d6695eb01793e2777f28ef6e0497d90"},{"author":{"_account_id":10215,"name":"Peter Stachowski","email":"peter@tesora.com","username":"peterstac"},"change_message_id":"920e144cca8c1ea5f4c1c5d6c9fa7b288c29690a","unresolved":false,"context_lines":[{"line_number":723,"context_line":"        # MongoDB require replicas require 5GB data volume."},{"line_number":724,"context_line":"        # Increase the Cinder backing file size to accommodate at least 8 such volumes."},{"line_number":725,"context_line":"        TROVE_VOLUME_SIZE\u003d\"5\""},{"line_number":726,"context_line":"        export VOLUME_BACKING_FILE_SIZE\u003d\"50G\""},{"line_number":727,"context_line":"    elif [[ \"$DATASTORE_TYPE\" \u003d \"couchbase\" || \"$DATASTORE_TYPE\" \u003d \"couchdb\" || \"$DATASTORE_TYPE\" \u003d \"redis\" ]]; then"},{"line_number":728,"context_line":"        INSTANCE_FLAVOR\u003d\"test.tiny\""},{"line_number":729,"context_line":"        BIGGER_INSTANCE_FLAVOR\u003d\"test.tiny.resize\""}],"source_content_type":"application/octet-stream","patch_set":5,"id":"7a5de9d1_f724660d","line":726,"in_reply_to":"7a5de9d1_67759396","updated":"2016-02-01 20:35:05.000000000","message":"Yes, this should be removed","commit_id":"8a02b59b9d6695eb01793e2777f28ef6e0497d90"},{"author":{"_account_id":10215,"name":"Peter Stachowski","email":"peter@tesora.com","username":"peterstac"},"change_message_id":"138f90b1b0534abf17697c51c9636dd76b21587a","unresolved":false,"context_lines":[{"line_number":741,"context_line":""},{"line_number":742,"context_line":"}"},{"line_number":743,"context_line":""},{"line_number":744,"context_line":"function setup_cluster_configs() {"},{"line_number":745,"context_line":"    # Some generic cluster-related configurations required by the tests to run."},{"line_number":746,"context_line":"    iniset $TROVE_TASKMANAGER_CONF DEFAULT agent_call_high_timeout 300"},{"line_number":747,"context_line":"    iniset $TROVE_TASKMANAGER_CONF DEFAULT cluster_usage_timeout 2000"},{"line_number":748,"context_line":"    iniset $TROVE_TASKMANAGER_CONF DEFAULT resize_time_out 900"},{"line_number":749,"context_line":"    iniset $TROVE_TASKMANAGER_CONF DEFAULT usage_timeout 1500"},{"line_number":750,"context_line":"}"},{"line_number":751,"context_line":""},{"line_number":752,"context_line":"function add_test_flavors() {"},{"line_number":753,"context_line":"    # name id ram root_vol vcpu eph_vol"}],"source_content_type":"application/octet-stream","patch_set":5,"id":"7a5de9d1_3f8e542e","line":750,"range":{"start_line":744,"start_character":0,"end_line":750,"end_character":1},"updated":"2016-02-01 21:32:07.000000000","message":"Speaking with Petr, these should probably be moved to the plugin as well.  I\u0027ve added them to the changeset I submitted (with the exception of cluster_usage_timeout, since it has a default of 36000 already).  So these could be added to redstack.rc as well:\n\n  TROVE_AGENT_CALL_HIGH_TIMEOUT\u003d${TROVE_AGENT_CALL_HIGH_TIMEOUT:-300}\n  TROVE_RESIZE_TIME_OUT\u003d${TROVE_RESIZE_TIME_OUT:-900}\n  TROVE_USAGE_TIMEOUT\u003d${TROVE_USAGE_TIMEOUT:-1500}\n  TROVE_STATE_CHANGE_WAIT_TIME\u003d${TROVE_STATE_CHANGE_WAIT_TIME:-180}","commit_id":"8a02b59b9d6695eb01793e2777f28ef6e0497d90"},{"author":{"_account_id":10215,"name":"Peter Stachowski","email":"peter@tesora.com","username":"peterstac"},"change_message_id":"920e144cca8c1ea5f4c1c5d6c9fa7b288c29690a","unresolved":false,"context_lines":[{"line_number":750,"context_line":"}"},{"line_number":751,"context_line":""},{"line_number":752,"context_line":"function add_test_flavors() {"},{"line_number":753,"context_line":"    # name id ram root_vol vcpu eph_vol"},{"line_number":754,"context_line":"    add_flavor \u0027test.tiny\u0027 11 512 3 1 0"},{"line_number":755,"context_line":"    add_flavor \u0027test.tiny.resize\u0027 12 528 3 2 0"},{"line_number":756,"context_line":""}],"source_content_type":"application/octet-stream","patch_set":5,"id":"7a5de9d1_1759028e","line":753,"updated":"2016-02-01 20:35:05.000000000","message":"You removed tinier - is this not used anymore?","commit_id":"8a02b59b9d6695eb01793e2777f28ef6e0497d90"},{"author":{"_account_id":14576,"name":"Petr Malik","email":"pmalik@tesora.com","username":"pmalik"},"change_message_id":"96f1164ade2d546fa91ee7768bacaee794e3257e","unresolved":false,"context_lines":[{"line_number":750,"context_line":"}"},{"line_number":751,"context_line":""},{"line_number":752,"context_line":"function add_test_flavors() {"},{"line_number":753,"context_line":"    # name id ram root_vol vcpu eph_vol"},{"line_number":754,"context_line":"    add_flavor \u0027test.tiny\u0027 11 512 3 1 0"},{"line_number":755,"context_line":"    add_flavor \u0027test.tiny.resize\u0027 12 528 3 2 0"},{"line_number":756,"context_line":""}],"source_content_type":"application/octet-stream","patch_set":5,"id":"7a5de9d1_2c9e30b9","line":753,"in_reply_to":"7a5de9d1_1759028e","updated":"2016-02-01 21:04:01.000000000","message":"The tests used to default to \u0027tiny\u0027 which could support MySQL 5.5, but MySQL (corresponding Percona/Maria variants) 5.6 no longer runs reliably on that flavor (certainly not while performing actual data operations). Other datastores require even larger flavors. That\u0027s why we increased the default to \u0027small\u0027. The only datastore that can run on \u0027tiny\u0027 is Redis (but is also very limited). I don\u0027t think we could realistically run anything on \u0027tinier\u0027 anymore...","commit_id":"8a02b59b9d6695eb01793e2777f28ef6e0497d90"},{"author":{"_account_id":10215,"name":"Peter Stachowski","email":"peter@tesora.com","username":"peterstac"},"change_message_id":"920e144cca8c1ea5f4c1c5d6c9fa7b288c29690a","unresolved":false,"context_lines":[{"line_number":751,"context_line":""},{"line_number":752,"context_line":"function add_test_flavors() {"},{"line_number":753,"context_line":"    # name id ram root_vol vcpu eph_vol"},{"line_number":754,"context_line":"    add_flavor \u0027test.tiny\u0027 11 512 3 1 0"},{"line_number":755,"context_line":"    add_flavor \u0027test.tiny.resize\u0027 12 528 3 2 0"},{"line_number":756,"context_line":""},{"line_number":757,"context_line":"    add_flavor \u0027test.eph.tiny\u0027 13 512 3 1 1"}],"source_content_type":"application/octet-stream","patch_set":5,"id":"7a5de9d1_57669a2a","line":754,"range":{"start_line":754,"start_character":27,"end_line":754,"end_character":29},"updated":"2016-02-01 20:35:05.000000000","message":"You changed the id from 7 to 11.  I know \u00277\u0027 is arbitrary, but I am used to typing that every time I manually create a mysql or redis instance.  I suspect it will annoy people (and by people I mean me ;) ) if they keep getting an error when creating an instance.  Can the id\u0027s be changed back to what they originally were?  Or maybe we could use \u00277\u0027 and \u00277r\u0027, \u00278\u0027 and \u00278r\u0027 etc.  That\u0027ll allow us to test string ID\u0027s as well.","commit_id":"8a02b59b9d6695eb01793e2777f28ef6e0497d90"},{"author":{"_account_id":4240,"name":"Craig Vyvial","email":"cp16net@gmail.com","username":"cp16net"},"change_message_id":"4421f93ee5adc5a9e581bf5633bd64c2c0018d04","unresolved":false,"context_lines":[{"line_number":751,"context_line":""},{"line_number":752,"context_line":"function add_test_flavors() {"},{"line_number":753,"context_line":"    # name id ram root_vol vcpu eph_vol"},{"line_number":754,"context_line":"    add_flavor \u0027test.tiny\u0027 11 512 3 1 0"},{"line_number":755,"context_line":"    add_flavor \u0027test.tiny.resize\u0027 12 528 3 2 0"},{"line_number":756,"context_line":""},{"line_number":757,"context_line":"    add_flavor \u0027test.eph.tiny\u0027 13 512 3 1 1"}],"source_content_type":"application/octet-stream","patch_set":5,"id":"7a5de9d1_09b80674","line":754,"range":{"start_line":754,"start_character":27,"end_line":754,"end_character":29},"in_reply_to":"7a5de9d1_57669a2a","updated":"2016-02-01 20:58:48.000000000","message":"i dont like the idea of changing the id here to something other than 7. But i like the idea of testing a string like peter mentioned, \u00277r\u0027 could we have the best of both worlds? meaning having a flavor id of \u00277\u0027 and \u00277-trove\u0027 with the same values but different names so we continue backward compatibility for this? Something like...\n\n    add_flavor \u0027test.tiny-old\u0027 7 512 3 1 0\n    add_flavor \u0027test.tiny\u0027 7-trove 512 3 1 0\n    add_flavor \u0027test.resize-old\u0027 8 768 3 1 0\n    add_flavor \u0027test.resize\u0027 8-trove-resize 768 3 1 0","commit_id":"8a02b59b9d6695eb01793e2777f28ef6e0497d90"},{"author":{"_account_id":14576,"name":"Petr Malik","email":"pmalik@tesora.com","username":"pmalik"},"change_message_id":"bcc8b3dd29a0ffe0673b9d47a408bb88bd155b90","unresolved":false,"context_lines":[{"line_number":763,"context_line":"    add_flavor \u0027test.eph.small\u0027 17 768 3 1 1"},{"line_number":764,"context_line":"    add_flavor \u0027test.eph.small.resize\u0027 18 784 3 2 1"},{"line_number":765,"context_line":""},{"line_number":766,"context_line":"    add_flavor \u0027test.large\u0027 19 2048 3 1 0"},{"line_number":767,"context_line":"    add_flavor \u0027test.large.resize\u0027 20 2064 3 2 0"},{"line_number":768,"context_line":""},{"line_number":769,"context_line":"    add_flavor \u0027test.eph.large\u0027 21 2048 3 1 1"},{"line_number":770,"context_line":"    add_flavor \u0027test.eph.large.resize\u0027 22 2064 3 2 1"},{"line_number":771,"context_line":"}"},{"line_number":772,"context_line":""},{"line_number":773,"context_line":"function cmd_test_init() {"}],"source_content_type":"application/octet-stream","patch_set":5,"id":"3a57f1b5_bfc9c0ae","line":770,"range":{"start_line":766,"start_character":4,"end_line":770,"end_character":52},"updated":"2016-02-09 21:21:52.000000000","message":"After running with more datastores locally I suggest increasing the root volume size (4th argument) on large flavors to 5 (certain Cassandra versions need more than 3GB).","commit_id":"8a02b59b9d6695eb01793e2777f28ef6e0497d90"},{"author":{"_account_id":4240,"name":"Craig Vyvial","email":"cp16net@gmail.com","username":"cp16net"},"change_message_id":"68be29e3540c5c9fbe0bea1558d238a570e745fc","unresolved":false,"context_lines":[{"line_number":718,"context_line":"function add_test_flavors() {"},{"line_number":719,"context_line":"    # name id ram root_vol vcpu eph_vol"},{"line_number":720,"context_line":"    add_flavor \u0027test.tiny\u0027 11 512 3 1 0"},{"line_number":721,"context_line":"    add_flavor \u0027test.tiny.resize\u0027 12 528 3 2 0"},{"line_number":722,"context_line":""},{"line_number":723,"context_line":"    add_flavor \u0027test.eph.tiny\u0027 13 512 3 1 1"},{"line_number":724,"context_line":"    add_flavor \u0027test.eph.tiny.resize\u0027 14 528 3 2 1"}],"source_content_type":"application/octet-stream","patch_set":17,"id":"dae33548_ed8bd51c","line":721,"range":{"start_line":721,"start_character":34,"end_line":721,"end_character":36},"updated":"2016-02-16 04:11:04.000000000","message":"we mentioned to make this id a string like \u002711r\u0027. I thought we agreed on that or change of plan?","commit_id":"168432a8a3d8b0a879c6be13c2411580b4126ef2"},{"author":{"_account_id":10215,"name":"Peter Stachowski","email":"peter@tesora.com","username":"peterstac"},"change_message_id":"90959818fffef13cff90ab350acaee820a3fe2a2","unresolved":false,"context_lines":[{"line_number":710,"context_line":""},{"line_number":711,"context_line":"}"},{"line_number":712,"context_line":""},{"line_number":713,"context_line":"function setup_cluster_configs() {"},{"line_number":714,"context_line":"    # Setting cluster_member_count to 2 to decrease cluster spawn time."},{"line_number":715,"context_line":"    iniset $TROVE_CONF $DATASTORE_TYPE cluster_member_count $1"},{"line_number":716,"context_line":"}"},{"line_number":717,"context_line":""},{"line_number":718,"context_line":"function add_test_flavors() {"},{"line_number":719,"context_line":"    # name id ram root_vol vcpu eph_vol"}],"source_content_type":"application/octet-stream","patch_set":19,"id":"dae33548_6c4e7970","line":716,"range":{"start_line":713,"start_character":0,"end_line":716,"end_character":1},"updated":"2016-02-16 19:52:14.000000000","message":"This is a per-datastore setting as well - shouldn\u0027t it be in the new \u0027\u003cdatastore\u003e.conf\u0027 files?","commit_id":"b449f1fb409ae95e1a7c267ea5dfa2b6bc0bb7ee"},{"author":{"_account_id":10215,"name":"Peter Stachowski","email":"peter@tesora.com","username":"peterstac"},"change_message_id":"d8218df8be9e7ffe88ae1e3ff79ff33d505563a0","unresolved":false,"context_lines":[{"line_number":710,"context_line":""},{"line_number":711,"context_line":"}"},{"line_number":712,"context_line":""},{"line_number":713,"context_line":"function setup_cluster_configs() {"},{"line_number":714,"context_line":"    # Setting cluster_member_count to 2 to decrease cluster spawn time."},{"line_number":715,"context_line":"    iniset $TROVE_CONF $DATASTORE_TYPE cluster_member_count $1"},{"line_number":716,"context_line":"}"},{"line_number":717,"context_line":""},{"line_number":718,"context_line":"function add_test_flavors() {"},{"line_number":719,"context_line":"    # name id ram root_vol vcpu eph_vol"}],"source_content_type":"application/octet-stream","patch_set":19,"id":"dae33548_ac452171","line":716,"range":{"start_line":713,"start_character":0,"end_line":716,"end_character":1},"in_reply_to":"dae33548_6c4e7970","updated":"2016-02-16 20:08:40.000000000","message":"Ah, never mind - this is in TROVE_CONF not TEST_CONF ...","commit_id":"b449f1fb409ae95e1a7c267ea5dfa2b6bc0bb7ee"}],"scripts/redstack.rc":[{"author":{"_account_id":4240,"name":"Craig Vyvial","email":"cp16net@gmail.com","username":"cp16net"},"change_message_id":"daf7dea377b8074179c23647e6ca08b88d898d69","unresolved":false,"context_lines":[{"line_number":66,"context_line":"# Cinder Volume Group Name"},{"line_number":67,"context_line":"VOLUME_GROUP\u003d${VOLUME_GROUP:-stack-volumes}"},{"line_number":68,"context_line":"VOLUME_BACKING_FILE\u003d${VOLUME_BACKING_FILE:-${DATA_DIR}/${VOLUME_GROUP}-backing-file}"},{"line_number":69,"context_line":"VOLUME_BACKING_FILE_SIZE\u003d${VOLUME_BACKING_FILE_SIZE:-51200M}"},{"line_number":70,"context_line":""},{"line_number":71,"context_line":"# Passwords used by devstack."},{"line_number":72,"context_line":"MYSQL_PASSWORD\u003de1a2c042c828d3566d0a"}],"source_content_type":"application/octet-stream","patch_set":5,"id":"7a5de9d1_2758fb31","line":69,"updated":"2016-01-29 08:09:44.000000000","message":"if you default this to 51.2GB whats the point of setting it for mongo to 50GB? Why make this an obscure size of 51.2GB? Why not just 50GB here and remove the mongo value that is set?","commit_id":"8a02b59b9d6695eb01793e2777f28ef6e0497d90"},{"author":{"_account_id":10215,"name":"Peter Stachowski","email":"peter@tesora.com","username":"peterstac"},"change_message_id":"920e144cca8c1ea5f4c1c5d6c9fa7b288c29690a","unresolved":false,"context_lines":[{"line_number":66,"context_line":"# Cinder Volume Group Name"},{"line_number":67,"context_line":"VOLUME_GROUP\u003d${VOLUME_GROUP:-stack-volumes}"},{"line_number":68,"context_line":"VOLUME_BACKING_FILE\u003d${VOLUME_BACKING_FILE:-${DATA_DIR}/${VOLUME_GROUP}-backing-file}"},{"line_number":69,"context_line":"VOLUME_BACKING_FILE_SIZE\u003d${VOLUME_BACKING_FILE_SIZE:-51200M}"},{"line_number":70,"context_line":""},{"line_number":71,"context_line":"# Passwords used by devstack."},{"line_number":72,"context_line":"MYSQL_PASSWORD\u003de1a2c042c828d3566d0a"}],"source_content_type":"application/octet-stream","patch_set":5,"id":"7a5de9d1_7795d630","line":69,"in_reply_to":"7a5de9d1_2758fb31","updated":"2016-02-01 20:35:05.000000000","message":"Well, 51200M \u003d 50G, but I agree that that\u0027s not always intuitive.  The default value in stackrc is given as \u002710250M\u0027 though, so maybe we should stick with \u0027M\u0027 since we know it works. ;)","commit_id":"8a02b59b9d6695eb01793e2777f28ef6e0497d90"},{"author":{"_account_id":4240,"name":"Craig Vyvial","email":"cp16net@gmail.com","username":"cp16net"},"change_message_id":"4421f93ee5adc5a9e581bf5633bd64c2c0018d04","unresolved":false,"context_lines":[{"line_number":66,"context_line":"# Cinder Volume Group Name"},{"line_number":67,"context_line":"VOLUME_GROUP\u003d${VOLUME_GROUP:-stack-volumes}"},{"line_number":68,"context_line":"VOLUME_BACKING_FILE\u003d${VOLUME_BACKING_FILE:-${DATA_DIR}/${VOLUME_GROUP}-backing-file}"},{"line_number":69,"context_line":"VOLUME_BACKING_FILE_SIZE\u003d${VOLUME_BACKING_FILE_SIZE:-51200M}"},{"line_number":70,"context_line":""},{"line_number":71,"context_line":"# Passwords used by devstack."},{"line_number":72,"context_line":"MYSQL_PASSWORD\u003de1a2c042c828d3566d0a"}],"source_content_type":"application/octet-stream","patch_set":5,"id":"7a5de9d1_09280632","line":69,"in_reply_to":"7a5de9d1_7795d630","updated":"2016-02-01 20:58:48.000000000","message":"oh man that didnt even click to me. nvm then","commit_id":"8a02b59b9d6695eb01793e2777f28ef6e0497d90"}]}