)]}'
{"scripts/conf/test_begin.conf":[{"author":{"_account_id":8415,"name":"Denis M.","email":"lildee1991@gmail.com","username":"dmakogon"},"change_message_id":"472d62a13acf309548c638da6d1cf0cce7008d8c","unresolved":false,"context_lines":[{"line_number":88,"context_line":"    \"rabbit_runs_locally\":true,"},{"line_number":89,"context_line":"    \"default_character_set\": \"latin2\","},{"line_number":90,"context_line":"    \"default_collate\": \"latin2_general_ci\","},{"line_number":91,"context_line":"    \"dbaas_datastore\": \"%datastore_type%\","},{"line_number":92,"context_line":"    \"dbaas_datastore_version\": \"%datastore_version%\","},{"line_number":93,"context_line":""}],"source_content_type":"text/plain","patch_set":16,"id":"1ae5cdf2_76ce44b9","line":91,"updated":"2014-06-16 11:03:33.000000000","message":"Please take a look at https://review.openstack.org/#/c/99682/.\nWas made a desicion that we should avoid use of datastores/version that were hardcoded in config file, instead of those Trove int-tests should use Datastores API to retrieve datastore/versions","commit_id":"5d405351d609a36e9752eeee0f6d94fa75f98b3b"},{"author":{"_account_id":9122,"name":"Shalini khandelwal","email":"shalini.khandelwal@globallogic.com","username":"shalini"},"change_message_id":"f7880e405e4b1b32ce74a7577886cf8059f123a2","unresolved":false,"context_lines":[{"line_number":88,"context_line":"    \"rabbit_runs_locally\":true,"},{"line_number":89,"context_line":"    \"default_character_set\": \"latin2\","},{"line_number":90,"context_line":"    \"default_collate\": \"latin2_general_ci\","},{"line_number":91,"context_line":"    \"dbaas_datastore\": \"%datastore_type%\","},{"line_number":92,"context_line":"    \"dbaas_datastore_version\": \"%datastore_version%\","},{"line_number":93,"context_line":""}],"source_content_type":"text/plain","patch_set":16,"id":"1ae5cdf2_36567c2b","line":91,"in_reply_to":"1ae5cdf2_76ce44b9","updated":"2014-06-16 11:28:56.000000000","message":"As i can see. This patch is under review and it is not yet reviewed by anyone.\nWill change this patch after merge.","commit_id":"5d405351d609a36e9752eeee0f6d94fa75f98b3b"}],"scripts/conf/test_end.conf":[{"author":{"_account_id":5293,"name":"Nikhil","email":"SlickNik@gmail.com","username":"slicknik"},"change_message_id":"c322e4f807b62dabc2626dcf4dcfb434897be86a","unresolved":false,"context_lines":[{"line_number":1,"context_line":"    \"default_character_set\": \"latin2\","},{"line_number":2,"context_line":"    \"default_collate\": \"latin2_general_ci\","},{"line_number":3,"context_line":"    \"dbaas_datastore\": \"%datastore_type%\","},{"line_number":4,"context_line":"    \"sentinel\": null"},{"line_number":5,"context_line":"}"}],"source_content_type":"text/plain","patch_set":15,"id":"1ae5cdf2_e0da184a","line":3,"range":{"start_line":3,"start_character":5,"end_line":3,"end_character":20},"updated":"2014-06-11 08:44:18.000000000","message":"Please add to test_begin.conf.\n\ntest_end.conf should only contain the sentinel value.","commit_id":"4f95ad5c156fc85fb12f0eddefb748015f663e25"}],"scripts/files/elements/ubuntu-vertica/install.d/97-vertica":[{"author":{"_account_id":5293,"name":"Nikhil","email":"SlickNik@gmail.com","username":"slicknik"},"change_message_id":"0e82fb0d7329708aa9b8816efd1c1fba6d7c8911","unresolved":false,"context_lines":[{"line_number":7,"context_line":"export DEBIAN_FRONTEND\u003dnoninteractive"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"apt-get update -y"},{"line_number":10,"context_line":"apt-get upgrade -y"},{"line_number":11,"context_line":"apt-get install build-essential bc iptables curl sysstat pstack mcelog python-dev python-pip g++ -y"},{"line_number":12,"context_line":"wget http://h30537.www3.hp.com/prdownloads/vertica_7.0.1-0_amd64.deb?downloadid\u003dkfgOdXyQ5mBk_H43Pf1HXu8MYy5ZJ0c3wy48_7ta84EdFRE0lvsWMyi3JcTzLoeK8Q2w65kh2GM_bT7Xrd5e51Hb6Leusrigugf83o4Cd2dFmFUM7FIs4XssR9jcFkNjaIQHD24o1II\u003d -O vertica_7.0.1-0_amd64.deb"},{"line_number":13,"context_line":"dpkg -i vertica_7.0.1-0_amd64.deb"}],"source_content_type":"application/x-shellscript","patch_set":11,"id":"3ae8d1ca_a3edf98d","line":10,"updated":"2014-05-27 08:16:31.000000000","message":"Is an upgrade of all packages really needed? Seems like a bad idea since your image won\u0027t have locked down versions of packages.","commit_id":"8138cd5b323865b36c2ff887bcf56c897bc83436"},{"author":{"_account_id":9122,"name":"Shalini khandelwal","email":"shalini.khandelwal@globallogic.com","username":"shalini"},"change_message_id":"62610e8152b6c703ce0012b7091f7be8103a7210","unresolved":false,"context_lines":[{"line_number":7,"context_line":"export DEBIAN_FRONTEND\u003dnoninteractive"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"apt-get update -y"},{"line_number":10,"context_line":"apt-get upgrade -y"},{"line_number":11,"context_line":"apt-get install build-essential bc iptables curl sysstat pstack mcelog python-dev python-pip g++ -y"},{"line_number":12,"context_line":"wget http://h30537.www3.hp.com/prdownloads/vertica_7.0.1-0_amd64.deb?downloadid\u003dkfgOdXyQ5mBk_H43Pf1HXu8MYy5ZJ0c3wy48_7ta84EdFRE0lvsWMyi3JcTzLoeK8Q2w65kh2GM_bT7Xrd5e51Hb6Leusrigugf83o4Cd2dFmFUM7FIs4XssR9jcFkNjaIQHD24o1II\u003d -O vertica_7.0.1-0_amd64.deb"},{"line_number":13,"context_line":"dpkg -i vertica_7.0.1-0_amd64.deb"}],"source_content_type":"application/x-shellscript","patch_set":11,"id":"3ae8d1ca_887c9474","line":10,"in_reply_to":"3ae8d1ca_a3edf98d","updated":"2014-05-27 11:38:25.000000000","message":"Yes it can be removed","commit_id":"8138cd5b323865b36c2ff887bcf56c897bc83436"},{"author":{"_account_id":5293,"name":"Nikhil","email":"SlickNik@gmail.com","username":"slicknik"},"change_message_id":"0e82fb0d7329708aa9b8816efd1c1fba6d7c8911","unresolved":false,"context_lines":[{"line_number":25,"context_line":"chown dbadmin:verticadba /var/lib/vertica"},{"line_number":26,"context_line":""},{"line_number":27,"context_line":"#Script to install vertica dependencies"},{"line_number":28,"context_line":"apt-get -y install unixODBC unixODBC-dev"},{"line_number":29,"context_line":"#check for directory existance"},{"line_number":30,"context_line":"if [ ! -d \u0027/opt/vertica/lib64\u0027 ]; then"},{"line_number":31,"context_line":"    exit \u0027lib64 missing\u0027"}],"source_content_type":"application/x-shellscript","patch_set":11,"id":"3ae8d1ca_839f1d03","line":28,"updated":"2014-05-27 08:16:31.000000000","message":"Why can\u0027t these be installed as part of the previous apt-get install?","commit_id":"8138cd5b323865b36c2ff887bcf56c897bc83436"},{"author":{"_account_id":5293,"name":"Nikhil","email":"SlickNik@gmail.com","username":"slicknik"},"change_message_id":"0e82fb0d7329708aa9b8816efd1c1fba6d7c8911","unresolved":false,"context_lines":[{"line_number":51,"context_line":"    mv /etc/vertica.ini /etc/vertica.ini.default"},{"line_number":52,"context_line":"fi"},{"line_number":53,"context_line":"mv odbcinst.ini /etc/"},{"line_number":54,"context_line":"mv vertica.ini /etc/"},{"line_number":55,"context_line":""},{"line_number":56,"context_line":"#installing vertica-sqlalchemy and dependencies"},{"line_number":57,"context_line":"pip install https://pyodbc.googlecode.com/files/pyodbc-3.0.6.zip"}],"source_content_type":"application/x-shellscript","patch_set":11,"id":"3ae8d1ca_86200b9d","line":54,"updated":"2014-05-27 08:16:31.000000000","message":"All of this moving and backup is easy to do using install. (man install)","commit_id":"8138cd5b323865b36c2ff887bcf56c897bc83436"},{"author":{"_account_id":9664,"name":"amrith","email":"amrith.kumar@gmail.com","username":"amrith"},"change_message_id":"5d3c12e1ebc28ac7fe78acd377500402d138e545","unresolved":false,"context_lines":[{"line_number":12,"context_line":""},{"line_number":13,"context_line":"# Creating dbadmin user"},{"line_number":14,"context_line":"groupadd verticadba"},{"line_number":15,"context_line":"useradd -g verticadba -d /home/dbadmin -s /bin/bash -m dbadmin"},{"line_number":16,"context_line":""},{"line_number":17,"context_line":"# INSTALL VERTICA"},{"line_number":18,"context_line":"touch /etc/mtab"}],"source_content_type":"application/x-shellscript","patch_set":12,"id":"3ae8d1ca_a9cce836","line":15,"updated":"2014-05-27 17:44:53.000000000","message":"this seems to imply that the guest image being built would have a user who could login?","commit_id":"a66a11c07927b0cd6cb484b4b3f5da554185a962"},{"author":{"_account_id":9122,"name":"Shalini khandelwal","email":"shalini.khandelwal@globallogic.com","username":"shalini"},"change_message_id":"1ac00b67a8a8ae63660214d42a48c5b8ac1d70d2","unresolved":false,"context_lines":[{"line_number":12,"context_line":""},{"line_number":13,"context_line":"# Creating dbadmin user"},{"line_number":14,"context_line":"groupadd verticadba"},{"line_number":15,"context_line":"useradd -g verticadba -d /home/dbadmin -s /bin/bash -m dbadmin"},{"line_number":16,"context_line":""},{"line_number":17,"context_line":"# INSTALL VERTICA"},{"line_number":18,"context_line":"touch /etc/mtab"}],"source_content_type":"application/x-shellscript","patch_set":12,"id":"3ae8d1ca_722c7248","line":15,"in_reply_to":"3ae8d1ca_a9cce836","updated":"2014-05-28 07:33:06.000000000","message":"Vertica uses \"dbadmin\" user for its database operations.","commit_id":"a66a11c07927b0cd6cb484b4b3f5da554185a962"},{"author":{"_account_id":9664,"name":"amrith","email":"amrith.kumar@gmail.com","username":"amrith"},"change_message_id":"5d3c12e1ebc28ac7fe78acd377500402d138e545","unresolved":false,"context_lines":[{"line_number":17,"context_line":"# INSTALL VERTICA"},{"line_number":18,"context_line":"touch /etc/mtab"},{"line_number":19,"context_line":"/opt/vertica/sbin/install_vertica -s localhost -X -N -S \u0027default\u0027 -r /vertica_7.0.1-0_amd64.deb -L CE -Y --failure-threshold NONE"},{"line_number":20,"context_line":"service vertica_agent stop"},{"line_number":21,"context_line":"# Base Directory for database creation"},{"line_number":22,"context_line":"mkdir /var/lib/vertica"},{"line_number":23,"context_line":"chown dbadmin:verticadba /var/lib/vertica"}],"source_content_type":"application/x-shellscript","patch_set":12,"id":"3ae8d1ca_89ee4c8a","line":20,"updated":"2014-05-27 17:44:53.000000000","message":"why are we running a service stop here? isn\u0027t that effectively a noop?","commit_id":"a66a11c07927b0cd6cb484b4b3f5da554185a962"},{"author":{"_account_id":9122,"name":"Shalini khandelwal","email":"shalini.khandelwal@globallogic.com","username":"shalini"},"change_message_id":"1ac00b67a8a8ae63660214d42a48c5b8ac1d70d2","unresolved":false,"context_lines":[{"line_number":17,"context_line":"# INSTALL VERTICA"},{"line_number":18,"context_line":"touch /etc/mtab"},{"line_number":19,"context_line":"/opt/vertica/sbin/install_vertica -s localhost -X -N -S \u0027default\u0027 -r /vertica_7.0.1-0_amd64.deb -L CE -Y --failure-threshold NONE"},{"line_number":20,"context_line":"service vertica_agent stop"},{"line_number":21,"context_line":"# Base Directory for database creation"},{"line_number":22,"context_line":"mkdir /var/lib/vertica"},{"line_number":23,"context_line":"chown dbadmin:verticadba /var/lib/vertica"}],"source_content_type":"application/x-shellscript","patch_set":12,"id":"3ae8d1ca_f8a7834d","line":20,"in_reply_to":"3ae8d1ca_89ee4c8a","updated":"2014-05-28 07:33:06.000000000","message":"No it is not.\nImage creation fails without it.","commit_id":"a66a11c07927b0cd6cb484b4b3f5da554185a962"},{"author":{"_account_id":9664,"name":"amrith","email":"amrith.kumar@gmail.com","username":"amrith"},"change_message_id":"5d3c12e1ebc28ac7fe78acd377500402d138e545","unresolved":false,"context_lines":[{"line_number":34,"context_line":"ODBCInstLib\u003d/usr/lib/x86_64-linux-gnu/libodbcinst.so"},{"line_number":35,"context_line":"ErrorMessagesPath\u003d/opt/vertica/lib64"},{"line_number":36,"context_line":"LogLevel\u003d4"},{"line_number":37,"context_line":"LogPath\u003d/tmp\" \u003e\u003e vertica.ini"},{"line_number":38,"context_line":"touch odbcinst.ini"},{"line_number":39,"context_line":"echo \"[vertica]"},{"line_number":40,"context_line":"Driver\u003d/opt/vertica/lib64/libverticaodbc.so"}],"source_content_type":"application/x-shellscript","patch_set":12,"id":"3ae8d1ca_e9bb5070","line":37,"updated":"2014-05-27 17:44:53.000000000","message":"Why touch a file and then concatenate to it?\n\nWhy not just echo ... \u003e vertica.ini?","commit_id":"a66a11c07927b0cd6cb484b4b3f5da554185a962"},{"author":{"_account_id":9664,"name":"amrith","email":"amrith.kumar@gmail.com","username":"amrith"},"change_message_id":"5d3c12e1ebc28ac7fe78acd377500402d138e545","unresolved":false,"context_lines":[{"line_number":35,"context_line":"ErrorMessagesPath\u003d/opt/vertica/lib64"},{"line_number":36,"context_line":"LogLevel\u003d4"},{"line_number":37,"context_line":"LogPath\u003d/tmp\" \u003e\u003e vertica.ini"},{"line_number":38,"context_line":"touch odbcinst.ini"},{"line_number":39,"context_line":"echo \"[vertica]"},{"line_number":40,"context_line":"Driver\u003d/opt/vertica/lib64/libverticaodbc.so"},{"line_number":41,"context_line":"Setup\u003d/opt/vertica/lib64/libverticaodbc.so\" \u003e\u003e odbcinst.ini"}],"source_content_type":"application/x-shellscript","patch_set":12,"id":"3ae8d1ca_09b07c8a","line":38,"updated":"2014-05-27 17:44:53.000000000","message":"same for this ...","commit_id":"a66a11c07927b0cd6cb484b4b3f5da554185a962"},{"author":{"_account_id":8415,"name":"Denis M.","email":"lildee1991@gmail.com","username":"dmakogon"},"change_message_id":"42ed85ed6b61382439b32957d6a9a6cf39e5dad1","unresolved":false,"context_lines":[{"line_number":7,"context_line":"export DEBIAN_FRONTEND\u003dnoninteractive"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"apt-get install build-essential bc iptables curl sysstat pstack mcelog python-dev python-pip g++ unixODBC unixODBC-dev -y"},{"line_number":10,"context_line":"wget http://h30537.www3.hp.com/prdownloads/vertica_7.0.1-0_amd64.deb?downloadid\u003dkfgOdXyQ5mBk_H43Pf1HXu8MYy5ZJ0c3wy48_7ta84EdFRE0lvsWMyi3JcTzLoeK8Q2w65kh2GM_bT7Xrd5e51Hb6Leusrigugf83o4Cd2dFmFUM7FIs4XssR9jcFkNjaIQHD24o1II\u003d -O vertica_7.0.1-0_amd64.deb"},{"line_number":11,"context_line":"dpkg -i vertica_7.0.1-0_amd64.deb"},{"line_number":12,"context_line":""},{"line_number":13,"context_line":"# Creating dbadmin user"}],"source_content_type":"application/x-shellscript","patch_set":14,"id":"1ae5cdf2_1614c94c","line":10,"updated":"2014-06-02 22:30:07.000000000","message":"Is this link stable?\nAre there any community PPA repos?","commit_id":"fdbbf9a43fbbd9a4df55b13d22342b5e256e35f1"},{"author":{"_account_id":9122,"name":"Shalini khandelwal","email":"shalini.khandelwal@globallogic.com","username":"shalini"},"change_message_id":"7630b46fbf3a85629af90588c7f0e0b33b2ff43e","unresolved":false,"context_lines":[{"line_number":7,"context_line":"export DEBIAN_FRONTEND\u003dnoninteractive"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"apt-get install build-essential bc iptables curl sysstat pstack mcelog python-dev python-pip g++ unixODBC unixODBC-dev -y"},{"line_number":10,"context_line":"wget http://h30537.www3.hp.com/prdownloads/vertica_7.0.1-0_amd64.deb?downloadid\u003dkfgOdXyQ5mBk_H43Pf1HXu8MYy5ZJ0c3wy48_7ta84EdFRE0lvsWMyi3JcTzLoeK8Q2w65kh2GM_bT7Xrd5e51Hb6Leusrigugf83o4Cd2dFmFUM7FIs4XssR9jcFkNjaIQHD24o1II\u003d -O vertica_7.0.1-0_amd64.deb"},{"line_number":11,"context_line":"dpkg -i vertica_7.0.1-0_amd64.deb"},{"line_number":12,"context_line":""},{"line_number":13,"context_line":"# Creating dbadmin user"}],"source_content_type":"application/x-shellscript","patch_set":14,"id":"1ae5cdf2_38144743","line":10,"in_reply_to":"1ae5cdf2_1614c94c","updated":"2014-06-03 07:14:36.000000000","message":"Yes this is very stable link and there is no such PPA repo for vertica","commit_id":"fdbbf9a43fbbd9a4df55b13d22342b5e256e35f1"},{"author":{"_account_id":5293,"name":"Nikhil","email":"SlickNik@gmail.com","username":"slicknik"},"change_message_id":"c322e4f807b62dabc2626dcf4dcfb434897be86a","unresolved":false,"context_lines":[{"line_number":42,"context_line":""},{"line_number":43,"context_line":"#installing vertica-sqlalchemy and dependencies"},{"line_number":44,"context_line":"pip install https://pyodbc.googlecode.com/files/pyodbc-3.0.6.zip"},{"line_number":45,"context_line":"pip install vertica-sqlalchemy"}],"source_content_type":"application/x-shellscript","patch_set":15,"id":"1ae5cdf2_00ae2498","line":45,"updated":"2014-06-11 08:44:18.000000000","message":"You can combine pip installs in the same line.","commit_id":"4f95ad5c156fc85fb12f0eddefb748015f663e25"},{"author":{"_account_id":9664,"name":"amrith","email":"amrith.kumar@gmail.com","username":"amrith"},"change_message_id":"2ccdaeb0ae78b1c47f3187d4e67dabb166b4c2ba","unresolved":false,"context_lines":[{"line_number":7,"context_line":"export DEBIAN_FRONTEND\u003dnoninteractive"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"apt-get install build-essential bc iptables curl sysstat pstack mcelog python-dev python-pip g++ unixODBC unixODBC-dev -y"},{"line_number":10,"context_line":"wget http://h30537.www3.hp.com/prdownloads/vertica_7.0.1-0_amd64.deb?downloadid\u003dkfgOdXyQ5mBk_H43Pf1HXu8MYy5ZJ0c3wy48_7ta84EdFRE0lvsWMyi3JcTzLoeK8Q2w65kh2GM_bT7Xrd5e51Hb6Leusrigugf83o4Cd2dFmFUM7FIs4XssR9jcFkNjaIQHD24o1II\u003d -O vertica_7.0.1-0_amd64.deb"},{"line_number":11,"context_line":"dpkg -i vertica_7.0.1-0_amd64.deb"},{"line_number":12,"context_line":""},{"line_number":13,"context_line":"# Creating dbadmin user"}],"source_content_type":"application/x-shellscript","patch_set":18,"id":"baada198_329220fd","line":10,"updated":"2014-07-22 09:56:08.000000000","message":"A problem I have with this chkin is the fact that I\u0027ve not been able to download this object. I\u0027ve consistently got a 403 (Forbidden) error.\n\nHow would this work for a person who decided to use this?","commit_id":"ee005b197876b52b80b441c08b4802a5d9103aaa"}],"scripts/redstack":[{"author":{"_account_id":5293,"name":"Nikhil","email":"SlickNik@gmail.com","username":"slicknik"},"change_message_id":"0e82fb0d7329708aa9b8816efd1c1fba6d7c8911","unresolved":false,"context_lines":[{"line_number":420,"context_line":"    sed -i \"s/%admin_password%/$ADMIN_PASSWORD/g\" $TROVE_CONF_DIR/test.conf"},{"line_number":421,"context_line":""},{"line_number":422,"context_line":"    # Add datastore type to the test conf"},{"line_number":423,"context_line":"    sed -i \"s/%datastore_type%/$DATASTORE_TYPE/g\" $TROVE_CONF_DIR/test.conf"},{"line_number":424,"context_line":""},{"line_number":425,"context_line":"    if [ $DATASTORE_TYPE \u003d \u0027vertica\u0027 ]; then"},{"line_number":426,"context_line":""}],"source_content_type":"application/octet-stream","patch_set":11,"id":"3ae8d1ca_46828353","line":423,"updated":"2014-05-27 08:16:31.000000000","message":"What\u0027s the rationale behind adding the datstore type to the test.conf?","commit_id":"8138cd5b323865b36c2ff887bcf56c897bc83436"},{"author":{"_account_id":9122,"name":"Shalini khandelwal","email":"shalini.khandelwal@globallogic.com","username":"shalini"},"change_message_id":"62610e8152b6c703ce0012b7091f7be8103a7210","unresolved":false,"context_lines":[{"line_number":420,"context_line":"    sed -i \"s/%admin_password%/$ADMIN_PASSWORD/g\" $TROVE_CONF_DIR/test.conf"},{"line_number":421,"context_line":""},{"line_number":422,"context_line":"    # Add datastore type to the test conf"},{"line_number":423,"context_line":"    sed -i \"s/%datastore_type%/$DATASTORE_TYPE/g\" $TROVE_CONF_DIR/test.conf"},{"line_number":424,"context_line":""},{"line_number":425,"context_line":"    if [ $DATASTORE_TYPE \u003d \u0027vertica\u0027 ]; then"},{"line_number":426,"context_line":""}],"source_content_type":"application/octet-stream","patch_set":11,"id":"3ae8d1ca_7cdd4e42","line":423,"in_reply_to":"3ae8d1ca_46828353","updated":"2014-05-27 11:38:25.000000000","message":"Int-tests uses this in instance creation. Without this change int-tests would fail for datastore other than \u0027mysql\u0027\nhttps://bugs.launchpad.net/trove/+bug/1301826","commit_id":"8138cd5b323865b36c2ff887bcf56c897bc83436"},{"author":{"_account_id":5293,"name":"Nikhil","email":"SlickNik@gmail.com","username":"slicknik"},"change_message_id":"0e82fb0d7329708aa9b8816efd1c1fba6d7c8911","unresolved":false,"context_lines":[{"line_number":447,"context_line":"    add_flavor \u0027Vr-m1.rd-tiny\u0027 21 2048 5 2 0"},{"line_number":448,"context_line":"    add_flavor \u0027Vr-m1.rd-smaller\u0027 22 4096 2 6 0"},{"line_number":449,"context_line":"    add_flavor \u0027Vr-eph.rd-tiny\u0027 23 2048 5 2 1"},{"line_number":450,"context_line":"    add_flavor \u0027Vr-eph.rd-smaller\u0027 24 4096 2 6 2"},{"line_number":451,"context_line":"}"},{"line_number":452,"context_line":""},{"line_number":453,"context_line":"function cmd_test_init() {"}],"source_content_type":"application/octet-stream","patch_set":11,"id":"3ae8d1ca_66793f93","line":450,"range":{"start_line":450,"start_character":19,"end_line":450,"end_character":22},"updated":"2014-05-27 08:16:31.000000000","message":"Can vertica use the standard nova L, XL, and 2XL flavors? Are these necessary for a test setup?","commit_id":"8138cd5b323865b36c2ff887bcf56c897bc83436"},{"author":{"_account_id":9122,"name":"Shalini khandelwal","email":"shalini.khandelwal@globallogic.com","username":"shalini"},"change_message_id":"62610e8152b6c703ce0012b7091f7be8103a7210","unresolved":false,"context_lines":[{"line_number":447,"context_line":"    add_flavor \u0027Vr-m1.rd-tiny\u0027 21 2048 5 2 0"},{"line_number":448,"context_line":"    add_flavor \u0027Vr-m1.rd-smaller\u0027 22 4096 2 6 0"},{"line_number":449,"context_line":"    add_flavor \u0027Vr-eph.rd-tiny\u0027 23 2048 5 2 1"},{"line_number":450,"context_line":"    add_flavor \u0027Vr-eph.rd-smaller\u0027 24 4096 2 6 2"},{"line_number":451,"context_line":"}"},{"line_number":452,"context_line":""},{"line_number":453,"context_line":"function cmd_test_init() {"}],"source_content_type":"application/octet-stream","patch_set":11,"id":"3ae8d1ca_1c00caa9","line":450,"in_reply_to":"3ae8d1ca_66793f93","updated":"2014-05-27 11:38:25.000000000","message":"Yes vertica can run on these flavors.\nOk, adding new flavors can be avoided by using m1.small flavor in int-tests.\nChanges done.","commit_id":"8138cd5b323865b36c2ff887bcf56c897bc83436"},{"author":{"_account_id":9664,"name":"amrith","email":"amrith.kumar@gmail.com","username":"amrith"},"change_message_id":"5d3c12e1ebc28ac7fe78acd377500402d138e545","unresolved":false,"context_lines":[{"line_number":422,"context_line":"    # Add datastore type to the test conf"},{"line_number":423,"context_line":"    sed -i \"s/%datastore_type%/$DATASTORE_TYPE/g\" $TROVE_CONF_DIR/test.conf"},{"line_number":424,"context_line":""},{"line_number":425,"context_line":"    if [ $DATASTORE_TYPE \u003d \u0027vertica\u0027 ]; then"},{"line_number":426,"context_line":""},{"line_number":427,"context_line":"        # Add Vertica flavors to the test conf"},{"line_number":428,"context_line":"        sed -i \"s/m1.rd-tiny/m1.small/g\" $TROVE_CONF_DIR/test.conf"}],"source_content_type":"application/octet-stream","patch_set":12,"id":"3ae8d1ca_4958c456","line":425,"updated":"2014-05-27 17:44:53.000000000","message":"would be good to quote $DATASTORE_TYPE. convention is to use \"\" in bash (which is what this is)","commit_id":"a66a11c07927b0cd6cb484b4b3f5da554185a962"},{"author":{"_account_id":8415,"name":"Denis M.","email":"lildee1991@gmail.com","username":"dmakogon"},"change_message_id":"42ed85ed6b61382439b32957d6a9a6cf39e5dad1","unresolved":false,"context_lines":[{"line_number":317,"context_line":"    rd_manage datastore_update Test_Datastore_1 \"\""},{"line_number":318,"context_line":"    cmd_stop"},{"line_number":319,"context_line":"    iniset $TROVE_CONF_DIR/trove.conf DEFAULT default_datastore \"$DATASTORE_TYPE\""},{"line_number":320,"context_line":"    iniset $TROVE_CONF_DIR/trove.conf DEFAULT datastore_manager \"$DATASTORE_TYPE\""},{"line_number":321,"context_line":"    sleep 1.5"},{"line_number":322,"context_line":"    cmd_start"},{"line_number":323,"context_line":"}"}],"source_content_type":"application/octet-stream","patch_set":14,"id":"1ae5cdf2_190b1a26","line":320,"updated":"2014-06-02 22:30:07.000000000","message":"no need to put this into config file, because this option is specific for guest-agent only","commit_id":"fdbbf9a43fbbd9a4df55b13d22342b5e256e35f1"},{"author":{"_account_id":9122,"name":"Shalini khandelwal","email":"shalini.khandelwal@globallogic.com","username":"shalini"},"change_message_id":"7630b46fbf3a85629af90588c7f0e0b33b2ff43e","unresolved":false,"context_lines":[{"line_number":317,"context_line":"    rd_manage datastore_update Test_Datastore_1 \"\""},{"line_number":318,"context_line":"    cmd_stop"},{"line_number":319,"context_line":"    iniset $TROVE_CONF_DIR/trove.conf DEFAULT default_datastore \"$DATASTORE_TYPE\""},{"line_number":320,"context_line":"    iniset $TROVE_CONF_DIR/trove.conf DEFAULT datastore_manager \"$DATASTORE_TYPE\""},{"line_number":321,"context_line":"    sleep 1.5"},{"line_number":322,"context_line":"    cmd_start"},{"line_number":323,"context_line":"}"}],"source_content_type":"application/octet-stream","patch_set":14,"id":"1ae5cdf2_9d546d22","line":320,"in_reply_to":"1ae5cdf2_190b1a26","updated":"2014-06-03 07:14:36.000000000","message":"This is defined in cfg.py\nAnd is not guestagent specific. This will only contain datastore type in it","commit_id":"fdbbf9a43fbbd9a4df55b13d22342b5e256e35f1"},{"author":{"_account_id":5293,"name":"Nikhil","email":"SlickNik@gmail.com","username":"slicknik"},"change_message_id":"c322e4f807b62dabc2626dcf4dcfb434897be86a","unresolved":false,"context_lines":[{"line_number":428,"context_line":""},{"line_number":429,"context_line":"    if [ \"$DATASTORE_TYPE\" \u003d \"vertica\" ]; then"},{"line_number":430,"context_line":"        # Add Vertica flavors to the test conf"},{"line_number":431,"context_line":"        sed -i \"s/m1.rd-tiny/m1.small/g\" $TROVE_CONF_DIR/test.conf"},{"line_number":432,"context_line":"        sed -i \"s/latin2/utf8/g\" $TROVE_CONF_DIR/test.conf"},{"line_number":433,"context_line":"        sed -i \"s/latin2_general_ci/utf8_general_ci/g\" $TROVE_CONF_DIR/test.conf"},{"line_number":434,"context_line":"    fi"}],"source_content_type":"application/octet-stream","patch_set":15,"id":"1ae5cdf2_c0e35cd7","line":431,"updated":"2014-06-11 08:44:18.000000000","message":"Why do we need to change the flavor in the conf for Vertica?","commit_id":"4f95ad5c156fc85fb12f0eddefb748015f663e25"},{"author":{"_account_id":9122,"name":"Shalini khandelwal","email":"shalini.khandelwal@globallogic.com","username":"shalini"},"change_message_id":"33a8ff55e7b69adf4bd2991ce56aa7bf14d2c414","unresolved":false,"context_lines":[{"line_number":428,"context_line":""},{"line_number":429,"context_line":"    if [ \"$DATASTORE_TYPE\" \u003d \"vertica\" ]; then"},{"line_number":430,"context_line":"        # Add Vertica flavors to the test conf"},{"line_number":431,"context_line":"        sed -i \"s/m1.rd-tiny/m1.small/g\" $TROVE_CONF_DIR/test.conf"},{"line_number":432,"context_line":"        sed -i \"s/latin2/utf8/g\" $TROVE_CONF_DIR/test.conf"},{"line_number":433,"context_line":"        sed -i \"s/latin2_general_ci/utf8_general_ci/g\" $TROVE_CONF_DIR/test.conf"},{"line_number":434,"context_line":"    fi"}],"source_content_type":"application/octet-stream","patch_set":15,"id":"1ae5cdf2_471e5d71","line":431,"in_reply_to":"1ae5cdf2_c0e35cd7","updated":"2014-06-12 11:21:46.000000000","message":"Because Vertica do not work with m1.rd-tiny flavor.\nNew flavors were added but as per your review comments in patch set:11, this change is done.","commit_id":"4f95ad5c156fc85fb12f0eddefb748015f663e25"},{"author":{"_account_id":5293,"name":"Nikhil","email":"SlickNik@gmail.com","username":"slicknik"},"change_message_id":"c322e4f807b62dabc2626dcf4dcfb434897be86a","unresolved":false,"context_lines":[{"line_number":430,"context_line":"        # Add Vertica flavors to the test conf"},{"line_number":431,"context_line":"        sed -i \"s/m1.rd-tiny/m1.small/g\" $TROVE_CONF_DIR/test.conf"},{"line_number":432,"context_line":"        sed -i \"s/latin2/utf8/g\" $TROVE_CONF_DIR/test.conf"},{"line_number":433,"context_line":"        sed -i \"s/latin2_general_ci/utf8_general_ci/g\" $TROVE_CONF_DIR/test.conf"},{"line_number":434,"context_line":"    fi"},{"line_number":435,"context_line":"}"},{"line_number":436,"context_line":""}],"source_content_type":"application/octet-stream","patch_set":15,"id":"1ae5cdf2_20d24000","line":433,"updated":"2014-06-11 08:44:18.000000000","message":"Same question for the encoding?","commit_id":"4f95ad5c156fc85fb12f0eddefb748015f663e25"},{"author":{"_account_id":9122,"name":"Shalini khandelwal","email":"shalini.khandelwal@globallogic.com","username":"shalini"},"change_message_id":"33a8ff55e7b69adf4bd2991ce56aa7bf14d2c414","unresolved":false,"context_lines":[{"line_number":430,"context_line":"        # Add Vertica flavors to the test conf"},{"line_number":431,"context_line":"        sed -i \"s/m1.rd-tiny/m1.small/g\" $TROVE_CONF_DIR/test.conf"},{"line_number":432,"context_line":"        sed -i \"s/latin2/utf8/g\" $TROVE_CONF_DIR/test.conf"},{"line_number":433,"context_line":"        sed -i \"s/latin2_general_ci/utf8_general_ci/g\" $TROVE_CONF_DIR/test.conf"},{"line_number":434,"context_line":"    fi"},{"line_number":435,"context_line":"}"},{"line_number":436,"context_line":""}],"source_content_type":"application/octet-stream","patch_set":15,"id":"1ae5cdf2_670fd939","line":433,"in_reply_to":"1ae5cdf2_20d24000","updated":"2014-06-12 11:21:46.000000000","message":"Because Vertica do not support latin2 character set. It supports utf8.\nAnd int tests fails without above changes (flavor and encoding change)","commit_id":"4f95ad5c156fc85fb12f0eddefb748015f663e25"},{"author":{"_account_id":8415,"name":"Denis M.","email":"lildee1991@gmail.com","username":"dmakogon"},"change_message_id":"472d62a13acf309548c638da6d1cf0cce7008d8c","unresolved":false,"context_lines":[{"line_number":311,"context_line":"        exit 1"},{"line_number":312,"context_line":"    fi"},{"line_number":313,"context_line":""},{"line_number":314,"context_line":"    sed -i \"s/%datastore_type%/$DATASTORE_TYPE/g\" $TROVE_CONF_DIR/test.conf"},{"line_number":315,"context_line":"    sed -i \"s/%datastore_version%/$VERSION/g\" $TROVE_CONF_DIR/test.conf"},{"line_number":316,"context_line":""},{"line_number":317,"context_line":"    #rd_manage datastore_version_update \u003cdatastore_name\u003e \u003cversion_name\u003e \u003cdatastore_manager\u003e \u003cimage_id\u003e \u003cpackages\u003e \u003cactive\u003e"}],"source_content_type":"application/octet-stream","patch_set":16,"id":"1ae5cdf2_f6c17482","line":314,"updated":"2014-06-16 11:03:33.000000000","message":"same","commit_id":"5d405351d609a36e9752eeee0f6d94fa75f98b3b"},{"author":{"_account_id":9122,"name":"Shalini khandelwal","email":"shalini.khandelwal@globallogic.com","username":"shalini"},"change_message_id":"f7880e405e4b1b32ce74a7577886cf8059f123a2","unresolved":false,"context_lines":[{"line_number":311,"context_line":"        exit 1"},{"line_number":312,"context_line":"    fi"},{"line_number":313,"context_line":""},{"line_number":314,"context_line":"    sed -i \"s/%datastore_type%/$DATASTORE_TYPE/g\" $TROVE_CONF_DIR/test.conf"},{"line_number":315,"context_line":"    sed -i \"s/%datastore_version%/$VERSION/g\" $TROVE_CONF_DIR/test.conf"},{"line_number":316,"context_line":""},{"line_number":317,"context_line":"    #rd_manage datastore_version_update \u003cdatastore_name\u003e \u003cversion_name\u003e \u003cdatastore_manager\u003e \u003cimage_id\u003e \u003cpackages\u003e \u003cactive\u003e"}],"source_content_type":"application/octet-stream","patch_set":16,"id":"1ae5cdf2_01ccdc60","line":314,"in_reply_to":"1ae5cdf2_f6c17482","updated":"2014-06-16 11:28:56.000000000","message":"SAME","commit_id":"5d405351d609a36e9752eeee0f6d94fa75f98b3b"}]}