)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":8449,"name":"Marios Andreou","email":"marios.andreou@gmail.com","username":"marios"},"change_message_id":"72f7eafd95e0fae8f428ad0e995514bef170291a","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"ec0518fc_78775216","updated":"2022-02-25 07:17:37.000000000","message":"cant spot the nits ;)","commit_id":"faf999e5c3dc3c4b25cfe2b6407f8a5770d49587"}],"zuul.d/content-provider.yaml":[{"author":{"_account_id":8449,"name":"Marios Andreou","email":"marios.andreou@gmail.com","username":"marios"},"change_message_id":"976bdfbfb326281c617d167405ac1d7740a2559e","unresolved":true,"context_lines":[{"line_number":28,"context_line":"- job:"},{"line_number":29,"context_line":"    name: tripleo-ci-centos-9-content-provider"},{"line_number":30,"context_line":"    parent: tripleo-ci-base-standalone-centos-8"},{"line_number":31,"context_line":"    branches: ^(?!stable/(train|ussuri|victoria|wallaby)).*$"},{"line_number":32,"context_line":"    nodeset: single-centos-9-node"},{"line_number":33,"context_line":"    required-projects:"},{"line_number":34,"context_line":"      - opendev.org/openstack/tripleo-common"}],"source_content_type":"text/x-yaml","patch_set":1,"id":"afd13de4_98f762f5","line":31,"range":{"start_line":31,"start_character":48,"end_line":31,"end_character":55},"updated":"2022-02-18 08:04:49.000000000","message":"we also need to remove this\nthen we will get tripleo-ci-centos-9-content-provider on stable/wallaby branches in those repos that have them.\n\n*this* is the job that will run in all our tripleo repos.\n\nthe new one you\u0027re adding below will be used with the upgrades jobs template and the multinode-branchful (tripleo-ci repos only) template","commit_id":"8dc70d6afedde0dbba9892621b191f9f561f4db9"},{"author":{"_account_id":12393,"name":"chandan kumar","display_name":"Chandan Kumar","email":"chkumar@redhat.com","username":"chkumar246"},"change_message_id":"1776322df82b28e313bdc4200377ef2ec67388d4","unresolved":true,"context_lines":[{"line_number":28,"context_line":"- job:"},{"line_number":29,"context_line":"    name: tripleo-ci-centos-9-content-provider"},{"line_number":30,"context_line":"    parent: tripleo-ci-base-standalone-centos-8"},{"line_number":31,"context_line":"    branches: ^(?!stable/(train|ussuri|victoria|wallaby)).*$"},{"line_number":32,"context_line":"    nodeset: single-centos-9-node"},{"line_number":33,"context_line":"    required-projects:"},{"line_number":34,"context_line":"      - opendev.org/openstack/tripleo-common"}],"source_content_type":"text/x-yaml","patch_set":1,"id":"9b63b408_3fbeb762","line":31,"range":{"start_line":31,"start_character":48,"end_line":31,"end_character":55},"in_reply_to":"624c53eb_19d3e00f","updated":"2022-02-22 10:26:14.000000000","message":"Adding them in upgrade and multinode template currently.","commit_id":"8dc70d6afedde0dbba9892621b191f9f561f4db9"},{"author":{"_account_id":12393,"name":"chandan kumar","display_name":"Chandan Kumar","email":"chkumar@redhat.com","username":"chkumar246"},"change_message_id":"1b5520ae7e5cbd833dceca26cca492586140b6bb","unresolved":false,"context_lines":[{"line_number":28,"context_line":"- job:"},{"line_number":29,"context_line":"    name: tripleo-ci-centos-9-content-provider"},{"line_number":30,"context_line":"    parent: tripleo-ci-base-standalone-centos-8"},{"line_number":31,"context_line":"    branches: ^(?!stable/(train|ussuri|victoria|wallaby)).*$"},{"line_number":32,"context_line":"    nodeset: single-centos-9-node"},{"line_number":33,"context_line":"    required-projects:"},{"line_number":34,"context_line":"      - opendev.org/openstack/tripleo-common"}],"source_content_type":"text/x-yaml","patch_set":1,"id":"123f10b5_c3ccc6c0","line":31,"range":{"start_line":31,"start_character":48,"end_line":31,"end_character":55},"in_reply_to":"9b63b408_3fbeb762","updated":"2022-02-22 10:26:39.000000000","message":"Done","commit_id":"8dc70d6afedde0dbba9892621b191f9f561f4db9"},{"author":{"_account_id":8449,"name":"Marios Andreou","email":"marios.andreou@gmail.com","username":"marios"},"change_message_id":"db5e94a681f1097ec57175e6e6176350a6ac4a93","unresolved":true,"context_lines":[{"line_number":28,"context_line":"- job:"},{"line_number":29,"context_line":"    name: tripleo-ci-centos-9-content-provider"},{"line_number":30,"context_line":"    parent: tripleo-ci-base-standalone-centos-8"},{"line_number":31,"context_line":"    branches: ^(?!stable/(train|ussuri|victoria|wallaby)).*$"},{"line_number":32,"context_line":"    nodeset: single-centos-9-node"},{"line_number":33,"context_line":"    required-projects:"},{"line_number":34,"context_line":"      - opendev.org/openstack/tripleo-common"}],"source_content_type":"text/x-yaml","patch_set":1,"id":"624c53eb_19d3e00f","line":31,"range":{"start_line":31,"start_character":48,"end_line":31,"end_character":55},"in_reply_to":"afd13de4_98f762f5","updated":"2022-02-21 10:12:49.000000000","message":"actually I\u0027m doing that there fyi https://review.opendev.org/c/openstack/tripleo-ci/+/830132/1/zuul.d/content-provider.yaml","commit_id":"8dc70d6afedde0dbba9892621b191f9f561f4db9"}],"zuul.d/periodic.yaml":[{"author":{"_account_id":8449,"name":"Marios Andreou","email":"marios.andreou@gmail.com","username":"marios"},"change_message_id":"72f7eafd95e0fae8f428ad0e995514bef170291a","unresolved":true,"context_lines":[{"line_number":189,"context_line":"    name: tripleo-periodic-upgrades-master-pipeline"},{"line_number":190,"context_line":"    periodic:"},{"line_number":191,"context_line":"      jobs:"},{"line_number":192,"context_line":"        - tripleo-ci-centos-9-content-provider-wallaby:"},{"line_number":193,"context_line":"            vars:"},{"line_number":194,"context_line":"              force_non_periodic: true"},{"line_number":195,"context_line":"        - tripleo-ci-centos-8-content-provider:"}],"source_content_type":"text/x-yaml","patch_set":4,"id":"7bd4795a_72aa278d","line":192,"range":{"start_line":192,"start_character":0,"end_line":192,"end_character":2},"updated":"2022-02-25 07:17:37.000000000","message":"i mean, this is OK, but IMO would be better to only add this when it is needed (after this merges the content provider is not providing for any job yet)\n\ni.e. in the parent patch where you are adding the upgrade job that will use it\n\nsince you have them lined up to merge together it should be OK.","commit_id":"faf999e5c3dc3c4b25cfe2b6407f8a5770d49587"}]}