)]}'
{"functions":[{"author":{"_account_id":970,"name":"Dean Troyer","email":"dtroyer@gmail.com","username":"dtroyer"},"change_message_id":"f72f4f2f0fe28dabee1e70d3ab0864820e078f28","unresolved":false,"context_lines":[{"line_number":1104,"context_line":"    echo $!"},{"line_number":1105,"context_line":"}"},{"line_number":1106,"context_line":""},{"line_number":1107,"context_line":"function uwsgi_it {"},{"line_number":1108,"context_line":"    local service_dir\u003d${SERVICE_DIR:-$DEST/status}"},{"line_number":1109,"context_line":"    local service\u003d$1"},{"line_number":1110,"context_line":"    local uwsgi_params\u003d$2"}],"source_content_type":"application/octet-stream","patch_set":1,"id":"AAAAUH%2F%2BIZM%3D","line":1107,"updated":"2014-01-29 21:52:11.000000000","message":"I need to understand the reasons for these functions better...it seems like the only difference from screen_it() are a) never uses screen no matter what USE_SCREEN is set to; b) location of pid file.\n\nHonestly I\u0027d just call run_process() directly, it\u0027s meant to be public. Maybe we need a stop_process() to go with it?","commit_id":"d5aceeb0502c155513214481ecb50c5c5dc90378"}],"lib/barbican":[{"author":{"_account_id":970,"name":"Dean Troyer","email":"dtroyer@gmail.com","username":"dtroyer"},"change_message_id":"f72f4f2f0fe28dabee1e70d3ab0864820e078f28","unresolved":false,"context_lines":[{"line_number":31,"context_line":"# Set up default directories"},{"line_number":32,"context_line":"BARBICAN_DIR\u003d$DEST/barbican"},{"line_number":33,"context_line":"BARBICANCLIENT_DIR\u003d$DEST/python-barbicanclient"},{"line_number":34,"context_line":"BARBICAN_CONF_DIR\u003d/etc/barbican"},{"line_number":35,"context_line":"BARBICAN_CONF\u003d$BARBICAN_CONF_DIR/barbican-api.conf"},{"line_number":36,"context_line":"BARBICAN_PASTE_CONF\u003d$BARBICAN_CONF_DIR/barbican-api-paste.ini"},{"line_number":37,"context_line":"BARBICAN_API_LOG_DIR\u003d/var/log/barbican"}],"source_content_type":"application/octet-stream","patch_set":1,"id":"AAAAUH%2F%2BIY8%3D","line":34,"updated":"2014-01-29 21:52:11.000000000","message":"This needs to be able to be overridden:\n BARBICAN_CONF_DIR\u003d${BARBICAN_CONF_DIR:-/etc/barbican}","commit_id":"d5aceeb0502c155513214481ecb50c5c5dc90378"},{"author":{"_account_id":970,"name":"Dean Troyer","email":"dtroyer@gmail.com","username":"dtroyer"},"change_message_id":"f72f4f2f0fe28dabee1e70d3ab0864820e078f28","unresolved":false,"context_lines":[{"line_number":34,"context_line":"BARBICAN_CONF_DIR\u003d/etc/barbican"},{"line_number":35,"context_line":"BARBICAN_CONF\u003d$BARBICAN_CONF_DIR/barbican-api.conf"},{"line_number":36,"context_line":"BARBICAN_PASTE_CONF\u003d$BARBICAN_CONF_DIR/barbican-api-paste.ini"},{"line_number":37,"context_line":"BARBICAN_API_LOG_DIR\u003d/var/log/barbican"},{"line_number":38,"context_line":"BARBICAN_AUTH_CACHE_DIR\u003d${BARBICAN_AUTH_CACHE_DIR:-/var/cache/barbican}"},{"line_number":39,"context_line":""},{"line_number":40,"context_line":"# Support potential entry-points console scripts"}],"source_content_type":"application/octet-stream","patch_set":1,"id":"AAAAUH%2F%2BITo%3D","line":37,"updated":"2014-01-29 21:52:11.000000000","message":"I\u0027d prefer to have service logs in $DEST/logs, do any other services do this?  It should be overrideable no matter what the default is though.","commit_id":"d5aceeb0502c155513214481ecb50c5c5dc90378"},{"author":{"_account_id":970,"name":"Dean Troyer","email":"dtroyer@gmail.com","username":"dtroyer"},"change_message_id":"f72f4f2f0fe28dabee1e70d3ab0864820e078f28","unresolved":false,"context_lines":[{"line_number":35,"context_line":"BARBICAN_CONF\u003d$BARBICAN_CONF_DIR/barbican-api.conf"},{"line_number":36,"context_line":"BARBICAN_PASTE_CONF\u003d$BARBICAN_CONF_DIR/barbican-api-paste.ini"},{"line_number":37,"context_line":"BARBICAN_API_LOG_DIR\u003d/var/log/barbican"},{"line_number":38,"context_line":"BARBICAN_AUTH_CACHE_DIR\u003d${BARBICAN_AUTH_CACHE_DIR:-/var/cache/barbican}"},{"line_number":39,"context_line":""},{"line_number":40,"context_line":"# Support potential entry-points console scripts"},{"line_number":41,"context_line":"BARBICAN_BIN_DIR\u003d$(get_python_exec_prefix)"}],"source_content_type":"application/octet-stream","patch_set":1,"id":"AAAAUH%2F%2BISk%3D","line":38,"updated":"2014-01-29 21:52:11.000000000","message":"On the other hand, I KNOW this is being done everywhere...I\u0027ll put it on my list to clean all services at once...it really isn\u0027t a big deal as it is just a cache that isn\u0027t large and we don\u0027t need for debugging...","commit_id":"d5aceeb0502c155513214481ecb50c5c5dc90378"},{"author":{"_account_id":970,"name":"Dean Troyer","email":"dtroyer@gmail.com","username":"dtroyer"},"change_message_id":"f72f4f2f0fe28dabee1e70d3ab0864820e078f28","unresolved":false,"context_lines":[{"line_number":41,"context_line":"BARBICAN_BIN_DIR\u003d$(get_python_exec_prefix)"},{"line_number":42,"context_line":""},{"line_number":43,"context_line":"# Set up database backend"},{"line_number":44,"context_line":"BARBICAN_BACKEND\u003d${BARBICAN_BACKEND:-mysql}"},{"line_number":45,"context_line":""},{"line_number":46,"context_line":"# Set Barbican repository"},{"line_number":47,"context_line":"BARBICAN_REPO\u003d${BARBICAN_REPO:-${GIT_BASE}/stackforge/barbican.git}"}],"source_content_type":"application/octet-stream","patch_set":1,"id":"AAAAUH%2F%2BIYU%3D","line":44,"updated":"2014-01-29 21:52:11.000000000","message":"Do you support postgresql?   Will this run OK when the other services are set to use it?","commit_id":"d5aceeb0502c155513214481ecb50c5c5dc90378"},{"author":{"_account_id":970,"name":"Dean Troyer","email":"dtroyer@gmail.com","username":"dtroyer"},"change_message_id":"f72f4f2f0fe28dabee1e70d3ab0864820e078f28","unresolved":false,"context_lines":[{"line_number":92,"context_line":""},{"line_number":93,"context_line":"    # Set the log file location"},{"line_number":94,"context_line":"    initset $BARBICAN_CONF DEFAULT log_file \"$BARBICAN_API_LOG_DIR/api.log\""},{"line_number":95,"context_line":"    iniuncomment $BARBICAN_CONF DEFAULT log_file"},{"line_number":96,"context_line":""},{"line_number":97,"context_line":"    # Install the policy file for the API server"},{"line_number":98,"context_line":"    cp $BARBICAN_DIR/etc/barbican/policy.json $BARBICAN_CONF_DIR"}],"source_content_type":"application/octet-stream","patch_set":1,"id":"AAAAUH%2F%2BIO8%3D","line":95,"updated":"2014-01-29 21:52:11.000000000","message":"Do you need both of these?  Also, the log file is on the command-line, Are both of those required?","commit_id":"d5aceeb0502c155513214481ecb50c5c5dc90378"},{"author":{"_account_id":970,"name":"Dean Troyer","email":"dtroyer@gmail.com","username":"dtroyer"},"change_message_id":"f72f4f2f0fe28dabee1e70d3ab0864820e078f28","unresolved":false,"context_lines":[{"line_number":102,"context_line":"    iniset $BARBICAN_CONF DEFAULT sql_connection `database_connection_url barbican`"},{"line_number":103,"context_line":""},{"line_number":104,"context_line":"    # Rabbit settings"},{"line_number":105,"context_line":"    iniset $BARBICAN_CONF \u0027secrets\u0027 broker rabbit://guest:$RABBIT_PASSWORD@$RABBIT_HOST"},{"line_number":106,"context_line":""},{"line_number":107,"context_line":"    ## Set up keystone"},{"line_number":108,"context_line":""}],"source_content_type":"application/octet-stream","patch_set":1,"id":"AAAAUH%2F%2BIPs%3D","line":105,"updated":"2014-01-29 21:52:11.000000000","message":"Is qpid supported?  If it is enabled instead of rabbit, you\u0027ll not know it here.","commit_id":"d5aceeb0502c155513214481ecb50c5c5dc90378"},{"author":{"_account_id":970,"name":"Dean Troyer","email":"dtroyer@gmail.com","username":"dtroyer"},"change_message_id":"f72f4f2f0fe28dabee1e70d3ab0864820e078f28","unresolved":false,"context_lines":[{"line_number":110,"context_line":"    iniset $BARBICAN_PASTE_CONF \u0027pipeline:main\u0027 pipeline \u0027keystone_authtoken context apiapp\u0027"},{"line_number":111,"context_line":""},{"line_number":112,"context_line":"    # Set the keystone parameters"},{"line_number":113,"context_line":"    iniset $BARBICAN_PASTE_CONF \u0027filter:keystone_authtoken\u0027 auth_protocol http"},{"line_number":114,"context_line":"    iniset $BARBICAN_PASTE_CONF \u0027filter:keystone_authtoken\u0027 admin_user barbican"},{"line_number":115,"context_line":"    iniset $BARBICAN_PASTE_CONF \u0027filter:keystone_authtoken\u0027 admin_password $SERVICE_PASSWORD"},{"line_number":116,"context_line":"    iniset $BARBICAN_PASTE_CONF \u0027filter:keystone_authtoken\u0027 admin_tenant_name $SERVICE_TENANT_NAME"}],"source_content_type":"application/octet-stream","patch_set":1,"id":"AAAAUH%2F%2BIQQ%3D","line":113,"updated":"2014-01-29 21:52:11.000000000","message":"You need to use $KEYSTONE_AUTH_PROTOCOL here for when keystone is run under SSL, which should be our default in the near future.\n\ndo you need to set auth_host and auth_port also?","commit_id":"d5aceeb0502c155513214481ecb50c5c5dc90378"},{"author":{"_account_id":970,"name":"Dean Troyer","email":"dtroyer@gmail.com","username":"dtroyer"},"change_message_id":"f72f4f2f0fe28dabee1e70d3ab0864820e078f28","unresolved":false,"context_lines":[{"line_number":143,"context_line":""},{"line_number":144,"context_line":"# start_barbican() - Start running processes, including screen"},{"line_number":145,"context_line":"function start_barbican() {"},{"line_number":146,"context_line":"    uwsgi_it barbican \"--master --emperor $BARBICAN_CONF_DIR/vassals --logto $BARBICAN_API_LOG_DIR/api.log\""},{"line_number":147,"context_line":"    }"},{"line_number":148,"context_line":""},{"line_number":149,"context_line":"# stop_barbican() - Stop running processes"}],"source_content_type":"application/octet-stream","patch_set":1,"id":"AAAAUH%2F%2BINs%3D","line":146,"updated":"2014-01-29 21:52:11.000000000","message":"As noted at the function declaration, what is the benefit of the uwisgi_it() function over just calling run_process() here?\n\nI can\u0027t let this go:  --logto is an odd option name, I don\u0027t think I\u0027ve ever seen it before...","commit_id":"d5aceeb0502c155513214481ecb50c5c5dc90378"}]}