)]}'
{".zuul.yaml":[{"author":{"_account_id":10459,"name":"Luigi Toscano","email":"ltoscano@redhat.com","username":"ltoscano"},"change_message_id":"e759085a13237d28f65b5a7b2cdb4b2d5de34b00","unresolved":false,"context_lines":[{"line_number":9,"context_line":"    gate:"},{"line_number":10,"context_line":"      jobs:"},{"line_number":11,"context_line":"        - cinder-tempest-plugin-lvm-lio-barbican"},{"line_number":12,"context_line":"        - devstack-plugin-ceph-tempest-py3"},{"line_number":13,"context_line":""},{"line_number":14,"context_line":"- job:"},{"line_number":15,"context_line":"    name: cinder-tempest-plugin-lvm-lio-barbican"}],"source_content_type":"text/x-yaml","patch_set":1,"id":"df33271e_14507dd4","line":12,"updated":"2020-04-03 23:29:34.000000000","message":"Maybe we can have a local job which inherits from devstack-plugin-ceph-tempest-py3 so that we can test with c-bak without enabling it for all the other users of that job.","commit_id":"3470bcb4b0da028f5f741d1beaa8c362325da954"},{"author":{"_account_id":10459,"name":"Luigi Toscano","email":"ltoscano@redhat.com","username":"ltoscano"},"change_message_id":"149888d7c0b335e79382ac139518b2c63d8db493","unresolved":false,"context_lines":[{"line_number":10,"context_line":"      jobs:"},{"line_number":11,"context_line":"        - cinder-tempest-plugin-lvm-lio-barbican"},{"line_number":12,"context_line":"        - devstack-plugin-ceph-tempest-py3"},{"line_number":13,"context_line":"        - devstack-plugin-ceph-tempest-py3-bak"},{"line_number":14,"context_line":""},{"line_number":15,"context_line":"- job:"},{"line_number":16,"context_line":"    name: cinder-tempest-plugin-lvm-lio-barbican"}],"source_content_type":"text/x-yaml","patch_set":3,"id":"df33271e_43e364a5","line":13,"updated":"2020-04-04 17:30:02.000000000","message":"It should be in the check job too, otherwise it can\u0027t be tested.\n\nEven better, it can replace devstack-plugin-ceph-tempest-py3 completely.\n\nFinally, as the job name depend on the repository it is defined in, it should be something like:\n\ncinder-tempest-plugin-ceph-cbak\ncinder-tempest-plugin-cbak-ceph\n\nor something else like that","commit_id":"0bdb1b311b34efc542d9d7bf3ad923acd9ab2103"},{"author":{"_account_id":10459,"name":"Luigi Toscano","email":"ltoscano@redhat.com","username":"ltoscano"},"change_message_id":"67cfaaa6654afc088269d67dcc8f73d27ef14faf","unresolved":false,"context_lines":[{"line_number":61,"context_line":"    branches: ^(?!stable/(ocata|pike|queens|rocky)).*$"},{"line_number":62,"context_line":"    vars:"},{"line_number":63,"context_line":"      devstack_services:"},{"line_number":64,"context_line":"        c-back: true"}],"source_content_type":"text/x-yaml","patch_set":4,"id":"df33271e_c4cf7a27","line":64,"updated":"2020-04-04 17:47:16.000000000","message":"c-bak","commit_id":"245f7187c7bd8c1778f294b6e7e9c9f1a4e3c7a5"},{"author":{"_account_id":27615,"name":"Rajat Dhasmana","email":"rajatdhasmana@gmail.com","username":"whoami-rajat"},"change_message_id":"fb1df68549d9fd75460c8ad804731cffeba93b9d","unresolved":false,"context_lines":[{"line_number":42,"context_line":"        CINDER_COORDINATION_URL: \u0027file://\\$state_path\u0027"},{"line_number":43,"context_line":"      devstack_services:"},{"line_number":44,"context_line":"        barbican: true"},{"line_number":45,"context_line":"        c-bak: true"},{"line_number":46,"context_line":"      tempest_plugins:"},{"line_number":47,"context_line":"        - cinder-tempest-plugin"},{"line_number":48,"context_line":"      fetch_subunit_output_additional_dirs:"}],"source_content_type":"text/x-yaml","patch_set":5,"id":"df33271e_5f3162ec","line":45,"range":{"start_line":45,"start_character":8,"end_line":45,"end_character":19},"updated":"2020-04-06 11:06:19.000000000","message":"The base job (devstack-minimal) already has c-bak enabled[1]\nand even without this patch, c-bak enabled in this job[2] (as it wasn\u0027t disabled by any of the parent jobs specifically devstack-tempest)\n\n[1] https://github.com/openstack/devstack/blob/master/.zuul.yaml#L431\n[2] https://zuul.opendev.org/t/openstack/build/3f5c2f02fcb349eca1a4cf826209cb16/log/controller/logs/devstacklog.txt#11919-11920","commit_id":"f7cb5a19d97b95a2b514277456085200d82d3406"},{"author":{"_account_id":27615,"name":"Rajat Dhasmana","email":"rajatdhasmana@gmail.com","username":"whoami-rajat"},"change_message_id":"fb1df68549d9fd75460c8ad804731cffeba93b9d","unresolved":false,"context_lines":[{"line_number":61,"context_line":"    branches: ^(?!stable/(ocata|pike|queens|rocky)).*$"},{"line_number":62,"context_line":"    vars:"},{"line_number":63,"context_line":"      devstack_services:"},{"line_number":64,"context_line":"        c-bak: true"}],"source_content_type":"text/x-yaml","patch_set":5,"id":"df33271e_ffcaf69e","line":64,"range":{"start_line":64,"start_character":8,"end_line":64,"end_character":19},"updated":"2020-04-06 11:06:19.000000000","message":"This has a different story, this job was derived from tempest-full-py3[1] (instead of devstack-tempest as the above job) which explicitly disables c-bak so we need to enable it here.\n\n[1] https://opendev.org/openstack/tempest/src/branch/master/.zuul.yaml#L192","commit_id":"f7cb5a19d97b95a2b514277456085200d82d3406"},{"author":{"_account_id":10459,"name":"Luigi Toscano","email":"ltoscano@redhat.com","username":"ltoscano"},"change_message_id":"47012b69b3dfb4b29c33d3bb0a16a4b32e701930","unresolved":false,"context_lines":[{"line_number":57,"context_line":"    description: |"},{"line_number":58,"context_line":"      Integration tests that runs with the ceph devstack plugin, py3"},{"line_number":59,"context_line":"      and enable the backup service."},{"line_number":60,"context_line":"    branches: ^(?!stable/(ocata|pike|queens|rocky)).*$"},{"line_number":61,"context_line":"    vars:"},{"line_number":62,"context_line":"      devstack_services:"},{"line_number":63,"context_line":"        c-bak: true"}],"source_content_type":"text/x-yaml","patch_set":6,"id":"df33271e_94b0e38d","line":60,"updated":"2020-04-06 21:23:37.000000000","message":"I\u0027ve just a question about this: do we need this rule if we are sure we won\u0027t use cinder-tempest-plugin-cbak-ceph on those branches? In fact this job is probably going to be used here only.","commit_id":"8f70c09bf85ccc995dfab808f8ac95fbd1a99bd6"}],"/COMMIT_MSG":[{"author":{"_account_id":27615,"name":"Rajat Dhasmana","email":"rajatdhasmana@gmail.com","username":"whoami-rajat"},"change_message_id":"cd11f75cf8832d92335274d4b3b25c6094c0ab24","unresolved":false,"context_lines":[{"line_number":7,"context_line":"Create new Ceph job with c-bak service enabled"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Currently backup service is disable in Ceph jobs."},{"line_number":10,"context_line":"In order to solve this cinder-tempest-plugin-cbak-ceph is"},{"line_number":11,"context_line":"defined with the c-bak service enabled."},{"line_number":12,"context_line":""},{"line_number":13,"context_line":"As a matter of fact the c-bak is redundant incase of lvm job"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":7,"id":"3f4c43b2_b774289b","line":10,"range":{"start_line":10,"start_character":22,"end_line":10,"end_character":23},"updated":"2020-04-13 15:35:33.000000000","message":"a new gate job","commit_id":"f44ff14f1a89732ab5890f124982555526c5bab5"},{"author":{"_account_id":27615,"name":"Rajat Dhasmana","email":"rajatdhasmana@gmail.com","username":"whoami-rajat"},"change_message_id":"cd11f75cf8832d92335274d4b3b25c6094c0ab24","unresolved":false,"context_lines":[{"line_number":10,"context_line":"In order to solve this cinder-tempest-plugin-cbak-ceph is"},{"line_number":11,"context_line":"defined with the c-bak service enabled."},{"line_number":12,"context_line":""},{"line_number":13,"context_line":"As a matter of fact the c-bak is redundant incase of lvm job"},{"line_number":14,"context_line":"because the parent job already has enabled the service."},{"line_number":15,"context_line":""},{"line_number":16,"context_line":"In addition, Tempest provides a blacklist file so that only"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":7,"id":"3f4c43b2_77ed8024","line":13,"range":{"start_line":13,"start_character":20,"end_line":13,"end_character":29},"updated":"2020-04-13 15:35:33.000000000","message":"enabling the c-bak service","commit_id":"f44ff14f1a89732ab5890f124982555526c5bab5"},{"author":{"_account_id":27615,"name":"Rajat Dhasmana","email":"rajatdhasmana@gmail.com","username":"whoami-rajat"},"change_message_id":"cd11f75cf8832d92335274d4b3b25c6094c0ab24","unresolved":false,"context_lines":[{"line_number":11,"context_line":"defined with the c-bak service enabled."},{"line_number":12,"context_line":""},{"line_number":13,"context_line":"As a matter of fact the c-bak is redundant incase of lvm job"},{"line_number":14,"context_line":"because the parent job already has enabled the service."},{"line_number":15,"context_line":""},{"line_number":16,"context_line":"In addition, Tempest provides a blacklist file so that only"},{"line_number":17,"context_line":"storage-relevant tests are executed. Use it instead of the"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":7,"id":"3f4c43b2_57c96476","line":14,"range":{"start_line":14,"start_character":12,"end_line":14,"end_character":23},"updated":"2020-04-13 15:35:33.000000000","message":"parent job (devstack-minimal)","commit_id":"f44ff14f1a89732ab5890f124982555526c5bab5"}]}