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

Auto refresh cache tweaks #5759

Closed
wants to merge 4 commits into from

clean up

6b62c9f
Select commit
Loading
Failed to load commit list.
Closed

Auto refresh cache tweaks #5759

clean up
6b62c9f
Select commit
Loading
Failed to load commit list.
Codecov / codecov/patch failed Sep 18, 2024 in 0s

0.00% of diff hit (target 36.30%)

View this Pull Request on Codecov

0.00% of diff hit (target 36.30%)

Annotations

Check warning on line 36 in flytepropeller/pkg/controller/nodes/subworkflow/handler.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

flytepropeller/pkg/controller/nodes/subworkflow/handler.go#L36 

Added line #L36 was not covered by tests

Check warning on line 127 in flytepropeller/pkg/controller/nodes/subworkflow/handler.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

flytepropeller/pkg/controller/nodes/subworkflow/handler.go#L123-L127 

Added lines #L123 - L127 were not covered by tests

Check warning on line 236 in flytepropeller/pkg/controller/nodes/subworkflow/launchplan.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

flytepropeller/pkg/controller/nodes/subworkflow/launchplan.go#L233-L236 

Added lines #L233 - L236 were not covered by tests

Check warning on line 242 in flytepropeller/pkg/controller/nodes/subworkflow/launchplan.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

flytepropeller/pkg/controller/nodes/subworkflow/launchplan.go#L238-L242 

Added lines #L238 - L242 were not covered by tests

Check warning on line 244 in flytepropeller/pkg/controller/nodes/subworkflow/launchplan.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

flytepropeller/pkg/controller/nodes/subworkflow/launchplan.go#L244 

Added line #L244 was not covered by tests

Check warning on line 247 in flytepropeller/pkg/controller/nodes/subworkflow/launchplan.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

flytepropeller/pkg/controller/nodes/subworkflow/launchplan.go#L247 

Added line #L247 was not covered by tests

Check warning on line 228 in flytepropeller/pkg/controller/nodes/subworkflow/launchplan/admin.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

flytepropeller/pkg/controller/nodes/subworkflow/launchplan/admin.go#L227-L228 

Added lines #L227 - L228 were not covered by tests

Check warning on line 27 in flytepropeller/pkg/controller/nodes/subworkflow/launchplan/mocks/executor.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

flytepropeller/pkg/controller/nodes/subworkflow/launchplan/mocks/executor.go#L26-L27 

Added lines #L26 - L27 were not covered by tests

Check warning on line 32 in flytepropeller/pkg/controller/nodes/subworkflow/launchplan/mocks/executor.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

flytepropeller/pkg/controller/nodes/subworkflow/launchplan/mocks/executor.go#L30-L32 

Added lines #L30 - L32 were not covered by tests

Check warning on line 37 in flytepropeller/pkg/controller/nodes/subworkflow/launchplan/mocks/executor.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

flytepropeller/pkg/controller/nodes/subworkflow/launchplan/mocks/executor.go#L35-L37 

Added lines #L35 - L37 were not covered by tests

Check warning on line 42 in flytepropeller/pkg/controller/nodes/subworkflow/launchplan/mocks/executor.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

flytepropeller/pkg/controller/nodes/subworkflow/launchplan/mocks/executor.go#L41-L42 

Added lines #L41 - L42 were not covered by tests

Check warning on line 48 in flytepropeller/pkg/controller/nodes/subworkflow/launchplan/mocks/executor.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

flytepropeller/pkg/controller/nodes/subworkflow/launchplan/mocks/executor.go#L44-L48 

Added lines #L44 - L48 were not covered by tests

Check warning on line 51 in flytepropeller/pkg/controller/nodes/subworkflow/launchplan/mocks/executor.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

flytepropeller/pkg/controller/nodes/subworkflow/launchplan/mocks/executor.go#L51 

Added line #L51 was not covered by tests

Check warning on line 27 in flytepropeller/pkg/controller/nodes/subworkflow/launchplan/mocks/flyte_admin.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

flytepropeller/pkg/controller/nodes/subworkflow/launchplan/mocks/flyte_admin.go#L26-L27 

Added lines #L26 - L27 were not covered by tests

Check warning on line 32 in flytepropeller/pkg/controller/nodes/subworkflow/launchplan/mocks/flyte_admin.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

flytepropeller/pkg/controller/nodes/subworkflow/launchplan/mocks/flyte_admin.go#L30-L32 

Added lines #L30 - L32 were not covered by tests

Check warning on line 37 in flytepropeller/pkg/controller/nodes/subworkflow/launchplan/mocks/flyte_admin.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

flytepropeller/pkg/controller/nodes/subworkflow/launchplan/mocks/flyte_admin.go#L35-L37 

Added lines #L35 - L37 were not covered by tests

Check warning on line 42 in flytepropeller/pkg/controller/nodes/subworkflow/launchplan/mocks/flyte_admin.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

flytepropeller/pkg/controller/nodes/subworkflow/launchplan/mocks/flyte_admin.go#L41-L42 

Added lines #L41 - L42 were not covered by tests

Check warning on line 48 in flytepropeller/pkg/controller/nodes/subworkflow/launchplan/mocks/flyte_admin.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

flytepropeller/pkg/controller/nodes/subworkflow/launchplan/mocks/flyte_admin.go#L44-L48 

Added lines #L44 - L48 were not covered by tests

Check warning on line 51 in flytepropeller/pkg/controller/nodes/subworkflow/launchplan/mocks/flyte_admin.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

flytepropeller/pkg/controller/nodes/subworkflow/launchplan/mocks/flyte_admin.go#L51 

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