)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":16643,"name":"Goutham Pacha Ravi","email":"gouthampravi@gmail.com","username":"gouthamr"},"change_message_id":"9fda1e1506cb756e8a78e038c0962ae217b6ee13","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":2,"id":"7abb9a99_ebf6338d","updated":"2022-07-18 11:03:37.000000000","message":"Hey Douglas, am i getting the syntax of job variants wrong with this change? ","commit_id":"a40b395cc6a18f63c1a05feb941d58f2e456d24d"},{"author":{"_account_id":30002,"name":"Douglas Viroel","email":"viroel@gmail.com","username":"dviroel"},"change_message_id":"77c8f7edd2f590300819839860e4b12907ef857a","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"2c0fb211_9b1b4246","updated":"2022-08-03 20:04:59.000000000","message":"LGTM, depends-on merged.\nUssuri won\u0027t be affected since this job doesn\u0027t run there.","commit_id":"9ac47cbab87b03e960f492b4a3a030eb068e9f23"},{"author":{"_account_id":29632,"name":"Carlos Eduardo","email":"ces.eduardo98@gmail.com","username":"silvacarlos"},"change_message_id":"008d13eaaa9d1aa2a5bed6e5072b4a36ac7e7a3e","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"d1e8932a_ffe23a6a","updated":"2022-07-28 20:11:42.000000000","message":"Looking good, thanks! I think we should merge the Ussuri change first, but I\u0027m on it :)","commit_id":"9ac47cbab87b03e960f492b4a3a030eb068e9f23"},{"author":{"_account_id":16643,"name":"Goutham Pacha Ravi","email":"gouthampravi@gmail.com","username":"gouthamr"},"change_message_id":"a08a4ce0f591c1ea8a2449475f99dd4207e9f5b6","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"6e73d2b6_6107f673","updated":"2022-07-19 18:45:24.000000000","message":"recheck","commit_id":"9ac47cbab87b03e960f492b4a3a030eb068e9f23"}],"zuul.d/manila-tempest-jobs.yaml":[{"author":{"_account_id":30002,"name":"Douglas Viroel","email":"viroel@gmail.com","username":"dviroel"},"change_message_id":"1b9ace8037bd469935128502440b3321e4a36649","unresolved":true,"context_lines":[{"line_number":64,"context_line":"      tempest_exclude_regex: \"(^manila_tempest_tests.tests.scenario.*IPv6.*)\""},{"line_number":65,"context_line":""},{"line_number":66,"context_line":"- job:"},{"line_number":67,"context_line":"    name: manila-tempest-plugin-base"},{"line_number":68,"context_line":"    abstract: true"},{"line_number":69,"context_line":"    branches: stable/victoria"},{"line_number":70,"context_line":"    description: Base job for devstack/tempest based manila jobs on stable/victoria."}],"source_content_type":"text/x-yaml","patch_set":2,"id":"6a0e6d48_f408fac0","line":67,"range":{"start_line":67,"start_character":10,"end_line":67,"end_character":36},"updated":"2022-07-18 16:30:22.000000000","message":"It won\u0027t work in job definition, you will need to define a new job with a different name. Zuul allow us to declare the same job multiple times and supports multi-parenting, so in this way, this definition is extending the original one.\n\nMaybe the easiest way is to create a new job for each one that runs on stable/victoria and just set project override:\n\n- job:\n    name: manila-tempest-plugin-generic-scenario-stable-victoria\n    description: |\n      Test the scenario test cases on the generic driver multibackend\n      (DHSS\u003dTrue) with NFS and CIFS\n    parent: manila-tempest-plugin-generic-scenario-base\n    branches: ^(stable/victoruia).*$\n    required-projects: *manila-tempest-required-projects\n      - name: openstack/manila-tempest-plugin\n        override-checkout: 1.8.0\n    vars: \u0026manila_tempest_image_pinned_vars\n      devstack_localrc:\n        # NOTE(carloss): Pinning manila service image to a Focal version,\n        # since on Zed we moved to Ubuntu Jammy (22), and it requires more\n        # VM resources.\n        MANILA_SERVICE_IMAGE_URL: https://tarballs.opendev.org/openstack/manila-image-elements/images/manila-service-image-1.3.0-76-ga216835.qcow2\n        MANILA_SERVICE_IMAGE_NAME: manila-service-image-1.3.0-76-ga216835\n\n\nIt can\u0027t be parent of manila-tempest-plugin-generic-scenario-stable, because there is a branch definition there, and zuul will apply all branch filters, from the job and their parents.","commit_id":"a40b395cc6a18f63c1a05feb941d58f2e456d24d"},{"author":{"_account_id":16643,"name":"Goutham Pacha Ravi","email":"gouthampravi@gmail.com","username":"gouthamr"},"change_message_id":"4a3d145654d13bacd0bc2c6d9781590d0093fc68","unresolved":true,"context_lines":[{"line_number":64,"context_line":"      tempest_exclude_regex: \"(^manila_tempest_tests.tests.scenario.*IPv6.*)\""},{"line_number":65,"context_line":""},{"line_number":66,"context_line":"- job:"},{"line_number":67,"context_line":"    name: manila-tempest-plugin-base"},{"line_number":68,"context_line":"    abstract: true"},{"line_number":69,"context_line":"    branches: stable/victoria"},{"line_number":70,"context_line":"    description: Base job for devstack/tempest based manila jobs on stable/victoria."}],"source_content_type":"text/x-yaml","patch_set":2,"id":"b1e2b3bc_e89f38b5","line":67,"range":{"start_line":67,"start_character":10,"end_line":67,"end_character":36},"in_reply_to":"6a0e6d48_f408fac0","updated":"2022-07-20 14:56:16.000000000","message":"Thanks for explaining this; it makes sense.. So, we\u0027re at a juncture where it makes no sense to have stable/victoria jobs in-tree here (because we\u0027re pinning the tempest version required). I cloned the jobs that we need onto the stable/victoria branch in manila: https://review.opendev.org/c/openstack/manila/+/850415 \n\nPTAL, and let me know what you think","commit_id":"a40b395cc6a18f63c1a05feb941d58f2e456d24d"},{"author":{"_account_id":16643,"name":"Goutham Pacha Ravi","email":"gouthampravi@gmail.com","username":"gouthamr"},"change_message_id":"f8dfca0bd4d3732a365f850266033df8ff1b6ac6","unresolved":false,"context_lines":[{"line_number":64,"context_line":"      tempest_exclude_regex: \"(^manila_tempest_tests.tests.scenario.*IPv6.*)\""},{"line_number":65,"context_line":""},{"line_number":66,"context_line":"- job:"},{"line_number":67,"context_line":"    name: manila-tempest-plugin-base"},{"line_number":68,"context_line":"    abstract: true"},{"line_number":69,"context_line":"    branches: stable/victoria"},{"line_number":70,"context_line":"    description: Base job for devstack/tempest based manila jobs on stable/victoria."}],"source_content_type":"text/x-yaml","patch_set":2,"id":"8189c554_45692966","line":67,"range":{"start_line":67,"start_character":10,"end_line":67,"end_character":36},"in_reply_to":"b1e2b3bc_e89f38b5","updated":"2022-07-20 14:56:29.000000000","message":"Done","commit_id":"a40b395cc6a18f63c1a05feb941d58f2e456d24d"}]}
