Skip to content

Commit

Permalink
fix test
Browse files Browse the repository at this point in the history
Signed-off-by: Kevin Su <[email protected]>
  • Loading branch information
pingsutw committed Dec 17, 2024
1 parent 466f35e commit cabe46a
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions flyteplugins/go/tasks/plugins/webapi/agent/plugin.go
Original file line number Diff line number Diff line change
Expand Up @@ -154,7 +154,7 @@ func (p *Plugin) ExecuteTaskSync(
stream, err := client.ExecuteTaskSync(ctx)
if err != nil {
logger.Errorf(ctx, "failed to execute task from agent with %v", err)
return nil, nil, fmt.Errorf("failed to execute task from agent with: %v", err)
return nil, nil, fmt.Errorf("failed to execute task from agent with %v", err)

Check warning on line 157 in flyteplugins/go/tasks/plugins/webapi/agent/plugin.go

View check run for this annotation

Codecov / codecov/patch

flyteplugins/go/tasks/plugins/webapi/agent/plugin.go#L156-L157

Added lines #L156 - L157 were not covered by tests
}

headerProto := &admin.ExecuteTaskSyncRequest{
Expand Down Expand Up @@ -203,7 +203,7 @@ func (p *Plugin) ExecuteTaskSync(
LogLinks: resource.GetLogLinks(),
CustomInfo: resource.GetCustomInfo(),
AgentError: resource.GetAgentError(),
}, fmt.Errorf("failed to execute task from agent with %v", err)
}, nil
}

func (p *Plugin) Get(ctx context.Context, taskCtx webapi.GetContext) (latest webapi.Resource, err error) {
Expand Down

0 comments on commit cabe46a

Please sign in to comment.