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

Backport #5788, #5855, and #5769 to release v1.13 #5890

Merged
merged 3 commits into from
Oct 23, 2024

Upstream contributions from Union.ai (#5769)

3aab668
Select commit
Loading
Failed to load commit list.
Merged

Backport #5788, #5855, and #5769 to release v1.13 #5890

Upstream contributions from Union.ai (#5769)
3aab668
Select commit
Loading
Failed to load commit list.
Codecov / codecov/patch succeeded Oct 22, 2024 in 1s

51.69% of diff hit (target 36.31%)

View this Pull Request on Codecov

51.69% of diff hit (target 36.31%)

Annotations

Check warning on line 133 in flyteadmin/auth/authzserver/provider.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

flyteadmin/auth/authzserver/provider.go#L127-L133

Added lines #L127 - L133 were not covered by tests

Check warning on line 142 in flyteadmin/auth/authzserver/provider.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

flyteadmin/auth/authzserver/provider.go#L140-L142

Added lines #L140 - L142 were not covered by tests

Check warning on line 305 in flyteadmin/auth/handlers.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

flyteadmin/auth/handlers.go#L304-L305

Added lines #L304 - L305 were not covered by tests

Check warning on line 309 in flyteadmin/auth/handlers.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

flyteadmin/auth/handlers.go#L309

Added line #L309 was not covered by tests

Check warning on line 311 in flyteadmin/auth/handlers.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

flyteadmin/auth/handlers.go#L311

Added line #L311 was not covered by tests

Check warning on line 314 in flyteadmin/auth/handlers.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

flyteadmin/auth/handlers.go#L314

Added line #L314 was not covered by tests

Check warning on line 317 in flyteadmin/auth/handlers.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

flyteadmin/auth/handlers.go#L317

Added line #L317 was not covered by tests

Check warning on line 322 in flyteadmin/auth/handlers.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

flyteadmin/auth/handlers.go#L322

Added line #L322 was not covered by tests

Check warning on line 537 in flyteadmin/pkg/manager/impl/execution_manager.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

flyteadmin/pkg/manager/impl/execution_manager.go#L536-L537

Added lines #L536 - L537 were not covered by tests

Check warning on line 631 in flyteadmin/pkg/manager/impl/execution_manager.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

flyteadmin/pkg/manager/impl/execution_manager.go#L630-L631

Added lines #L630 - L631 were not covered by tests

Check warning on line 25 in flyteadmin/pkg/manager/mocks/resource_interface.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

flyteadmin/pkg/manager/mocks/resource_interface.go#L24-L25

Added lines #L24 - L25 were not covered by tests

Check warning on line 30 in flyteadmin/pkg/manager/mocks/resource_interface.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

flyteadmin/pkg/manager/mocks/resource_interface.go#L28-L30

Added lines #L28 - L30 were not covered by tests

Check warning on line 35 in flyteadmin/pkg/manager/mocks/resource_interface.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

flyteadmin/pkg/manager/mocks/resource_interface.go#L33-L35

Added lines #L33 - L35 were not covered by tests

Check warning on line 48 in flyteadmin/pkg/manager/mocks/resource_interface.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

flyteadmin/pkg/manager/mocks/resource_interface.go#L39-L48

Added lines #L39 - L48 were not covered by tests

Check warning on line 56 in flyteadmin/pkg/manager/mocks/resource_interface.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

flyteadmin/pkg/manager/mocks/resource_interface.go#L51-L56

Added lines #L51 - L56 were not covered by tests

Check warning on line 58 in flyteadmin/pkg/manager/mocks/resource_interface.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

flyteadmin/pkg/manager/mocks/resource_interface.go#L58

Added line #L58 was not covered by tests

Check warning on line 66 in flyteadmin/pkg/manager/mocks/resource_interface.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

flyteadmin/pkg/manager/mocks/resource_interface.go#L65-L66

Added lines #L65 - L66 were not covered by tests

Check warning on line 71 in flyteadmin/pkg/manager/mocks/resource_interface.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

flyteadmin/pkg/manager/mocks/resource_interface.go#L69-L71

Added lines #L69 - L71 were not covered by tests

Check warning on line 76 in flyteadmin/pkg/manager/mocks/resource_interface.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

flyteadmin/pkg/manager/mocks/resource_interface.go#L74-L76

Added lines #L74 - L76 were not covered by tests

Check warning on line 89 in flyteadmin/pkg/manager/mocks/resource_interface.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

flyteadmin/pkg/manager/mocks/resource_interface.go#L80-L89

Added lines #L80 - L89 were not covered by tests

Check warning on line 97 in flyteadmin/pkg/manager/mocks/resource_interface.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

flyteadmin/pkg/manager/mocks/resource_interface.go#L92-L97

Added lines #L92 - L97 were not covered by tests

Check warning on line 99 in flyteadmin/pkg/manager/mocks/resource_interface.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

flyteadmin/pkg/manager/mocks/resource_interface.go#L99

Added line #L99 was not covered by tests

Check warning on line 107 in flyteadmin/pkg/manager/mocks/resource_interface.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

flyteadmin/pkg/manager/mocks/resource_interface.go#L106-L107

Added lines #L106 - L107 were not covered by tests

Check warning on line 112 in flyteadmin/pkg/manager/mocks/resource_interface.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

flyteadmin/pkg/manager/mocks/resource_interface.go#L110-L112

Added lines #L110 - L112 were not covered by tests

Check warning on line 117 in flyteadmin/pkg/manager/mocks/resource_interface.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

flyteadmin/pkg/manager/mocks/resource_interface.go#L115-L117

Added lines #L115 - L117 were not covered by tests