)]}'
{"id":"openstack%2Freleases~715967","triplet_id":"openstack%2Freleases~master~I310e30898816d2c83a3f52c22feb550420fc11ea","project":"openstack/releases","branch":"master","topic":"validations","hashtags":[],"change_id":"I310e30898816d2c83a3f52c22feb550420fc11ea","subject":"Add validations-common and validations-lib deliverables to train","status":"ABANDONED","created":"2020-03-30 14:45:28.000000000","updated":"2020-04-01 12:53:30.000000000","total_comment_count":4,"unresolved_comment_count":0,"has_review_started":true,"meta_rev_id":"3336371740addec7c1460d9586eb5ada5f2c08d0","_number":715967,"virtual_id_number":715967,"owner":{"_account_id":28223,"name":"Cedric Jeanneret","display_name":"cjeanner (Tengu)","email":"cjeanner@redhat.com","username":"cjeanner"},"actions":{},"labels":{"Verified":{"recommended":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"all":[{"_account_id":11904,"name":"Sean McGinnis","email":"sean.mcginnis@gmail.com","username":"SeanM"},{"_account_id":1955,"name":"Alan Pevec","email":"alan.pevec@redhat.com","username":"apevec"},{"_account_id":16515,"name":"mbu","email":"mat.bultel@gmail.com","username":"matbu"},{"_account_id":11491,"name":"Gael Chamoulaud","email":"gchamoul@redhat.com","username":"gchamoul"},{"_account_id":308,"name":"Thierry Carrez","email":"thierry@openstack.org","username":"ttx"},{"_account_id":9592,"name":"wes hayutin","email":"whayutin@redhat.com","username":"weshayutin"},{"_account_id":28522,"name":"Hervé Beraud","email":"herveberaud.pro@gmail.com","username":"hberaud"},{"tag":"autogenerated:zuul:check","value":1,"date":"2020-03-30 15:23:36.000000000","permitted_voting_range":{"min":-2,"max":2},"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},{"date":"2020-04-01 10:36:13.000000000","_account_id":28223,"name":"Cedric Jeanneret","display_name":"cjeanner (Tengu)","email":"cjeanner@redhat.com","username":"cjeanner"}],"values":{"-2":"Fails","-1":"Doesn\u0027t seem to work"," 0":"No score","+1":"Works for me","+2":"Verified"},"description":"","value":1,"default_value":0,"optional":true},"Code-Review":{"recommended":{"_account_id":9592,"name":"wes hayutin","email":"whayutin@redhat.com","username":"weshayutin"},"all":[{"value":0,"permitted_voting_range":{"min":-2,"max":2},"_account_id":11904,"name":"Sean McGinnis","email":"sean.mcginnis@gmail.com","username":"SeanM"},{"value":0,"permitted_voting_range":{"min":-2,"max":2},"_account_id":1955,"name":"Alan Pevec","email":"alan.pevec@redhat.com","username":"apevec"},{"value":0,"permitted_voting_range":{"min":-1,"max":1},"_account_id":16515,"name":"mbu","email":"mat.bultel@gmail.com","username":"matbu"},{"value":0,"permitted_voting_range":{"min":-1,"max":1},"_account_id":11491,"name":"Gael Chamoulaud","email":"gchamoul@redhat.com","username":"gchamoul"},{"value":0,"date":"2020-04-01 09:46:04.000000000","permitted_voting_range":{"min":-2,"max":2},"_account_id":308,"name":"Thierry Carrez","email":"thierry@openstack.org","username":"ttx"},{"value":1,"date":"2020-03-31 14:49:00.000000000","permitted_voting_range":{"min":-1,"max":1},"_account_id":9592,"name":"wes hayutin","email":"whayutin@redhat.com","username":"weshayutin"},{"value":1,"date":"2020-03-30 16:02:25.000000000","permitted_voting_range":{"min":-2,"max":2},"_account_id":28522,"name":"Hervé Beraud","email":"herveberaud.pro@gmail.com","username":"hberaud"},{"value":0,"permitted_voting_range":{"min":-1,"max":1},"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},{"value":0,"permitted_voting_range":{"min":-1,"max":1},"_account_id":28223,"name":"Cedric Jeanneret","display_name":"cjeanner (Tengu)","email":"cjeanner@redhat.com","username":"cjeanner"}],"values":{"-2":"Do not merge","-1":"This patch needs further work before it can be merged"," 0":"No score","+1":"Looks good to me, but someone else must approve","+2":"Looks good to me (core reviewer)"},"description":"","value":1,"default_value":0,"optional":true},"Workflow":{"all":[{"value":0,"permitted_voting_range":{"min":-1,"max":1},"_account_id":11904,"name":"Sean McGinnis","email":"sean.mcginnis@gmail.com","username":"SeanM"},{"value":0,"permitted_voting_range":{"min":-1,"max":1},"_account_id":1955,"name":"Alan Pevec","email":"alan.pevec@redhat.com","username":"apevec"},{"_account_id":16515,"name":"mbu","email":"mat.bultel@gmail.com","username":"matbu"},{"_account_id":11491,"name":"Gael Chamoulaud","email":"gchamoul@redhat.com","username":"gchamoul"},{"value":0,"permitted_voting_range":{"min":-1,"max":1},"_account_id":308,"name":"Thierry Carrez","email":"thierry@openstack.org","username":"ttx"},{"_account_id":9592,"name":"wes hayutin","email":"whayutin@redhat.com","username":"weshayutin"},{"value":0,"permitted_voting_range":{"min":-1,"max":1},"_account_id":28522,"name":"Hervé Beraud","email":"herveberaud.pro@gmail.com","username":"hberaud"},{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},{"value":0,"permitted_voting_range":{"min":-1,"max":0},"_account_id":28223,"name":"Cedric Jeanneret","display_name":"cjeanner (Tengu)","email":"cjeanner@redhat.com","username":"cjeanner"}],"values":{"-1":"Work in progress"," 0":"Ready for reviews","+1":"Approved"},"description":"","default_value":0,"optional":true},"PTL-Approved":{"all":[{"value":0,"permitted_voting_range":{"min":0,"max":1},"_account_id":11904,"name":"Sean McGinnis","email":"sean.mcginnis@gmail.com","username":"SeanM"},{"value":0,"permitted_voting_range":{"min":0,"max":1},"_account_id":1955,"name":"Alan Pevec","email":"alan.pevec@redhat.com","username":"apevec"},{"_account_id":16515,"name":"mbu","email":"mat.bultel@gmail.com","username":"matbu"},{"_account_id":11491,"name":"Gael Chamoulaud","email":"gchamoul@redhat.com","username":"gchamoul"},{"value":0,"permitted_voting_range":{"min":0,"max":1},"_account_id":308,"name":"Thierry Carrez","email":"thierry@openstack.org","username":"ttx"},{"_account_id":9592,"name":"wes hayutin","email":"whayutin@redhat.com","username":"weshayutin"},{"value":0,"permitted_voting_range":{"min":0,"max":1},"_account_id":28522,"name":"Hervé Beraud","email":"herveberaud.pro@gmail.com","username":"hberaud"},{"value":0,"permitted_voting_range":{"min":0,"max":1},"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},{"_account_id":28223,"name":"Cedric Jeanneret","display_name":"cjeanner (Tengu)","email":"cjeanner@redhat.com","username":"cjeanner"}],"values":{" 0":"Not approved yet","+1":"Approved"},"description":"","default_value":0,"optional":true}},"removable_reviewers":[],"reviewers":{"REVIEWER":[{"_account_id":308,"name":"Thierry Carrez","email":"thierry@openstack.org","username":"ttx"},{"_account_id":1955,"name":"Alan Pevec","email":"alan.pevec@redhat.com","username":"apevec"},{"_account_id":9592,"name":"wes hayutin","email":"whayutin@redhat.com","username":"weshayutin"},{"_account_id":11491,"name":"Gael Chamoulaud","email":"gchamoul@redhat.com","username":"gchamoul"},{"_account_id":11904,"name":"Sean McGinnis","email":"sean.mcginnis@gmail.com","username":"SeanM"},{"_account_id":16515,"name":"mbu","email":"mat.bultel@gmail.com","username":"matbu"},{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},{"_account_id":28223,"name":"Cedric Jeanneret","display_name":"cjeanner (Tengu)","email":"cjeanner@redhat.com","username":"cjeanner"},{"_account_id":28522,"name":"Hervé Beraud","email":"herveberaud.pro@gmail.com","username":"hberaud"}]},"pending_reviewers":{},"reviewer_updates":[{"updated":"2020-03-30 14:50:04.000000000","updated_by":{"_account_id":11491,"name":"Gael Chamoulaud","email":"gchamoul@redhat.com","username":"gchamoul"},"reviewer":{"_account_id":11491,"name":"Gael Chamoulaud","email":"gchamoul@redhat.com","username":"gchamoul"},"state":"REVIEWER"},{"updated":"2020-03-30 14:50:07.000000000","updated_by":{"_account_id":16515,"name":"mbu","email":"mat.bultel@gmail.com","username":"matbu"},"reviewer":{"_account_id":16515,"name":"mbu","email":"mat.bultel@gmail.com","username":"matbu"},"state":"REVIEWER"},{"updated":"2020-03-30 14:51:10.000000000","updated_by":{"_account_id":1955,"name":"Alan Pevec","email":"alan.pevec@redhat.com","username":"apevec"},"reviewer":{"_account_id":1955,"name":"Alan Pevec","email":"alan.pevec@redhat.com","username":"apevec"},"state":"REVIEWER"},{"updated":"2020-03-30 14:51:20.000000000","updated_by":{"_account_id":11904,"name":"Sean McGinnis","email":"sean.mcginnis@gmail.com","username":"SeanM"},"reviewer":{"_account_id":11904,"name":"Sean McGinnis","email":"sean.mcginnis@gmail.com","username":"SeanM"},"state":"REVIEWER"},{"updated":"2020-03-30 15:23:36.000000000","updated_by":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"reviewer":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"state":"REVIEWER"},{"updated":"2020-03-30 16:02:25.000000000","updated_by":{"_account_id":28522,"name":"Hervé Beraud","email":"herveberaud.pro@gmail.com","username":"hberaud"},"reviewer":{"_account_id":28522,"name":"Hervé Beraud","email":"herveberaud.pro@gmail.com","username":"hberaud"},"state":"REVIEWER"},{"updated":"2020-03-31 14:49:00.000000000","updated_by":{"_account_id":9592,"name":"wes hayutin","email":"whayutin@redhat.com","username":"weshayutin"},"reviewer":{"_account_id":9592,"name":"wes hayutin","email":"whayutin@redhat.com","username":"weshayutin"},"state":"REVIEWER"},{"updated":"2020-04-01 09:46:04.000000000","updated_by":{"_account_id":308,"name":"Thierry Carrez","email":"thierry@openstack.org","username":"ttx"},"reviewer":{"_account_id":308,"name":"Thierry Carrez","email":"thierry@openstack.org","username":"ttx"},"state":"REVIEWER"}],"messages":[{"id":"bec78439bd262d04dc01347276ec69de0d692697","author":{"_account_id":28223,"name":"Cedric Jeanneret","display_name":"cjeanner (Tengu)","email":"cjeanner@redhat.com","username":"cjeanner"},"date":"2020-03-30 14:45:28.000000000","message":"Uploaded patch set 1.","accounts_in_message":[],"_revision_number":1},{"id":"6ea057b5da4532b6820827f06a2b617036c3bc37","author":{"_account_id":11904,"name":"Sean McGinnis","email":"sean.mcginnis@gmail.com","username":"SeanM"},"date":"2020-03-30 14:51:20.000000000","message":"Patch Set 1: Code-Review-1\n\nWe can\u0027t really add new deliverables that were not included in the train cycle originally. Can you provide a little more context on this?","accounts_in_message":[],"_revision_number":1},{"id":"83f8c38a4523cc3e65be7a42c376f27ef32f9318","author":{"_account_id":28522,"name":"Hervé Beraud","email":"herveberaud.pro@gmail.com","username":"hberaud"},"date":"2020-03-30 14:59:00.000000000","message":"Patch Set 1: Code-Review-1\n\n(2 comments)\n\nI agree with Sean, also you\u0027ve some nits inline","accounts_in_message":[],"_revision_number":1},{"id":"45135fa87102e1f2bc151a102a4ffa52743058e7","author":{"_account_id":28223,"name":"Cedric Jeanneret","display_name":"cjeanner (Tengu)","email":"cjeanner@redhat.com","username":"cjeanner"},"date":"2020-03-30 14:59:20.000000000","message":"Patch Set 1:\n\n\u003e We can\u0027t really add new deliverables that were not included in the\n \u003e train cycle originally. Can you provide a little more context on\n \u003e this?\n\nHello Sean - of course, here are the info:\n- this is part of the Validation Framework, which is a new way to run validations (for the records, they were to be launched via the UI, which has been deprecated since stein (or even earlier? not sure)).\n- those \"new\" packages are a clean split of what we currently have in tripleoclient and part of the tripleo-validations content - the code itself isn\u0027t that new, more like a cleanup and refactoring\n\nIn addition, we have a need in OSP-16.1 (Red Hat OpenStack Platform). \"unfortunately\" this version will be based on stable/train - it would therefore need the upstream branch in order to be in a clean and consistent state, especially in order to avoid to do some ugly \"downstream-only\" patches within OSP code.\n\nDoes it answer your concerns? I fully understand them - but if this branching is refused, it will lead to a lot of trouble and inconsistency :/.\n\nFeel free to ask for more details!","accounts_in_message":[],"_revision_number":1},{"id":"673fa520769b4ca33cef93834ec2861b03aad159","tag":"autogenerated:zuul:check","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2020-03-30 14:59:30.000000000","message":"Patch Set 1: Verified-1\n\nBuild failed (check pipeline).  For information on how to proceed, see\nhttps://docs.opendev.org/opendev/infra-manual/latest/developers.html#automated-testing\n\n\n- openstack-tox-docs https://zuul.opendev.org/t/openstack/build/e083a8f20446424aba54d04b5030b366 : SUCCESS in 12m 00s\n- openstack-tox-validate https://zuul.opendev.org/t/openstack/build/42c442409765406f9a4cf46b7a02928e : FAILURE in 8m 13s\n- releases-tox-list-changes https://zuul.opendev.org/t/openstack/build/c8d5b756dc9e4d14ae605f3907119c97 : SUCCESS in 7m 38s","accounts_in_message":[],"_revision_number":1},{"id":"a1a9e50a4259776617d8a674dccb3441c828a8ff","author":{"_account_id":28223,"name":"Cedric Jeanneret","display_name":"cjeanner (Tengu)","email":"cjeanner@redhat.com","username":"cjeanner"},"date":"2020-03-30 15:01:15.000000000","message":"Uploaded patch set 2.","accounts_in_message":[],"_revision_number":2},{"id":"57394538e577a74d24ed884dcb8c403ae7c49250","author":{"_account_id":28223,"name":"Cedric Jeanneret","display_name":"cjeanner (Tengu)","email":"cjeanner@redhat.com","username":"cjeanner"},"date":"2020-03-30 15:02:11.000000000","message":"Patch Set 2:\n\n(2 comments)\n\nThanks Hervé - as said to Sean, feel free to ping me if you need more details!","accounts_in_message":[],"_revision_number":2},{"id":"fb344052bf92bb95467471aed2d15df72c0b4c71","tag":"autogenerated:zuul:check","author":{"_account_id":22348,"name":"Zuul","username":"zuul","tags":["SERVICE_USER"]},"date":"2020-03-30 15:23:36.000000000","message":"Patch Set 2: Verified+1\n\nBuild succeeded (check pipeline).\n\n- openstack-tox-docs https://zuul.opendev.org/t/openstack/build/45fd844574cc4121b23164db331b4c15 : SUCCESS in 13m 40s\n- openstack-tox-validate https://zuul.opendev.org/t/openstack/build/ec3e15c1806d46a09122451c6a1f88ee : SUCCESS in 10m 21s\n- releases-tox-list-changes https://zuul.opendev.org/t/openstack/build/6d8bf5e9e4844d289661a714b48660f3 : SUCCESS in 6m 54s","accounts_in_message":[],"_revision_number":2},{"id":"ea0fca69a57ce5ce4d3e9bb0a50af2c0d865552f","author":{"_account_id":28522,"name":"Hervé Beraud","email":"herveberaud.pro@gmail.com","username":"hberaud"},"date":"2020-03-30 16:02:25.000000000","message":"Patch Set 2: Code-Review+1\n\n\u003e \u003e We can\u0027t really add new deliverables that were not included in\n \u003e the\n \u003e \u003e train cycle originally. Can you provide a little more context on\n \u003e \u003e this?\n \u003e \n \u003e Hello Sean - of course, here are the info:\n \u003e - this is part of the Validation Framework, which is a new way to\n \u003e run validations (for the records, they were to be launched via the\n \u003e UI, which has been deprecated since stein (or even earlier? not\n \u003e sure)).\n \u003e - those \"new\" packages are a clean split of what we currently have\n \u003e in tripleoclient and part of the tripleo-validations content - the\n \u003e code itself isn\u0027t that new, more like a cleanup and refactoring\n\nI suppose the corresponding code was removed from tripleoclient on train, can you confirm? If not It could be worth to release this drop too, to avoid confusion.\n\n \u003e \n \u003e In addition, we have a need in OSP-16.1 (Red Hat OpenStack\n \u003e Platform). \"unfortunately\" this version will be based on\n \u003e stable/train - it would therefore need the upstream branch in order\n \u003e to be in a clean and consistent state, especially in order to avoid\n \u003e to do some ugly \"downstream-only\" patches within OSP code.\n\nI\u0027m not against introduce these new packages, if I\u0027m right they are only used by RedHat so if RedHat ask for something specific in their product I don\u0027t see any reasons to refuse them if they know what they do downstream.\n\nHowever I prefer to wait for Sean and Thierry feedbacks first.\n\n \u003e \n \u003e Does it answer your concerns? \n\npersonally yes\n\n \u003e I fully understand them - but if this\n \u003e branching is refused, it will lead to a lot of trouble and\n \u003e inconsistency :/.\n\nAs explained before I don\u0027t see reasons to refuse specific distro stuff especially if they are only used by a specific distro.\n\n \u003e \n \u003e Feel free to ask for more details!\n\nOnly +1 for now to lets time to discute about.","accounts_in_message":[],"_revision_number":2},{"id":"fe34a520f0188cce99cb2eb6e3c0e6b4fe41acdc","author":{"_account_id":28223,"name":"Cedric Jeanneret","display_name":"cjeanner (Tengu)","email":"cjeanner@redhat.com","username":"cjeanner"},"date":"2020-03-31 06:07:10.000000000","message":"Patch Set 2:\n\n\u003e \u003e \u003e We can\u0027t really add new deliverables that were not included in\n \u003e \u003e the\n \u003e \u003e \u003e train cycle originally. Can you provide a little more context\n \u003e on\n \u003e \u003e \u003e this?\n \u003e \u003e\n \u003e \u003e Hello Sean - of course, here are the info:\n \u003e \u003e - this is part of the Validation Framework, which is a new way to\n \u003e \u003e run validations (for the records, they were to be launched via\n \u003e the\n \u003e \u003e UI, which has been deprecated since stein (or even earlier? not\n \u003e \u003e sure)).\n \u003e \u003e - those \"new\" packages are a clean split of what we currently\n \u003e have\n \u003e \u003e in tripleoclient and part of the tripleo-validations content -\n \u003e the\n \u003e \u003e code itself isn\u0027t that new, more like a cleanup and refactoring\n \u003e \n \u003e I suppose the corresponding code was removed from tripleoclient on\n \u003e train, can you confirm? If not It could be worth to release this\n \u003e drop too, to avoid confusion.\n\nIt is intended yes. Not all is done yet on that side, the -libs is under dev in order to ensure a 1:1 match and proper unit testing.\nSame goes for the validations that will be moved to -common.\n\n \u003e \n \u003e \u003e\n \u003e \u003e In addition, we have a need in OSP-16.1 (Red Hat OpenStack\n \u003e \u003e Platform). \"unfortunately\" this version will be based on\n \u003e \u003e stable/train - it would therefore need the upstream branch in\n \u003e order\n \u003e \u003e to be in a clean and consistent state, especially in order to\n \u003e avoid\n \u003e \u003e to do some ugly \"downstream-only\" patches within OSP code.\n \u003e \n \u003e I\u0027m not against introduce these new packages, if I\u0027m right they are\n \u003e only used by RedHat so if RedHat ask for something specific in\n \u003e their product I don\u0027t see any reasons to refuse them if they know\n \u003e what they do downstream.\n \u003e \n \u003e However I prefer to wait for Sean and Thierry feedbacks first.\n \u003e \n \u003e \u003e\n \u003e \u003e Does it answer your concerns?\n \u003e \n \u003e personally yes\n \u003e \n \u003e \u003e I fully understand them - but if this\n \u003e \u003e branching is refused, it will lead to a lot of trouble and\n \u003e \u003e inconsistency :/.\n \u003e \n \u003e As explained before I don\u0027t see reasons to refuse specific distro\n \u003e stuff especially if they are only used by a specific distro.\n \u003e \n \u003e \u003e\n \u003e \u003e Feel free to ask for more details!\n \u003e \n \u003e Only +1 for now to lets time to discute about.","accounts_in_message":[],"_revision_number":2},{"id":"422d47dba8ba3e99cbb2975e40e26b5979a73829","author":{"_account_id":9592,"name":"wes hayutin","email":"whayutin@redhat.com","username":"weshayutin"},"date":"2020-03-31 14:49:00.000000000","message":"Patch Set 2: Code-Review+1","accounts_in_message":[],"_revision_number":2},{"id":"16e5777e735fa0a8eb6ed941bdfc88f1bfb72da0","author":{"_account_id":28522,"name":"Hervé Beraud","email":"herveberaud.pro@gmail.com","username":"hberaud"},"date":"2020-03-31 14:52:26.000000000","message":"Patch Set 2:\n\n\u003e \u003e \u003e \u003e We can\u0027t really add new deliverables that were not included\n \u003e in\n \u003e \u003e \u003e the\n \u003e \u003e \u003e \u003e train cycle originally. Can you provide a little more context\n \u003e \u003e on\n \u003e \u003e \u003e \u003e this?\n \u003e \u003e \u003e\n \u003e \u003e \u003e Hello Sean - of course, here are the info:\n \u003e \u003e \u003e - this is part of the Validation Framework, which is a new way\n \u003e to\n \u003e \u003e \u003e run validations (for the records, they were to be launched via\n \u003e \u003e the\n \u003e \u003e \u003e UI, which has been deprecated since stein (or even earlier? not\n \u003e \u003e \u003e sure)).\n \u003e \u003e \u003e - those \"new\" packages are a clean split of what we currently\n \u003e \u003e have\n \u003e \u003e \u003e in tripleoclient and part of the tripleo-validations content -\n \u003e \u003e the\n \u003e \u003e \u003e code itself isn\u0027t that new, more like a cleanup and refactoring\n \u003e \u003e\n \u003e \u003e I suppose the corresponding code was removed from tripleoclient\n \u003e on\n \u003e \u003e train, can you confirm? If not It could be worth to release this\n \u003e \u003e drop too, to avoid confusion.\n \u003e \n \u003e It is intended yes. Not all is done yet on that side, the -libs is\n \u003e under dev in order to ensure a 1:1 match and proper unit testing.\n \u003e Same goes for the validations that will be moved to -common.\n \u003e \n \u003e \u003e\n \u003e \u003e \u003e\n \u003e \u003e \u003e In addition, we have a need in OSP-16.1 (Red Hat OpenStack\n \u003e \u003e \u003e Platform). \"unfortunately\" this version will be based on\n \u003e \u003e \u003e stable/train - it would therefore need the upstream branch in\n \u003e \u003e order\n \u003e \u003e \u003e to be in a clean and consistent state, especially in order to\n \u003e \u003e avoid\n \u003e \u003e \u003e to do some ugly \"downstream-only\" patches within OSP code.\n \u003e \u003e\n \u003e \u003e I\u0027m not against introduce these new packages, if I\u0027m right they\n \u003e are\n \u003e \u003e only used by RedHat so if RedHat ask for something specific in\n \u003e \u003e their product I don\u0027t see any reasons to refuse them if they know\n \u003e \u003e what they do downstream.\n \u003e \u003e\n \u003e \u003e However I prefer to wait for Sean and Thierry feedbacks first.\n \u003e \u003e\n \u003e \u003e \u003e\n \u003e \u003e \u003e Does it answer your concerns?\n \u003e \u003e\n \u003e \u003e personally yes\n \u003e \u003e\n \u003e \u003e \u003e I fully understand them - but if this\n \u003e \u003e \u003e branching is refused, it will lead to a lot of trouble and\n \u003e \u003e \u003e inconsistency :/.\n \u003e \u003e\n \u003e \u003e As explained before I don\u0027t see reasons to refuse specific distro\n \u003e \u003e stuff especially if they are only used by a specific distro.\n \u003e \u003e\n \u003e \u003e \u003e\n \u003e \u003e \u003e Feel free to ask for more details!\n \u003e \u003e\n \u003e \u003e Only +1 for now to lets time to discute about.\n\nThanks for your responses.\n\nI\u0027ll +2 if other release team cores members are ok with that.","accounts_in_message":[],"_revision_number":2},{"id":"3d96507c7a2a3ed42e0aaf0bfc09b24666075a3c","author":{"_account_id":308,"name":"Thierry Carrez","email":"thierry@openstack.org","username":"ttx"},"date":"2020-04-01 09:18:13.000000000","message":"Patch Set 2:\n\nI\u0027m fine with granting an MembershipFreeze exception on this one as it is a cycle-trailing project","accounts_in_message":[],"_revision_number":2},{"id":"217b5adaacb7dcf610eeceb3b153c3f7afed1a4c","author":{"_account_id":308,"name":"Thierry Carrez","email":"thierry@openstack.org","username":"ttx"},"date":"2020-04-01 09:18:28.000000000","message":"Patch Set 2: Code-Review+2","accounts_in_message":[],"_revision_number":2},{"id":"39ec7eb2616dd8f5abd661f7b9eb12949d0dc883","author":{"_account_id":308,"name":"Thierry Carrez","email":"thierry@openstack.org","username":"ttx"},"date":"2020-04-01 09:19:05.000000000","message":"Patch Set 2: -Code-Review\n\nOh, misread that.. thought it was a ussuri thing. let me think about it a bit more","accounts_in_message":[],"_revision_number":2},{"id":"b27fb3e53f2d43276c126f4347bd1520613d234b","author":{"_account_id":308,"name":"Thierry Carrez","email":"thierry@openstack.org","username":"ttx"},"date":"2020-04-01 09:46:04.000000000","message":"Patch Set 2:\n\nSo.. As Sean said, adding new deliverables to old OpenStack releases is a big no-no. For us it\u0027s akin to rewriting history and pretending it happened differently, confusing downstream consumers of the code as there is no way for us to communicate that type of change.\n\nObviously, here we have a corner case, since the only known downstream consumer of TripleO is the Red Hat product. It is unlikely that this change would create any confusion. The do-not-rewrite-history rule is part of a set of practices designed to enable multi-vendor projects, so they can appear weird in a single-vendor scenario.\n\nWe\u0027ll discuss this at the release meeting tomorrow, potentially punting to the TC for a final call. Personally I see three options on the table:\n\n1- Granting an exception to the rule, recognizing it\u0027s a corner case. As downstream usage is centralized, confusion is unlikely.\n\n2- Handle train releases of validations-* out of the openstack/releases repository. Like for Kayobe (which had its own \u0027train\u0027 releases before it was added officially to OpenStack), use direct tags to produce releases on that specific branch. That means the tripleo team has to special-case how to do train releases, but avoids setting a precedent.\n\n3- Fully recognize that TripleO is not likely to attract multiple vendors, and push it out of OpenStack multi-vendor governance and rules. TripleO could have its own domain on opendev and write its own rulebook as to what is acceptable to do in old releases.\n\nNone of those options is ideal.","accounts_in_message":[],"_revision_number":2},{"id":"4e70c3efa4c75a059ad63182ce3bd373f38ddcdb","author":{"_account_id":28223,"name":"Cedric Jeanneret","display_name":"cjeanner (Tengu)","email":"cjeanner@redhat.com","username":"cjeanner"},"date":"2020-04-01 10:36:13.000000000","message":"Patch Set 2:\n\nHello there,\n\nhm, seeing the amount of questions|issues this raises, I\u0027ll check with our side if we can take what\u0027s coming from master (and, when branched, from ussuri) - the release cycle of osp has changed a bit and will heavily rely on backports anyway... Hopefully I\u0027ll get some more inputs today already and be able to update this change - maybe even drop it. That would probably be the best.","accounts_in_message":[],"_revision_number":2},{"id":"2852b18c4505a156cbd2d435f78217f567554bc3","author":{"_account_id":28223,"name":"Cedric Jeanneret","display_name":"cjeanner (Tengu)","email":"cjeanner@redhat.com","username":"cjeanner"},"date":"2020-04-01 12:38:50.000000000","message":"Abandoned\n\nAfter some discussions downstream, we\u0027re able to track the master branch for now and wait for the split to ussuri.\n\nSo we won\u0027t be the exception :).\n\nThank you for all your feedbacks and info!","accounts_in_message":[],"_revision_number":2},{"id":"cd67b15e3b100099602913e6d96bb97857a3bddb","author":{"_account_id":11904,"name":"Sean McGinnis","email":"sean.mcginnis@gmail.com","username":"SeanM"},"date":"2020-04-01 12:53:30.000000000","message":"Patch Set 2:\n\nThanks Cédric!","accounts_in_message":[],"_revision_number":2}],"current_revision_number":2,"current_revision":"0ee36ec62de2d4aba5266075bd63507c20958ba4","revisions":{"1a029670e8ae9c3db0ad6e52cafb33222411609c":{"kind":"REWORK","_number":1,"created":"2020-03-30 14:45:28.000000000","uploader":{"_account_id":28223,"name":"Cedric Jeanneret","display_name":"cjeanner (Tengu)","email":"cjeanner@redhat.com","username":"cjeanner"},"ref":"refs/changes/67/715967/1","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/releases","ref":"refs/changes/67/715967/1","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/releases refs/changes/67/715967/1 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/releases refs/changes/67/715967/1 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/releases refs/changes/67/715967/1 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/releases refs/changes/67/715967/1"}}},"commit":{"parents":[{"commit":"7b4aac601cdd3c8d147fd9e029a42d8a756c89ce","subject":"Merge \"Release OSA Rocky\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/releases/commit/7b4aac601cdd3c8d147fd9e029a42d8a756c89ce"}]}],"author":{"name":"Cédric Jeanneret","email":"cjeanner@redhat.com","date":"2020-03-30 14:44:34.000000000","tz":120},"committer":{"name":"Cédric Jeanneret","email":"cjeanner@redhat.com","date":"2020-03-30 14:45:26.000000000","tz":120},"subject":"Add validations-common and validations-lib deliverables to train","message":"Add validations-common and validations-lib deliverables to train\n\nChange-Id: I310e30898816d2c83a3f52c22feb550420fc11ea\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/releases/commit/1a029670e8ae9c3db0ad6e52cafb33222411609c"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/releases/commit/1a029670e8ae9c3db0ad6e52cafb33222411609c"}]},"branch":"refs/heads/master"},"0ee36ec62de2d4aba5266075bd63507c20958ba4":{"kind":"REWORK","_number":2,"created":"2020-03-30 15:01:15.000000000","uploader":{"_account_id":28223,"name":"Cedric Jeanneret","display_name":"cjeanner (Tengu)","email":"cjeanner@redhat.com","username":"cjeanner"},"ref":"refs/changes/67/715967/2","fetch":{"anonymous http":{"url":"https://review.opendev.org/openstack/releases","ref":"refs/changes/67/715967/2","commands":{"Checkout":"git fetch https://review.opendev.org/openstack/releases refs/changes/67/715967/2 \u0026\u0026 git checkout FETCH_HEAD","Cherry Pick":"git fetch https://review.opendev.org/openstack/releases refs/changes/67/715967/2 \u0026\u0026 git cherry-pick FETCH_HEAD","Format Patch":"git fetch https://review.opendev.org/openstack/releases refs/changes/67/715967/2 \u0026\u0026 git format-patch -1 --stdout FETCH_HEAD","Pull":"git pull https://review.opendev.org/openstack/releases refs/changes/67/715967/2"}}},"commit":{"parents":[{"commit":"7b4aac601cdd3c8d147fd9e029a42d8a756c89ce","subject":"Merge \"Release OSA Rocky\"","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/releases/commit/7b4aac601cdd3c8d147fd9e029a42d8a756c89ce"}]}],"author":{"name":"Cédric Jeanneret","email":"cjeanner@redhat.com","date":"2020-03-30 14:44:34.000000000","tz":120},"committer":{"name":"Cédric Jeanneret","email":"cjeanner@redhat.com","date":"2020-03-30 15:00:32.000000000","tz":120},"subject":"Add validations-common and validations-lib deliverables to train","message":"Add validations-common and validations-lib deliverables to train\n\nChange-Id: I310e30898816d2c83a3f52c22feb550420fc11ea\n","web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/releases/commit/0ee36ec62de2d4aba5266075bd63507c20958ba4"}],"resolve_conflicts_web_links":[{"name":"gitea","tooltip":"Open in GitWeb","url":"https://opendev.org/openstack/releases/commit/0ee36ec62de2d4aba5266075bd63507c20958ba4"}]},"branch":"refs/heads/master"}},"requirements":[],"submit_records":[],"submit_requirements":[]}
