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

Bug/abort map task subtasks #4506

Merged
merged 17 commits into from
Dec 15, 2023

handling PhaseAbortSubTasks in awsbatch plugin

b34f6d3
Select commit
Loading
Failed to load commit list.
Merged

Bug/abort map task subtasks #4506

handling PhaseAbortSubTasks in awsbatch plugin
b34f6d3
Select commit
Loading
Failed to load commit list.
Codecov / codecov/patch failed Dec 15, 2023 in 1s

40.96% of diff hit (target 59.00%)

View this Pull Request on Codecov

40.96% of diff hit (target 59.00%)

Annotations

Check warning on line 59 in flyteplugins/go/tasks/pluginmachinery/core/phase.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

flyteplugins/go/tasks/pluginmachinery/core/phase.go#L59

Added line #L59 was not covered by tests

Check warning on line 71 in flyteplugins/go/tasks/pluginmachinery/core/phase.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

flyteplugins/go/tasks/pluginmachinery/core/phase.go#L70-L71

Added lines #L70 - L71 were not covered by tests

Check warning on line 268 in flyteplugins/go/tasks/pluginmachinery/core/phase.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

flyteplugins/go/tasks/pluginmachinery/core/phase.go#L267-L268

Added lines #L267 - L268 were not covered by tests

Check warning on line 276 in flyteplugins/go/tasks/pluginmachinery/core/phase.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

flyteplugins/go/tasks/pluginmachinery/core/phase.go#L275-L276

Added lines #L275 - L276 were not covered by tests

Check warning on line 83 in flyteplugins/go/tasks/plugins/array/awsbatch/executor.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

flyteplugins/go/tasks/plugins/array/awsbatch/executor.go#L82-L83

Added lines #L82 - L83 were not covered by tests

Check warning on line 121 in flyteplugins/go/tasks/plugins/array/k8s/executor.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

flyteplugins/go/tasks/plugins/array/k8s/executor.go#L120-L121

Added lines #L120 - L121 were not covered by tests

Check warning on line 162 in flyteplugins/go/tasks/plugins/array/k8s/executor.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

flyteplugins/go/tasks/plugins/array/k8s/executor.go#L162

Added line #L162 was not covered by tests

Check warning on line 172 in flyteplugins/go/tasks/plugins/array/k8s/executor.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

flyteplugins/go/tasks/plugins/array/k8s/executor.go#L171-L172

Added lines #L171 - L172 were not covered by tests

Check warning on line 355 in flyteplugins/go/tasks/plugins/array/k8s/management.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

flyteplugins/go/tasks/plugins/array/k8s/management.go#L355

Added line #L355 was not covered by tests

Check warning on line 357 in flyteplugins/go/tasks/plugins/array/k8s/management.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

flyteplugins/go/tasks/plugins/array/k8s/management.go#L357

Added line #L357 was not covered by tests

Check warning on line 377 in flyteplugins/go/tasks/plugins/array/k8s/management.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

flyteplugins/go/tasks/plugins/array/k8s/management.go#L377

Added line #L377 was not covered by tests

Check warning on line 824 in flytepropeller/pkg/controller/nodes/task/handler.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

flytepropeller/pkg/controller/nodes/task/handler.go#L798-L824

Added lines #L798 - L824 were not covered by tests

Check warning on line 48 in flytepropeller/pkg/controller/nodes/task/transformer.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

flytepropeller/pkg/controller/nodes/task/transformer.go#L47-L48

Added lines #L47 - L48 were not covered by tests

Check warning on line 122 in flytepropeller/pkg/controller/nodes/task/transformer.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

flytepropeller/pkg/controller/nodes/task/transformer.go#L122

Added line #L122 was not covered by tests

Check warning on line 129 in flytepropeller/pkg/controller/nodes/task/transformer.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

flytepropeller/pkg/controller/nodes/task/transformer.go#L129

Added line #L129 was not covered by tests