On Fri, Jun 9, 2023 at 1:01 PM Ilya Maximets <[email protected]> wrote: > > On 6/9/23 18:39, Mark Michelson wrote: > > Thanks for the reviews. I added Han's, Numan's, and Dumitru's acks to > > all patches, updated the issue Han pointed out in patch 3, and pushed > > the series to main. > > Hey, Mark. This series broke the documentation build in CI: > > https://github.com/ovsrobot/ovn/actions/runs/5225327862/jobs/9434647125#step:7:3650 > > Could you take a look? GHA is all red. > > Best regards, Ilya Maximets.
I just pushed a fix to unblock the build. https://github.com/ovn-org/ovn/commit/38644f0590ccc1ad4a09e794c8507430a4529cad Thanks, Han > > > > > On 6/8/23 11:27, Numan Siddique wrote: > >> On Thu, Jun 8, 2023 at 2:04 AM Han Zhou <[email protected]> wrote: > >>> > >>> On Wed, Jun 7, 2023 at 12:47 PM Mark Michelson <[email protected]> wrote: > >>>> > >>>> The current state of development in OVN has resulted in many discussions > >>>> happening about how things should change. The main points of the > >>>> discussions have been: > >>>> > >>>> * Recent releases have been much more bare than releases made two or > >>>> three years ago. > >>>> * Supporting as many versions of OVN as we have is a maintenance burden. > >>>> * It is not well-defined what the lifetime of non-LTS OVN versions is. > >>>> * We rarely, if ever, make sub-releases of released OVN versions, so it > >>>> sometimes feels like backporting bugfixes is pointless. > >>>> * It is not always clear as a maintainer which branches should receive > >>>> backports of fixes. > >>>> > >>>> This set of changes aims to address these issues, while also correcting > >>>> some existing errors in the documentation. > >>>> > >>>> The reason for the large number of patches is to allow for easy > >>>> discussion about each aspect of the change. This can allow for us to > >>>> reach conclusions for certain aspects and get them integrated into the > >>>> documentation while allowing for other aspects to continue to be fleshed > >>>> out. > >>>> > >>> > >>> Thanks Mark. LGTM except a nit in patch-3, which can be fixed while > >>> merging. For the series: > >>> > >>> Acked-by: Han Zhou <[email protected]> > >> > >> LGTM too > >> > >> Acked-by: Numan Siddique <[email protected]> > >> > >> Numan > >> > >>> > >>>> --- > >>>> V3 -> V4 differences: > >>>> * Rebased > >>>> * Added Dumitru's acks from v1 to patches 1, 4, 5, and 8. > >>>> * Updated patch 7 to have fewer scheduled point-releases based on > >>>> feedback from IRC development meetings. > >>>> > >>>> V2 -> V3 differences: > >>>> * Patch 2: Updated commit message to not mention giving longer for soft > >>>> freeze. > >>>> * Patch 3: Use "branch" instead of "version" when referring to the > >>>> development code from which releases/versions are made. > >>>> * Patch 4: Made it more explicit that LTS versions' receive bug fixes > >>>> for two years. > >>>> * Patch 9: Use "branch" instead of "version" here as well. > >>>> > >>>> V1 -> V2 differences > >>>> * Rebased > >>>> * Patch 2: Changed from one month to two week soft freeze timeframe. > >>>> * Patch 7: Updated more files to change "master" to "main". > >>>> * Patch 9: Added an example for clarification. > >>>> > >>>> V2 does not address Dumitru's questions from patch 6 about automating > >>>> minor releases. > >>>> > >>>> *** BLURB HERE *** > >>>> > >>>> Mark Michelson (9): > >>>> release-process: Use more accurate example releases. > >>>> release-process: Switch to two releases per year. > >>>> release-policy: Change "release" to "branch" in LTS section. > >>>> release-process: State release policy for LTS versions. > >>>> release-process: Add section for standard-term support releases. > >>>> release-policy: Document when versions receive new releases. > >>>> Documentation: Change "master" to "main". > >>>> backporting-patches: Fix inaccuracies in the document. > >>>> backporting-patches: Add concrete policy for where backports go. > >>>> > >>>> .../internals/committer-grant-revocation.rst | 2 +- > >>>> .../internals/committer-responsibilities.rst | 8 +- > >>>> .../contributing/backporting-patches.rst | 146 ++++++++++++------ > >>>> Documentation/internals/release-process.rst | 86 +++++++---- > >>>> 4 files changed, 158 insertions(+), 84 deletions(-) > >>>> > >>>> -- > >>>> 2.39.2 > >>>> > >>>> _______________________________________________ > >>>> dev mailing list > >>>> [email protected] > >>>> https://mail.openvswitch.org/mailman/listinfo/ovs-dev > >>> _______________________________________________ > >>> dev mailing list > >>> [email protected] > >>> https://mail.openvswitch.org/mailman/listinfo/ovs-dev > >> > > > > _______________________________________________ > > dev mailing list > > [email protected] > > https://mail.openvswitch.org/mailman/listinfo/ovs-dev > _______________________________________________ dev mailing list [email protected] https://mail.openvswitch.org/mailman/listinfo/ovs-dev
