)]}'
{".zuul.yaml":[{"author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"change_message_id":"ca9a4e2c6e713ad92fa9cd0f162cb960ebc688f8","unresolved":false,"context_lines":[{"line_number":20,"context_line":"      devstack_localrc:"},{"line_number":21,"context_line":"        ENABLE_FILE_INJECTION: false"},{"line_number":22,"context_line":"      tox_envlist: all"},{"line_number":23,"context_line":"      tempest_test_regex: |"},{"line_number":24,"context_line":"               (?x)                           # Ignore comments and whitespaces"},{"line_number":25,"context_line":"               (?!.*\\[.*\\bslow\\b.*\\])         # Do not run slow tests"},{"line_number":26,"context_line":"                (^tempest\\.(api|scenario)     # Tempest API and scenario"},{"line_number":27,"context_line":"               |(^cinder\\.tests.tempest))     # Cinder plugin Tempest tests"},{"line_number":28,"context_line":""},{"line_number":29,"context_line":"- job:"},{"line_number":30,"context_line":"    name: devstack-plugin-ceph-tempest-full-py3"}],"source_content_type":"text/x-yaml","patch_set":5,"id":"df7087c5_8db6a3d9","line":27,"range":{"start_line":23,"start_character":6,"end_line":27,"end_character":75},"updated":"2018-03-07 15:11:30.000000000","message":"Looking at https://review.openstack.org/#/c/459774/4/tempest-conf.sh - this regex should also be in the py3 job.","commit_id":"d653c28e918e7f8ffb93681f22988d648ef7c98e"},{"author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"change_message_id":"023760c4b794ef335ef816bae5fcf08bc718b1e9","unresolved":false,"context_lines":[{"line_number":27,"context_line":"               |(^cinder\\.tests.tempest))     # Cinder plugin Tempest tests"},{"line_number":28,"context_line":""},{"line_number":29,"context_line":"- job:"},{"line_number":30,"context_line":"    name: devstack-plugin-ceph-tempest-full-py3"},{"line_number":31,"context_line":"    parent: tempest-full-py3"},{"line_number":32,"context_line":"    description: |"},{"line_number":33,"context_line":"      Integration tests that runs with the ceph devstack plugin and py3."}],"source_content_type":"text/x-yaml","patch_set":5,"id":"df7087c5_cd8edb2d","line":30,"range":{"start_line":30,"start_character":10,"end_line":30,"end_character":47},"updated":"2018-03-07 15:09:10.000000000","message":"Why isn\u0027t this job getting run?","commit_id":"d653c28e918e7f8ffb93681f22988d648ef7c98e"},{"author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"change_message_id":"2ac02e093f88705358586dd2f563c8deae23fc9b","unresolved":false,"context_lines":[{"line_number":27,"context_line":"               |(^cinder\\.tests.tempest))     # Cinder plugin Tempest tests"},{"line_number":28,"context_line":""},{"line_number":29,"context_line":"- job:"},{"line_number":30,"context_line":"    name: devstack-plugin-ceph-tempest-full-py3"},{"line_number":31,"context_line":"    parent: tempest-full-py3"},{"line_number":32,"context_line":"    description: |"},{"line_number":33,"context_line":"      Integration tests that runs with the ceph devstack plugin and py3."}],"source_content_type":"text/x-yaml","patch_set":5,"id":"df7087c5_ed763f3f","line":30,"range":{"start_line":30,"start_character":10,"end_line":30,"end_character":47},"in_reply_to":"df7087c5_cd8edb2d","updated":"2018-03-07 15:09:45.000000000","message":"Oh nevermind it\u0027s in the experimental queue.","commit_id":"d653c28e918e7f8ffb93681f22988d648ef7c98e"},{"author":{"_account_id":11904,"name":"Sean McGinnis","email":"sean.mcginnis@gmail.com","username":"SeanM"},"change_message_id":"c0341a5c8c3d7442681d82ee83050551bf6dec20","unresolved":false,"context_lines":[{"line_number":51,"context_line":"      Runs devstack-plugin-ceph-tempest jobs"},{"line_number":52,"context_line":"    check:"},{"line_number":53,"context_line":"      jobs:"},{"line_number":54,"context_line":"        - devstack-plugin-ceph-tempest-full:"},{"line_number":55,"context_line":"            branches: ^(?!driverfixes/).*$"},{"line_number":56,"context_line":"            irrelevant-files:"},{"line_number":57,"context_line":"              - ^.*\\.rst$"}],"source_content_type":"text/x-yaml","patch_set":5,"id":"3fa0c359_b320f1ad","line":54,"updated":"2018-02-12 13:17:21.000000000","message":"If we have this voting in check, shouldn\u0027t we also have it in gate?","commit_id":"d653c28e918e7f8ffb93681f22988d648ef7c98e"},{"author":{"_account_id":1921,"name":"Andrea Frittoli","email":"andrea.frittoli@gmail.com","username":"andrea-frittoli"},"change_message_id":"ea7f3ab7dbd8bfdfa59df5f4d99e4c5ff414b5ad","unresolved":false,"context_lines":[{"line_number":51,"context_line":"      Runs devstack-plugin-ceph-tempest jobs"},{"line_number":52,"context_line":"    check:"},{"line_number":53,"context_line":"      jobs:"},{"line_number":54,"context_line":"        - devstack-plugin-ceph-tempest-full:"},{"line_number":55,"context_line":"            branches: ^(?!driverfixes/).*$"},{"line_number":56,"context_line":"            irrelevant-files:"},{"line_number":57,"context_line":"              - ^.*\\.rst$"}],"source_content_type":"text/x-yaml","patch_set":5,"id":"3fa0c359_bf150842","line":54,"in_reply_to":"3fa0c359_b320f1ad","updated":"2018-02-12 21:00:23.000000000","message":"it\u0027s not voting, the job is defined as non-voting since it\u0027s not voting everywhere today.  I don\u0027t have the background to say whether it should be voting","commit_id":"d653c28e918e7f8ffb93681f22988d648ef7c98e"},{"author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"change_message_id":"f061b18fb25c74050561a492f15c9e2b177bb9de","unresolved":false,"context_lines":[{"line_number":44,"context_line":"      devstack_plugins:"},{"line_number":45,"context_line":"        devstack-plugin-ceph: git://git.openstack.org/openstack/devstack-plugin-ceph"},{"line_number":46,"context_line":"      tox_envlist: all"},{"line_number":47,"context_line":"      tempest_test_regex: |"},{"line_number":48,"context_line":"               (?x)                           # Ignore comments and whitespaces"},{"line_number":49,"context_line":"               (?!.*\\[.*\\bslow\\b.*\\])         # Do not run slow tests"},{"line_number":50,"context_line":"                (^tempest\\.(api|scenario)     # Tempest API and scenario"}],"source_content_type":"text/x-yaml","patch_set":6,"id":"df7087c5_f09138f3","line":47,"range":{"start_line":47,"start_character":6,"end_line":47,"end_character":24},"updated":"2018-03-07 15:53:39.000000000","message":"This is failing:\n\nhttp://logs.openstack.org/48/543048/6/check/devstack-plugin-ceph-tempest-full/0398bcb/job-output.txt.gz#_2018-03-07_15_47_31_602709\n\n2018-03-07 15:47:28.883899 | controller | all runtests: commands[1] | tempest run --regex (?x)                           # Ignore comments and whitespaces\n2018-03-07 15:47:28.883978 | controller | (?!.*\\[.*\\bslow\\b.*\\])         # Do not run slow tests\n2018-03-07 15:47:28.884016 | controller |  (^tempest\\.(api|scenario)     # Tempest API and scenario\n2018-03-07 15:47:28.884053 | controller | |(^cinder\\.tests.tempest))     # Cinder plugin Tempest tests\n2018-03-07 15:47:28.884069 | controller |  --concurrency\u003d4\n2018-03-07 15:47:31.602709 | controller | unbalanced parenthesis","commit_id":"cc433354b1a892d8195e1edf4cccf48f400f70f0"},{"author":{"_account_id":4523,"name":"Eric Harney","email":"eharney@redhat.com","username":"eharney"},"change_message_id":"59bfeedc1569a088a61c60663afe79e9a1311f66","unresolved":false,"context_lines":[{"line_number":44,"context_line":"      devstack_plugins:"},{"line_number":45,"context_line":"        devstack-plugin-ceph: git://git.openstack.org/openstack/devstack-plugin-ceph"},{"line_number":46,"context_line":"      tox_envlist: all"},{"line_number":47,"context_line":"      tempest_test_regex: |"},{"line_number":48,"context_line":"               (?x)                           # Ignore comments and whitespaces"},{"line_number":49,"context_line":"               (?!.*\\[.*\\bslow\\b.*\\])         # Do not run slow tests"},{"line_number":50,"context_line":"                (^tempest\\.(api|scenario)     # Tempest API and scenario"}],"source_content_type":"text/x-yaml","patch_set":6,"id":"df7087c5_c7c29859","line":47,"range":{"start_line":47,"start_character":6,"end_line":47,"end_character":24},"in_reply_to":"df7087c5_566c9c07","updated":"2018-03-14 13:44:26.000000000","message":"This same problem has cropped up over here too:\n    https://review.openstack.org/#/c/531206/\n\nIt\u0027s not clear yet what has changed around this.","commit_id":"cc433354b1a892d8195e1edf4cccf48f400f70f0"},{"author":{"_account_id":13252,"name":"Dr. Jens Harbott","display_name":"Jens Harbott (frickler)","email":"frickler@offenerstapel.de","username":"jrosenboom"},"change_message_id":"a74d9fef2e0bb226210d493352e075a5af3404cf","unresolved":false,"context_lines":[{"line_number":44,"context_line":"      devstack_plugins:"},{"line_number":45,"context_line":"        devstack-plugin-ceph: git://git.openstack.org/openstack/devstack-plugin-ceph"},{"line_number":46,"context_line":"      tox_envlist: all"},{"line_number":47,"context_line":"      tempest_test_regex: |"},{"line_number":48,"context_line":"               (?x)                           # Ignore comments and whitespaces"},{"line_number":49,"context_line":"               (?!.*\\[.*\\bslow\\b.*\\])         # Do not run slow tests"},{"line_number":50,"context_line":"                (^tempest\\.(api|scenario)     # Tempest API and scenario"}],"source_content_type":"text/x-yaml","patch_set":6,"id":"5f7c97a3_62431113","line":47,"range":{"start_line":47,"start_character":6,"end_line":47,"end_character":24},"in_reply_to":"df7087c5_c7c29859","updated":"2018-06-28 11:39:56.000000000","message":"It seems that \"tempest run\" doesn\u0027t work well on a regex that contains comments and newlines, at least that\u0027s what I found when testing locally. Dropped these.","commit_id":"cc433354b1a892d8195e1edf4cccf48f400f70f0"},{"author":{"_account_id":1921,"name":"Andrea Frittoli","email":"andrea.frittoli@gmail.com","username":"andrea-frittoli"},"change_message_id":"f6db613893cfdd0e5ecffc333dd68563b3413f8e","unresolved":false,"context_lines":[{"line_number":44,"context_line":"      devstack_plugins:"},{"line_number":45,"context_line":"        devstack-plugin-ceph: git://git.openstack.org/openstack/devstack-plugin-ceph"},{"line_number":46,"context_line":"      tox_envlist: all"},{"line_number":47,"context_line":"      tempest_test_regex: |"},{"line_number":48,"context_line":"               (?x)                           # Ignore comments and whitespaces"},{"line_number":49,"context_line":"               (?!.*\\[.*\\bslow\\b.*\\])         # Do not run slow tests"},{"line_number":50,"context_line":"                (^tempest\\.(api|scenario)     # Tempest API and scenario"}],"source_content_type":"text/x-yaml","patch_set":6,"id":"df7087c5_566c9c07","line":47,"range":{"start_line":47,"start_character":6,"end_line":47,"end_character":24},"in_reply_to":"df7087c5_f09138f3","updated":"2018-03-07 16:43:23.000000000","message":"OK I\u0027ll check this, strange that it works on the other job, it may be a py version specific thing or a copy paste issue.","commit_id":"cc433354b1a892d8195e1edf4cccf48f400f70f0"},{"author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"change_message_id":"23a77391601266abdfc710af9c024d6d2bf8d03c","unresolved":false,"context_lines":[{"line_number":47,"context_line":"      tempest_test_regex: |"},{"line_number":48,"context_line":"               (?x)                           # Ignore comments and whitespaces"},{"line_number":49,"context_line":"               (?!.*\\[.*\\bslow\\b.*\\])         # Do not run slow tests"},{"line_number":50,"context_line":"                (^tempest\\.(api|scenario)     # Tempest API and scenario"},{"line_number":51,"context_line":"               |(^cinder\\.tests.tempest))     # Cinder plugin Tempest tests"},{"line_number":52,"context_line":""},{"line_number":53,"context_line":"- project-template:"}],"source_content_type":"text/x-yaml","patch_set":6,"id":"df7087c5_edfcff3f","line":50,"range":{"start_line":50,"start_character":16,"end_line":50,"end_character":72},"updated":"2018-03-07 15:21:05.000000000","message":"Is there a way with zuulv3 to run the api tests concurrently but the scenario tests in serial?","commit_id":"cc433354b1a892d8195e1edf4cccf48f400f70f0"},{"author":{"_account_id":1921,"name":"Andrea Frittoli","email":"andrea.frittoli@gmail.com","username":"andrea-frittoli"},"change_message_id":"f6db613893cfdd0e5ecffc333dd68563b3413f8e","unresolved":false,"context_lines":[{"line_number":47,"context_line":"      tempest_test_regex: |"},{"line_number":48,"context_line":"               (?x)                           # Ignore comments and whitespaces"},{"line_number":49,"context_line":"               (?!.*\\[.*\\bslow\\b.*\\])         # Do not run slow tests"},{"line_number":50,"context_line":"                (^tempest\\.(api|scenario)     # Tempest API and scenario"},{"line_number":51,"context_line":"               |(^cinder\\.tests.tempest))     # Cinder plugin Tempest tests"},{"line_number":52,"context_line":""},{"line_number":53,"context_line":"- project-template:"}],"source_content_type":"text/x-yaml","patch_set":6,"id":"df7087c5_56825cca","line":50,"range":{"start_line":50,"start_character":16,"end_line":50,"end_character":72},"in_reply_to":"df7087c5_edfcff3f","updated":"2018-03-07 16:43:23.000000000","message":"We never controlled that in the Job definitions, we customized the tox env for that. \nSo you could  have a tox environment all-scenario-serial or so and use that instead.\nElse you could build your play where you trigger the run-tempest role twice with different regex and concurrency, but then you would also have to make sure that the test result collection bit is working fine.","commit_id":"cc433354b1a892d8195e1edf4cccf48f400f70f0"},{"author":{"_account_id":8556,"name":"Ghanshyam Maan","display_name":"Ghanshyam Maan","email":"gmaan.os14@gmail.com","username":"ghanshyam"},"change_message_id":"2169dd71c6d23b4c2bae2cf453b3859d37405da6","unresolved":false,"context_lines":[{"line_number":44,"context_line":"      devstack_plugins:"},{"line_number":45,"context_line":"        devstack-plugin-ceph: git://git.openstack.org/openstack/devstack-plugin-ceph"},{"line_number":46,"context_line":"      tox_envlist: all"},{"line_number":47,"context_line":"      tempest_test_regex: |"},{"line_number":48,"context_line":"               (?x)                           # Ignore comments and whitespaces"},{"line_number":49,"context_line":"               (?!.*\\[.*\\bslow\\b.*\\])         # Do not run slow tests"},{"line_number":50,"context_line":"                (^tempest\\.(api|scenario)     # Tempest API and scenario"},{"line_number":51,"context_line":"               |(^cinder\\.tests.tempest))     # Cinder plugin Tempest tests"},{"line_number":52,"context_line":""},{"line_number":53,"context_line":"- project-template:"},{"line_number":54,"context_line":"    name: devstack-plugin-ceph-tempest"}],"source_content_type":"text/x-yaml","patch_set":6,"id":"5f7c97a3_245e65e9","line":51,"range":{"start_line":47,"start_character":27,"end_line":51,"end_character":75},"updated":"2018-06-06 05:12:27.000000000","message":"i think we keep the regex file as separate file (https://review.openstack.org/#/c/459774/4) and feed that in zuulv3 job definition. Reason why i like the separate file is that blacklist of tests for ceph will be large like - https://review.openstack.org/#/c/517228/ and having all these blacklisted tests in zuul definition is little complex to maintain.","commit_id":"cc433354b1a892d8195e1edf4cccf48f400f70f0"},{"author":{"_account_id":13252,"name":"Dr. Jens Harbott","display_name":"Jens Harbott (frickler)","email":"frickler@offenerstapel.de","username":"jrosenboom"},"change_message_id":"a74d9fef2e0bb226210d493352e075a5af3404cf","unresolved":false,"context_lines":[{"line_number":44,"context_line":"      devstack_plugins:"},{"line_number":45,"context_line":"        devstack-plugin-ceph: git://git.openstack.org/openstack/devstack-plugin-ceph"},{"line_number":46,"context_line":"      tox_envlist: all"},{"line_number":47,"context_line":"      tempest_test_regex: |"},{"line_number":48,"context_line":"               (?x)                           # Ignore comments and whitespaces"},{"line_number":49,"context_line":"               (?!.*\\[.*\\bslow\\b.*\\])         # Do not run slow tests"},{"line_number":50,"context_line":"                (^tempest\\.(api|scenario)     # Tempest API and scenario"},{"line_number":51,"context_line":"               |(^cinder\\.tests.tempest))     # Cinder plugin Tempest tests"},{"line_number":52,"context_line":""},{"line_number":53,"context_line":"- project-template:"},{"line_number":54,"context_line":"    name: devstack-plugin-ceph-tempest"}],"source_content_type":"text/x-yaml","patch_set":6,"id":"5f7c97a3_e26081a7","line":51,"range":{"start_line":47,"start_character":27,"end_line":51,"end_character":75},"in_reply_to":"5f7c97a3_245e65e9","updated":"2018-06-28 11:39:56.000000000","message":"Done","commit_id":"cc433354b1a892d8195e1edf4cccf48f400f70f0"},{"author":{"_account_id":6873,"name":"Matt Riedemann","email":"mriedem.os@gmail.com","username":"mriedem"},"change_message_id":"23a77391601266abdfc710af9c024d6d2bf8d03c","unresolved":false,"context_lines":[{"line_number":64,"context_line":"    experimental:"},{"line_number":65,"context_line":"      jobs:"},{"line_number":66,"context_line":"        - devstack-plugin-ceph-tempest-full-py3:"},{"line_number":67,"context_line":"            branches: ^(?!(stable/(newton|ocata)|driverfixes/)).*$"},{"line_number":68,"context_line":"            irrelevant-files:"},{"line_number":69,"context_line":"              - ^.*\\.rst$"},{"line_number":70,"context_line":"              - ^doc/.*$"}],"source_content_type":"text/x-yaml","patch_set":6,"id":"df7087c5_6d9eaf08","line":67,"range":{"start_line":67,"start_character":35,"end_line":67,"end_character":47},"updated":"2018-03-07 15:21:05.000000000","message":"Do you want queens in here now too?","commit_id":"cc433354b1a892d8195e1edf4cccf48f400f70f0"},{"author":{"_account_id":10459,"name":"Luigi Toscano","email":"ltoscano@redhat.com","username":"ltoscano"},"change_message_id":"873f15620e443f0b488e771e2dac895027618c21","unresolved":false,"context_lines":[{"line_number":1,"context_line":"- job:"},{"line_number":2,"context_line":"    name: devstack-plugin-ceph-tempest-full"},{"line_number":3,"context_line":"    parent: tempest-full"},{"line_number":4,"context_line":"    description: |"},{"line_number":5,"context_line":"      Integration tests that runs with the ceph devstack plugin."}],"source_content_type":"text/x-yaml","patch_set":7,"id":"5f7c97a3_25e9a3e2","line":2,"updated":"2018-06-28 12:32:00.000000000","message":"I would drop -full from the name (here are for the py3 job); it\u0027s not running the full set of tempest jobs.","commit_id":"c3ba0a4044edaa81f5556d28305c4ea1030bbe72"},{"author":{"_account_id":13252,"name":"Dr. Jens Harbott","display_name":"Jens Harbott (frickler)","email":"frickler@offenerstapel.de","username":"jrosenboom"},"change_message_id":"603d40cefef349118d92f90ef2ff6f2755d5bd18","unresolved":false,"context_lines":[{"line_number":1,"context_line":"- job:"},{"line_number":2,"context_line":"    name: devstack-plugin-ceph-tempest-full"},{"line_number":3,"context_line":"    parent: tempest-full"},{"line_number":4,"context_line":"    description: |"},{"line_number":5,"context_line":"      Integration tests that runs with the ceph devstack plugin."}],"source_content_type":"text/x-yaml","patch_set":7,"id":"5f7c97a3_56d57746","line":2,"in_reply_to":"5f7c97a3_25e9a3e2","updated":"2018-06-28 14:09:18.000000000","message":"Done","commit_id":"c3ba0a4044edaa81f5556d28305c4ea1030bbe72"},{"author":{"_account_id":10459,"name":"Luigi Toscano","email":"ltoscano@redhat.com","username":"ltoscano"},"change_message_id":"873f15620e443f0b488e771e2dac895027618c21","unresolved":false,"context_lines":[{"line_number":11,"context_line":"        # override DEVSTACK_GATE_TEMPEST_REGEX to further refine the set of"},{"line_number":12,"context_line":"        # tests being run."},{"line_number":13,"context_line":"    required-projects:"},{"line_number":14,"context_line":"      - openstack/devstack-plugin-ceph"},{"line_number":15,"context_line":"    timeout: 7200"},{"line_number":16,"context_line":"    voting: false"},{"line_number":17,"context_line":"    vars:"}],"source_content_type":"text/x-yaml","patch_set":7,"id":"5f7c97a3_e58acb18","line":14,"updated":"2018-06-28 12:32:00.000000000","message":"we need also\nopenstack/cinder-tempest-plugin\n\nand later\n      devstack_localrc:\n        TEMPEST_PLUGINS: \u0027/opt/stack/cinder-tempest-plugin\u0027","commit_id":"c3ba0a4044edaa81f5556d28305c4ea1030bbe72"},{"author":{"_account_id":13252,"name":"Dr. Jens Harbott","display_name":"Jens Harbott (frickler)","email":"frickler@offenerstapel.de","username":"jrosenboom"},"change_message_id":"603d40cefef349118d92f90ef2ff6f2755d5bd18","unresolved":false,"context_lines":[{"line_number":11,"context_line":"        # override DEVSTACK_GATE_TEMPEST_REGEX to further refine the set of"},{"line_number":12,"context_line":"        # tests being run."},{"line_number":13,"context_line":"    required-projects:"},{"line_number":14,"context_line":"      - openstack/devstack-plugin-ceph"},{"line_number":15,"context_line":"    timeout: 7200"},{"line_number":16,"context_line":"    voting: false"},{"line_number":17,"context_line":"    vars:"}],"source_content_type":"text/x-yaml","patch_set":7,"id":"5f7c97a3_16dfff66","line":14,"in_reply_to":"5f7c97a3_e58acb18","updated":"2018-06-28 14:09:18.000000000","message":"Done","commit_id":"c3ba0a4044edaa81f5556d28305c4ea1030bbe72"},{"author":{"_account_id":10459,"name":"Luigi Toscano","email":"ltoscano@redhat.com","username":"ltoscano"},"change_message_id":"873f15620e443f0b488e771e2dac895027618c21","unresolved":false,"context_lines":[{"line_number":18,"context_line":"      devstack_plugins:"},{"line_number":19,"context_line":"        devstack-plugin-ceph: git://git.openstack.org/openstack/devstack-plugin-ceph"},{"line_number":20,"context_line":"      devstack_localrc:"},{"line_number":21,"context_line":"        ENABLE_FILE_INJECTION: false"},{"line_number":22,"context_line":"      tox_envlist: all"},{"line_number":23,"context_line":"      tempest_test_regex: |"},{"line_number":24,"context_line":"              (?x)(^tempest\\.(api|scenario)|(^cinder\\.tests.tempest))"}],"source_content_type":"text/x-yaml","patch_set":7,"id":"5f7c97a3_c59c6f72","line":21,"updated":"2018-06-28 12:32:00.000000000","message":"Is this really needed?","commit_id":"c3ba0a4044edaa81f5556d28305c4ea1030bbe72"},{"author":{"_account_id":13252,"name":"Dr. Jens Harbott","display_name":"Jens Harbott (frickler)","email":"frickler@offenerstapel.de","username":"jrosenboom"},"change_message_id":"d04d418a4f6280382e3a4f64683c0ae7700d2c2b","unresolved":false,"context_lines":[{"line_number":18,"context_line":"      devstack_plugins:"},{"line_number":19,"context_line":"        devstack-plugin-ceph: git://git.openstack.org/openstack/devstack-plugin-ceph"},{"line_number":20,"context_line":"      devstack_localrc:"},{"line_number":21,"context_line":"        ENABLE_FILE_INJECTION: false"},{"line_number":22,"context_line":"      tox_envlist: all"},{"line_number":23,"context_line":"      tempest_test_regex: |"},{"line_number":24,"context_line":"              (?x)(^tempest\\.(api|scenario)|(^cinder\\.tests.tempest))"}],"source_content_type":"text/x-yaml","patch_set":7,"id":"5f7c97a3_8a318df0","line":21,"in_reply_to":"5f7c97a3_bb9634a0","updated":"2018-06-29 06:02:50.000000000","message":"So this is indeed still needed, see\n\nhttp://logs.openstack.org/48/543048/9/check/devstack-plugin-ceph-tempest/49f1c85/testr_results.html.gz\n\nfor the failure mode when trying to run with file injection enabled.","commit_id":"c3ba0a4044edaa81f5556d28305c4ea1030bbe72"},{"author":{"_account_id":13252,"name":"Dr. Jens Harbott","display_name":"Jens Harbott (frickler)","email":"frickler@offenerstapel.de","username":"jrosenboom"},"change_message_id":"b9c35d75147dfd7671b6db958533deb3455d366e","unresolved":false,"context_lines":[{"line_number":18,"context_line":"      devstack_plugins:"},{"line_number":19,"context_line":"        devstack-plugin-ceph: git://git.openstack.org/openstack/devstack-plugin-ceph"},{"line_number":20,"context_line":"      devstack_localrc:"},{"line_number":21,"context_line":"        ENABLE_FILE_INJECTION: false"},{"line_number":22,"context_line":"      tox_envlist: all"},{"line_number":23,"context_line":"      tempest_test_regex: |"},{"line_number":24,"context_line":"              (?x)(^tempest\\.(api|scenario)|(^cinder\\.tests.tempest))"}],"source_content_type":"text/x-yaml","patch_set":7,"id":"5f7c97a3_bb9634a0","line":21,"in_reply_to":"5f7c97a3_c59c6f72","updated":"2018-06-28 13:34:39.000000000","message":"IIRC there was some issue with file injection on ceph, but it may also have been fixed in the meantime, we can test without it, too.","commit_id":"c3ba0a4044edaa81f5556d28305c4ea1030bbe72"},{"author":{"_account_id":10459,"name":"Luigi Toscano","email":"ltoscano@redhat.com","username":"ltoscano"},"change_message_id":"873f15620e443f0b488e771e2dac895027618c21","unresolved":false,"context_lines":[{"line_number":21,"context_line":"        ENABLE_FILE_INJECTION: false"},{"line_number":22,"context_line":"      tox_envlist: all"},{"line_number":23,"context_line":"      tempest_test_regex: |"},{"line_number":24,"context_line":"              (?x)(^tempest\\.(api|scenario)|(^cinder\\.tests.tempest))"},{"line_number":25,"context_line":"      tempest_test_blacklist: |"},{"line_number":26,"context_line":"              \"{{ ansible_user_dir }}/openstack/devstack-plugin-ceph/tempest_blacklist.txt\""},{"line_number":27,"context_line":""}],"source_content_type":"text/x-yaml","patch_set":7,"id":"5f7c97a3_85b19704","line":24,"updated":"2018-06-28 12:32:00.000000000","message":"The second part should be (^cinder_tempest_plugin), because the plugin was split out from the cinder repository.\n\nIf this is all on one line, do we still need (?x) ?","commit_id":"c3ba0a4044edaa81f5556d28305c4ea1030bbe72"},{"author":{"_account_id":13252,"name":"Dr. Jens Harbott","display_name":"Jens Harbott (frickler)","email":"frickler@offenerstapel.de","username":"jrosenboom"},"change_message_id":"603d40cefef349118d92f90ef2ff6f2755d5bd18","unresolved":false,"context_lines":[{"line_number":21,"context_line":"        ENABLE_FILE_INJECTION: false"},{"line_number":22,"context_line":"      tox_envlist: all"},{"line_number":23,"context_line":"      tempest_test_regex: |"},{"line_number":24,"context_line":"              (?x)(^tempest\\.(api|scenario)|(^cinder\\.tests.tempest))"},{"line_number":25,"context_line":"      tempest_test_blacklist: |"},{"line_number":26,"context_line":"              \"{{ ansible_user_dir }}/openstack/devstack-plugin-ceph/tempest_blacklist.txt\""},{"line_number":27,"context_line":""}],"source_content_type":"text/x-yaml","patch_set":7,"id":"5f7c97a3_96f30f0a","line":24,"in_reply_to":"5f7c97a3_3b8464b2","updated":"2018-06-28 14:09:18.000000000","message":"Oh, thanks. Maybe it breaks when the comment/newline happen inside a () block. But I think we are back to a simple enough regex that we can afford to omit the comments now anyway.","commit_id":"c3ba0a4044edaa81f5556d28305c4ea1030bbe72"},{"author":{"_account_id":13252,"name":"Dr. Jens Harbott","display_name":"Jens Harbott (frickler)","email":"frickler@offenerstapel.de","username":"jrosenboom"},"change_message_id":"b9c35d75147dfd7671b6db958533deb3455d366e","unresolved":false,"context_lines":[{"line_number":21,"context_line":"        ENABLE_FILE_INJECTION: false"},{"line_number":22,"context_line":"      tox_envlist: all"},{"line_number":23,"context_line":"      tempest_test_regex: |"},{"line_number":24,"context_line":"              (?x)(^tempest\\.(api|scenario)|(^cinder\\.tests.tempest))"},{"line_number":25,"context_line":"      tempest_test_blacklist: |"},{"line_number":26,"context_line":"              \"{{ ansible_user_dir }}/openstack/devstack-plugin-ceph/tempest_blacklist.txt\""},{"line_number":27,"context_line":""}],"source_content_type":"text/x-yaml","patch_set":7,"id":"5f7c97a3_db7690a4","line":24,"in_reply_to":"5f7c97a3_85b19704","updated":"2018-06-28 13:34:39.000000000","message":"I must admit that I don\u0027t understand the \"(?x)\" part yet, just copied what andreaf had there. Do you have some reference for this? Will fix the cinder part.","commit_id":"c3ba0a4044edaa81f5556d28305c4ea1030bbe72"},{"author":{"_account_id":10459,"name":"Luigi Toscano","email":"ltoscano@redhat.com","username":"ltoscano"},"change_message_id":"51401c853158fa0b369ffa9a321a7a1806799c60","unresolved":false,"context_lines":[{"line_number":21,"context_line":"        ENABLE_FILE_INJECTION: false"},{"line_number":22,"context_line":"      tox_envlist: all"},{"line_number":23,"context_line":"      tempest_test_regex: |"},{"line_number":24,"context_line":"              (?x)(^tempest\\.(api|scenario)|(^cinder\\.tests.tempest))"},{"line_number":25,"context_line":"      tempest_test_blacklist: |"},{"line_number":26,"context_line":"              \"{{ ansible_user_dir }}/openstack/devstack-plugin-ceph/tempest_blacklist.txt\""},{"line_number":27,"context_line":""}],"source_content_type":"text/x-yaml","patch_set":7,"id":"5f7c97a3_3b8464b2","line":24,"in_reply_to":"5f7c97a3_db7690a4","updated":"2018-06-28 13:55:23.000000000","message":"It comes from here:\nhttp://git.openstack.org/cgit/openstack/tempest/tree/roles/run-tempest/README.rst#n15\n\nTechnically it\u0027s the magic trick that would have allowed for the definition us to split the definition on multiple lines, as in the original patch.\nI\u0027m not sure why it did not work.\nA bit of history here:\nhttps://review.openstack.org/#/c/529624/\nBut apparently it was tested only with one line:\nhttps://review.openstack.org/#/c/529633/","commit_id":"c3ba0a4044edaa81f5556d28305c4ea1030bbe72"},{"author":{"_account_id":10459,"name":"Luigi Toscano","email":"ltoscano@redhat.com","username":"ltoscano"},"change_message_id":"873f15620e443f0b488e771e2dac895027618c21","unresolved":false,"context_lines":[{"line_number":27,"context_line":""},{"line_number":28,"context_line":"- job:"},{"line_number":29,"context_line":"    name: devstack-plugin-ceph-tempest-full-py3"},{"line_number":30,"context_line":"    parent: tempest-full-py3"},{"line_number":31,"context_line":"    description: |"},{"line_number":32,"context_line":"      Integration tests that runs with the ceph devstack plugin and py3."},{"line_number":33,"context_line":"      Former names for this job where:"}],"source_content_type":"text/x-yaml","patch_set":7,"id":"5f7c97a3_65df9b38","line":30,"updated":"2018-06-28 12:32:00.000000000","message":"Can\u0027t we make devstack-plugin-ceph-tempest-full-py3 inherit from devstack-plugin-ceph-tempest-full, and redefine only:\n\n      devstack_localrc:\n        USE_PYTHON3: \u0027True\u0027\n\nLess duplication...","commit_id":"c3ba0a4044edaa81f5556d28305c4ea1030bbe72"},{"author":{"_account_id":13252,"name":"Dr. Jens Harbott","display_name":"Jens Harbott (frickler)","email":"frickler@offenerstapel.de","username":"jrosenboom"},"change_message_id":"6e5290a6663ea63dbcc28afe36c79c03f650e2de","unresolved":false,"context_lines":[{"line_number":27,"context_line":""},{"line_number":28,"context_line":"- job:"},{"line_number":29,"context_line":"    name: devstack-plugin-ceph-tempest-full-py3"},{"line_number":30,"context_line":"    parent: tempest-full-py3"},{"line_number":31,"context_line":"    description: |"},{"line_number":32,"context_line":"      Integration tests that runs with the ceph devstack plugin and py3."},{"line_number":33,"context_line":"      Former names for this job where:"}],"source_content_type":"text/x-yaml","patch_set":7,"id":"5f7c97a3_ea436110","line":30,"in_reply_to":"5f7c97a3_3b3784e7","updated":"2018-06-29 06:13:37.000000000","message":"I did take a closer look and it looks like we would have to duplicate all the other settings from full-py3 like disabling swift which still is lacking py3 support. So until zuul supports multiple inheritance, we are forced to have at least some duplication and in this case I think it is better to have the duplication inside the same file, making it easier to keep things in sync.","commit_id":"c3ba0a4044edaa81f5556d28305c4ea1030bbe72"},{"author":{"_account_id":13252,"name":"Dr. Jens Harbott","display_name":"Jens Harbott (frickler)","email":"frickler@offenerstapel.de","username":"jrosenboom"},"change_message_id":"b9c35d75147dfd7671b6db958533deb3455d366e","unresolved":false,"context_lines":[{"line_number":27,"context_line":""},{"line_number":28,"context_line":"- job:"},{"line_number":29,"context_line":"    name: devstack-plugin-ceph-tempest-full-py3"},{"line_number":30,"context_line":"    parent: tempest-full-py3"},{"line_number":31,"context_line":"    description: |"},{"line_number":32,"context_line":"      Integration tests that runs with the ceph devstack plugin and py3."},{"line_number":33,"context_line":"      Former names for this job where:"}],"source_content_type":"text/x-yaml","patch_set":7,"id":"5f7c97a3_3b3784e7","line":30,"in_reply_to":"5f7c97a3_65df9b38","updated":"2018-06-28 13:34:39.000000000","message":"I will look at this once the first job is working, yes.","commit_id":"c3ba0a4044edaa81f5556d28305c4ea1030bbe72"},{"author":{"_account_id":13252,"name":"Dr. Jens Harbott","display_name":"Jens Harbott (frickler)","email":"frickler@offenerstapel.de","username":"jrosenboom"},"change_message_id":"33dcccda5a9b1bd2e1a0bcbffa2b8055de77dd09","unresolved":false,"context_lines":[{"line_number":14,"context_line":"      - openstack/cinder-tempest-plugin"},{"line_number":15,"context_line":"      - openstack/devstack-plugin-ceph"},{"line_number":16,"context_line":"    timeout: 7200"},{"line_number":17,"context_line":"    voting: false"},{"line_number":18,"context_line":"    vars:"},{"line_number":19,"context_line":"      devstack_plugins:"},{"line_number":20,"context_line":"        devstack-plugin-ceph: git://git.openstack.org/openstack/devstack-plugin-ceph"}],"source_content_type":"text/x-yaml","patch_set":10,"id":"5f7c97a3_215bbcf6","line":17,"updated":"2018-06-29 10:22:55.000000000","message":"I\u0027m wondering whether we really want to have this test non-voting everywhere. For this repo it might IMO make sense to have it voting, possibly also the -py3 variant. We could still provide a second project-template adding a non-voting options if consumers like nova want that.","commit_id":"579532481d67f7d34dfa28d708789d4c64d5a074"},{"author":{"_account_id":10459,"name":"Luigi Toscano","email":"ltoscano@redhat.com","username":"ltoscano"},"change_message_id":"9c63ea4282e9c25c2b50d91be9f48563ef02a2e9","unresolved":false,"context_lines":[{"line_number":14,"context_line":"      - openstack/cinder-tempest-plugin"},{"line_number":15,"context_line":"      - openstack/devstack-plugin-ceph"},{"line_number":16,"context_line":"    timeout: 7200"},{"line_number":17,"context_line":"    voting: false"},{"line_number":18,"context_line":"    vars:"},{"line_number":19,"context_line":"      devstack_plugins:"},{"line_number":20,"context_line":"        devstack-plugin-ceph: git://git.openstack.org/openstack/devstack-plugin-ceph"}],"source_content_type":"text/x-yaml","patch_set":10,"id":"5f7c97a3_41b838e7","line":17,"in_reply_to":"5f7c97a3_215bbcf6","updated":"2018-06-29 10:30:14.000000000","message":"I would vote (ahah) to remove voting: false from here. The test should be IMHO voting for this repository, other repositories will decide about it.","commit_id":"579532481d67f7d34dfa28d708789d4c64d5a074"},{"author":{"_account_id":10459,"name":"Luigi Toscano","email":"ltoscano@redhat.com","username":"ltoscano"},"change_message_id":"9c63ea4282e9c25c2b50d91be9f48563ef02a2e9","unresolved":false,"context_lines":[{"line_number":41,"context_line":"      - openstack/cinder-tempest-plugin"},{"line_number":42,"context_line":"      - openstack/devstack-plugin-ceph"},{"line_number":43,"context_line":"    timeout: 7200"},{"line_number":44,"context_line":"    voting: false"},{"line_number":45,"context_line":"    vars:"},{"line_number":46,"context_line":"      devstack_localrc:"},{"line_number":47,"context_line":"        ENABLE_FILE_INJECTION: false"}],"source_content_type":"text/x-yaml","patch_set":10,"id":"5f7c97a3_01b24007","line":44,"updated":"2018-06-29 10:30:14.000000000","message":"same here","commit_id":"579532481d67f7d34dfa28d708789d4c64d5a074"}],"tempest_blacklist.txt":[{"author":{"_account_id":13252,"name":"Dr. Jens Harbott","display_name":"Jens Harbott (frickler)","email":"frickler@offenerstapel.de","username":"jrosenboom"},"change_message_id":"33dcccda5a9b1bd2e1a0bcbffa2b8055de77dd09","unresolved":false,"context_lines":[{"line_number":1,"context_line":"# List of tests getting skipped"},{"line_number":2,"context_line":".*\\[.*\\bslow\\b.*\\]"}],"source_content_type":"text/plain","patch_set":10,"id":"5f7c97a3_013d608d","line":2,"updated":"2018-06-29 10:22:55.000000000","message":"This doesn\u0027t seem to work currently, I see lots of slow tests being executed. Will need to dig a bit more.","commit_id":"579532481d67f7d34dfa28d708789d4c64d5a074"},{"author":{"_account_id":10459,"name":"Luigi Toscano","email":"ltoscano@redhat.com","username":"ltoscano"},"change_message_id":"9c63ea4282e9c25c2b50d91be9f48563ef02a2e9","unresolved":false,"context_lines":[{"line_number":1,"context_line":"# List of tests getting skipped"},{"line_number":2,"context_line":".*\\[.*\\bslow\\b.*\\]"}],"source_content_type":"text/plain","patch_set":10,"id":"5f7c97a3_21b73cf8","line":2,"in_reply_to":"5f7c97a3_013d608d","updated":"2018-06-29 10:30:14.000000000","message":"Maybe when listed here it does not need some quoting?","commit_id":"579532481d67f7d34dfa28d708789d4c64d5a074"}]}