From cf4f91295f318fa97b2b5952bd4a8885c498a609 Mon Sep 17 00:00:00 2001 From: Giovanni Harting <539@idlegandalf.com> Date: Tue, 14 Mar 2023 01:36:11 +0100 Subject: [PATCH] only record rusage if build was successful --- buildmanager.go | 1 - proto_package.go | 10 +++++----- 2 files changed, 5 insertions(+), 6 deletions(-) diff --git a/buildmanager.go b/buildmanager.go index fe198ee..c1f0521 100644 --- a/buildmanager.go +++ b/buildmanager.go @@ -220,7 +220,6 @@ func (b *BuildManager) buildQueue(queue []*ProtoPackage, ctx context.Context) er unknownBuilds = true } b.buildingLock.RUnlock() - up = 0 } return nil } diff --git a/proto_package.go b/proto_package.go index af86f77..0d766ba 100644 --- a/proto_package.go +++ b/proto_package.go @@ -263,11 +263,11 @@ func (p *ProtoPackage) build(ctx context.Context) (time.Duration, error) { SetStatus(dbpackage.StatusFailed). ClearSkipReason(). SetBuildTimeStart(start). - SetMaxRss(Rusage.Maxrss). - SetIoOut(Rusage.Oublock). - SetIoIn(Rusage.Inblock). - SetUTime(Rusage.Utime.Sec). - SetSTime(Rusage.Stime.Sec). + ClearMaxRss(). + ClearIoOut(). + ClearIoIn(). + ClearUTime(). + ClearSTime(). SetHash(p.Hash). ExecX(ctx) return time.Since(start), fmt.Errorf("build failed: exit code %d", cmd.ProcessState.ExitCode())