)]}'
{".zuul.yaml":[{"author":{"_account_id":13252,"name":"Dr. Jens Harbott","display_name":"Jens Harbott (frickler)","email":"frickler@offenerstapel.de","username":"jrosenboom"},"change_message_id":"7cd1628d0f8062ef006f4fbedcd3455dea2b3e51","unresolved":false,"context_lines":[{"line_number":24,"context_line":"    vars:"},{"line_number":25,"context_line":"      tox_venvlist: full"},{"line_number":26,"context_line":"      devstack_localrc:"},{"line_number":27,"context_line":"        ENABLE_FILE_INJECTION: True"},{"line_number":28,"context_line":""},{"line_number":29,"context_line":"- job:"},{"line_number":30,"context_line":"    name: tempest-tox-plugin-sanity-check"}],"source_content_type":"text/x-yaml","patch_set":1,"id":"ff82abbf_2b18457f","line":27,"updated":"2017-11-23 06:53:08.000000000","message":"I don\u0027t know the history of this, but assuming that this is needed for some tests to succeed, what prevents us from making it the default in devstack?\n\nOr can we have a base job without this option and then a variant for the tests that need it?","commit_id":"b23794a351b580bbeb953e457c8ca3779a9ddd28"},{"author":{"_account_id":8556,"name":"Ghanshyam Maan","display_name":"Ghanshyam Maan","email":"gmaan.os14@gmail.com","username":"ghanshyam"},"change_message_id":"fa60b1e6018cb190783c41ceb732e3052362bbf4","unresolved":false,"context_lines":[{"line_number":111,"context_line":"              - ^playbooks/"},{"line_number":112,"context_line":"              - ^roles/"},{"line_number":113,"context_line":"              - ^.zuul.yaml$"},{"line_number":114,"context_line":"        - tempest-full:"},{"line_number":115,"context_line":"            voting: false"},{"line_number":116,"context_line":"            irrelevant-files:"},{"line_number":117,"context_line":"              - ^(test-|)requirements.txt$"}],"source_content_type":"text/x-yaml","patch_set":3,"id":"df87a7cf_23a0b2f4","line":114,"range":{"start_line":114,"start_character":8,"end_line":114,"end_character":23},"updated":"2017-12-04 04:06:17.000000000","message":"legacy-tempest-dsvm-neutron-full was used for gate pipeline also, we need to add gate pipeline also.","commit_id":"8e9e7f74baecf60c6e501fff47ee19c0e972d25e"},{"author":{"_account_id":1921,"name":"Andrea Frittoli","email":"andrea.frittoli@gmail.com","username":"andrea-frittoli"},"change_message_id":"a5658d16da187052c473f8086918eae84ccb636a","unresolved":false,"context_lines":[{"line_number":111,"context_line":"              - ^playbooks/"},{"line_number":112,"context_line":"              - ^roles/"},{"line_number":113,"context_line":"              - ^.zuul.yaml$"},{"line_number":114,"context_line":"        - tempest-full:"},{"line_number":115,"context_line":"            voting: false"},{"line_number":116,"context_line":"            irrelevant-files:"},{"line_number":117,"context_line":"              - ^(test-|)requirements.txt$"}],"source_content_type":"text/x-yaml","patch_set":3,"id":"df87a7cf_497dd0af","line":114,"range":{"start_line":114,"start_character":8,"end_line":114,"end_character":23},"in_reply_to":"df87a7cf_23a0b2f4","updated":"2017-12-04 10:27:49.000000000","message":"It will be in gate once it\u0027s ready to replace its legacy counterpart.","commit_id":"8e9e7f74baecf60c6e501fff47ee19c0e972d25e"},{"author":{"_account_id":8556,"name":"Ghanshyam Maan","display_name":"Ghanshyam Maan","email":"gmaan.os14@gmail.com","username":"ghanshyam"},"change_message_id":"6b575f71cb6f1c7d09dd1f7eb9dd96bc403b8b34","unresolved":false,"context_lines":[{"line_number":111,"context_line":"              - ^playbooks/"},{"line_number":112,"context_line":"              - ^roles/"},{"line_number":113,"context_line":"              - ^.zuul.yaml$"},{"line_number":114,"context_line":"        - tempest-full:"},{"line_number":115,"context_line":"            voting: false"},{"line_number":116,"context_line":"            irrelevant-files:"},{"line_number":117,"context_line":"              - ^(test-|)requirements.txt$"}],"source_content_type":"text/x-yaml","patch_set":3,"id":"df87a7cf_de0d0291","line":114,"range":{"start_line":114,"start_character":8,"end_line":114,"end_character":23},"in_reply_to":"df87a7cf_497dd0af","updated":"2017-12-05 07:42:55.000000000","message":"ok","commit_id":"8e9e7f74baecf60c6e501fff47ee19c0e972d25e"},{"author":{"_account_id":8556,"name":"Ghanshyam Maan","display_name":"Ghanshyam Maan","email":"gmaan.os14@gmail.com","username":"ghanshyam"},"change_message_id":"fa60b1e6018cb190783c41ceb732e3052362bbf4","unresolved":false,"context_lines":[{"line_number":112,"context_line":"              - ^roles/"},{"line_number":113,"context_line":"              - ^.zuul.yaml$"},{"line_number":114,"context_line":"        - tempest-full:"},{"line_number":115,"context_line":"            voting: false"},{"line_number":116,"context_line":"            irrelevant-files:"},{"line_number":117,"context_line":"              - ^(test-|)requirements.txt$"},{"line_number":118,"context_line":"              - ^.*\\.rst$"}],"source_content_type":"text/x-yaml","patch_set":3,"id":"df87a7cf_237bd28c","line":115,"range":{"start_line":115,"start_character":12,"end_line":115,"end_character":25},"updated":"2017-12-04 04:06:17.000000000","message":"legacy-tempest-dsvm-neutron-full was not non voting","commit_id":"8e9e7f74baecf60c6e501fff47ee19c0e972d25e"},{"author":{"_account_id":1921,"name":"Andrea Frittoli","email":"andrea.frittoli@gmail.com","username":"andrea-frittoli"},"change_message_id":"a5658d16da187052c473f8086918eae84ccb636a","unresolved":false,"context_lines":[{"line_number":112,"context_line":"              - ^roles/"},{"line_number":113,"context_line":"              - ^.zuul.yaml$"},{"line_number":114,"context_line":"        - tempest-full:"},{"line_number":115,"context_line":"            voting: false"},{"line_number":116,"context_line":"            irrelevant-files:"},{"line_number":117,"context_line":"              - ^(test-|)requirements.txt$"},{"line_number":118,"context_line":"              - ^.*\\.rst$"}],"source_content_type":"text/x-yaml","patch_set":3,"id":"df87a7cf_c94a407f","line":115,"range":{"start_line":115,"start_character":12,"end_line":115,"end_character":25},"in_reply_to":"df87a7cf_237bd28c","updated":"2017-12-04 10:27:49.000000000","message":"It will be voting once it\u0027s ready.","commit_id":"8e9e7f74baecf60c6e501fff47ee19c0e972d25e"},{"author":{"_account_id":8556,"name":"Ghanshyam Maan","display_name":"Ghanshyam Maan","email":"gmaan.os14@gmail.com","username":"ghanshyam"},"change_message_id":"6b575f71cb6f1c7d09dd1f7eb9dd96bc403b8b34","unresolved":false,"context_lines":[{"line_number":112,"context_line":"              - ^roles/"},{"line_number":113,"context_line":"              - ^.zuul.yaml$"},{"line_number":114,"context_line":"        - tempest-full:"},{"line_number":115,"context_line":"            voting: false"},{"line_number":116,"context_line":"            irrelevant-files:"},{"line_number":117,"context_line":"              - ^(test-|)requirements.txt$"},{"line_number":118,"context_line":"              - ^.*\\.rst$"}],"source_content_type":"text/x-yaml","patch_set":3,"id":"df87a7cf_fe1246b1","line":115,"range":{"start_line":115,"start_character":12,"end_line":115,"end_character":25},"in_reply_to":"df87a7cf_c94a407f","updated":"2017-12-05 07:42:55.000000000","message":"ok","commit_id":"8e9e7f74baecf60c6e501fff47ee19c0e972d25e"},{"author":{"_account_id":8556,"name":"Ghanshyam Maan","display_name":"Ghanshyam Maan","email":"gmaan.os14@gmail.com","username":"ghanshyam"},"change_message_id":"fa60b1e6018cb190783c41ceb732e3052362bbf4","unresolved":false,"context_lines":[{"line_number":114,"context_line":"        - tempest-full:"},{"line_number":115,"context_line":"            voting: false"},{"line_number":116,"context_line":"            irrelevant-files:"},{"line_number":117,"context_line":"              - ^(test-|)requirements.txt$"},{"line_number":118,"context_line":"              - ^.*\\.rst$"},{"line_number":119,"context_line":"              - ^doc/.*$"},{"line_number":120,"context_line":"              - ^etc/.*$"},{"line_number":121,"context_line":"              - ^releasenotes/.*$"},{"line_number":122,"context_line":"              - ^setup.cfg$"},{"line_number":123,"context_line":"              - ^tempest/hacking/.*$"},{"line_number":124,"context_line":"              - ^tempest/tests/.*$"},{"line_number":125,"context_line":"        - tempest-tox-plugin-sanity-check"}],"source_content_type":"text/x-yaml","patch_set":3,"id":"df87a7cf_a36602ed","line":124,"range":{"start_line":117,"start_character":0,"end_line":124,"end_character":34},"updated":"2017-12-04 04:06:17.000000000","message":"can we move this to job definition, which is easy to find the complete definition of job. moving previous job in https://review.openstack.org/#/c/525038/1","commit_id":"8e9e7f74baecf60c6e501fff47ee19c0e972d25e"},{"author":{"_account_id":10459,"name":"Luigi Toscano","email":"ltoscano@redhat.com","username":"ltoscano"},"change_message_id":"480b458bc6b183b8a9acd7047a097e136eb02f6e","unresolved":false,"context_lines":[{"line_number":114,"context_line":"        - tempest-full:"},{"line_number":115,"context_line":"            voting: false"},{"line_number":116,"context_line":"            irrelevant-files:"},{"line_number":117,"context_line":"              - ^(test-|)requirements.txt$"},{"line_number":118,"context_line":"              - ^.*\\.rst$"},{"line_number":119,"context_line":"              - ^doc/.*$"},{"line_number":120,"context_line":"              - ^etc/.*$"},{"line_number":121,"context_line":"              - ^releasenotes/.*$"},{"line_number":122,"context_line":"              - ^setup.cfg$"},{"line_number":123,"context_line":"              - ^tempest/hacking/.*$"},{"line_number":124,"context_line":"              - ^tempest/tests/.*$"},{"line_number":125,"context_line":"        - tempest-tox-plugin-sanity-check"}],"source_content_type":"text/x-yaml","patch_set":3,"id":"df87a7cf_38dbd8ed","line":124,"range":{"start_line":117,"start_character":0,"end_line":124,"end_character":34},"in_reply_to":"df87a7cf_280e2b3b","updated":"2017-12-05 10:28:03.000000000","message":"Fine for rst, doc, releasenotes, and maybe etc, but are you sure that you don\u0027t want to re-run the integration tests when the requirements change or setup.cfg changes?","commit_id":"8e9e7f74baecf60c6e501fff47ee19c0e972d25e"},{"author":{"_account_id":8556,"name":"Ghanshyam Maan","display_name":"Ghanshyam Maan","email":"gmaan.os14@gmail.com","username":"ghanshyam"},"change_message_id":"e758c244ea241bb634cd35075acf033bf9765630","unresolved":false,"context_lines":[{"line_number":114,"context_line":"        - tempest-full:"},{"line_number":115,"context_line":"            voting: false"},{"line_number":116,"context_line":"            irrelevant-files:"},{"line_number":117,"context_line":"              - ^(test-|)requirements.txt$"},{"line_number":118,"context_line":"              - ^.*\\.rst$"},{"line_number":119,"context_line":"              - ^doc/.*$"},{"line_number":120,"context_line":"              - ^etc/.*$"},{"line_number":121,"context_line":"              - ^releasenotes/.*$"},{"line_number":122,"context_line":"              - ^setup.cfg$"},{"line_number":123,"context_line":"              - ^tempest/hacking/.*$"},{"line_number":124,"context_line":"              - ^tempest/tests/.*$"},{"line_number":125,"context_line":"        - tempest-tox-plugin-sanity-check"}],"source_content_type":"text/x-yaml","patch_set":3,"id":"df87a7cf_bbd2825b","line":124,"range":{"start_line":117,"start_character":0,"end_line":124,"end_character":34},"in_reply_to":"df87a7cf_280e2b3b","updated":"2017-12-05 11:17:17.000000000","message":"I see another point where it can leads to have \u0027files\u0027 and \u0027irrelevant-files\u0027 both in same job which is wrong so i agree with you not to put these in job definition. \n\nBase job like this can leave files related things and concrete job should be able to make it in job definition.","commit_id":"8e9e7f74baecf60c6e501fff47ee19c0e972d25e"},{"author":{"_account_id":8556,"name":"Ghanshyam Maan","display_name":"Ghanshyam Maan","email":"gmaan.os14@gmail.com","username":"ghanshyam"},"change_message_id":"e758c244ea241bb634cd35075acf033bf9765630","unresolved":false,"context_lines":[{"line_number":114,"context_line":"        - tempest-full:"},{"line_number":115,"context_line":"            voting: false"},{"line_number":116,"context_line":"            irrelevant-files:"},{"line_number":117,"context_line":"              - ^(test-|)requirements.txt$"},{"line_number":118,"context_line":"              - ^.*\\.rst$"},{"line_number":119,"context_line":"              - ^doc/.*$"},{"line_number":120,"context_line":"              - ^etc/.*$"},{"line_number":121,"context_line":"              - ^releasenotes/.*$"},{"line_number":122,"context_line":"              - ^setup.cfg$"},{"line_number":123,"context_line":"              - ^tempest/hacking/.*$"},{"line_number":124,"context_line":"              - ^tempest/tests/.*$"},{"line_number":125,"context_line":"        - tempest-tox-plugin-sanity-check"}],"source_content_type":"text/x-yaml","patch_set":3,"id":"df87a7cf_5b46d623","line":124,"range":{"start_line":117,"start_character":0,"end_line":124,"end_character":34},"in_reply_to":"df87a7cf_38dbd8ed","updated":"2017-12-05 11:17:17.000000000","message":"most of the lib are testing integration test as part of \u0027lib-forward-testing\u0027 so requirement files should be ok to skip. setup.cfg is not tested by integration test do should be ok.","commit_id":"8e9e7f74baecf60c6e501fff47ee19c0e972d25e"},{"author":{"_account_id":1921,"name":"Andrea Frittoli","email":"andrea.frittoli@gmail.com","username":"andrea-frittoli"},"change_message_id":"a5658d16da187052c473f8086918eae84ccb636a","unresolved":false,"context_lines":[{"line_number":114,"context_line":"        - tempest-full:"},{"line_number":115,"context_line":"            voting: false"},{"line_number":116,"context_line":"            irrelevant-files:"},{"line_number":117,"context_line":"              - ^(test-|)requirements.txt$"},{"line_number":118,"context_line":"              - ^.*\\.rst$"},{"line_number":119,"context_line":"              - ^doc/.*$"},{"line_number":120,"context_line":"              - ^etc/.*$"},{"line_number":121,"context_line":"              - ^releasenotes/.*$"},{"line_number":122,"context_line":"              - ^setup.cfg$"},{"line_number":123,"context_line":"              - ^tempest/hacking/.*$"},{"line_number":124,"context_line":"              - ^tempest/tests/.*$"},{"line_number":125,"context_line":"        - tempest-tox-plugin-sanity-check"}],"source_content_type":"text/x-yaml","patch_set":3,"id":"df87a7cf_e930640a","line":124,"range":{"start_line":117,"start_character":0,"end_line":124,"end_character":34},"in_reply_to":"df87a7cf_a36602ed","updated":"2017-12-04 10:27:49.000000000","message":"I\u0027d rather not. The job definition is going to be used by many different projects, and the irrelevant-files section is a project specific decision as it depends on the structure of their repo.","commit_id":"8e9e7f74baecf60c6e501fff47ee19c0e972d25e"},{"author":{"_account_id":8556,"name":"Ghanshyam Maan","display_name":"Ghanshyam Maan","email":"gmaan.os14@gmail.com","username":"ghanshyam"},"change_message_id":"6b575f71cb6f1c7d09dd1f7eb9dd96bc403b8b34","unresolved":false,"context_lines":[{"line_number":114,"context_line":"        - tempest-full:"},{"line_number":115,"context_line":"            voting: false"},{"line_number":116,"context_line":"            irrelevant-files:"},{"line_number":117,"context_line":"              - ^(test-|)requirements.txt$"},{"line_number":118,"context_line":"              - ^.*\\.rst$"},{"line_number":119,"context_line":"              - ^doc/.*$"},{"line_number":120,"context_line":"              - ^etc/.*$"},{"line_number":121,"context_line":"              - ^releasenotes/.*$"},{"line_number":122,"context_line":"              - ^setup.cfg$"},{"line_number":123,"context_line":"              - ^tempest/hacking/.*$"},{"line_number":124,"context_line":"              - ^tempest/tests/.*$"},{"line_number":125,"context_line":"        - tempest-tox-plugin-sanity-check"}],"source_content_type":"text/x-yaml","patch_set":3,"id":"df87a7cf_280e2b3b","line":124,"range":{"start_line":117,"start_character":0,"end_line":124,"end_character":34},"in_reply_to":"df87a7cf_e930640a","updated":"2017-12-05 07:42:55.000000000","message":"actually there are below files for which none of the project want to run integration test.\n\n              - ^(test-|)requirements.txt$\n              - ^.*\\.rst$\n              - ^doc/.*$\n              - ^etc/.*$\n              - ^releasenotes/.*$\n              - ^setup.cfg$\n\nIf this var can be appended then we can define those in job definition to avoid duplicating in all projects. Checking with infra if there is possibility.","commit_id":"8e9e7f74baecf60c6e501fff47ee19c0e972d25e"}]}