)]}'
{".zuul.yaml":[{"author":{"_account_id":11975,"name":"Slawek Kaplonski","email":"skaplons@redhat.com","username":"slaweq"},"change_message_id":"889da9ea7b35838722ccacef3fc6932dbe03969f","unresolved":true,"context_lines":[{"line_number":100,"context_line":"    gate:"},{"line_number":101,"context_line":"      jobs:"},{"line_number":102,"context_line":"        - openstack-tox-lower-constraints:"},{"line_number":103,"context_line":"            voting: false"},{"line_number":104,"context_line":"            required-projects:"},{"line_number":105,"context_line":"              - openstack/neutron"},{"line_number":106,"context_line":"              - openstack/networking-bagpipe"}],"source_content_type":"text/x-yaml","patch_set":9,"id":"c0a48ab2_58a9c5c4","line":103,"updated":"2021-06-22 06:45:49.000000000","message":"do we need to keep that lower-constraints job in stable branches? I think we agreed some time ago to drop it completely from stable branches, didn\u0027t we?","commit_id":"c2342dacdebbd4f7ea71f402f150540fc888fe99"},{"author":{"_account_id":21798,"name":"Bernard Cafarelli","email":"bcafarel@redhat.com","username":"bcafarel"},"change_message_id":"d5db945aa5d8c4815db456ec6f4d82ff05390cc2","unresolved":true,"context_lines":[{"line_number":100,"context_line":"    gate:"},{"line_number":101,"context_line":"      jobs:"},{"line_number":102,"context_line":"        - openstack-tox-lower-constraints:"},{"line_number":103,"context_line":"            voting: false"},{"line_number":104,"context_line":"            required-projects:"},{"line_number":105,"context_line":"              - openstack/neutron"},{"line_number":106,"context_line":"              - openstack/networking-bagpipe"}],"source_content_type":"text/x-yaml","patch_set":9,"id":"c397af90_69cfe609","line":103,"in_reply_to":"c0a48ab2_58a9c5c4","updated":"2021-06-30 13:30:22.000000000","message":"Indeed, we can merge that drop with the gate fix or follow-up, either is fine for me","commit_id":"c2342dacdebbd4f7ea71f402f150540fc888fe99"},{"author":{"_account_id":8313,"name":"Lajos Katona","display_name":"lajoskatona","email":"katonalala@gmail.com","username":"elajkat","status":"Ericsson Software Technology"},"change_message_id":"56f85e494077b5926876b7a76660634357f2d6a2","unresolved":true,"context_lines":[{"line_number":100,"context_line":"    gate:"},{"line_number":101,"context_line":"      jobs:"},{"line_number":102,"context_line":"        - openstack-tox-lower-constraints:"},{"line_number":103,"context_line":"            voting: false"},{"line_number":104,"context_line":"            required-projects:"},{"line_number":105,"context_line":"              - openstack/neutron"},{"line_number":106,"context_line":"              - openstack/networking-bagpipe"}],"source_content_type":"text/x-yaml","patch_set":9,"id":"246b0f2e_6324369c","line":103,"in_reply_to":"c397af90_69cfe609","updated":"2021-07-06 13:02:08.000000000","message":"I removed l-c job","commit_id":"c2342dacdebbd4f7ea71f402f150540fc888fe99"}],"/COMMIT_MSG":[{"author":{"_account_id":17685,"name":"Elod Illes","email":"elod.illes@est.tech","username":"elod.illes"},"change_message_id":"a46b153713ca1efe8a69f418f5cc27c6de809cc0","unresolved":true,"context_lines":[{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Fix pep8 with missing isort, networking-bgpvpn-dsvm-functional with"},{"line_number":10,"context_line":"ensuring tox, and make lower-constraints job non-voting."},{"line_number":11,"context_line":"Remove tools/test-setup.s and let zuul setup database."},{"line_number":12,"context_line":""},{"line_number":13,"context_line":"Change-Id: I3d49c6dcf004d1ae216d20aa9b59e199c21d61f1"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":11,"id":"2e24a4d7_8c117073","line":11,"range":{"start_line":11,"start_character":0,"end_line":11,"end_character":54},"updated":"2021-07-07 15:55:53.000000000","message":"Nice! I did not know that this can be removed and trust in zuul setup instead.\nActually, for local devstack deployers the test-setup.sh could be useful, though, but the question is how actively used networking-odl\u0027s stable/train.","commit_id":"a871a9bdfb23666a501be30ba7b08b7b9f8bbc63"}],"devstack/post_test_hook.sh":[{"author":{"_account_id":8313,"name":"Lajos Katona","display_name":"lajoskatona","email":"katonalala@gmail.com","username":"elajkat","status":"Ericsson Software Technology"},"change_message_id":"bd33e5990cbe25c763e263565e075afd523d9852","unresolved":true,"context_lines":[{"line_number":29,"context_line":"    venv\u003d${typetest/dsvm-bagpipe/dsvm}"},{"line_number":30,"context_line":"    echo \"Running networking-bgpvpn $venv test suite\""},{"line_number":31,"context_line":"    set +e"},{"line_number":32,"context_line":"    sudo -E -H -u $GATE_STACK_USER /home/$GATE_STACK_USER/.local/tox/bin/tox -ve ${venv}"},{"line_number":33,"context_line":"    testr_exit_code\u003d$?"},{"line_number":34,"context_line":"    set -e"},{"line_number":35,"context_line":""}],"source_content_type":"text/x-sh","patch_set":9,"id":"d8ac8460_b7b58501","line":32,"range":{"start_line":32,"start_character":35,"end_line":32,"end_character":77},"updated":"2021-06-22 05:52:34.000000000","message":"ok it\u0027s not nice....","commit_id":"c2342dacdebbd4f7ea71f402f150540fc888fe99"},{"author":{"_account_id":841,"name":"Akihiro Motoki","email":"amotoki@gmail.com","username":"amotoki"},"change_message_id":"93f2212b80ac13e6343bba2d56670851f41dadf1","unresolved":true,"context_lines":[{"line_number":29,"context_line":"    venv\u003d${typetest/dsvm-bagpipe/dsvm}"},{"line_number":30,"context_line":"    echo \"Running networking-bgpvpn $venv test suite\""},{"line_number":31,"context_line":"    set +e"},{"line_number":32,"context_line":"    sudo -E -H -u $GATE_STACK_USER /home/$GATE_STACK_USER/.local/tox/bin/tox -ve ${venv}"},{"line_number":33,"context_line":"    testr_exit_code\u003d$?"},{"line_number":34,"context_line":"    set -e"},{"line_number":35,"context_line":""}],"source_content_type":"text/x-sh","patch_set":9,"id":"8910a702_a4292a10","line":32,"range":{"start_line":32,"start_character":35,"end_line":32,"end_character":77},"in_reply_to":"1be231c2_232f2c95","updated":"2021-07-06 12:17:03.000000000","message":"Instead I would suggest to pass ensure_global_symlinks\u003dTrue to ensure-tox role. It installs /usr/local/bin/tox symlink.\n\nhttps://zuul-ci.org/docs/zuul-jobs/python-roles.html#role-ensure-tox","commit_id":"c2342dacdebbd4f7ea71f402f150540fc888fe99"},{"author":{"_account_id":8313,"name":"Lajos Katona","display_name":"lajoskatona","email":"katonalala@gmail.com","username":"elajkat","status":"Ericsson Software Technology"},"change_message_id":"56f85e494077b5926876b7a76660634357f2d6a2","unresolved":true,"context_lines":[{"line_number":29,"context_line":"    venv\u003d${typetest/dsvm-bagpipe/dsvm}"},{"line_number":30,"context_line":"    echo \"Running networking-bgpvpn $venv test suite\""},{"line_number":31,"context_line":"    set +e"},{"line_number":32,"context_line":"    sudo -E -H -u $GATE_STACK_USER /home/$GATE_STACK_USER/.local/tox/bin/tox -ve ${venv}"},{"line_number":33,"context_line":"    testr_exit_code\u003d$?"},{"line_number":34,"context_line":"    set -e"},{"line_number":35,"context_line":""}],"source_content_type":"text/x-sh","patch_set":9,"id":"b97ed74e_a8eabcea","line":32,"range":{"start_line":32,"start_character":35,"end_line":32,"end_character":77},"in_reply_to":"8910a702_a4292a10","updated":"2021-07-06 13:02:08.000000000","message":"Thanks, I pushed a try","commit_id":"c2342dacdebbd4f7ea71f402f150540fc888fe99"},{"author":{"_account_id":21798,"name":"Bernard Cafarelli","email":"bcafarel@redhat.com","username":"bcafarel"},"change_message_id":"d5db945aa5d8c4815db456ec6f4d82ff05390cc2","unresolved":true,"context_lines":[{"line_number":29,"context_line":"    venv\u003d${typetest/dsvm-bagpipe/dsvm}"},{"line_number":30,"context_line":"    echo \"Running networking-bgpvpn $venv test suite\""},{"line_number":31,"context_line":"    set +e"},{"line_number":32,"context_line":"    sudo -E -H -u $GATE_STACK_USER /home/$GATE_STACK_USER/.local/tox/bin/tox -ve ${venv}"},{"line_number":33,"context_line":"    testr_exit_code\u003d$?"},{"line_number":34,"context_line":"    set -e"},{"line_number":35,"context_line":""}],"source_content_type":"text/x-sh","patch_set":9,"id":"1be231c2_232f2c95","line":32,"range":{"start_line":32,"start_character":35,"end_line":32,"end_character":77},"in_reply_to":"d8ac8460_b7b58501","updated":"2021-06-30 13:30:22.000000000","message":"But it fixes gate in EM branch, so at least no strong objection from me","commit_id":"c2342dacdebbd4f7ea71f402f150540fc888fe99"},{"author":{"_account_id":17685,"name":"Elod Illes","email":"elod.illes@est.tech","username":"elod.illes"},"change_message_id":"a46b153713ca1efe8a69f418f5cc27c6de809cc0","unresolved":true,"context_lines":[{"line_number":4,"context_line":""},{"line_number":5,"context_line":"NETWORKING_BGPVPN_DIR\u003d\"$BASE/new/networking-bgpvpn\""},{"line_number":6,"context_line":"LOGS\u003d/opt/stack/logs"},{"line_number":7,"context_line":"GATE_STACK_USER\u003dzuul"},{"line_number":8,"context_line":"SCRIPTS_DIR\u003d\"/usr/os-testr-env/bin/\""},{"line_number":9,"context_line":""},{"line_number":10,"context_line":"typetest\u003d${1:-\"unknown\"}"}],"source_content_type":"text/x-sh","patch_set":11,"id":"e3cdb086_175aa64f","line":7,"range":{"start_line":7,"start_character":16,"end_line":7,"end_character":20},"updated":"2021-07-07 15:55:53.000000000","message":"Isn\u0027t this causing problems for people who deploys their devstack locally with stack user?","commit_id":"a871a9bdfb23666a501be30ba7b08b7b9f8bbc63"},{"author":{"_account_id":841,"name":"Akihiro Motoki","email":"amotoki@gmail.com","username":"amotoki"},"change_message_id":"2a51d4c9ddc01eab37f816025e739e87ff042a7c","unresolved":true,"context_lines":[{"line_number":4,"context_line":""},{"line_number":5,"context_line":"NETWORKING_BGPVPN_DIR\u003d\"$BASE/new/networking-bgpvpn\""},{"line_number":6,"context_line":"LOGS\u003d/opt/stack/logs"},{"line_number":7,"context_line":"GATE_STACK_USER\u003dzuul"},{"line_number":8,"context_line":"SCRIPTS_DIR\u003d\"/usr/os-testr-env/bin/\""},{"line_number":9,"context_line":""},{"line_number":10,"context_line":"typetest\u003d${1:-\"unknown\"}"}],"source_content_type":"text/x-sh","patch_set":11,"id":"8c0683e2_65087575","line":7,"range":{"start_line":7,"start_character":16,"end_line":7,"end_character":20},"in_reply_to":"e3cdb086_175aa64f","updated":"2021-07-07 17:32:27.000000000","message":"In my understanding, the script is located under \"devstack\" directory but it is not directly related to devstack. It is just related to our CI. We see the similar script in neutron stadium projects but they are usually located under \"tests\" directories.\n\nOf course, it would be nice if the script supports local runs, but I am not sure it was part of the scope of the script.\n\nIn most cases, when we try to reproduce what our CI does, we need to adjust several? many? things like this... so I don\u0027t think this is the only thing.","commit_id":"a871a9bdfb23666a501be30ba7b08b7b9f8bbc63"}],"playbooks/networking-bgpvpn-dsvm-functional/pre.yaml":[{"author":{"_account_id":841,"name":"Akihiro Motoki","email":"amotoki@gmail.com","username":"amotoki"},"change_message_id":"e8f7e7c56c34c6ecc5508c16602d9095ed60ac31","unresolved":true,"context_lines":[{"line_number":1,"context_line":"- hosts: all"},{"line_number":2,"context_line":"  roles:"},{"line_number":3,"context_line":"    - ensure-tox"},{"line_number":4,"context_line":"      ensure_global_symlinks: true"}],"source_content_type":"text/x-yaml","patch_set":10,"id":"257cd9f0_727b9b93","line":4,"updated":"2021-07-06 13:51:37.000000000","message":"This is a good example on how to pass variables to a role.\nhttps://opendev.org/openstack/openstack-zuul-jobs/src/branch/master/playbooks/release/pre.yaml#L9-L10","commit_id":"039d5d2ea02093422d7fc0cef68c779563a18c43"},{"author":{"_account_id":21798,"name":"Bernard Cafarelli","email":"bcafarel@redhat.com","username":"bcafarel"},"change_message_id":"864200ed4d18ae1a43b14553ad36d2fc4ebf1397","unresolved":true,"context_lines":[{"line_number":1,"context_line":"- hosts: all"},{"line_number":2,"context_line":"  roles:"},{"line_number":3,"context_line":"    - ensure-tox"},{"line_number":4,"context_line":"      ensure_global_symlinks: true"}],"source_content_type":"text/x-yaml","patch_set":10,"id":"f7747f83_951ccdb4","line":4,"in_reply_to":"257cd9f0_727b9b93","updated":"2021-07-06 14:34:50.000000000","message":"Indeed current run failed with \"mapping values are not allowed in this context\" here","commit_id":"039d5d2ea02093422d7fc0cef68c779563a18c43"}]}