Re: [Xen-devel] [PATCH] Introduce a description of a new optional tag for Backports
Hi, I have noticed Wei began to use the tag today. It reminded me that I never followed-up on the patch, sorry for that. On 08/11/2019 19:09, Stefano Stabellini wrote: Signed-off-by: Stefano Stabellini CC: jbeul...@suse.com CC: george.dun...@citrix.com CC: jul...@xen.org CC: lars.ku...@citrix.com CC: andrew.coop...@citrix.com CC: ian.jack...@eu.citrix.com CC: konrad.w...@oracle.com CC: w...@xen.org --- docs/process/backport-tag.pandoc | 23 +++ 1 file changed, 23 insertions(+) create mode 100644 docs/process/backport-tag.pandoc diff --git a/docs/process/backport-tag.pandoc b/docs/process/backport-tag.pandoc new file mode 100644 index 00..e570efdcc8 --- /dev/null +++ b/docs/process/backport-tag.pandoc @@ -0,0 +1,23 @@ +Backport Tag + + +A backport tag is an optional tag in the commit message to request a +given commit to be backported to the stable trees: + +Backport: all + +It marks a commit for being a candidate for backports to all relevant +trees. + +Backport: 4.9+ + +It marks a commit for being a candidate for backports to all stable +trees from 4.9 onward. + +Maintainers request the Backport tag to be added on commit. +Contributors are also welcome to mark their patches with the Backport +tag when they deem appropriate. Maintainers will request for it to be +removed when that is not the case. + +Please note that the Backport tag is a **request** for backport, which +will still need to be evaluated by the stable tree maintainers. This proposal look good to me. Are you planning to resend the patch with George's suggestion? Cheers, -- Julien Grall ___ Xen-devel mailing list Xen-devel@lists.xenproject.org https://lists.xenproject.org/mailman/listinfo/xen-devel
Re: [Xen-devel] [PATCH] Introduce a description of a new optional tag for Backports
On 12/11/2019, 11:10, "Stefano Stabellini" wrote:
On Tue, 12 Nov 2019, Ian Jackson wrote:
> Anthony PERARD writes ("Re: [Xen-devel] [PATCH] Introduce a description
of a new optional tag for Backports"):
> > Should we describe the Fixes: tag as well? These would have a similar
> > purpose to the backport tag, I mean it could help figure out which
> > commit to backport to which tree.
>
> Good point.
Yes, good idea.
Lars, I think we are already in agreement.
You can find the description of "Fixes" here in Linux
Documentation/process/submitting-patches.rst.
It would be good to get Jan's ACK at least
Lars
___
Xen-devel mailing list
Xen-devel@lists.xenproject.org
https://lists.xenproject.org/mailman/listinfo/xen-devel
Re: [Xen-devel] [PATCH] Introduce a description of a new optional tag for Backports
On Tue, 12 Nov 2019, Ian Jackson wrote:
> Anthony PERARD writes ("Re: [Xen-devel] [PATCH] Introduce a description of a
> new optional tag for Backports"):
> > Should we describe the Fixes: tag as well? These would have a similar
> > purpose to the backport tag, I mean it could help figure out which
> > commit to backport to which tree.
>
> Good point.
Yes, good idea.
Lars, I think we are already in agreement.
You can find the description of "Fixes" here in Linux
Documentation/process/submitting-patches.rst.
___
Xen-devel mailing list
Xen-devel@lists.xenproject.org
https://lists.xenproject.org/mailman/listinfo/xen-devel
Re: [Xen-devel] [PATCH] Introduce a description of a new optional tag for Backports
Anthony PERARD writes ("Re: [Xen-devel] [PATCH] Introduce a description of a
new optional tag for Backports"):
> Should we describe the Fixes: tag as well? These would have a similar
> purpose to the backport tag, I mean it could help figure out which
> commit to backport to which tree.
Good point.
Ian.
___
Xen-devel mailing list
Xen-devel@lists.xenproject.org
https://lists.xenproject.org/mailman/listinfo/xen-devel
Re: [Xen-devel] [PATCH] Introduce a description of a new optional tag for Backports
On Fri, Nov 08, 2019 at 11:09:52AM -0800, Stefano Stabellini wrote: > +Backport Tag > + > + > +A backport tag is an optional tag in the commit message to request a > +given commit to be backported to the stable trees: > + > +Backport: all [...] Should we describe the Fixes: tag as well? These would have a similar purpose to the backport tag, I mean it could help figure out which commit to backport to which tree. -- Anthony PERARD ___ Xen-devel mailing list Xen-devel@lists.xenproject.org https://lists.xenproject.org/mailman/listinfo/xen-devel
Re: [Xen-devel] [PATCH] Introduce a description of a new optional tag for Backports
On 11/11/2019, 11:03, "Stefano Stabellini" wrote: On Mon, 11 Nov 2019, Lars Kurth wrote: > On 11/11/2019, 08:12, "George Dunlap" wrote: > > On 11/8/19 7:09 PM, Stefano Stabellini wrote: > > Signed-off-by: Stefano Stabellini > > CC: jbeul...@suse.com > > CC: george.dun...@citrix.com > > CC: jul...@xen.org > > CC: lars.ku...@citrix.com > > CC: andrew.coop...@citrix.com > > CC: ian.jack...@eu.citrix.com > > CC: konrad.w...@oracle.com > > CC: w...@xen.org > > --- > > docs/process/backport-tag.pandoc | 23 +++ > > 1 file changed, 23 insertions(+) > > create mode 100644 docs/process/backport-tag.pandoc > > > > diff --git a/docs/process/backport-tag.pandoc b/docs/process/backport-tag.pandoc > > new file mode 100644 > > index 00..e570efdcc8 > > --- /dev/null > > +++ b/docs/process/backport-tag.pandoc > > @@ -0,0 +1,23 @@ > > +Backport Tag > > + > > + > > +A backport tag is an optional tag in the commit message to request a > > +given commit to be backported to the stable trees: > > + > > +Backport: all > > + > > +It marks a commit for being a candidate for backports to all relevant > > +trees. > > + > > +Backport: 4.9+ > > + > > +It marks a commit for being a candidate for backports to all stable > > +trees from 4.9 onward. > > + > > +Maintainers request the Backport tag to be added on commit. > > +Contributors are also welcome to mark their patches with the Backport > > +tag when they deem appropriate. Maintainers will request for it to be > > +removed when that is not the case. > > + > > +Please note that the Backport tag is a **request** for backport, which > > +will still need to be evaluated by the stable tree maintainers. > > The text and the idea both look good to me. Thank you! > But it seems kind of balkanized to put it in its own file. Would it be > better to try to make a slightly more general bit of content? Either > about the backport process, or about tags in general? Yeah, it was never meant to stay in its own separate file. I thought it would get merged into a bigger file about the whole process when it gets submitted. > It should be in https://wiki.xenproject.org/wiki/Submitting_Xen_Project_Patches#What_is_in_a_patch.3F > What is currently missing is > - Release-Acked-by > - The new proposed tag > > But maybe we should have a master document in tree, which defines the tags in use > And then I can refer to it from https://wiki.xenproject.org/wiki/Submitting_Xen_Project_Patches#What_is_in_a_patch.3F For now, would you like me to add the text to the wiki at https://wiki.xenproject.org/wiki/Submitting_Xen_Project_Patches#What_is_in_a_patch.3F ? Or would you rather do it? No: I can do it. Just ping me when we are in agreement about the proposal Lars ___ Xen-devel mailing list Xen-devel@lists.xenproject.org https://lists.xenproject.org/mailman/listinfo/xen-devel
Re: [Xen-devel] [PATCH] Introduce a description of a new optional tag for Backports
On Mon, 11 Nov 2019, Lars Kurth wrote: > On 11/11/2019, 08:12, "George Dunlap" wrote: > > On 11/8/19 7:09 PM, Stefano Stabellini wrote: > > Signed-off-by: Stefano Stabellini > > CC: jbeul...@suse.com > > CC: george.dun...@citrix.com > > CC: jul...@xen.org > > CC: lars.ku...@citrix.com > > CC: andrew.coop...@citrix.com > > CC: ian.jack...@eu.citrix.com > > CC: konrad.w...@oracle.com > > CC: w...@xen.org > > --- > > docs/process/backport-tag.pandoc | 23 +++ > > 1 file changed, 23 insertions(+) > > create mode 100644 docs/process/backport-tag.pandoc > > > > diff --git a/docs/process/backport-tag.pandoc > b/docs/process/backport-tag.pandoc > > new file mode 100644 > > index 00..e570efdcc8 > > --- /dev/null > > +++ b/docs/process/backport-tag.pandoc > > @@ -0,0 +1,23 @@ > > +Backport Tag > > + > > + > > +A backport tag is an optional tag in the commit message to request a > > +given commit to be backported to the stable trees: > > + > > +Backport: all > > + > > +It marks a commit for being a candidate for backports to all relevant > > +trees. > > + > > +Backport: 4.9+ > > + > > +It marks a commit for being a candidate for backports to all stable > > +trees from 4.9 onward. > > + > > +Maintainers request the Backport tag to be added on commit. > > +Contributors are also welcome to mark their patches with the Backport > > +tag when they deem appropriate. Maintainers will request for it to be > > +removed when that is not the case. > > + > > +Please note that the Backport tag is a **request** for backport, which > > +will still need to be evaluated by the stable tree maintainers. > > The text and the idea both look good to me. Thank you! > But it seems kind of balkanized to put it in its own file. Would it be > better to try to make a slightly more general bit of content? Either > about the backport process, or about tags in general? Yeah, it was never meant to stay in its own separate file. I thought it would get merged into a bigger file about the whole process when it gets submitted. > It should be in > https://wiki.xenproject.org/wiki/Submitting_Xen_Project_Patches#What_is_in_a_patch.3F > What is currently missing is > - Release-Acked-by > - The new proposed tag > > But maybe we should have a master document in tree, which defines the tags in > use > And then I can refer to it from > https://wiki.xenproject.org/wiki/Submitting_Xen_Project_Patches#What_is_in_a_patch.3F > For now, would you like me to add the text to the wiki at https://wiki.xenproject.org/wiki/Submitting_Xen_Project_Patches#What_is_in_a_patch.3F ? Or would you rather do it?___ Xen-devel mailing list Xen-devel@lists.xenproject.org https://lists.xenproject.org/mailman/listinfo/xen-devel
Re: [Xen-devel] [PATCH] Introduce a description of a new optional tag for Backports
On 11/11/2019, 08:12, "George Dunlap" wrote: On 11/8/19 7:09 PM, Stefano Stabellini wrote: > Signed-off-by: Stefano Stabellini > CC: jbeul...@suse.com > CC: george.dun...@citrix.com > CC: jul...@xen.org > CC: lars.ku...@citrix.com > CC: andrew.coop...@citrix.com > CC: ian.jack...@eu.citrix.com > CC: konrad.w...@oracle.com > CC: w...@xen.org > --- > docs/process/backport-tag.pandoc | 23 +++ > 1 file changed, 23 insertions(+) > create mode 100644 docs/process/backport-tag.pandoc > > diff --git a/docs/process/backport-tag.pandoc b/docs/process/backport-tag.pandoc > new file mode 100644 > index 00..e570efdcc8 > --- /dev/null > +++ b/docs/process/backport-tag.pandoc > @@ -0,0 +1,23 @@ > +Backport Tag > + > + > +A backport tag is an optional tag in the commit message to request a > +given commit to be backported to the stable trees: > + > +Backport: all > + > +It marks a commit for being a candidate for backports to all relevant > +trees. > + > +Backport: 4.9+ > + > +It marks a commit for being a candidate for backports to all stable > +trees from 4.9 onward. > + > +Maintainers request the Backport tag to be added on commit. > +Contributors are also welcome to mark their patches with the Backport > +tag when they deem appropriate. Maintainers will request for it to be > +removed when that is not the case. > + > +Please note that the Backport tag is a **request** for backport, which > +will still need to be evaluated by the stable tree maintainers. The text and the idea both look good to me. But it seems kind of balkanized to put it in its own file. Would it be better to try to make a slightly more general bit of content? Either about the backport process, or about tags in general? It should be in https://wiki.xenproject.org/wiki/Submitting_Xen_Project_Patches#What_is_in_a_patch.3F What is currently missing is - Release-Acked-by - The new proposed tag But maybe we should have a master document in tree, which defines the tags in use And then I can refer to it from https://wiki.xenproject.org/wiki/Submitting_Xen_Project_Patches#What_is_in_a_patch.3F Regards Lars ___ Xen-devel mailing list Xen-devel@lists.xenproject.org https://lists.xenproject.org/mailman/listinfo/xen-devel
Re: [Xen-devel] [PATCH] Introduce a description of a new optional tag for Backports
On 11/8/19 7:09 PM, Stefano Stabellini wrote: > Signed-off-by: Stefano Stabellini > CC: jbeul...@suse.com > CC: george.dun...@citrix.com > CC: jul...@xen.org > CC: lars.ku...@citrix.com > CC: andrew.coop...@citrix.com > CC: ian.jack...@eu.citrix.com > CC: konrad.w...@oracle.com > CC: w...@xen.org > --- > docs/process/backport-tag.pandoc | 23 +++ > 1 file changed, 23 insertions(+) > create mode 100644 docs/process/backport-tag.pandoc > > diff --git a/docs/process/backport-tag.pandoc > b/docs/process/backport-tag.pandoc > new file mode 100644 > index 00..e570efdcc8 > --- /dev/null > +++ b/docs/process/backport-tag.pandoc > @@ -0,0 +1,23 @@ > +Backport Tag > + > + > +A backport tag is an optional tag in the commit message to request a > +given commit to be backported to the stable trees: > + > +Backport: all > + > +It marks a commit for being a candidate for backports to all relevant > +trees. > + > +Backport: 4.9+ > + > +It marks a commit for being a candidate for backports to all stable > +trees from 4.9 onward. > + > +Maintainers request the Backport tag to be added on commit. > +Contributors are also welcome to mark their patches with the Backport > +tag when they deem appropriate. Maintainers will request for it to be > +removed when that is not the case. > + > +Please note that the Backport tag is a **request** for backport, which > +will still need to be evaluated by the stable tree maintainers. The text and the idea both look good to me. But it seems kind of balkanized to put it in its own file. Would it be better to try to make a slightly more general bit of content? Either about the backport process, or about tags in general? (This would simply be renaming the file; not expecting you to generate extra content.) -George ___ Xen-devel mailing list Xen-devel@lists.xenproject.org https://lists.xenproject.org/mailman/listinfo/xen-devel
Re: [Xen-devel] [PATCH] Introduce a description of a new optional tag for Backports
On Fri, Nov 08, 2019 at 11:09:52AM -0800, Stefano Stabellini wrote: > Signed-off-by: Stefano Stabellini Acked-by: Wei Liu ___ Xen-devel mailing list Xen-devel@lists.xenproject.org https://lists.xenproject.org/mailman/listinfo/xen-devel
Re: [Xen-devel] [PATCH] Introduce a description of a new optional tag for Backports
Stefano Stabellini writes ("[PATCH] Introduce a description of a new optional
tag for Backports"):
> Signed-off-by: Stefano Stabellini
+2
Acked-by: Ian Jackson
___
Xen-devel mailing list
Xen-devel@lists.xenproject.org
https://lists.xenproject.org/mailman/listinfo/xen-devel
[Xen-devel] [PATCH] Introduce a description of a new optional tag for Backports
Signed-off-by: Stefano Stabellini CC: jbeul...@suse.com CC: george.dun...@citrix.com CC: jul...@xen.org CC: lars.ku...@citrix.com CC: andrew.coop...@citrix.com CC: ian.jack...@eu.citrix.com CC: konrad.w...@oracle.com CC: w...@xen.org --- docs/process/backport-tag.pandoc | 23 +++ 1 file changed, 23 insertions(+) create mode 100644 docs/process/backport-tag.pandoc diff --git a/docs/process/backport-tag.pandoc b/docs/process/backport-tag.pandoc new file mode 100644 index 00..e570efdcc8 --- /dev/null +++ b/docs/process/backport-tag.pandoc @@ -0,0 +1,23 @@ +Backport Tag + + +A backport tag is an optional tag in the commit message to request a +given commit to be backported to the stable trees: + +Backport: all + +It marks a commit for being a candidate for backports to all relevant +trees. + +Backport: 4.9+ + +It marks a commit for being a candidate for backports to all stable +trees from 4.9 onward. + +Maintainers request the Backport tag to be added on commit. +Contributors are also welcome to mark their patches with the Backport +tag when they deem appropriate. Maintainers will request for it to be +removed when that is not the case. + +Please note that the Backport tag is a **request** for backport, which +will still need to be evaluated by the stable tree maintainers. -- 2.17.1 ___ Xen-devel mailing list Xen-devel@lists.xenproject.org https://lists.xenproject.org/mailman/listinfo/xen-devel
