public inbox for buildbot@sourceware.org
 help / color / mirror / Atom feed
* [PATCH] stap: exclude testsuite/.systemtap-root from bunsen upload
@ 2024-04-24 14:00 Serhei Makarov
  2024-04-24 14:08 ` Serhei Makarov
  0 siblings, 1 reply; 2+ messages in thread
From: Serhei Makarov @ 2024-04-24 14:00 UTC (permalink / raw)
  To: buildbot

The testsuite leaves a heap of .log files under
testsuite/.systemtap-root/cache, which are then picked up by bunsen.
They shouldn't be.
---
 builder/master.cfg | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/builder/master.cfg b/builder/master.cfg
index 22102b5..1b65e22 100644
--- a/builder/master.cfg
+++ b/builder/master.cfg
@@ -4745,7 +4745,8 @@ systemtap_privileged_factory.addStep(steps.Test(
         haltOnFailure=False, flunkOnFailure=False)) # some failures are normal for stap
 systemtap_privileged_factory.addStep(steps.ShellCommand(
         workdir="stap-build",
-        command=["rm", "-rf", "testsuite/.systemtap-buildbot/cache"], # ugly hardcoding bot username :-(
+        command=["rm", "-rf", "testsuite/.systemtap-buildbot/cache", # ugly hardcoding bot username :-(
+                 "testsuite/.systemtap-root"], # contains .log files, should be excluded from bunsen upload
         name="cleanup caches",
         haltOnFailure=True))
 systemtap_privileged_factory.addSteps(bunsen_logfile_upload_cpio_steps(
--
2.43.0

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

* Re: [PATCH] stap: exclude testsuite/.systemtap-root from bunsen upload
  2024-04-24 14:00 [PATCH] stap: exclude testsuite/.systemtap-root from bunsen upload Serhei Makarov
@ 2024-04-24 14:08 ` Serhei Makarov
  0 siblings, 0 replies; 2+ messages in thread
From: Serhei Makarov @ 2024-04-24 14:08 UTC (permalink / raw)
  To: buildbot

> The testsuite leaves a heap of .log files under
> testsuite/.systemtap-root/cache, which are then picked up by bunsen.
> They shouldn't be.
> ---
>  builder/master.cfg | 3 ++-
>  1 file changed, 2 insertions(+), 1 deletion(-)
>
> diff --git a/builder/master.cfg b/builder/master.cfg
> index 22102b5..1b65e22 100644
> --- a/builder/master.cfg
> +++ b/builder/master.cfg
> @@ -4745,7 +4745,8 @@ systemtap_privileged_factory.addStep(steps.Test(
>          haltOnFailure=False, flunkOnFailure=False)) # some failures 
> are normal for stap
>  systemtap_privileged_factory.addStep(steps.ShellCommand(
>          workdir="stap-build",
> -        command=["rm", "-rf", "testsuite/.systemtap-buildbot/cache"], 
> # ugly hardcoding bot username :-(
> +        command=["rm", "-rf", "testsuite/.systemtap-buildbot/cache", # 
> ugly hardcoding bot username :-(
> +                 "testsuite/.systemtap-root"], # contains .log files, 
> should be excluded from bunsen upload
>          name="cleanup caches",
>          haltOnFailure=True))
>  systemtap_privileged_factory.addSteps(bunsen_logfile_upload_cpio_steps(
> --
> 2.43.0

(Used my own privileges to commit this after checking with Frank.)

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

end of thread, other threads:[~2024-04-24 14:08 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2024-04-24 14:00 [PATCH] stap: exclude testsuite/.systemtap-root from bunsen upload Serhei Makarov
2024-04-24 14:08 ` Serhei Makarov

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 read-only IMAP folder(s) and NNTP newsgroup(s).