)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":9976,"name":"Ronelle Landy","email":"rlandy@redhat.com","username":"rlandy"},"change_message_id":"fdb7c0dcfd1e6602bc2760b68ee0721660b5098e","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"4533e68d_3bd3a543","updated":"2022-02-24 19:17:51.000000000","message":"I like this approach in general! It\u0027s safer and does not require so much reparenting.\nThe error is an issue","commit_id":"bc7dc56dc49b12818a992c6d4f60cd534279bfb7"},{"author":{"_account_id":9976,"name":"Ronelle Landy","email":"rlandy@redhat.com","username":"rlandy"},"change_message_id":"aa35a89bb6cfa309b675a7d3af3e7526879a9d72","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":3,"id":"8859fd41_b1b654e6","updated":"2022-02-24 19:14:57.000000000","message":"weird standalone error in tempest:\n\nfatal: [undercloud]: FAILED! \u003d\u003e {\"msg\": \"No file was found when using first_found. Use errors\u003d\u0027ignore\u0027 to allow this task to be skipped if no files are found\"}\n","commit_id":"bc7dc56dc49b12818a992c6d4f60cd534279bfb7"},{"author":{"_account_id":8449,"name":"Marios Andreou","email":"marios.andreou@gmail.com","username":"marios"},"change_message_id":"4e826b645485e0db9889eb4826d730af33b744d7","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":8,"id":"5e44248a_fb99e898","updated":"2022-02-28 07:36:16.000000000","message":"ok I think one more discussion today in the review time and let\u0027s go ","commit_id":"3bf71212509f00efea5f48a011d48487db240a6b"},{"author":{"_account_id":29775,"name":"Sandeep Yadav","email":"sandyada@redhat.com","username":"sandeepyadav93"},"change_message_id":"8d2364cbc0bf6b368741b3e2bd298be471b9da46","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":9,"id":"760c28ed_254477c9","updated":"2022-03-01 11:56:38.000000000","message":"Looks good to me.","commit_id":"1662814677834b1e4761c2d1f1a89c12938c73ec"},{"author":{"_account_id":8449,"name":"Marios Andreou","email":"marios.andreou@gmail.com","username":"marios"},"change_message_id":"1e8e0a2a145a7002519d43e1e4f24dd23df3758d","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":9,"id":"868c1a06_49b8243e","updated":"2022-03-01 06:53:35.000000000","message":"ack looks good as discussed this adds the two parents... they aren\u0027t wired up to anything here so this should be safe to merge ","commit_id":"1662814677834b1e4761c2d1f1a89c12938c73ec"}],"zuul.d/base-upstream.yaml":[{"author":{"_account_id":30750,"name":"amolkahat","display_name":"Amol Kahat","email":"amolkahat@gmail.com","username":"amolkahat"},"change_message_id":"1e45c1b6ac708f16deb12cf8c0655eb80bee3114","unresolved":true,"context_lines":[{"line_number":84,"context_line":"      bridge_address_subnet: 24"},{"line_number":85,"context_line":"      bridge_address_offset: 2"},{"line_number":86,"context_line":"      bridge_mtu: 1350"},{"line_number":87,"context_line":"    nodeset: two-centos-8-nodes"},{"line_number":88,"context_line":"    roles:"},{"line_number":89,"context_line":"      - zuul: opendev.org/zuul/zuul-jobs"},{"line_number":90,"context_line":"      - zuul: opendev.org/openstack/openstack-zuul-jobs"}],"source_content_type":"text/x-yaml","patch_set":6,"id":"d4b1e565_1c1c7ede","line":87,"range":{"start_line":87,"start_character":4,"end_line":87,"end_character":31},"updated":"2022-02-25 13:57:42.000000000","message":"We don\u0027t need to define nodesets in standard/option jobs.","commit_id":"7f0019065caca0826cab61cdb10834c9a180ea8a"},{"author":{"_account_id":12393,"name":"chandan kumar","display_name":"Chandan Kumar","email":"chkumar@redhat.com","username":"chkumar246"},"change_message_id":"4c2ff4009207ddfb1f2377e5fefb08731561741c","unresolved":false,"context_lines":[{"line_number":84,"context_line":"      bridge_address_subnet: 24"},{"line_number":85,"context_line":"      bridge_address_offset: 2"},{"line_number":86,"context_line":"      bridge_mtu: 1350"},{"line_number":87,"context_line":"    nodeset: two-centos-8-nodes"},{"line_number":88,"context_line":"    roles:"},{"line_number":89,"context_line":"      - zuul: opendev.org/zuul/zuul-jobs"},{"line_number":90,"context_line":"      - zuul: opendev.org/openstack/openstack-zuul-jobs"}],"source_content_type":"text/x-yaml","patch_set":6,"id":"97a39695_c3eea167","line":87,"range":{"start_line":87,"start_character":4,"end_line":87,"end_character":31},"in_reply_to":"d4b1e565_1c1c7ede","updated":"2022-02-28 04:34:36.000000000","message":"Done","commit_id":"7f0019065caca0826cab61cdb10834c9a180ea8a"},{"author":{"_account_id":8449,"name":"Marios Andreou","email":"marios.andreou@gmail.com","username":"marios"},"change_message_id":"4e826b645485e0db9889eb4826d730af33b744d7","unresolved":true,"context_lines":[{"line_number":67,"context_line":"# in the base job"},{"line_number":68,"context_line":""},{"line_number":69,"context_line":"- job:"},{"line_number":70,"context_line":"    name: tripleo-ci-base-standard"},{"line_number":71,"context_line":"    # min version to use (needed for proper python interpreter detection)"},{"line_number":72,"context_line":"    ansible-version: 2.9"},{"line_number":73,"context_line":"    abstract: true"}],"source_content_type":"text/x-yaml","patch_set":8,"id":"98a881a5_40d4e0c1","line":70,"range":{"start_line":70,"start_character":0,"end_line":70,"end_character":4},"updated":"2022-02-28 07:36:16.000000000","message":"why do we need this one. can\u0027t we use the existing tripleo-ci-base definition  https://opendev.org/openstack/tripleo-ci/src/commit/6b4d6200ab2c161887c525247ce6cc337b06469c/zuul.d/base.yaml#L53 \n\nand change the parent of that to use tripleo-ci-base-required-projects-upstream\n\n\nIf we keep this -standard notation will be then eventually remove things like \"tripleo-ci-base\" ? I think we should if nothing will be using them perhaps the last patch in the series should do that.","commit_id":"3bf71212509f00efea5f48a011d48487db240a6b"},{"author":{"_account_id":8449,"name":"Marios Andreou","email":"marios.andreou@gmail.com","username":"marios"},"change_message_id":"1e8e0a2a145a7002519d43e1e4f24dd23df3758d","unresolved":false,"context_lines":[{"line_number":67,"context_line":"# in the base job"},{"line_number":68,"context_line":""},{"line_number":69,"context_line":"- job:"},{"line_number":70,"context_line":"    name: tripleo-ci-base-standard"},{"line_number":71,"context_line":"    # min version to use (needed for proper python interpreter detection)"},{"line_number":72,"context_line":"    ansible-version: 2.9"},{"line_number":73,"context_line":"    abstract: true"}],"source_content_type":"text/x-yaml","patch_set":8,"id":"b1ad4ee8_8b9ec0dd","line":70,"range":{"start_line":70,"start_character":0,"end_line":70,"end_character":4},"in_reply_to":"35fb5789_425b1426","updated":"2022-03-01 06:53:35.000000000","message":"OK marking comment resolved then the -standard has been removed in the latest version of this patch","commit_id":"3bf71212509f00efea5f48a011d48487db240a6b"},{"author":{"_account_id":12393,"name":"chandan kumar","display_name":"Chandan Kumar","email":"chkumar@redhat.com","username":"chkumar246"},"change_message_id":"02d7a415998dcc6625fcaec2b200e7d9bd875182","unresolved":true,"context_lines":[{"line_number":67,"context_line":"# in the base job"},{"line_number":68,"context_line":""},{"line_number":69,"context_line":"- job:"},{"line_number":70,"context_line":"    name: tripleo-ci-base-standard"},{"line_number":71,"context_line":"    # min version to use (needed for proper python interpreter detection)"},{"line_number":72,"context_line":"    ansible-version: 2.9"},{"line_number":73,"context_line":"    abstract: true"}],"source_content_type":"text/x-yaml","patch_set":8,"id":"35fb5789_425b1426","line":70,"range":{"start_line":70,"start_character":0,"end_line":70,"end_character":4},"in_reply_to":"60467207_396ad323","updated":"2022-02-28 11:35:34.000000000","message":"Moving it to a seperate patch.","commit_id":"3bf71212509f00efea5f48a011d48487db240a6b"},{"author":{"_account_id":12393,"name":"chandan kumar","display_name":"Chandan Kumar","email":"chkumar@redhat.com","username":"chkumar246"},"change_message_id":"e0c4bae1699332d992679446b12c8de218d097ef","unresolved":true,"context_lines":[{"line_number":67,"context_line":"# in the base job"},{"line_number":68,"context_line":""},{"line_number":69,"context_line":"- job:"},{"line_number":70,"context_line":"    name: tripleo-ci-base-standard"},{"line_number":71,"context_line":"    # min version to use (needed for proper python interpreter detection)"},{"line_number":72,"context_line":"    ansible-version: 2.9"},{"line_number":73,"context_line":"    abstract: true"}],"source_content_type":"text/x-yaml","patch_set":8,"id":"60467207_396ad323","line":70,"range":{"start_line":70,"start_character":0,"end_line":70,"end_character":4},"in_reply_to":"98a881a5_40d4e0c1","updated":"2022-02-28 11:11:03.000000000","message":"Thank you Marios for valuable comments.\nYes, It make sense to use the existing job definition.\nI have started using -standard to keep the things seperate in order to avoid breaking anything.","commit_id":"3bf71212509f00efea5f48a011d48487db240a6b"},{"author":{"_account_id":29775,"name":"Sandeep Yadav","email":"sandyada@redhat.com","username":"sandeepyadav93"},"change_message_id":"8d2364cbc0bf6b368741b3e2bd298be471b9da46","unresolved":true,"context_lines":[{"line_number":1,"context_line":"---"},{"line_number":2,"context_line":"# zuul.d/basei-upstream.yaml"},{"line_number":3,"context_line":"# This file contains parent layouts for all TripleO jobs."},{"line_number":4,"context_line":"# Do not add children specific layout here."},{"line_number":5,"context_line":""}],"source_content_type":"text/x-yaml","patch_set":9,"id":"3cb212ff_6ee55077","line":2,"range":{"start_line":2,"start_character":13,"end_line":2,"end_character":14},"updated":"2022-03-01 11:56:38.000000000","message":"nit","commit_id":"1662814677834b1e4761c2d1f1a89c12938c73ec"}],"zuul.d/base.yaml":[{"author":{"_account_id":12393,"name":"chandan kumar","display_name":"Chandan Kumar","email":"chkumar@redhat.com","username":"chkumar246"},"change_message_id":"b6a2b79e39fb7df4b7a1919264d7989463e7363c","unresolved":true,"context_lines":[{"line_number":96,"context_line":"      - ^.pre-commit-config.yaml$"},{"line_number":97,"context_line":"      - ^.yamllint$"},{"line_number":98,"context_line":""},{"line_number":99,"context_line":"# tripleo-ci base common required projects"},{"line_number":100,"context_line":"# It contains a minimum set of common projects which needs to be"},{"line_number":101,"context_line":"# clonned in upstream, rdo and downstream to perform the deployment"},{"line_number":102,"context_line":"- job:"}],"source_content_type":"text/x-yaml","patch_set":3,"id":"1d2f5abd_f1642e0e","line":99,"updated":"2022-02-24 15:57:54.000000000","message":"Keeping a common required projects (which are also used everywhere without much change) will allow to use the same parent in the downstream also.","commit_id":"bc7dc56dc49b12818a992c6d4f60cd534279bfb7"},{"author":{"_account_id":30750,"name":"amolkahat","display_name":"Amol Kahat","email":"amolkahat@gmail.com","username":"amolkahat"},"change_message_id":"56f9c392db847965b77e31402d99599cdea6e2f6","unresolved":true,"context_lines":[{"line_number":105,"context_line":"        Base abstract job to add common required-projects for tripleo-ci-base"},{"line_number":106,"context_line":"    abstract: true"},{"line_number":107,"context_line":"    parent: multinode"},{"line_number":108,"context_line":"    required-projects:"},{"line_number":109,"context_line":"      - opendev.org/openstack/tripleo-ci"},{"line_number":110,"context_line":"      - opendev.org/openstack/tripleo-quickstart"},{"line_number":111,"context_line":"      - opendev.org/openstack/tripleo-quickstart-extras"}],"source_content_type":"text/x-yaml","patch_set":3,"id":"56b84f31_71ecfee5","line":108,"range":{"start_line":108,"start_character":0,"end_line":108,"end_character":12},"updated":"2022-02-25 09:28:30.000000000","message":"We can create common project for all repos which are used in tripleo deployment.\n\novb repos will go to another job:\n- opendev.org/openstack-virtual-baremetal\n\nUpgrade repos will go to another job:\n- opendev.org/openstack/tripleo-upgrade\n\n\nIn short it will be like\n```\n- tripleo-ci-base-common-required-projects\n  - tripleo-ci-base-common-ovb-projects\n\n- tripleo-ci-base-common-required-projects\n  - tripleo-ci-base-common-upgrade-projects\n```","commit_id":"bc7dc56dc49b12818a992c6d4f60cd534279bfb7"},{"author":{"_account_id":12393,"name":"chandan kumar","display_name":"Chandan Kumar","email":"chkumar@redhat.com","username":"chkumar246"},"change_message_id":"73d1f522351e3e650c5ed86749a648163f4ddc55","unresolved":true,"context_lines":[{"line_number":105,"context_line":"        Base abstract job to add common required-projects for tripleo-ci-base"},{"line_number":106,"context_line":"    abstract: true"},{"line_number":107,"context_line":"    parent: multinode"},{"line_number":108,"context_line":"    required-projects:"},{"line_number":109,"context_line":"      - opendev.org/openstack/tripleo-ci"},{"line_number":110,"context_line":"      - opendev.org/openstack/tripleo-quickstart"},{"line_number":111,"context_line":"      - opendev.org/openstack/tripleo-quickstart-extras"}],"source_content_type":"text/x-yaml","patch_set":3,"id":"d5e027d6_ee06c42f","line":108,"range":{"start_line":108,"start_character":0,"end_line":108,"end_character":12},"in_reply_to":"56b84f31_71ecfee5","updated":"2022-02-25 09:33:50.000000000","message":"Thank you Amol for the comments.\n\nfor ovb case:\nhttps://opendev.org/openstack/tripleo-ci/src/branch/master/zuul.d/base.yaml#L248\ntripleo-ci-base-ovb-centos-8 is the base job\n\nand here we can add ovb as a required projects.\n\nFor upgrade,\nWe can follow the similar approach.\nhttps://opendev.org/openstack/tripleo-ci/src/branch/master/zuul.d/base.yaml#L515\n\nand here we can add tripleo-upgrade as a required project for tripleo-ci-base-standalone-upgrade-centos-8 job.","commit_id":"bc7dc56dc49b12818a992c6d4f60cd534279bfb7"},{"author":{"_account_id":12393,"name":"chandan kumar","display_name":"Chandan Kumar","email":"chkumar@redhat.com","username":"chkumar246"},"change_message_id":"b8932cbe94fb5a8bbb8325e5c6de09cec2cacd03","unresolved":true,"context_lines":[{"line_number":105,"context_line":"        Base abstract job to add common required-projects for tripleo-ci-base"},{"line_number":106,"context_line":"    abstract: true"},{"line_number":107,"context_line":"    parent: multinode"},{"line_number":108,"context_line":"    required-projects:"},{"line_number":109,"context_line":"      - opendev.org/openstack/tripleo-ci"},{"line_number":110,"context_line":"      - opendev.org/openstack/tripleo-quickstart"},{"line_number":111,"context_line":"      - opendev.org/openstack/tripleo-quickstart-extras"}],"source_content_type":"text/x-yaml","patch_set":3,"id":"b8d4e114_f1fa4655","line":108,"range":{"start_line":108,"start_character":0,"end_line":108,"end_character":12},"in_reply_to":"d5e027d6_ee06c42f","updated":"2022-02-25 11:38:45.000000000","message":"Let\u0027s try out later","commit_id":"bc7dc56dc49b12818a992c6d4f60cd534279bfb7"},{"author":{"_account_id":12393,"name":"chandan kumar","display_name":"Chandan Kumar","email":"chkumar@redhat.com","username":"chkumar246"},"change_message_id":"b6a2b79e39fb7df4b7a1919264d7989463e7363c","unresolved":true,"context_lines":[{"line_number":156,"context_line":"      - opendev.org/openstack/tripleo-operator-ansible"},{"line_number":157,"context_line":"      - name: opendev.org/openstack/ansible-config_template"},{"line_number":158,"context_line":"        override-checkout: master"},{"line_number":159,"context_line":"      - name: opendev.org/openstack/openstack-virtual-baremetal"},{"line_number":160,"context_line":"        override-checkout: stable/2.0"},{"line_number":161,"context_line":"      - name: opendev.org/openstack/openstack-ansible-os_tempest"},{"line_number":162,"context_line":"        override-checkout: stable/wallaby"},{"line_number":163,"context_line":""},{"line_number":164,"context_line":"- job:"},{"line_number":165,"context_line":"    name: tripleo-ci-base-centos-8"}],"source_content_type":"text/x-yaml","patch_set":3,"id":"37662275_c0fe060d","line":162,"range":{"start_line":159,"start_character":6,"end_line":162,"end_character":41},"updated":"2022-02-24 15:57:54.000000000","message":"It will be checked out by zuul in each of the job. So we can override required projects anywhere it will solve the issue of handling multiple projects with multiple releases.","commit_id":"bc7dc56dc49b12818a992c6d4f60cd534279bfb7"},{"author":{"_account_id":12393,"name":"chandan kumar","display_name":"Chandan Kumar","email":"chkumar@redhat.com","username":"chkumar246"},"change_message_id":"657d987c7fff8dcc5b81a1b0c3620bed754068d0","unresolved":true,"context_lines":[{"line_number":160,"context_line":"        override-checkout: stable/2.0"},{"line_number":161,"context_line":"      - name: opendev.org/openstack/openstack-ansible-os_tempest"},{"line_number":162,"context_line":"        override-checkout: stable/wallaby"},{"line_number":163,"context_line":""},{"line_number":164,"context_line":"- job:"},{"line_number":165,"context_line":"    name: tripleo-ci-base-centos-8"},{"line_number":166,"context_line":"    # min version to use (needed for proper python interpreter detection)"}],"source_content_type":"text/x-yaml","patch_set":3,"id":"149eeefb_50d572af","line":163,"updated":"2022-02-25 09:13:52.000000000","message":"based on the job results:\nhttps://storage.bhs.cloud.ovh.net/v1/AUTH_dcaab5e32b234d56b626f72581e3644c/zuul_opendev_logs_f4c/830836/3/check/tripleo-buildimage-overcloud-full-centos-9/f4c0581/job-output.txt\n```\nSwitched to branch \u0027stable/wallaby\u0027\n2022-02-24 15:43:38.322586 | primary | opendev.org/openstack/openstack-ansible-os_tempest checked out to:\n2022-02-24 15:43:38.322658 | primary | bd399b80e9de12452f1bee620bc60a69c142bf40 Pin neutron-tempest-plugin to v1.6.0\n```\n\nBased on the above experimentation, we can override a project under required projects with needed branch/version/tag.\n\nIt solves the issue of handling multiple project releases.","commit_id":"bc7dc56dc49b12818a992c6d4f60cd534279bfb7"}]}