)]}'
{".zuul.yaml":[{"author":{"_account_id":11564,"name":"Chris Dent","email":"cdent@anticdent.org","username":"chdent"},"change_message_id":"729ceef3c6f6c24f52e3d9812fca9e99c6717ce4","unresolved":false,"context_lines":[{"line_number":2,"context_line":"# we get things working."},{"line_number":3,"context_line":"- project:"},{"line_number":4,"context_line":"    templates:"},{"line_number":5,"context_line":"      - integrated-gate-py35"},{"line_number":6,"context_line":"      - publish-openstack-docs-pti"},{"line_number":7,"context_line":"      - openstack-cover-jobs"},{"line_number":8,"context_line":"      - openstack-lower-constraints-jobs"}],"source_content_type":"text/x-yaml","patch_set":1,"id":"3f79a3b5_4a27f9cf","line":5,"updated":"2018-12-13 15:08:57.000000000","message":"this can perhaps go away since you\u0027ve overridden it below?","commit_id":"0414880b1f1cf0060a5dde673360f2d8c66b67fb"},{"author":{"_account_id":8556,"name":"Ghanshyam Maan","display_name":"Ghanshyam Maan","email":"gmaan.os14@gmail.com","username":"ghanshyam"},"change_message_id":"3adc7142c3be2488f7eed1f4f96c4efd7442f664","unresolved":false,"context_lines":[{"line_number":2,"context_line":"# we get things working."},{"line_number":3,"context_line":"- project:"},{"line_number":4,"context_line":"    templates:"},{"line_number":5,"context_line":"      - integrated-gate-py35"},{"line_number":6,"context_line":"      - publish-openstack-docs-pti"},{"line_number":7,"context_line":"      - openstack-cover-jobs"},{"line_number":8,"context_line":"      - openstack-lower-constraints-jobs"}],"source_content_type":"text/x-yaml","patch_set":1,"id":"3f79a3b5_5688f3c2","line":5,"in_reply_to":"3f79a3b5_4a27f9cf","updated":"2018-12-14 05:31:13.000000000","message":"i recommend not to remove the template. By adding integrated-gate* template :\n\n1. we can make sure all the integrated projects like nova, cinder, neutron, keystone, glance etc have same kind of integrated tests running on CI. It is something infra/QA can decide about integrated jobs running in their gate instead of project team itself.\n\n2. There are always chance to add/remove the integrated jobs there, in that case we do not need to change every project which has chance to miss some of them. Doing the changes in template in single place is always good idea.\n\n3. There are also few branch related variant on those jobs so doing it in centralized place in template is good idea.\n\nI agree that we do not have perfect way handle the irrelevant-files in template and that\u0027s why we need to add integrated jobs on project pipeline job list. But i see the template mentioned here and jobs in pipeline list for irrelevant-files are  both different purpose. Obviously we can remove the template if same jobs are listed down but with all 3 points i mentioned above is the reason I would not like to remove it.","commit_id":"0414880b1f1cf0060a5dde673360f2d8c66b67fb"},{"author":{"_account_id":11564,"name":"Chris Dent","email":"cdent@anticdent.org","username":"chdent"},"change_message_id":"729ceef3c6f6c24f52e3d9812fca9e99c6717ce4","unresolved":false,"context_lines":[{"line_number":17,"context_line":"        - placement-perfload:"},{"line_number":18,"context_line":"            voting: false"},{"line_number":19,"context_line":"        - tempest-full-py3:"},{"line_number":20,"context_line":"            irrelevant-files: \u0026gate-irrelevant-files"},{"line_number":21,"context_line":"              - ^api-.*$"},{"line_number":22,"context_line":"              - ^(test-|)requirements.txt$"},{"line_number":23,"context_line":"              - ^.*\\.rst$"}],"source_content_type":"text/x-yaml","patch_set":1,"id":"3f79a3b5_2d041713","line":20,"updated":"2018-12-13 15:08:57.000000000","message":"I\u0027m glad you named the anchor in this way, it would be unfortunate if people chose to use this anchor in non-gate places. If you feel like it, perhaps add a warning stating that?","commit_id":"0414880b1f1cf0060a5dde673360f2d8c66b67fb"},{"author":{"_account_id":8556,"name":"Ghanshyam Maan","display_name":"Ghanshyam Maan","email":"gmaan.os14@gmail.com","username":"ghanshyam"},"change_message_id":"3adc7142c3be2488f7eed1f4f96c4efd7442f664","unresolved":false,"context_lines":[{"line_number":17,"context_line":"        - placement-perfload:"},{"line_number":18,"context_line":"            voting: false"},{"line_number":19,"context_line":"        - tempest-full-py3:"},{"line_number":20,"context_line":"            irrelevant-files: \u0026gate-irrelevant-files"},{"line_number":21,"context_line":"              - ^api-.*$"},{"line_number":22,"context_line":"              - ^(test-|)requirements.txt$"},{"line_number":23,"context_line":"              - ^.*\\.rst$"}],"source_content_type":"text/x-yaml","patch_set":1,"id":"3f79a3b5_96552b23","line":20,"in_reply_to":"3f79a3b5_2d041713","updated":"2018-12-14 05:31:13.000000000","message":"Done","commit_id":"0414880b1f1cf0060a5dde673360f2d8c66b67fb"},{"author":{"_account_id":11564,"name":"Chris Dent","email":"cdent@anticdent.org","username":"chdent"},"change_message_id":"729ceef3c6f6c24f52e3d9812fca9e99c6717ce4","unresolved":false,"context_lines":[{"line_number":25,"context_line":"              - ^doc/.*$"},{"line_number":26,"context_line":"              - ^placement/tests/.*$"},{"line_number":27,"context_line":"              - ^releasenotes/.*$"},{"line_number":28,"context_line":"              - ^setup.cfg$"},{"line_number":29,"context_line":"              - ^tools/.*$"},{"line_number":30,"context_line":"              - ^tox.ini$"},{"line_number":31,"context_line":"        - grenade-py3:"}],"source_content_type":"text/x-yaml","patch_set":1,"id":"3f79a3b5_4a10595c","line":28,"updated":"2018-12-13 15:08:57.000000000","message":"while it is probably safe to ignore tox.ini, it doesn\u0027t seem safe to setup.cfg\n\nThings like entry points will be in there, which can come into play in devstack and grenade.","commit_id":"0414880b1f1cf0060a5dde673360f2d8c66b67fb"},{"author":{"_account_id":8556,"name":"Ghanshyam Maan","display_name":"Ghanshyam Maan","email":"gmaan.os14@gmail.com","username":"ghanshyam"},"change_message_id":"3adc7142c3be2488f7eed1f4f96c4efd7442f664","unresolved":false,"context_lines":[{"line_number":25,"context_line":"              - ^doc/.*$"},{"line_number":26,"context_line":"              - ^placement/tests/.*$"},{"line_number":27,"context_line":"              - ^releasenotes/.*$"},{"line_number":28,"context_line":"              - ^setup.cfg$"},{"line_number":29,"context_line":"              - ^tools/.*$"},{"line_number":30,"context_line":"              - ^tox.ini$"},{"line_number":31,"context_line":"        - grenade-py3:"}],"source_content_type":"text/x-yaml","patch_set":1,"id":"3f79a3b5_36b9b77a","line":28,"in_reply_to":"3f79a3b5_4a10595c","updated":"2018-12-14 05:31:13.000000000","message":"sure.","commit_id":"0414880b1f1cf0060a5dde673360f2d8c66b67fb"},{"author":{"_account_id":9708,"name":"Balazs Gibizer","display_name":"gibi","email":"gibizer@gmail.com","username":"gibi"},"change_message_id":"2282f4d6acefdd044b8d47f1acef983b64706500","unresolved":false,"context_lines":[{"line_number":26,"context_line":"            # jobs."},{"line_number":27,"context_line":"            irrelevant-files: \u0026gate-irrelevant-files"},{"line_number":28,"context_line":"              - ^api-.*$"},{"line_number":29,"context_line":"              - ^(test-|)requirements.txt$"},{"line_number":30,"context_line":"              - ^.*\\.rst$"},{"line_number":31,"context_line":"              - ^.git.*$"},{"line_number":32,"context_line":"              - ^doc/.*$"}],"source_content_type":"text/x-yaml","patch_set":2,"id":"3f79a3b5_46e7c2af","line":29,"updated":"2018-12-17 11:11:42.000000000","message":"it seems to me that tempest-full-py3 does use the test-requirements.txt from the placement repo so a change there could theoretically break the job run.\n\nhttp://logs.openstack.org/02/625002/2/check/tempest-full-py3/29085d1/job-output.txt.gz#_2018-12-14_05_57_46_270115","commit_id":"2ed5b2d9a6d66ae4a6a066621510d1112da5fef8"},{"author":{"_account_id":8556,"name":"Ghanshyam Maan","display_name":"Ghanshyam Maan","email":"gmaan.os14@gmail.com","username":"ghanshyam"},"change_message_id":"8d0c4582d57703be476c1a2ba8269fc67fc8790c","unresolved":false,"context_lines":[{"line_number":26,"context_line":"            # jobs."},{"line_number":27,"context_line":"            irrelevant-files: \u0026gate-irrelevant-files"},{"line_number":28,"context_line":"              - ^api-.*$"},{"line_number":29,"context_line":"              - ^(test-|)requirements.txt$"},{"line_number":30,"context_line":"              - ^.*\\.rst$"},{"line_number":31,"context_line":"              - ^.git.*$"},{"line_number":32,"context_line":"              - ^doc/.*$"}],"source_content_type":"text/x-yaml","patch_set":2,"id":"3f79a3b5_49afd57e","line":29,"in_reply_to":"3f79a3b5_46e7c2af","updated":"2018-12-17 11:43:00.000000000","message":"test-requirements and requirements can impact of testing and installation of service But that part is covered by functional/unittest jobs itself. \n\nExample - http://logs.openstack.org/02/625002/2/check/openstack-tox-py36/0b7c354/job-output.txt.gz#_2018-12-14_05_50_07_248896","commit_id":"2ed5b2d9a6d66ae4a6a066621510d1112da5fef8"},{"author":{"_account_id":9708,"name":"Balazs Gibizer","display_name":"gibi","email":"gibizer@gmail.com","username":"gibi"},"change_message_id":"cbc7b3eb70ffaffdf94db48b62b61eff18854087","unresolved":false,"context_lines":[{"line_number":26,"context_line":"            # jobs."},{"line_number":27,"context_line":"            irrelevant-files: \u0026gate-irrelevant-files"},{"line_number":28,"context_line":"              - ^api-.*$"},{"line_number":29,"context_line":"              - ^(test-|)requirements.txt$"},{"line_number":30,"context_line":"              - ^.*\\.rst$"},{"line_number":31,"context_line":"              - ^.git.*$"},{"line_number":32,"context_line":"              - ^doc/.*$"}],"source_content_type":"text/x-yaml","patch_set":2,"id":"3f79a3b5_d03912fa","line":29,"in_reply_to":"3f79a3b5_49afd57e","updated":"2018-12-19 09:15:47.000000000","message":"Do we assume that every dependency related code change are fully covered with unit and functional test? I\u0027m a bit hesitant to assume that. How frequently the requirements are changing? Do we spare a lot of CI cycle by not triggering tempest for these changes?","commit_id":"2ed5b2d9a6d66ae4a6a066621510d1112da5fef8"},{"author":{"_account_id":8556,"name":"Ghanshyam Maan","display_name":"Ghanshyam Maan","email":"gmaan.os14@gmail.com","username":"ghanshyam"},"change_message_id":"136002d67889a3f3524cdbebb02055f06c9d225c","unresolved":false,"context_lines":[{"line_number":26,"context_line":"            # jobs."},{"line_number":27,"context_line":"            irrelevant-files: \u0026gate-irrelevant-files"},{"line_number":28,"context_line":"              - ^api-.*$"},{"line_number":29,"context_line":"              - ^(test-|)requirements.txt$"},{"line_number":30,"context_line":"              - ^.*\\.rst$"},{"line_number":31,"context_line":"              - ^.git.*$"},{"line_number":32,"context_line":"              - ^doc/.*$"}],"source_content_type":"text/x-yaml","patch_set":2,"id":"bfdaf3ff_f291fd9a","line":29,"in_reply_to":"3f79a3b5_d03912fa","updated":"2019-01-15 08:39:50.000000000","message":"For dependency installable wise, unit/functional tests jobs are enough. But to cover all version updated code and scenario, I agree with you. To make sure integrated testing of your project is working fine on any dependency version update etc, it is good to run gate jobs for requirement file also.","commit_id":"2ed5b2d9a6d66ae4a6a066621510d1112da5fef8"},{"author":{"_account_id":9708,"name":"Balazs Gibizer","display_name":"gibi","email":"gibizer@gmail.com","username":"gibi"},"change_message_id":"2282f4d6acefdd044b8d47f1acef983b64706500","unresolved":false,"context_lines":[{"line_number":32,"context_line":"              - ^doc/.*$"},{"line_number":33,"context_line":"              - ^placement/tests/.*$"},{"line_number":34,"context_line":"              - ^releasenotes/.*$"},{"line_number":35,"context_line":"              - ^tools/.*$"},{"line_number":36,"context_line":"              - ^tox.ini$"},{"line_number":37,"context_line":"        - grenade-py3:"},{"line_number":38,"context_line":"            irrelevant-files: *gate-irrelevant-files"}],"source_content_type":"text/x-yaml","patch_set":2,"id":"3f79a3b5_26cba61c","line":35,"updated":"2018-12-17 11:11:42.000000000","message":"there are scripts for grenade there but not for tempest-full so it is OK.","commit_id":"2ed5b2d9a6d66ae4a6a066621510d1112da5fef8"}]}