* Monitoring the status of your own patches on patchwork? @ 2021-12-11 19:06 Toke Høiland-Jørgensen 2021-12-13 15:08 ` Jakub Kicinski 2021-12-13 18:37 ` Simon Glass 0 siblings, 2 replies; 11+ messages in thread From: Toke Høiland-Jørgensen @ 2021-12-11 19:06 UTC (permalink / raw) To: workflows Hi everyone! I was wondering if anyone had any nifty tricks or utilities to help monitor the status of your own patch submissions on a patchwork instance? Things like getting a notification when a patch changes status or when one of the automated checks on that particular patchwork instance fails for a patch you submitted? I can just filter the patchwork web interface to my name, of course, but I would like something that can be automated so I can make the stuff I care about show up in my inbox :) Many thanks in advance! -Toke ^ permalink raw reply [flat|nested] 11+ messages in thread
* Re: Monitoring the status of your own patches on patchwork? 2021-12-11 19:06 Monitoring the status of your own patches on patchwork? Toke Høiland-Jørgensen @ 2021-12-13 15:08 ` Jakub Kicinski 2021-12-13 15:48 ` Toke Høiland-Jørgensen 2021-12-13 18:37 ` Simon Glass 1 sibling, 1 reply; 11+ messages in thread From: Jakub Kicinski @ 2021-12-13 15:08 UTC (permalink / raw) To: Toke Høiland-Jørgensen; +Cc: workflows On Sat, 11 Dec 2021 20:06:00 +0100 Toke Høiland-Jørgensen wrote: > Hi everyone! > > I was wondering if anyone had any nifty tricks or utilities to help > monitor the status of your own patch submissions on a patchwork > instance? Things like getting a notification when a patch changes status > or when one of the automated checks on that particular patchwork > instance fails for a patch you submitted? Status notification when checks are failing? Hopefully not, we don't want people posting patches just to get them tested... > I can just filter the patchwork web interface to my name, of course, but > I would like something that can be automated so I can make the stuff I > care about show up in my inbox :) > > Many thanks in advance! ^ permalink raw reply [flat|nested] 11+ messages in thread
* Re: Monitoring the status of your own patches on patchwork? 2021-12-13 15:08 ` Jakub Kicinski @ 2021-12-13 15:48 ` Toke Høiland-Jørgensen 2021-12-13 16:06 ` Jakub Kicinski 0 siblings, 1 reply; 11+ messages in thread From: Toke Høiland-Jørgensen @ 2021-12-13 15:48 UTC (permalink / raw) To: Jakub Kicinski; +Cc: workflows Jakub Kicinski <kuba@kernel.org> writes: > On Sat, 11 Dec 2021 20:06:00 +0100 Toke Høiland-Jørgensen wrote: >> Hi everyone! >> >> I was wondering if anyone had any nifty tricks or utilities to help >> monitor the status of your own patch submissions on a patchwork >> instance? Things like getting a notification when a patch changes status >> or when one of the automated checks on that particular patchwork >> instance fails for a patch you submitted? > > Status notification when checks are failing? Hopefully not, we don't > want people posting patches just to get them tested... Well no, but sometimes a patch will have failures despite the best efforts of the submitter (otherwise what's the point of the checks?). Right now the only way for me to discover that there's an issue is to go look at the patchwork web interface, and I wanted something that better suits my workflow (i.e., that's not in a web browser). I wasn't asking for patchwork to send out automatic notifications (yikes!), I just wanted to know if anyone else had done something similar before I go play around with the patchwork API myself... :) -Toke ^ permalink raw reply [flat|nested] 11+ messages in thread
* Re: Monitoring the status of your own patches on patchwork? 2021-12-13 15:48 ` Toke Høiland-Jørgensen @ 2021-12-13 16:06 ` Jakub Kicinski 2021-12-13 18:39 ` Toke Høiland-Jørgensen 0 siblings, 1 reply; 11+ messages in thread From: Jakub Kicinski @ 2021-12-13 16:06 UTC (permalink / raw) To: Toke Høiland-Jørgensen; +Cc: workflows On Mon, 13 Dec 2021 16:48:37 +0100 Toke Høiland-Jørgensen wrote: > > Status notification when checks are failing? Hopefully not, we don't > > want people posting patches just to get them tested... > > Well no, but sometimes a patch will have failures despite the best > efforts of the submitter (otherwise what's the point of the checks?). > Right now the only way for me to discover that there's an issue is to go > look at the patchwork web interface, and I wanted something that better > suits my workflow (i.e., that's not in a web browser). I think that the maintainer should notify the submitter about the reason the patch state was changed (with the exception of patches for a different tree, maybe). I know Kees has been trying to add more meaningful states to patchwork but I can never guess the meaning of those either :S So no automated state checker can replace the maintainer's reply. > I wasn't asking for patchwork to send out automatic notifications > (yikes!), I just wanted to know if anyone else had done something > similar before I go play around with the patchwork API myself... :) Despite the promise of "best effort" I fear such automation. It's pretty common in (let's call them) modern workflows to submit PRs / post changes just to get them tested by a CI. We don't want to give people the impression that the mailing list can serve this purpose. Obviously I can't stop you from writing your local code :) just trying to widen the perspective. ^ permalink raw reply [flat|nested] 11+ messages in thread
* Re: Monitoring the status of your own patches on patchwork? 2021-12-13 16:06 ` Jakub Kicinski @ 2021-12-13 18:39 ` Toke Høiland-Jørgensen 2021-12-13 22:56 ` Kees Cook 0 siblings, 1 reply; 11+ messages in thread From: Toke Høiland-Jørgensen @ 2021-12-13 18:39 UTC (permalink / raw) To: Jakub Kicinski; +Cc: workflows Jakub Kicinski <kuba@kernel.org> writes: > On Mon, 13 Dec 2021 16:48:37 +0100 Toke Høiland-Jørgensen wrote: >> > Status notification when checks are failing? Hopefully not, we don't >> > want people posting patches just to get them tested... >> >> Well no, but sometimes a patch will have failures despite the best >> efforts of the submitter (otherwise what's the point of the checks?). >> Right now the only way for me to discover that there's an issue is to go >> look at the patchwork web interface, and I wanted something that better >> suits my workflow (i.e., that's not in a web browser). > > I think that the maintainer should notify the submitter about > the reason the patch state was changed (with the exception of > patches for a different tree, maybe). I know Kees has been > trying to add more meaningful states to patchwork but I can > never guess the meaning of those either :S So no automated > state checker can replace the maintainer's reply. Well, sometimes the maintainers forget to reply entirely :) And yeah, I do realise that no bot is going to be able to tell me exactly what the current status is, I just want a tool to help me manually keep track... >> I wasn't asking for patchwork to send out automatic notifications >> (yikes!), I just wanted to know if anyone else had done something >> similar before I go play around with the patchwork API myself... :) > > Despite the promise of "best effort" I fear such automation. > It's pretty common in (let's call them) modern workflows to > submit PRs / post changes just to get them tested by a CI. > We don't want to give people the impression that the mailing > list can serve this purpose. People could already do that, though, they just have to look at the patchwork site. And I really don't think it's that convenient of an interface that this is a real risk. But don't worry, I'm not planning to publish this as a generally-available service :) -Toke ^ permalink raw reply [flat|nested] 11+ messages in thread
* Re: Monitoring the status of your own patches on patchwork? 2021-12-13 18:39 ` Toke Høiland-Jørgensen @ 2021-12-13 22:56 ` Kees Cook 0 siblings, 0 replies; 11+ messages in thread From: Kees Cook @ 2021-12-13 22:56 UTC (permalink / raw) To: Toke Høiland-Jørgensen; +Cc: Jakub Kicinski, workflows On Mon, Dec 13, 2021 at 07:39:27PM +0100, Toke Høiland-Jørgensen wrote: > Jakub Kicinski <kuba@kernel.org> writes: > > > On Mon, 13 Dec 2021 16:48:37 +0100 Toke Høiland-Jørgensen wrote: > >> > Status notification when checks are failing? Hopefully not, we don't > >> > want people posting patches just to get them tested... > >> > >> Well no, but sometimes a patch will have failures despite the best > >> efforts of the submitter (otherwise what's the point of the checks?). > >> Right now the only way for me to discover that there's an issue is to go > >> look at the patchwork web interface, and I wanted something that better > >> suits my workflow (i.e., that's not in a web browser). > > > > I think that the maintainer should notify the submitter about > > the reason the patch state was changed (with the exception of > > patches for a different tree, maybe). I know Kees has been > > trying to add more meaningful states to patchwork but I can > > never guess the meaning of those either :S So no automated > > state checker can replace the maintainer's reply. > > Well, sometimes the maintainers forget to reply entirely :) > > And yeah, I do realise that no bot is going to be able to tell me > exactly what the current status is, I just want a tool to help me > manually keep track... I've mostly been trying to capture the state of the patch review conversation with some kind of flag in patchwork. For KSPP, we've documented it here: https://kernsec.org/wiki/index.php/Kernel_Self_Protection_Project/Patch_Tracking -- Kees Cook ^ permalink raw reply [flat|nested] 11+ messages in thread
* Re: Monitoring the status of your own patches on patchwork? 2021-12-11 19:06 Monitoring the status of your own patches on patchwork? Toke Høiland-Jørgensen 2021-12-13 15:08 ` Jakub Kicinski @ 2021-12-13 18:37 ` Simon Glass 2021-12-13 21:43 ` Toke Høiland-Jørgensen 2021-12-14 2:12 ` Randy Dunlap 1 sibling, 2 replies; 11+ messages in thread From: Simon Glass @ 2021-12-13 18:37 UTC (permalink / raw) To: Toke Høiland-Jørgensen; +Cc: workflows Hi, On Sat, 11 Dec 2021 at 12:06, Toke Høiland-Jørgensen <toke@redhat.com> wrote: > > Hi everyone! > > I was wondering if anyone had any nifty tricks or utilities to help > monitor the status of your own patch submissions on a patchwork > instance? Things like getting a notification when a patch changes status > or when one of the automated checks on that particular patchwork > instance fails for a patch you submitted? > > I can just filter the patchwork web interface to my name, of course, but > I would like something that can be automated so I can make the stuff I > care about show up in my inbox :) I'll just mention 'patman status' here, which collects tags from patchwork so you can send them with the next version. It also shows you code-review snippets on the cmdline which I find much faster to process, particularly for large patches. See below, although it's a bit hard to read without the colours that patman uses. I would like something that can track all the series and tell me what is outstanding. At the moment I keep a manual list but often forget things. $ patman status 1 efi: Rename UCLASS_EFI and IF_TYPE_EFI + Reviewed-by: Heinrich Schuchardt <xypron.glpk@gmx.de> 2 efi: Add EFI uclass for media + Reviewed-by: Heinrich Schuchardt <xypron.glpk@gmx.de> 3 efi: Add a media/block driver for EFI block device + Reviewed-by: Heinrich Schuchardt <xypron.glpk@gmx.de> 4 efi: Locate all block devices in the app 5 efi: serial: Support arrow keys [...] 3 new responses available in patchwork (use -d to write them to a new branch) $ patman status -C 1 efi: Rename UCLASS_EFI and IF_TYPE_EFI + Reviewed-by: Heinrich Schuchardt <xypron.glpk@gmx.de> Review: Heinrich Schuchardt <xypron.glpk@gmx.de> > File: include/dm/uclass-id.h > Line: 48 / 48: enum uclass_id { > UCLASS_DMA, /* Direct Memory Access */ > UCLASS_DSA, /* Distributed (Ethernet) Switch Architecture */ > UCLASS_ECDSA, /* Elliptic curve cryptographic device */ > - UCLASS_EFI, /* EFI managed devices */ > + UCLASS_EFI_LOADER, /* Devices managed by EFI_LOADER */ "managed by EFI_LOADER" does not catch it. I will change this to "devices created by UEFI applications and drivers" when merging. 2 efi: Add EFI uclass for media + Reviewed-by: Heinrich Schuchardt <xypron.glpk@gmx.de> Review: Heinrich Schuchardt <xypron.glpk@gmx.de> > At present UCLASS_EFI is used to represent an EFI filesystem among other %s/UCLASS_EFI/UCLASS_EFI_LOADER/ > pretty vague. The only driver that uses this uclass is in fact not a real > U-Boot driver, since its operations do not include a struct udevice. > > Rather than mess with this, create a new UCLASS_EFI_MEDIA uclass to handle > EFI media such as disks. Make this the uclass to use for EFI media so that The new uclass is for devices provided by an UEFI implementation loading U-Boot as an EFI application. > it can be used with 'part list', for example. > > The existing implementation using UCLASS_EFI remains as is, for %s/UCLASS_EFI/UCLASS_EFI_LOADER/ The existing uclass is for devices created by UEFI applications loaded by U-Boot. > File: include/dm/uclass-id.h > Line: 49 / 49: enum uclass_id { > +++ b/include/dm/uclass-id.h > UCLASS_DSA, /* Distributed (Ethernet) Switch Architecture */ > UCLASS_ECDSA, /* Elliptic curve cryptographic device */ > UCLASS_EFI_LOADER, /* Devices managed by EFI_LOADER */ > + UCLASS_EFI_MEDIA, /* EFI media (e.g. a disk) */ %s/EFI media (e.g. a disk)/devices provided by UEFI firmware */ With those changes: 3 efi: Add a media/block driver for EFI block device + Reviewed-by: Heinrich Schuchardt <xypron.glpk@gmx.de> Regards, Simon ^ permalink raw reply [flat|nested] 11+ messages in thread
* Re: Monitoring the status of your own patches on patchwork? 2021-12-13 18:37 ` Simon Glass @ 2021-12-13 21:43 ` Toke Høiland-Jørgensen 2021-12-13 23:51 ` Simon Glass 2021-12-14 2:12 ` Randy Dunlap 1 sibling, 1 reply; 11+ messages in thread From: Toke Høiland-Jørgensen @ 2021-12-13 21:43 UTC (permalink / raw) To: Simon Glass; +Cc: workflows Simon Glass <sjg@chromium.org> writes: > Hi, > > On Sat, 11 Dec 2021 at 12:06, Toke Høiland-Jørgensen <toke@redhat.com> wrote: >> >> Hi everyone! >> >> I was wondering if anyone had any nifty tricks or utilities to help >> monitor the status of your own patch submissions on a patchwork >> instance? Things like getting a notification when a patch changes status >> or when one of the automated checks on that particular patchwork >> instance fails for a patch you submitted? >> >> I can just filter the patchwork web interface to my name, of course, but >> I would like something that can be automated so I can make the stuff I >> care about show up in my inbox :) > > I'll just mention 'patman status' here, which collects tags from > patchwork so you can send them with the next version. It also shows > you code-review snippets on the cmdline which I find much faster to > process, particularly for large patches. See below, although it's a > bit hard to read without the colours that patman uses. Ah, this is cool, didn't know about patman! Will take a closer look, it seems to be solving some of the same issues I've been rolling my own solutions for - thanks for the pointer! :) -Toke ^ permalink raw reply [flat|nested] 11+ messages in thread
* Re: Monitoring the status of your own patches on patchwork? 2021-12-13 21:43 ` Toke Høiland-Jørgensen @ 2021-12-13 23:51 ` Simon Glass 0 siblings, 0 replies; 11+ messages in thread From: Simon Glass @ 2021-12-13 23:51 UTC (permalink / raw) To: Toke Høiland-Jørgensen; +Cc: workflows Hi Toke, On Mon, 13 Dec 2021 at 14:43, Toke Høiland-Jørgensen <toke@redhat.com> wrote: > > Simon Glass <sjg@chromium.org> writes: > > > Hi, > > > > On Sat, 11 Dec 2021 at 12:06, Toke Høiland-Jørgensen <toke@redhat.com> wrote: > >> > >> Hi everyone! > >> > >> I was wondering if anyone had any nifty tricks or utilities to help > >> monitor the status of your own patch submissions on a patchwork > >> instance? Things like getting a notification when a patch changes status > >> or when one of the automated checks on that particular patchwork > >> instance fails for a patch you submitted? > >> > >> I can just filter the patchwork web interface to my name, of course, but > >> I would like something that can be automated so I can make the stuff I > >> care about show up in my inbox :) > > > > I'll just mention 'patman status' here, which collects tags from > > patchwork so you can send them with the next version. It also shows > > you code-review snippets on the cmdline which I find much faster to > > process, particularly for large patches. See below, although it's a > > bit hard to read without the colours that patman uses. > > Ah, this is cool, didn't know about patman! Will take a closer look, it > seems to be solving some of the same issues I've been rolling my own > solutions for - thanks for the pointer! :) You're welcome. I do hope things continue to grow in this area. Regards, Simon ^ permalink raw reply [flat|nested] 11+ messages in thread
* Re: Monitoring the status of your own patches on patchwork? 2021-12-13 18:37 ` Simon Glass 2021-12-13 21:43 ` Toke Høiland-Jørgensen @ 2021-12-14 2:12 ` Randy Dunlap 2021-12-14 16:31 ` Simon Glass 1 sibling, 1 reply; 11+ messages in thread From: Randy Dunlap @ 2021-12-14 2:12 UTC (permalink / raw) To: Simon Glass, Toke Høiland-Jørgensen; +Cc: workflows On 12/13/21 10:37, Simon Glass wrote: > Hi, > > On Sat, 11 Dec 2021 at 12:06, Toke Høiland-Jørgensen <toke@redhat.com> wrote: >> >> Hi everyone! >> >> I was wondering if anyone had any nifty tricks or utilities to help >> monitor the status of your own patch submissions on a patchwork >> instance? Things like getting a notification when a patch changes status >> or when one of the automated checks on that particular patchwork >> instance fails for a patch you submitted? >> >> I can just filter the patchwork web interface to my name, of course, but >> I would like something that can be automated so I can make the stuff I >> care about show up in my inbox :) > > I'll just mention 'patman status' here, which collects tags from > patchwork so you can send them with the next version. It also shows > you code-review snippets on the cmdline which I find much faster to > process, particularly for large patches. See below, although it's a > bit hard to read without the colours that patman uses. > > I would like something that can track all the series and tell me what > is outstanding. At the moment I keep a manual list but often forget > things. Hi, Where is this 'patman' found, please? I see https://source.denx.de/u-boot/custodians/u-boot-dm/-/tree/patman-working and https://github.com/siemens/u-boot/tree/master/tools/patman (the latter one seems to be untouched recently). Is there another URL or is the denx.de URL the one to use? thanks. -- ~Randy ^ permalink raw reply [flat|nested] 11+ messages in thread
* Re: Monitoring the status of your own patches on patchwork? 2021-12-14 2:12 ` Randy Dunlap @ 2021-12-14 16:31 ` Simon Glass 0 siblings, 0 replies; 11+ messages in thread From: Simon Glass @ 2021-12-14 16:31 UTC (permalink / raw) To: Randy Dunlap; +Cc: Toke Høiland-Jørgensen, workflows Hi Randy, On Mon, 13 Dec 2021 at 19:12, Randy Dunlap <rdunlap@infradead.org> wrote: > > > > On 12/13/21 10:37, Simon Glass wrote: > > Hi, > > > > On Sat, 11 Dec 2021 at 12:06, Toke Høiland-Jørgensen <toke@redhat.com> wrote: > >> > >> Hi everyone! > >> > >> I was wondering if anyone had any nifty tricks or utilities to help > >> monitor the status of your own patch submissions on a patchwork > >> instance? Things like getting a notification when a patch changes status > >> or when one of the automated checks on that particular patchwork > >> instance fails for a patch you submitted? > >> > >> I can just filter the patchwork web interface to my name, of course, but > >> I would like something that can be automated so I can make the stuff I > >> care about show up in my inbox :) > > > > I'll just mention 'patman status' here, which collects tags from > > patchwork so you can send them with the next version. It also shows > > you code-review snippets on the cmdline which I find much faster to > > process, particularly for large patches. See below, although it's a > > bit hard to read without the colours that patman uses. > > > > I would like something that can track all the series and tell me what > > is outstanding. At the moment I keep a manual list but often forget > > things. > > Hi, > Where is this 'patman' found, please? > > I see https://source.denx.de/u-boot/custodians/u-boot-dm/-/tree/patman-working > and https://github.com/siemens/u-boot/tree/master/tools/patman > (the latter one seems to be untouched recently). > > Is there another URL or is the denx.de URL the one to use? You can use Denx but I suggest github: https://github.com/u-boot/u-boot I should figure out how to get patman into pip. Regards, Simon ^ permalink raw reply [flat|nested] 11+ messages in thread
end of thread, other threads:[~2021-12-14 16:31 UTC | newest] Thread overview: 11+ messages (download: mbox.gz / follow: Atom feed) -- links below jump to the message on this page -- 2021-12-11 19:06 Monitoring the status of your own patches on patchwork? Toke Høiland-Jørgensen 2021-12-13 15:08 ` Jakub Kicinski 2021-12-13 15:48 ` Toke Høiland-Jørgensen 2021-12-13 16:06 ` Jakub Kicinski 2021-12-13 18:39 ` Toke Høiland-Jørgensen 2021-12-13 22:56 ` Kees Cook 2021-12-13 18:37 ` Simon Glass 2021-12-13 21:43 ` Toke Høiland-Jørgensen 2021-12-13 23:51 ` Simon Glass 2021-12-14 2:12 ` Randy Dunlap 2021-12-14 16:31 ` Simon Glass
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox; as well as URLs for NNTP newsgroup(s).