)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"change_message_id":"d701c142e54ee3c1a202d5876e5243b1b079bf9d","unresolved":false,"context_lines":[{"line_number":35,"context_line":"API smoke tests but the savings might not be worth the complexity in"},{"line_number":36,"context_line":"the job configuration."},{"line_number":37,"context_line":""},{"line_number":38,"context_line":"[1] https://zuul.opendev.org/t/openstack/build/377affdf97b04022a321087352e1cc4c/log/logs/grenade.sh.txt.gz#41965"},{"line_number":39,"context_line":""},{"line_number":40,"context_line":"Change-Id: Icac785eec824da5146efe0ea8ecd01383f18398e"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"3fa7e38b_da228354","line":38,"updated":"2019-11-18 15:03:34.000000000","message":"And this is the smoke run against the new side:\n\nhttps://zuul.opendev.org/t/openstack/build/377affdf97b04022a321087352e1cc4c/log/job-output.txt#5900\n\nSo that\u0027s about ~15 minutes overall in tempest test runtime. So maybe we should consider controlling the tests we run, something like:\n\nDEVSTACK_GATE_TEMPEST_REGEX\u003d\"(tempest.api.compute.*|tempest.scenario.*)([.*\\bsmoke\\b.*])\"\n\n?","commit_id":"66da85481d8216f4fa9868c31291bf4d53438ecf"},{"author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"change_message_id":"0d3f9e64da0beaed994be62cb8828b92204d8b40","unresolved":false,"context_lines":[{"line_number":35,"context_line":"API smoke tests but the savings might not be worth the complexity in"},{"line_number":36,"context_line":"the job configuration."},{"line_number":37,"context_line":""},{"line_number":38,"context_line":"[1] https://zuul.opendev.org/t/openstack/build/377affdf97b04022a321087352e1cc4c/log/logs/grenade.sh.txt.gz#41965"},{"line_number":39,"context_line":""},{"line_number":40,"context_line":"Change-Id: Icac785eec824da5146efe0ea8ecd01383f18398e"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"3fa7e38b_9ad96bf5","line":38,"in_reply_to":"3fa7e38b_da228354","updated":"2019-11-18 15:27:10.000000000","message":"This is likely the regex we\u0027d want:\n\ntempest.(api.compute|scenario)\\..*smoke.*","commit_id":"66da85481d8216f4fa9868c31291bf4d53438ecf"}],"playbooks/legacy/nova-grenade-multinode/run.yaml":[{"author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"change_message_id":"c8c965a7a42bc5344ce31f1b6115320c061da3dd","unresolved":false,"context_lines":[{"line_number":32,"context_line":"          export DEVSTACK_GATE_CONFIGDRIVE\u003d0"},{"line_number":33,"context_line":"          export DEVSTACK_GATE_NEUTRON\u003d1"},{"line_number":34,"context_line":"          # NOTE(mriedem): Run tempest smoke tests specific to compute on the"},{"line_number":35,"context_line":"          # old and new side of the grenade environment. The post-test hook"},{"line_number":36,"context_line":"          # script will run non-smoke migration tests in a local/block and"},{"line_number":37,"context_line":"          # shared/ceph setup."},{"line_number":38,"context_line":"          export DEVSTACK_GATE_TEMPEST\u003d1"}],"source_content_type":"text/x-yaml","patch_set":2,"id":"3fa7e38b_a7ca0017","line":35,"range":{"start_line":35,"start_character":12,"end_line":35,"end_character":20},"updated":"2019-11-19 15:43:19.000000000","message":"Could drop this given we don\u0027t control it:\n\nhttps://github.com/openstack/grenade/blob/fad62595bff3ae55b5b428a5ea00a9a168390fd2/grenade.sh#L285","commit_id":"47765c074a438042c68fb1e1a09bc4c57ffb6226"},{"author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"change_message_id":"8da02f3cd687d4b0debcfee1e3d51cf71499aca1","unresolved":false,"context_lines":[{"line_number":36,"context_line":"          # script will run non-smoke migration tests in a local/block and"},{"line_number":37,"context_line":"          # shared/ceph setup."},{"line_number":38,"context_line":"          export DEVSTACK_GATE_TEMPEST\u003d1"},{"line_number":39,"context_line":"          export DEVSTACK_GATE_TEMPEST_REGEX\u003d\"tempest\\.(api\\.compute|scenario)\\..*smoke.*\""},{"line_number":40,"context_line":"          export DEVSTACK_GATE_GRENADE\u003dpullup"},{"line_number":41,"context_line":"          export DEVSTACK_GATE_USE_PYTHON3\u003dTrue"},{"line_number":42,"context_line":"          # By default grenade runs only smoke tests so we need to set"}],"source_content_type":"text/x-yaml","patch_set":2,"id":"3fa7e38b_07543453","line":39,"updated":"2019-11-19 15:42:37.000000000","message":"The regex works:\n\nhttps://zuul.opendev.org/t/openstack/build/edea84e438bb45a1b389aeea64a1602e/log/job-output.txt#5811\n\nThough we could stand to omit this test:\n\ntempest.scenario.test_network_basic_ops.TestNetworkBasicOps.test_network_basic_ops [173.010598s] ... ok\n\nNote that the regex isn\u0027t applied to the old side tempest run:\n\nhttps://zuul.opendev.org/t/openstack/build/edea84e438bb45a1b389aeea64a1602e/log/logs/grenade.sh.txt.gz#41852\n\nBecause of this:\n\nhttps://github.com/openstack/grenade/blob/fad62595bff3ae55b5b428a5ea00a9a168390fd2/grenade.sh#L285","commit_id":"47765c074a438042c68fb1e1a09bc4c57ffb6226"},{"author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"change_message_id":"402d9abad40e7ab3522604848bd20acbb3daba8c","unresolved":false,"context_lines":[{"line_number":36,"context_line":"          # script will run non-smoke migration tests in a local/block and"},{"line_number":37,"context_line":"          # shared/ceph setup."},{"line_number":38,"context_line":"          export DEVSTACK_GATE_TEMPEST\u003d1"},{"line_number":39,"context_line":"          export DEVSTACK_GATE_TEMPEST_REGEX\u003d\"tempest\\.(api\\.compute|scenario)\\..*smoke.*\""},{"line_number":40,"context_line":"          export DEVSTACK_GATE_GRENADE\u003dpullup"},{"line_number":41,"context_line":"          export DEVSTACK_GATE_USE_PYTHON3\u003dTrue"},{"line_number":42,"context_line":"          # By default grenade runs only smoke tests so we need to set"}],"source_content_type":"text/x-yaml","patch_set":2,"id":"3fa7e38b_a798c01a","line":39,"in_reply_to":"3fa7e38b_07543453","updated":"2019-11-19 15:49:56.000000000","message":"This should filter out that network scenario test:\n\ntempest\\.(api\\.compute|scenario(?!.*test_network_basic_ops))\\..*smoke.*\n\nThough I guess that scenario test does create a VM and ssh into it so we should likely just keep it and avoid the added complexity in the regex.","commit_id":"47765c074a438042c68fb1e1a09bc4c57ffb6226"},{"author":{"_account_id":15334,"name":"Stephen Finucane","display_name":"stephenfin","email":"stephenfin@redhat.com","username":"sfinucan"},"change_message_id":"3b8f9997d18c80599f3ac454b96fd687a9d56b8b","unresolved":false,"context_lines":[{"line_number":35,"context_line":"          # new side of the grenade environment. The post-test hook script will"},{"line_number":36,"context_line":"          # run non-smoke migration tests in a local/block and shared/ceph"},{"line_number":37,"context_line":"          # setup. Note that grenade hard-codes \"tox -esmoke\" for tempest on"},{"line_number":38,"context_line":"          # the old side so the regex is not appied there."},{"line_number":39,"context_line":"          export DEVSTACK_GATE_TEMPEST\u003d1"},{"line_number":40,"context_line":"          export DEVSTACK_GATE_TEMPEST_REGEX\u003d\"tempest\\.(api\\.compute|scenario)\\..*smoke.*\""},{"line_number":41,"context_line":"          export DEVSTACK_GATE_GRENADE\u003dpullup"}],"source_content_type":"text/x-yaml","patch_set":4,"id":"3fa7e38b_7227e464","line":38,"range":{"start_line":38,"start_character":45,"end_line":38,"end_character":51},"updated":"2019-11-19 16:19:33.000000000","message":"applied","commit_id":"8cb6f074446b21af6c0198b3130c22cdc38ac389"},{"author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"change_message_id":"83b5848a423f4bfed2e05803db250fd500e4c04e","unresolved":false,"context_lines":[{"line_number":35,"context_line":"          # new side of the grenade environment. The post-test hook script will"},{"line_number":36,"context_line":"          # run non-smoke migration tests in a local/block and shared/ceph"},{"line_number":37,"context_line":"          # setup. Note that grenade hard-codes \"tox -esmoke\" for tempest on"},{"line_number":38,"context_line":"          # the old side so the regex is not appied there."},{"line_number":39,"context_line":"          export DEVSTACK_GATE_TEMPEST\u003d1"},{"line_number":40,"context_line":"          export DEVSTACK_GATE_TEMPEST_REGEX\u003d\"tempest\\.(api\\.compute|scenario)\\..*smoke.*\""},{"line_number":41,"context_line":"          export DEVSTACK_GATE_GRENADE\u003dpullup"}],"source_content_type":"text/x-yaml","patch_set":4,"id":"3fa7e38b_3fd054c8","line":38,"range":{"start_line":38,"start_character":45,"end_line":38,"end_character":51},"in_reply_to":"3fa7e38b_7227e464","updated":"2019-11-19 17:56:35.000000000","message":"Guh. Whoever approves this should just fix this and then fast approve.","commit_id":"8cb6f074446b21af6c0198b3130c22cdc38ac389"}]}