Skip to content

Commit 508a8cb

Browse files
committed
make the template reason step optional
1 parent a8cbd5c commit 508a8cb

File tree

3 files changed

+59
-57
lines changed

3 files changed

+59
-57
lines changed

packages/orchestrator/internal/template/build/builderrors/errors.go

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -10,7 +10,7 @@ func UnwrapUserError(err error) *template_manager.TemplateBuildStatusReason {
1010
if phaseBuildError != nil {
1111
return &template_manager.TemplateBuildStatusReason{
1212
Message: phaseBuildError.Error(),
13-
Step: phaseBuildError.Step,
13+
Step: &phaseBuildError.Step,
1414
}
1515
}
1616

packages/orchestrator/template-manager.proto

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -99,7 +99,7 @@ message TemplateBuildLogEntry {
9999

100100
message TemplateBuildStatusReason {
101101
string message = 1;
102-
string step = 2;
102+
optional string step = 2;
103103
}
104104

105105
// Logs from template build

packages/shared/pkg/grpc/template-manager/template-manager.pb.go

Lines changed: 57 additions & 55 deletions
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

0 commit comments

Comments
 (0)