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

Agents protocol refactor #4874

Merged
merged 41 commits into from
Feb 23, 2024

nit

672bc3e
Select commit
Loading
Failed to load commit list.
Merged

Agents protocol refactor #4874

nit
672bc3e
Select commit
Loading
Failed to load commit list.
Codecov / codecov/patch succeeded Feb 22, 2024 in 0s

64.74% of diff hit (target 58.93%)

View this Pull Request on Codecov

64.74% of diff hit (target 58.93%)

Annotations

Check warning on line 142 in flyteplugins/go/tasks/plugins/webapi/agent/client.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

flyteplugins/go/tasks/plugins/webapi/agent/client.go#L141-L142

Added lines #L141 - L142 were not covered by tests

Check warning on line 104 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#L103-L104

Added lines #L103 - L104 were not covered by tests

Check warning on line 136 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#L135-L136

Added lines #L135 - L136 were not covered by tests

Check warning on line 147 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#L146-L147

Added lines #L146 - L147 were 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 162 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#L161-L162

Added lines #L161 - L162 were not covered by tests

Check warning on line 165 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#L164-L165

Added lines #L164 - L165 were not covered by tests

Check warning on line 172 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#L171-L172

Added lines #L171 - L172 were not covered by tests

Check warning on line 177 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#L175-L177

Added lines #L175 - L177 were not covered by tests

Check warning on line 194 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#L193-L194

Added lines #L193 - L194 were not covered by tests

Check warning on line 222 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#L222

Added line #L222 was 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#L224-L227

Added lines #L224 - L227 were not covered by tests

Check warning on line 236 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#L231-L236

Added lines #L231 - L236 were not covered by tests

Check warning on line 282 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#L282

Added line #L282 was not covered by tests

Check warning on line 300 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#L295-L300

Added lines #L295 - L300 were 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#L310-L311

Added lines #L310 - L311 were not covered by tests

Check warning on line 334 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#L334

Added line #L334 was not covered by tests

Check warning on line 374 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#L374

Added line #L374 was not covered by tests