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.

Sign up for GitHub

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

[House keeping] remove setting max size bytes in node context #5092

Merged
merged 2 commits into from
Apr 19, 2024

remove setting max size bytes in node context

5a3945f
Select commit
Loading
Failed to load commit list.
Merged

[House keeping] remove setting max size bytes in node context #5092

remove setting max size bytes in node context
5a3945f
Select commit
Loading
Failed to load commit list.
Codecov / codecov/patch failed Mar 22, 2024 in 0s

54.54% of diff hit (target 58.99%)

View this Pull Request on Codecov

54.54% of diff hit (target 58.99%)

Annotations

Check warning on line 129 in flyteplugins/go/tasks/pluginmachinery/ioutils/remote_file_output_reader.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

flyteplugins/go/tasks/pluginmachinery/ioutils/remote_file_output_reader.go#L126-L129 

Added lines #L126 - L129 were not covered by tests

Check warning on line 133 in flyteplugins/go/tasks/pluginmachinery/ioutils/remote_file_output_reader.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

flyteplugins/go/tasks/pluginmachinery/ioutils/remote_file_output_reader.go#L133 

Added line #L133 was not covered by tests

Check warning on line 79 in flyteplugins/go/tasks/plugins/testing/echo.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

flyteplugins/go/tasks/plugins/testing/echo.go#L79 

Added line #L79 was not covered by tests

Check warning on line 335 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#L335 

Added line #L335 was not covered by tests

Check warning on line 295 in flyteplugins/go/tasks/plugins/webapi/databricks/plugin.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

flyteplugins/go/tasks/plugins/webapi/databricks/plugin.go#L295 

Added line #L295 was not covered by tests

Check warning on line 442 in flytepropeller/pkg/controller/controller.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

flytepropeller/pkg/controller/controller.go#L442 

Added line #L442 was not covered by tests

Check warning on line 508 in flytepropeller/pkg/controller/nodes/array/handler.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

flytepropeller/pkg/controller/nodes/array/handler.go#L508 

Added line #L508 was not covered by tests
View more details on Codecov