)]}'
{"functions.sh":[{"author":{"_account_id":4146,"name":"Clark Boylan","email":"cboylan@sapwetik.org","username":"cboylan"},"change_message_id":"834945f71ac22801cdb386b29748517239d9a61e","unresolved":false,"context_lines":[{"line_number":201,"context_line":""},{"line_number":202,"context_line":"    # first get all keys that match our filter and then output the whole line"},{"line_number":203,"context_line":"    # that will ensure that multi-line env vars are set properly"},{"line_number":204,"context_line":"    for KEY in $(printenv -0 | grep -z -Z \u0027\\(DEVSTACK\\|GRENADE_PLUGINRC\\|TEMPEST_CONCURRENCY\\|ZUUL\\)\u0027 | sed -z -n \u0027s/^\\([^\u003d]\\+\\)\u003d.*/\\1\\n/p\u0027); do"},{"line_number":205,"context_line":"        echo \"declare -x ${KEY}\u003d\\\"${!KEY}\\\"\" \u003e\u003e $WORKSPACE/logs/reproduce.sh"},{"line_number":206,"context_line":"    done"},{"line_number":207,"context_line":"    if [ -n \"$JOB_PROJECTS\" ] ; then"}],"source_content_type":"text/x-sh","patch_set":1,"id":"9a629dbe_5d949d89","line":204,"updated":"2016-11-11 20:54:23.000000000","message":"We shouldn\u0027t always write TEMPEST_CONCURRENCY as it is supposed to scale up with cpu count which may differ on different machines. If ironic requires this be set to 1 then I think its probably safe to treat concurrency \u003d 1 as a special case and only write it out if it matches that.","commit_id":"0b842c06737d86caf7cc13a6b4cf45ec40815681"},{"author":{"_account_id":14760,"name":"John L. Villalovos","email":"openstack.org@sodarock.com","username":"jlvillal"},"change_message_id":"e86d20833e475985e2f4974b0f273543f12c902a","unresolved":false,"context_lines":[{"line_number":201,"context_line":""},{"line_number":202,"context_line":"    # first get all keys that match our filter and then output the whole line"},{"line_number":203,"context_line":"    # that will ensure that multi-line env vars are set properly"},{"line_number":204,"context_line":"    for KEY in $(printenv -0 | grep -z -Z \u0027\\(DEVSTACK\\|GRENADE_PLUGINRC\\|TEMPEST_CONCURRENCY\\|ZUUL\\)\u0027 | sed -z -n \u0027s/^\\([^\u003d]\\+\\)\u003d.*/\\1\\n/p\u0027); do"},{"line_number":205,"context_line":"        echo \"declare -x ${KEY}\u003d\\\"${!KEY}\\\"\" \u003e\u003e $WORKSPACE/logs/reproduce.sh"},{"line_number":206,"context_line":"    done"},{"line_number":207,"context_line":"    if [ -n \"$JOB_PROJECTS\" ] ; then"}],"source_content_type":"text/x-sh","patch_set":1,"id":"9a629dbe_5d873db4","line":204,"in_reply_to":"9a629dbe_5d949d89","updated":"2016-11-11 21:13:33.000000000","message":"Thanks. I changed it to only output if is set to 1","commit_id":"0b842c06737d86caf7cc13a6b4cf45ec40815681"},{"author":{"_account_id":7118,"name":"Ian Wienand","email":"iwienand@redhat.com","username":"iwienand"},"change_message_id":"91506188267142c2ae1a5340156dfcdc149b8edb","unresolved":false,"context_lines":[{"line_number":204,"context_line":"    for KEY in $(printenv -0 | grep -z -Z \u0027\\(DEVSTACK\\|GRENADE_PLUGINRC\\|ZUUL\\)\u0027 | sed -z -n \u0027s/^\\([^\u003d]\\+\\)\u003d.*/\\1\\n/p\u0027); do"},{"line_number":205,"context_line":"        echo \"declare -x ${KEY}\u003d\\\"${!KEY}\\\"\" \u003e\u003e $WORKSPACE/logs/reproduce.sh"},{"line_number":206,"context_line":"    done"},{"line_number":207,"context_line":"    # If TEMPEST_CONCURRENCY has been explicitly set to 1, then save it in reproduce.sh"},{"line_number":208,"context_line":"    if [ \"${TEMPEST_CONCURRENCY}\" -eq 1 ]; then"},{"line_number":209,"context_line":"        echo \"declare -x TEMPEST_CONCURRENCY\u003d\\\"${TEMPEST_CONCURRENCY}\\\"\" \u003e\u003e $WORKSPACE/logs/reproduce.sh"},{"line_number":210,"context_line":"    fi"}],"source_content_type":"text/x-sh","patch_set":2,"id":"3a71b18c_b3bdf603","line":207,"updated":"2016-12-06 19:55:15.000000000","message":"a better comment might have described that having it set at 1 is a special case and can be important for some jobs. but ok","commit_id":"913fd7e2bf36d2ca008543f2d5c4454d3eb05745"}]}