Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

improve error message in the agent plugin #6114

Merged
merged 3 commits into from
Dec 18, 2024

fix test

cabe46a
Select commit
Loading
Failed to load commit list.
Merged

improve error message in the agent plugin #6114

fix test
cabe46a
Select commit
Loading
Failed to load commit list.
Codecov / codecov/patch failed Dec 17, 2024 in 1s

33.33% of diff hit (target 36.98%)

View this Pull Request on Codecov

33.33% of diff hit (target 36.98%)

Annotations

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

See this annotation in the file changed.

@codecov codecov / codecov/patch

flyteplugins/go/tasks/plugins/webapi/agent/plugin.go#L101

Added line #L101 was not covered by tests

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

See this annotation in the file changed.

@codecov codecov / codecov/patch

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

Added lines #L156 - L157 were not covered by tests

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

See this annotation in the file changed.

@codecov codecov / codecov/patch

flyteplugins/go/tasks/plugins/webapi/agent/plugin.go#L189-L190

Added lines #L189 - L190 were not covered by tests

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

See this annotation in the file changed.

@codecov codecov / codecov/patch

flyteplugins/go/tasks/plugins/webapi/agent/plugin.go#L227

Added line #L227 was not covered by tests

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

See this annotation in the file changed.

@codecov codecov / codecov/patch

flyteplugins/go/tasks/plugins/webapi/agent/plugin.go#L260

Added line #L260 was not covered by tests

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

See this annotation in the file changed.

@codecov codecov / codecov/patch

flyteplugins/go/tasks/plugins/webapi/agent/plugin.go#L311

Added line #L311 was not covered by tests