)]}'
{"devstack/plugin.sh":[{"author":{"_account_id":14517,"name":"Ryan Brandt","email":"ryan.brandt@hpe.com","username":"ryan-brandt"},"change_message_id":"906ef1610ef414dd53bc372c09e95a1fada897d7","unresolved":false,"context_lines":[{"line_number":867,"context_line":""},{"line_number":868,"context_line":"    PIP_VIRTUAL_ENV\u003d/opt/monasca-api"},{"line_number":869,"context_line":""},{"line_number":870,"context_line":"    (cd /opt/monasca-api ; sudo ./bin/pip install -c $DEST/new/requirements/upper-constraints.txt $MONASCA_COMMON_SRC_DIST)"},{"line_number":871,"context_line":""},{"line_number":872,"context_line":"    (cd /opt/monasca-api ; sudo ./bin/pip install -c $DEST/new/requirements/upper-constraints.txt $MONASCA_STATSD_SRC_DIST)"},{"line_number":873,"context_line":""}],"source_content_type":"text/x-sh","patch_set":2,"id":"fa31d9ce_6a23a5a7","line":870,"range":{"start_line":870,"start_character":58,"end_line":870,"end_character":62},"updated":"2017-02-14 18:00:48.000000000","message":"need to remove this, DEST will have it if it is needed.","commit_id":"89d990fb29da8a41fb29de7dcad5a6c7836c9435"},{"author":{"_account_id":16168,"name":"Tomasz Trębski","email":"kornicameister@gmail.com","username":"trebskit"},"change_message_id":"e2688c161b9c4d388e3d98e2ce651ca689283d27","unresolved":false,"context_lines":[{"line_number":884,"context_line":"      pip_install psycopg2\u003d\u003d2.6.2"},{"line_number":885,"context_line":"    elif is_service_enabled mysql; then"},{"line_number":886,"context_line":"      apt_get -y install libmysqlclient-dev"},{"line_number":887,"context_line":"      pip_install PyMySQL"},{"line_number":888,"context_line":"    fi"},{"line_number":889,"context_line":""},{"line_number":890,"context_line":"    (cd \"${MONASCA_API_DIR}\" ; sudo python setup.py sdist)"}],"source_content_type":"text/x-sh","patch_set":4,"id":"fa31d9ce_7c3a819a","line":887,"updated":"2017-02-15 05:33:22.000000000","message":"I get a feeling that modifying this line to\n  \n  pip_install_gr \n\nmight suffice.\n\nI will test this locally to see if I am right.","commit_id":"c58a292bbec064b422bb455af75febfa655e5e58"},{"author":{"_account_id":14273,"name":"Joe Keen","email":"joe.keen@hpe.com","username":"jkeen"},"change_message_id":"721eaad2ae6b03fabbe8e74b6200bb460b5188fe","unresolved":false,"context_lines":[{"line_number":890,"context_line":"    (cd \"${MONASCA_API_DIR}\" ; sudo python setup.py sdist)"},{"line_number":891,"context_line":"    MONASCA_API_SRC_DIST\u003d$(ls -td \"${MONASCA_API_DIR}\"/dist/monasca-api-*.tar.gz)"},{"line_number":892,"context_line":""},{"line_number":893,"context_line":"    (cd /opt/monasca-api ; ./bin/pip install $MONASCA_API_SRC_DIST)"},{"line_number":894,"context_line":""},{"line_number":895,"context_line":"    unset PIP_VIRTUAL_ENV"},{"line_number":896,"context_line":""}],"source_content_type":"text/x-sh","patch_set":4,"id":"fa31d9ce_90c63f66","line":893,"range":{"start_line":893,"start_character":13,"end_line":893,"end_character":20},"updated":"2017-02-15 00:33:21.000000000","message":"I think we still need setup_install here to apply the constraints to the monasca-api source install.","commit_id":"c58a292bbec064b422bb455af75febfa655e5e58"},{"author":{"_account_id":14273,"name":"Joe Keen","email":"joe.keen@hpe.com","username":"jkeen"},"change_message_id":"721eaad2ae6b03fabbe8e74b6200bb460b5188fe","unresolved":false,"context_lines":[{"line_number":1113,"context_line":""},{"line_number":1114,"context_line":"    setup_install $MONASCA_COMMON_DIR"},{"line_number":1115,"context_line":""},{"line_number":1116,"context_line":"    (cd /opt/monasca-persister ; ./bin/pip install $MONASCA_PERSISTER_SRC_DIST)"},{"line_number":1117,"context_line":""},{"line_number":1118,"context_line":"    if [[ \"${MONASCA_METRICS_DB,,}\" \u003d\u003d \u0027influxdb\u0027 ]]; then"},{"line_number":1119,"context_line":""}],"source_content_type":"text/x-sh","patch_set":4,"id":"fa31d9ce_909d5f68","line":1116,"range":{"start_line":1116,"start_character":13,"end_line":1116,"end_character":30},"updated":"2017-02-15 00:33:21.000000000","message":"Same as the API above I think we need setup_install here.","commit_id":"c58a292bbec064b422bb455af75febfa655e5e58"},{"author":{"_account_id":16168,"name":"Tomasz Trębski","email":"kornicameister@gmail.com","username":"trebskit"},"change_message_id":"c258b72df383c3128abfc8084aed772d1d8ebbc7","unresolved":false,"context_lines":[{"line_number":1090,"context_line":"    echo_summary \"Install Monasca monasca_persister_python\""},{"line_number":1091,"context_line":""},{"line_number":1092,"context_line":"    git_clone $MONASCA_PERSISTER_REPO $MONASCA_PERSISTER_DIR $MONASCA_PERSISTER_BRANCH"},{"line_number":1093,"context_line":"    (cd \"${MONASCA_PERSISTER_DIR}\"; sudo python setup.py sdist)"},{"line_number":1094,"context_line":""},{"line_number":1095,"context_line":"    MONASCA_PERSISTER_SRC_DIST\u003d$(ls -td \"${MONASCA_PERSISTER_DIR}\"/dist/monasca-persister-*.tar.gz | head -1)"},{"line_number":1096,"context_line":""}],"source_content_type":"text/x-sh","patch_set":5,"id":"fa31d9ce_d6525e66","line":1093,"updated":"2017-02-15 11:06:30.000000000","message":"Are those lines here needed ? I think we ought to delete them now since everything is based on setup_install ?","commit_id":"734e5553e214fd3145eceaf82a6184de529a255b"},{"author":{"_account_id":16168,"name":"Tomasz Trębski","email":"kornicameister@gmail.com","username":"trebskit"},"change_message_id":"c258b72df383c3128abfc8084aed772d1d8ebbc7","unresolved":false,"context_lines":[{"line_number":1244,"context_line":"    apt_get -y install python-dev"},{"line_number":1245,"context_line":"    apt_get -y install build-essential"},{"line_number":1246,"context_line":""},{"line_number":1247,"context_line":"    git_clone $MONASCA_NOTIFICATION_REPO $MONASCA_NOTIFICATION_DIR $MONASCA_NOTIFICATION_BRANCH"},{"line_number":1248,"context_line":"    (cd \"${MONASCA_NOTIFICATION_DIR}\" ; sudo python setup.py sdist)"},{"line_number":1249,"context_line":""},{"line_number":1250,"context_line":"    MONASCA_NOTIFICATION_SRC_DIST\u003d$(ls -td \"${MONASCA_NOTIFICATION_DIR}\"/dist/monasca-notification-*.tar.gz | head -1)"}],"source_content_type":"text/x-sh","patch_set":5,"id":"fa31d9ce_96d386d4","line":1247,"updated":"2017-02-15 11:06:30.000000000","message":"Also I think that these 2 lines here are now redundant.","commit_id":"734e5553e214fd3145eceaf82a6184de529a255b"}]}