)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":17685,"name":"Elod Illes","email":"elod.illes@est.tech","username":"elod.illes"},"change_message_id":"b60419f156181e5e8444b4abccc9d4bbad34e703","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"8ba1dd16_c24066d4","updated":"2022-01-21 16:49:16.000000000","message":"Giving -1 due to the version bump, but i\u0027m still not quite sure whether the branch cut is valid at independent release model.","commit_id":"1d9b62f6d075850371dd223f59ac9e168516193b"},{"author":{"_account_id":16515,"name":"mbu","email":"mat.bultel@gmail.com","username":"matbu"},"change_message_id":"570c4ae7b796d7edada030627a8491a5dac597f6","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"1b2b92a0_da85d5fa","updated":"2022-01-19 07:26:28.000000000","message":"recheck","commit_id":"1d9b62f6d075850371dd223f59ac9e168516193b"},{"author":{"_account_id":16515,"name":"mbu","email":"mat.bultel@gmail.com","username":"matbu"},"change_message_id":"7b01aa55b0137bf4e334679b7a7267ac522f03e5","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":6,"id":"1438d231_27f34421","updated":"2022-02-21 15:31:13.000000000","message":"@elod @herve is it fine for you now and can we merged it ?\n\nThank you folks.","commit_id":"7c1865432452db7c5d774caaf3d4e55644dd61de"},{"author":{"_account_id":17685,"name":"Elod Illes","email":"elod.illes@est.tech","username":"elod.illes"},"change_message_id":"5e3c98a208ec3395ce71c911dcd3c1c887bded45","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":6,"id":"255820c2_e7cf0eb5","updated":"2022-02-01 16:51:21.000000000","message":"Downgrading to +1 until we get answer to Hervé\u0027s question.","commit_id":"7c1865432452db7c5d774caaf3d4e55644dd61de"},{"author":{"_account_id":17685,"name":"Elod Illes","email":"elod.illes@est.tech","username":"elod.illes"},"change_message_id":"320a441456bc68ed455b893e9ee5c133ac6cd8d4","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":6,"id":"63337d93_e870c347","updated":"2022-02-01 11:38:02.000000000","message":"Thanks for the update! LGTM.","commit_id":"7c1865432452db7c5d774caaf3d4e55644dd61de"},{"author":{"_account_id":17685,"name":"Elod Illes","email":"elod.illes@est.tech","username":"elod.illes"},"change_message_id":"6038609f4a60bb750e99ed0676400419dee85235","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":6,"id":"46839846_f26dd04c","in_reply_to":"1438d231_27f34421","updated":"2022-02-22 11:24:37.000000000","message":"Sure, sorry, Just waited for acknowledgement to what Hervé wrote. But then as I understand it is acceptable here as well.","commit_id":"7c1865432452db7c5d774caaf3d4e55644dd61de"}],"deliverables/_independent/validations-libs.yaml":[{"author":{"_account_id":17685,"name":"Elod Illes","email":"elod.illes@est.tech","username":"elod.illes"},"change_message_id":"b60419f156181e5e8444b4abccc9d4bbad34e703","unresolved":true,"context_lines":[{"line_number":1,"context_line":"---"},{"line_number":2,"context_line":"launchpad: tripleo"},{"line_number":3,"context_line":"release-type: python-pypi"},{"line_number":4,"context_line":"stable-branch-type: upstream"},{"line_number":5,"context_line":"team: tripleo"},{"line_number":6,"context_line":"type: other"},{"line_number":7,"context_line":"repository-settings:"}],"source_content_type":"text/x-yaml","patch_set":4,"id":"70578195_8addc2e3","line":4,"updated":"2022-01-21 16:49:16.000000000","message":"This seems valid for \u0027stable/1.5\u0027 to me according to documentation [1]:\n\n\"upstream: Stable branch names track upstream release names, rather than OpenStack series names.\"\n\nBut i\u0027m not sure whether it\u0027s valid to use here, at an independent release.\n\n[1] https://releases.openstack.org/reference/using.html#deliverables-file-schema","commit_id":"1d9b62f6d075850371dd223f59ac9e168516193b"},{"author":{"_account_id":28522,"name":"Hervé Beraud","email":"herveberaud.pro@gmail.com","username":"hberaud"},"change_message_id":"6c7d92685ae87885ad3e9cd518f46158b936c035","unresolved":true,"context_lines":[{"line_number":1,"context_line":"---"},{"line_number":2,"context_line":"launchpad: tripleo"},{"line_number":3,"context_line":"release-type: python-pypi"},{"line_number":4,"context_line":"stable-branch-type: upstream"},{"line_number":5,"context_line":"team: tripleo"},{"line_number":6,"context_line":"type: other"},{"line_number":7,"context_line":"repository-settings:"}],"source_content_type":"text/x-yaml","patch_set":4,"id":"a39687dd_cffeb7d0","line":4,"in_reply_to":"70578195_8addc2e3","updated":"2022-02-01 16:43:00.000000000","message":"Well, our validation won\u0027t work without this line, so they can\u0027t generate a new branch named (more or less) like a specific tag without this line.\n\nIf we remove this line the validation will exit in error and output:\n\n```\n1 warnings found\ndeliverables/_independent/validations-libs.yaml: validate_release_branch_membership: skipping descendant test for independent project, verify branch manually\n\n\n1 errors found\ndeliverables/_independent/validations-libs.yaml: validate_stable_branches: stable branches must be named for known series but stable/1.6 was not found in [\u0027austin\u0027, \u0027bexar\u0027, \u0027cactus\u0027, \u0027diablo\u0027, \u0027essex\u0027, \u0027folsom\u0027, \u0027grizzly\u0027, \u0027havana\u0027, \u0027icehouse\u0027, \u0027juno\u0027, \u0027kilo\u0027, \u0027liberty\u0027, \u0027mitaka\u0027, \u0027newton\u0027, \u0027ocata\u0027, \u0027pike\u0027, \u0027queens\u0027, \u0027rocky\u0027, \u0027stein\u0027, \u0027train\u0027, \u0027ussuri\u0027, \u0027victoria\u0027, \u0027wallaby\u0027, \u0027xena\u0027, \u0027yoga\u0027]\n```\n\nSo if we agree to create a new branch for an independent deliverable then we should also agree the adding of this line.","commit_id":"1d9b62f6d075850371dd223f59ac9e168516193b"},{"author":{"_account_id":17685,"name":"Elod Illes","email":"elod.illes@est.tech","username":"elod.illes"},"change_message_id":"5e3c98a208ec3395ce71c911dcd3c1c887bded45","unresolved":true,"context_lines":[{"line_number":1,"context_line":"---"},{"line_number":2,"context_line":"launchpad: tripleo"},{"line_number":3,"context_line":"release-type: python-pypi"},{"line_number":4,"context_line":"stable-branch-type: upstream"},{"line_number":5,"context_line":"team: tripleo"},{"line_number":6,"context_line":"type: other"},{"line_number":7,"context_line":"repository-settings:"}],"source_content_type":"text/x-yaml","patch_set":4,"id":"f31e596d_e4f407ef","line":4,"in_reply_to":"a39687dd_cffeb7d0","updated":"2022-02-01 16:51:21.000000000","message":"Thanks for the clarification Hervé!","commit_id":"1d9b62f6d075850371dd223f59ac9e168516193b"},{"author":{"_account_id":17685,"name":"Elod Illes","email":"elod.illes@est.tech","username":"elod.illes"},"change_message_id":"b60419f156181e5e8444b4abccc9d4bbad34e703","unresolved":true,"context_lines":[{"line_number":47,"context_line":"    projects:"},{"line_number":48,"context_line":"      - repo: openstack/validations-libs"},{"line_number":49,"context_line":"        hash: 17e689536f1c0e2269434d99c9e83555ef918511"},{"line_number":50,"context_line":"  - version: 1.5.1"},{"line_number":51,"context_line":"    projects:"},{"line_number":52,"context_line":"      - repo: openstack/validations-libs"},{"line_number":53,"context_line":"        hash: eb5c38ee62e8bf3e13a8d10e1dd48ba5320e53b2"}],"source_content_type":"text/x-yaml","patch_set":4,"id":"b9d9dfe7_fa8da2c5","line":50,"range":{"start_line":50,"start_character":13,"end_line":50,"end_character":18},"updated":"2022-01-21 16:49:16.000000000","message":"requirements change [1] warrant MINOR version bump, so we need 1.6.0 here. (it\u0027s strange that requirements lower bound is moved even lower in a new release, usually it happens in the other direction. anyway, if that works, i\u0027m not against it)\n\n[1] https://zuul.opendev.org/t/openstack/build/07e2e9084012468393751e04a187b8ed/log/tox/list-changes-results.log#210-213","commit_id":"1d9b62f6d075850371dd223f59ac9e168516193b"},{"author":{"_account_id":17685,"name":"Elod Illes","email":"elod.illes@est.tech","username":"elod.illes"},"change_message_id":"b60419f156181e5e8444b4abccc9d4bbad34e703","unresolved":true,"context_lines":[{"line_number":52,"context_line":"      - repo: openstack/validations-libs"},{"line_number":53,"context_line":"        hash: eb5c38ee62e8bf3e13a8d10e1dd48ba5320e53b2"},{"line_number":54,"context_line":"branches:"},{"line_number":55,"context_line":"  - name: stable/1.5"},{"line_number":56,"context_line":"    location: 1.5.1"}],"source_content_type":"text/x-yaml","patch_set":4,"id":"5f9dcb92_4b376482","line":56,"range":{"start_line":55,"start_character":0,"end_line":56,"end_character":19},"updated":"2022-01-21 16:49:16.000000000","message":"Didn\u0027t we say, that stable branch in independent model is not really acceptable?\nBesides, i think the stable branch name should represent the location of the branch cut (e.g. stable/1.6 should have \u0027location: 1.6.0\u0027)","commit_id":"1d9b62f6d075850371dd223f59ac9e168516193b"},{"author":{"_account_id":28522,"name":"Hervé Beraud","email":"herveberaud.pro@gmail.com","username":"hberaud"},"change_message_id":"67adad8b6c6f3a3d2a1ca4339567c3c03b890195","unresolved":true,"context_lines":[{"line_number":52,"context_line":"      - repo: openstack/validations-libs"},{"line_number":53,"context_line":"        hash: eb5c38ee62e8bf3e13a8d10e1dd48ba5320e53b2"},{"line_number":54,"context_line":"branches:"},{"line_number":55,"context_line":"  - name: stable/1.5"},{"line_number":56,"context_line":"    location: 1.5.1"}],"source_content_type":"text/x-yaml","patch_set":4,"id":"5b7da429_6d843135","line":56,"range":{"start_line":55,"start_character":0,"end_line":56,"end_character":19},"in_reply_to":"2dceab5e_6affd941","updated":"2022-02-01 16:46:46.000000000","message":"@James and @mbu ^","commit_id":"1d9b62f6d075850371dd223f59ac9e168516193b"},{"author":{"_account_id":17685,"name":"Elod Illes","email":"elod.illes@est.tech","username":"elod.illes"},"change_message_id":"65c7173f3c662cce2aeb8ca8d74ecc95318f3cef","unresolved":true,"context_lines":[{"line_number":52,"context_line":"      - repo: openstack/validations-libs"},{"line_number":53,"context_line":"        hash: eb5c38ee62e8bf3e13a8d10e1dd48ba5320e53b2"},{"line_number":54,"context_line":"branches:"},{"line_number":55,"context_line":"  - name: stable/1.5"},{"line_number":56,"context_line":"    location: 1.5.1"}],"source_content_type":"text/x-yaml","patch_set":4,"id":"f1d76cf6_fd496d6b","line":56,"range":{"start_line":55,"start_character":0,"end_line":56,"end_character":19},"in_reply_to":"416fe69d_3829e281","updated":"2022-01-28 08:46:46.000000000","message":"Sorry, my bad. I see that there are branch cuts even for independent deliverables.\n\nAnyway, it is still strange for me to cut branch 1.5 from 1.5.1 instead of 1.5.0. (besides, as i commented above, i\u0027d rather bump to version 1.6.0 instead of 1.5.1), but again, maybe it\u0027s just me...","commit_id":"1d9b62f6d075850371dd223f59ac9e168516193b"},{"author":{"_account_id":16515,"name":"mbu","email":"mat.bultel@gmail.com","username":"matbu"},"change_message_id":"dd1ac917c0be4d66200ee73197f8fdefdb10bad0","unresolved":true,"context_lines":[{"line_number":52,"context_line":"      - repo: openstack/validations-libs"},{"line_number":53,"context_line":"        hash: eb5c38ee62e8bf3e13a8d10e1dd48ba5320e53b2"},{"line_number":54,"context_line":"branches:"},{"line_number":55,"context_line":"  - name: stable/1.5"},{"line_number":56,"context_line":"    location: 1.5.1"}],"source_content_type":"text/x-yaml","patch_set":4,"id":"5cbc441b_f28676fc","line":56,"range":{"start_line":55,"start_character":0,"end_line":56,"end_character":19},"in_reply_to":"4f1ecfd6_c4aa3b2d","updated":"2022-01-31 14:52:42.000000000","message":"Yep,\nI\u0027m ok with that, I move it the 1.6.\n\nThank you guys for the answers.","commit_id":"1d9b62f6d075850371dd223f59ac9e168516193b"},{"author":{"_account_id":16515,"name":"mbu","email":"mat.bultel@gmail.com","username":"matbu"},"change_message_id":"1e320838aa427c941ae516e62dca4fb0facf0b67","unresolved":true,"context_lines":[{"line_number":52,"context_line":"      - repo: openstack/validations-libs"},{"line_number":53,"context_line":"        hash: eb5c38ee62e8bf3e13a8d10e1dd48ba5320e53b2"},{"line_number":54,"context_line":"branches:"},{"line_number":55,"context_line":"  - name: stable/1.5"},{"line_number":56,"context_line":"    location: 1.5.1"}],"source_content_type":"text/x-yaml","patch_set":4,"id":"df41a741_bf16e398","line":56,"range":{"start_line":55,"start_character":0,"end_line":56,"end_character":19},"in_reply_to":"5b7da429_6d843135","updated":"2022-02-01 17:33:43.000000000","message":"Ack, actually from this point, we will increase our release tags with 2.x and keep this stable/1.6 for maintenance.\nThe 1.x version supports a large scope (from py27 to actual current dependencies).\n\nThe goal is to evolve like this deliverable for example:\nhttps://github.com/openstack/releases/blob/master/deliverables/_independent/puppet-ceph.yaml","commit_id":"1d9b62f6d075850371dd223f59ac9e168516193b"},{"author":{"_account_id":28522,"name":"Hervé Beraud","email":"herveberaud.pro@gmail.com","username":"hberaud"},"change_message_id":"31f80e55f18d1e4d66ac75c85fc2caed3a8bbaa6","unresolved":true,"context_lines":[{"line_number":52,"context_line":"      - repo: openstack/validations-libs"},{"line_number":53,"context_line":"        hash: eb5c38ee62e8bf3e13a8d10e1dd48ba5320e53b2"},{"line_number":54,"context_line":"branches:"},{"line_number":55,"context_line":"  - name: stable/1.5"},{"line_number":56,"context_line":"    location: 1.5.1"}],"source_content_type":"text/x-yaml","patch_set":4,"id":"2dceab5e_6affd941","line":56,"range":{"start_line":55,"start_character":0,"end_line":56,"end_character":19},"in_reply_to":"5cbc441b_f28676fc","updated":"2022-02-01 16:45:55.000000000","message":"You should be aware that that\u0027s won\u0027t be possible to create new tag for this deliverable from this new branch.\n\nThis is ok for you?","commit_id":"1d9b62f6d075850371dd223f59ac9e168516193b"},{"author":{"_account_id":7144,"name":"James Slagle","email":"jslagle@redhat.com","username":"slagle"},"change_message_id":"f0a84466985e1f4edcceb1ca1aa61cfc2f90f5bd","unresolved":true,"context_lines":[{"line_number":52,"context_line":"      - repo: openstack/validations-libs"},{"line_number":53,"context_line":"        hash: eb5c38ee62e8bf3e13a8d10e1dd48ba5320e53b2"},{"line_number":54,"context_line":"branches:"},{"line_number":55,"context_line":"  - name: stable/1.5"},{"line_number":56,"context_line":"    location: 1.5.1"}],"source_content_type":"text/x-yaml","patch_set":4,"id":"416fe69d_3829e281","line":56,"range":{"start_line":55,"start_character":0,"end_line":56,"end_character":19},"in_reply_to":"5f9dcb92_4b376482","updated":"2022-01-27 20:46:09.000000000","message":"there are other instances of independent deliverables with stable branches. why would it not be acceptable?","commit_id":"1d9b62f6d075850371dd223f59ac9e168516193b"},{"author":{"_account_id":28522,"name":"Hervé Beraud","email":"herveberaud.pro@gmail.com","username":"hberaud"},"change_message_id":"0dd63228def42c90bd1cb20e38bcf83ef5bbbf5b","unresolved":true,"context_lines":[{"line_number":52,"context_line":"      - repo: openstack/validations-libs"},{"line_number":53,"context_line":"        hash: eb5c38ee62e8bf3e13a8d10e1dd48ba5320e53b2"},{"line_number":54,"context_line":"branches:"},{"line_number":55,"context_line":"  - name: stable/1.5"},{"line_number":56,"context_line":"    location: 1.5.1"}],"source_content_type":"text/x-yaml","patch_set":4,"id":"1b213706_ba7632be","line":56,"range":{"start_line":55,"start_character":0,"end_line":56,"end_character":19},"in_reply_to":"df41a741_bf16e398","updated":"2022-02-03 15:04:23.000000000","message":"okay. Note that even with puppet-ceph all the versions were released well ordered and never a version 2.x was released after the release of the 3.x.\n\nhttps://opendev.org/openstack/releases/commits/branch/master/deliverables/_independent/puppet-ceph.yaml","commit_id":"1d9b62f6d075850371dd223f59ac9e168516193b"},{"author":{"_account_id":7144,"name":"James Slagle","email":"jslagle@redhat.com","username":"slagle"},"change_message_id":"148ccea1463f236c1c26cf9a183a8e119645f6f5","unresolved":true,"context_lines":[{"line_number":52,"context_line":"      - repo: openstack/validations-libs"},{"line_number":53,"context_line":"        hash: eb5c38ee62e8bf3e13a8d10e1dd48ba5320e53b2"},{"line_number":54,"context_line":"branches:"},{"line_number":55,"context_line":"  - name: stable/1.5"},{"line_number":56,"context_line":"    location: 1.5.1"}],"source_content_type":"text/x-yaml","patch_set":4,"id":"4f1ecfd6_c4aa3b2d","line":56,"range":{"start_line":55,"start_character":0,"end_line":56,"end_character":19},"in_reply_to":"f1d76cf6_fd496d6b","updated":"2022-01-29 22:58:16.000000000","message":"i will let mathieu comment on that and if he is happy with the branch location.","commit_id":"1d9b62f6d075850371dd223f59ac9e168516193b"}]}