DPDK website maintenance
 help / color / mirror / Atom feed
* [PATCH] Add CI testing depends-on support status
@ 2024-05-31 22:01 Patrick Robb
  2024-05-31 22:06 ` Patrick Robb
  2024-06-03 18:52 ` Aaron Conole
  0 siblings, 2 replies; 5+ messages in thread
From: Patrick Robb @ 2024-05-31 22:01 UTC (permalink / raw)
  To: web; +Cc: aconole, mb, thomas, zhoumin, rajesh.t.puttaswamy, Patrick Robb

Currently, the DPDK submission guidelines allow for contributors to
indicate patch dependencies with their patchseries submission. This is
not broadly supported in CI testing today, although we can expect this
to change soon, as an initiative is about to begin with the goal of
supporting patch dependencies on the patchwork server side. The DPDK
Community Lab is going to allocate developer time towards this effort.
Once that is complete, it should be relatively simple for the CI labs
to begin supporting patch dependencies. In the meantime, it has been
suggested that we begin tracking that support by lab. Once support is
at 100%, this note on the testing page can be removed.

Tested with local Hugo instance.

Signed-off-by: Patrick Robb <probb@iol.unh.edu>
---
 content/testing/_index.md | 2 ++
 1 file changed, 2 insertions(+)

diff --git a/content/testing/_index.md b/content/testing/_index.md
index 394287b..b7d5e36 100644
--- a/content/testing/_index.md
+++ b/content/testing/_index.md
@@ -103,6 +103,8 @@ referencing the report archive and incrementing one of the counters
 A maintainer should wait the end of the tests (should be less than a day),
 and check the results before accepting a patch.
 
+Note: At this time, the [depends-on](https://doc.dpdk.org/guides/contributing/patches.html#patch-dependencies) tag is supported in CI testing by only the Github Robot. Patchwork test reports under labels iol, intel, and loongson do not yet support this feature.
+
 #### Requesting a Patch Retest
 
 If CI reports a failure for a test label on Patchwork,
-- 
2.40.0


^ permalink raw reply	[flat|nested] 5+ messages in thread

* Re: [PATCH] Add CI testing depends-on support status
  2024-05-31 22:01 [PATCH] Add CI testing depends-on support status Patrick Robb
@ 2024-05-31 22:06 ` Patrick Robb
  2024-06-01 14:03   ` zhoumin
  2024-06-03 18:52 ` Aaron Conole
  1 sibling, 1 reply; 5+ messages in thread
From: Patrick Robb @ 2024-05-31 22:06 UTC (permalink / raw)
  To: zhoumin; +Cc: aconole, mb, thomas, rajesh.t.puttaswamy, web

[-- Attachment #1: Type: text/plain, Size: 233 bytes --]

Zhoumin, are you supporting depends-on at loongson CI right now, or do you
just apply the patch alone in all cases and run testing?

If you do actually have depends-on support, then I'll submit a v2 here with
the correction. Thanks.

[-- Attachment #2: Type: text/html, Size: 307 bytes --]

^ permalink raw reply	[flat|nested] 5+ messages in thread

* Re: [PATCH] Add CI testing depends-on support status
  2024-05-31 22:06 ` Patrick Robb
@ 2024-06-01 14:03   ` zhoumin
  0 siblings, 0 replies; 5+ messages in thread
From: zhoumin @ 2024-06-01 14:03 UTC (permalink / raw)
  To: Patrick Robb; +Cc: aconole, mb, thomas, rajesh.t.puttaswamy, web

Hi, Patrick,

   The Loongson CI doesn't support depends-on right now and we just 
apply the patch alone to test it. Maybe we can support it in the future.

On Sat, June 1, 2024 at 6:06AM, Patrick Robb wrote:
> Zhoumin, are you supporting depends-on at loongson CI right now, or do 
> you just apply the patch alone in all cases and run testing?
>
> If you do actually have depends-on support, then I'll submit a v2 here 
> with the correction. Thanks.


^ permalink raw reply	[flat|nested] 5+ messages in thread

* Re: [PATCH] Add CI testing depends-on support status
  2024-05-31 22:01 [PATCH] Add CI testing depends-on support status Patrick Robb
  2024-05-31 22:06 ` Patrick Robb
@ 2024-06-03 18:52 ` Aaron Conole
  2024-07-30 16:05   ` Thomas Monjalon
  1 sibling, 1 reply; 5+ messages in thread
From: Aaron Conole @ 2024-06-03 18:52 UTC (permalink / raw)
  To: Patrick Robb; +Cc: web, mb, thomas, zhoumin, rajesh.t.puttaswamy

Patrick Robb <probb@iol.unh.edu> writes:

> Currently, the DPDK submission guidelines allow for contributors to
> indicate patch dependencies with their patchseries submission. This is
> not broadly supported in CI testing today, although we can expect this
> to change soon, as an initiative is about to begin with the goal of
> supporting patch dependencies on the patchwork server side. The DPDK
> Community Lab is going to allocate developer time towards this effort.
> Once that is complete, it should be relatively simple for the CI labs
> to begin supporting patch dependencies. In the meantime, it has been
> suggested that we begin tracking that support by lab. Once support is
> at 100%, this note on the testing page can be removed.
>
> Tested with local Hugo instance.
>
> Signed-off-by: Patrick Robb <probb@iol.unh.edu>
> ---

Acked-by: Aaron Conole <aconole@redhat.com>

>  content/testing/_index.md | 2 ++
>  1 file changed, 2 insertions(+)
>
> diff --git a/content/testing/_index.md b/content/testing/_index.md
> index 394287b..b7d5e36 100644
> --- a/content/testing/_index.md
> +++ b/content/testing/_index.md
> @@ -103,6 +103,8 @@ referencing the report archive and incrementing one of the counters
>  A maintainer should wait the end of the tests (should be less than a day),
>  and check the results before accepting a patch.
>  
> +Note: At this time, the [depends-on](https://doc.dpdk.org/guides/contributing/patches.html#patch-dependencies) tag is supported in CI testing by only the Github Robot. Patchwork test reports under labels iol, intel, and loongson do not yet support this feature.
> +
>  #### Requesting a Patch Retest
>  
>  If CI reports a failure for a test label on Patchwork,


^ permalink raw reply	[flat|nested] 5+ messages in thread

* Re: [PATCH] Add CI testing depends-on support status
  2024-06-03 18:52 ` Aaron Conole
@ 2024-07-30 16:05   ` Thomas Monjalon
  0 siblings, 0 replies; 5+ messages in thread
From: Thomas Monjalon @ 2024-07-30 16:05 UTC (permalink / raw)
  To: Patrick Robb; +Cc: Aaron Conole, web, mb, zhoumin, rajesh.t.puttaswamy

03/06/2024 20:52, Aaron Conole:
> Patrick Robb <probb@iol.unh.edu> writes:
> 
> > Currently, the DPDK submission guidelines allow for contributors to
> > indicate patch dependencies with their patchseries submission. This is
> > not broadly supported in CI testing today, although we can expect this
> > to change soon, as an initiative is about to begin with the goal of
> > supporting patch dependencies on the patchwork server side. The DPDK
> > Community Lab is going to allocate developer time towards this effort.
> > Once that is complete, it should be relatively simple for the CI labs
> > to begin supporting patch dependencies. In the meantime, it has been
> > suggested that we begin tracking that support by lab. Once support is
> > at 100%, this note on the testing page can be removed.
> >
> > Tested with local Hugo instance.
> >
> > Signed-off-by: Patrick Robb <probb@iol.unh.edu>
> > ---
> 
> Acked-by: Aaron Conole <aconole@redhat.com>

Applied, thanks.




^ permalink raw reply	[flat|nested] 5+ messages in thread

end of thread, other threads:[~2024-07-30 16:08 UTC | newest]

Thread overview: 5+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2024-05-31 22:01 [PATCH] Add CI testing depends-on support status Patrick Robb
2024-05-31 22:06 ` Patrick Robb
2024-06-01 14:03   ` zhoumin
2024-06-03 18:52 ` Aaron Conole
2024-07-30 16:05   ` Thomas Monjalon

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).