)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":25625,"name":"Tetsuro Nakamura","email":"tetsuro.nakamura.bc@hco.ntt.co.jp","username":"tetsuro0907"},"change_message_id":"07057acaa789586f2ea7e49b3c4f38ad785cb71f","unresolved":false,"context_lines":[{"line_number":8,"context_line":""},{"line_number":9,"context_line":"While experimenting with expanding the nested perfload tests,"},{"line_number":10,"context_line":"it became clear that the call to parallel was not working as"},{"line_number":11,"context_line":"documented. What\u0027s supposed to be happening is that 3*nprocs"},{"line_number":12,"context_line":"processes are run, but in local testing that\u0027s not what happens:"},{"line_number":13,"context_line":"instead just one, in sequence."},{"line_number":14,"context_line":""},{"line_number":15,"context_line":"This changes it to use available processors."}],"source_content_type":"text/x-gerrit-commit-message","patch_set":2,"id":"7faddb67_d7d44a22","line":12,"range":{"start_line":11,"start_character":52,"end_line":12,"end_character":17},"updated":"2019-08-06 05:13:10.000000000","message":"The manpage says \"Multiply N% with the number of CPU cores.  Run this many jobs in parallel. See also --use-cpus-instead-of-cores.\"\n\nso I assume the actual of this is not (3 * nprocs) but (0.03 * nprocs)\n\nFWIW, examples I\u0027ve tested locally:\n\n    tetsuro@ubuntu01:~$ nproc\n    4\n    tetsuro@ubuntu01:~$ parallel --no-notice -P 75% \"sleep {}; echo {}\" ::: 5 4 3 2 1\n    3\n    4\n    5\n    2\n    1\n    tetsuro@ubuntu01:~$ parallel --no-notice -P 50% \"sleep {}; echo {}\" ::: 5 4 3 2 1\n    4\n    5\n    3\n    2\n    1\n    tetsuro@ubuntu01:~$ parallel --no-notice -P 25% \"sleep {}; echo {}\" ::: 5 4 3 2 1\n    5\n    4\n    3\n    2\n    1","commit_id":"735e8d52a51a5acb31463cd4c4a4e5a81440e8a8"},{"author":{"_account_id":25625,"name":"Tetsuro Nakamura","email":"tetsuro.nakamura.bc@hco.ntt.co.jp","username":"tetsuro0907"},"change_message_id":"7c2db98c761d71e2113d98d524f37addef680787","unresolved":false,"context_lines":[{"line_number":17,"context_line":""},{"line_number":18,"context_line":"Subsequent patches will add a more complicated nested structure."},{"line_number":19,"context_line":""},{"line_number":20,"context_line":"Co-Authored-By: Tetsuro Nakamura \u003ctetsuro.nakamura.bc@hco.ntt.co.jp\u003e"},{"line_number":21,"context_line":"Change-Id: Ie4809abc31212711b96f69e5f291104ae761059e"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":3,"id":"7faddb67_265479fd","line":20,"range":{"start_line":20,"start_character":0,"end_line":20,"end_character":68},"updated":"2019-08-07 00:44:45.000000000","message":"Well, I haven\u0027t written any part of the change, but meh, okay, I\u0027ll leave someone else to +W.","commit_id":"7464ff6e24c34e412ec3def799edbc9dedd3a2ef"},{"author":{"_account_id":11564,"name":"Chris Dent","email":"cdent@anticdent.org","username":"chdent"},"change_message_id":"69664c49783c35460139fabf46b8bf1d27e11f81","unresolved":false,"context_lines":[{"line_number":17,"context_line":""},{"line_number":18,"context_line":"Subsequent patches will add a more complicated nested structure."},{"line_number":19,"context_line":""},{"line_number":20,"context_line":"Co-Authored-By: Tetsuro Nakamura \u003ctetsuro.nakamura.bc@hco.ntt.co.jp\u003e"},{"line_number":21,"context_line":"Change-Id: Ie4809abc31212711b96f69e5f291104ae761059e"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":3,"id":"7faddb67_ac012709","line":20,"range":{"start_line":20,"start_character":0,"end_line":20,"end_character":68},"in_reply_to":"7faddb67_265479fd","updated":"2019-08-07 08:33:37.000000000","message":"You\u0027re responsible for the \"correctly\" being actually correct so I thought you deserved some credit for that since it is really the only change in the patch.","commit_id":"7464ff6e24c34e412ec3def799edbc9dedd3a2ef"}],"gate/perfload-nested-runner.sh":[{"author":{"_account_id":14070,"name":"Eric Fried","email":"openstack@fried.cc","username":"efried"},"change_message_id":"f80b6a9e871b0a456ab39d70fc796d52183943c9","unresolved":false,"context_lines":[{"line_number":72,"context_line":"    # LOADER is called $ITERATIONS times in parallel by 3 * number"},{"line_number":73,"context_line":"    # of processors on the host."},{"line_number":74,"context_line":"    echo \"##### Creating $TOTAL_PROVIDER_COUNT providers\" | tee -a $LOG"},{"line_number":75,"context_line":"    seq 1 $ITERATIONS | parallel -P 3% $LOADER $PLACEMENT_URL $GABBIT"},{"line_number":76,"context_line":""},{"line_number":77,"context_line":"    set +x"},{"line_number":78,"context_line":"    rp_count\u003d$(curl -H \u0027x-auth-token: admin\u0027 ${PLACEMENT_URL}/resource_providers |json_pp|grep -c \u0027\"name\"\u0027)"}],"source_content_type":"text/x-sh","patch_set":1,"id":"7faddb67_6bdb91ec","side":"PARENT","line":75,"range":{"start_line":75,"start_character":36,"end_line":75,"end_character":38},"updated":"2019-07-30 15:50:39.000000000","message":"I wonder if shell was grabbing the % and quoting this would have helped","commit_id":"2f56f379a568d427ac9057086386074a7801b00c"},{"author":{"_account_id":11564,"name":"Chris Dent","email":"cdent@anticdent.org","username":"chdent"},"change_message_id":"ac30bbb3ca257bf4229b37ee978ffdcee821acdf","unresolved":false,"context_lines":[{"line_number":72,"context_line":"    # LOADER is called $ITERATIONS times in parallel by 3 * number"},{"line_number":73,"context_line":"    # of processors on the host."},{"line_number":74,"context_line":"    echo \"##### Creating $TOTAL_PROVIDER_COUNT providers\" | tee -a $LOG"},{"line_number":75,"context_line":"    seq 1 $ITERATIONS | parallel -P 3% $LOADER $PLACEMENT_URL $GABBIT"},{"line_number":76,"context_line":""},{"line_number":77,"context_line":"    set +x"},{"line_number":78,"context_line":"    rp_count\u003d$(curl -H \u0027x-auth-token: admin\u0027 ${PLACEMENT_URL}/resource_providers |json_pp|grep -c \u0027\"name\"\u0027)"}],"source_content_type":"text/x-sh","patch_set":1,"id":"7faddb67_ab42e966","side":"PARENT","line":75,"range":{"start_line":75,"start_character":36,"end_line":75,"end_character":38},"in_reply_to":"7faddb67_6bdb91ec","updated":"2019-07-30 15:54:11.000000000","message":"I did a fair bit of testing locally and as far as I was able to tell, it simply doesn\u0027t do what the man page says. The consistent way to get \"lots of processes\" was to do nothing.","commit_id":"2f56f379a568d427ac9057086386074a7801b00c"},{"author":{"_account_id":25625,"name":"Tetsuro Nakamura","email":"tetsuro.nakamura.bc@hco.ntt.co.jp","username":"tetsuro0907"},"change_message_id":"07057acaa789586f2ea7e49b3c4f38ad785cb71f","unresolved":false,"context_lines":[{"line_number":69,"context_line":""},{"line_number":70,"context_line":"    # Create $TOTAL_PROVIDER_COUNT nested resource provider trees,"},{"line_number":71,"context_line":"    # each tree having $PROVIDER_TOPOLOGY_COUNT resource providers."},{"line_number":72,"context_line":"    # LOADER is called $ITERATIONS times in parallel by 3 * number"},{"line_number":73,"context_line":"    # of processors on the host."},{"line_number":74,"context_line":"    echo \"##### Creating $TOTAL_PROVIDER_COUNT providers\" | tee -a $LOG"},{"line_number":75,"context_line":"    seq 1 $ITERATIONS | parallel $LOADER $PLACEMENT_URL $GABBIT"},{"line_number":76,"context_line":""}],"source_content_type":"text/x-sh","patch_set":2,"id":"7faddb67_77e31606","line":73,"range":{"start_line":72,"start_character":53,"end_line":73,"end_character":32},"updated":"2019-08-06 05:13:10.000000000","message":"Can you update this please?","commit_id":"735e8d52a51a5acb31463cd4c4a4e5a81440e8a8"},{"author":{"_account_id":9708,"name":"Balazs Gibizer","display_name":"gibi","email":"gibizer@gmail.com","username":"gibi"},"change_message_id":"656c2e6a1bcdd3c4b280f6fad9f1fcbf077e4dc8","unresolved":false,"context_lines":[{"line_number":72,"context_line":"    # LOADER is called $ITERATIONS times in parallel using 50% of"},{"line_number":73,"context_line":"    # the number of processors on the host."},{"line_number":74,"context_line":"    echo \"##### Creating $TOTAL_PROVIDER_COUNT providers\" | tee -a $LOG"},{"line_number":75,"context_line":"    seq 1 $ITERATIONS | parallel -P 50% $LOADER $PLACEMENT_URL $GABBIT"},{"line_number":76,"context_line":""},{"line_number":77,"context_line":"    set +x"},{"line_number":78,"context_line":"    rp_count\u003d$(curl -H \u0027x-auth-token: admin\u0027 ${PLACEMENT_URL}/resource_providers |json_pp|grep -c \u0027\"name\"\u0027)"}],"source_content_type":"text/x-sh","patch_set":3,"id":"7faddb67_6feb2112","line":75,"updated":"2019-08-07 09:09:45.000000000","message":"-P doc checks out:\n\n -P N%    Multiply N% with the number of CPU cores.  Run this many jobs in parallel","commit_id":"7464ff6e24c34e412ec3def799edbc9dedd3a2ef"}]}