Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(574)

Side by Side Diff: milo/appengine/logdog/logDogBuild.go

Issue 2254993002: Milo: Add task expired as a failure status (Closed) Base URL: https://chromium.googlesource.com/external/github.com/luci/luci-go@master
Patch Set: Retrain, remove build-hang test case Created 4 years, 4 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View unified diff | Download patch
OLDNEW
1 // Copyright 2016 The LUCI Authors. All rights reserved. 1 // Copyright 2016 The LUCI Authors. All rights reserved.
2 // Use of this source code is governed under the Apache License, Version 2.0 2 // Use of this source code is governed under the Apache License, Version 2.0
3 // that can be found in the LICENSE file. 3 // that can be found in the LICENSE file.
4 4
5 package logdog 5 package logdog
6 6
7 import ( 7 import (
8 "fmt" 8 "fmt"
9 "time" 9 "time"
10 10
(...skipping 31 matching lines...) Expand 10 before | Expand all | Expand 10 after
42 42
43 case miloProto.Status_SUCCESS: 43 case miloProto.Status_SUCCESS:
44 comp.Status = resp.Success 44 comp.Status = resp.Success
45 45
46 case miloProto.Status_FAILURE: 46 case miloProto.Status_FAILURE:
47 if fd := anno.GetFailureDetails(); fd != nil { 47 if fd := anno.GetFailureDetails(); fd != nil {
48 switch fd.Type { 48 switch fd.Type {
49 case miloProto.FailureDetails_EXCEPTION, miloProto.Failu reDetails_INFRA: 49 case miloProto.FailureDetails_EXCEPTION, miloProto.Failu reDetails_INFRA:
50 comp.Status = resp.InfraFailure 50 comp.Status = resp.InfraFailure
51 51
52 case miloProto.FailureDetails_EXPIRED:
53 comp.Status = resp.Expired
54
52 case miloProto.FailureDetails_DM_DEPENDENCY_FAILED: 55 case miloProto.FailureDetails_DM_DEPENDENCY_FAILED:
53 comp.Status = resp.DependencyFailure 56 comp.Status = resp.DependencyFailure
54 57
55 default: 58 default:
56 comp.Status = resp.Failure 59 comp.Status = resp.Failure
57 } 60 }
58 61
59 if fd.Text != "" { 62 if fd.Text != "" {
60 comp.Text = append(comp.Text, fd.Text) 63 comp.Text = append(comp.Text, fd.Text)
61 } 64 }
(...skipping 124 matching lines...) Expand 10 before | Expand all | Expand 10 after
186 for _, prop := range mainAnno.Property { 189 for _, prop := range mainAnno.Property {
187 propGroup.Property = append(propGroup.Property, &resp.Property{ 190 propGroup.Property = append(propGroup.Property, &resp.Property{
188 Key: prop.Name, 191 Key: prop.Name,
189 Value: prop.Value, 192 Value: prop.Value,
190 }) 193 })
191 } 194 }
192 build.PropertyGroup = append(build.PropertyGroup, propGroup) 195 build.PropertyGroup = append(build.PropertyGroup, propGroup)
193 196
194 return 197 return
195 } 198 }
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698