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

[Monorepo] Enable proxy-authorization in admin client #4189

Merged
merged 19 commits into from
Oct 11, 2023

Fix flytepropeller's getAdminClient

d2013e9
Select commit
Loading
Failed to load commit list.
Merged

[Monorepo] Enable proxy-authorization in admin client #4189

Fix flytepropeller's getAdminClient
d2013e9
Select commit
Loading
Failed to load commit list.
Codecov / codecov/patch succeeded Oct 11, 2023 in 0s

68.75% of diff hit (target 59.45%)

View this Pull Request on Codecov

68.75% of diff hit (target 59.45%)

Annotations

Check warning on line 58 in flyteidl/clients/go/admin/auth_interceptor.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

flyteidl/clients/go/admin/auth_interceptor.go#L57-L58 

Added lines #L57 - L58 were not covered by tests

Check warning on line 62 in flyteidl/clients/go/admin/auth_interceptor.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

flyteidl/clients/go/admin/auth_interceptor.go#L61-L62 

Added lines #L61 - L62 were not covered by tests

Check warning on line 70 in flyteidl/clients/go/admin/auth_interceptor.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

flyteidl/clients/go/admin/auth_interceptor.go#L69-L70 

Added lines #L69 - L70 were not covered by tests

Check warning on line 91 in flyteidl/clients/go/admin/auth_interceptor.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

flyteidl/clients/go/admin/auth_interceptor.go#L90-L91 

Added lines #L90 - L91 were not covered by tests

Check warning on line 96 in flyteidl/clients/go/admin/auth_interceptor.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

flyteidl/clients/go/admin/auth_interceptor.go#L95-L96 

Added lines #L95 - L96 were not covered by tests

Check warning on line 168 in flyteidl/clients/go/admin/auth_interceptor.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

flyteidl/clients/go/admin/auth_interceptor.go#L167-L168 

Added lines #L167 - L168 were not covered by tests

Check warning on line 171 in flyteidl/clients/go/admin/auth_interceptor.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

flyteidl/clients/go/admin/auth_interceptor.go#L171 

Added line #L171 was not covered by tests

Check warning on line 159 in flyteidl/clients/go/admin/client.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

flyteidl/clients/go/admin/client.go#L157-L159 

Added lines #L157 - L159 were not covered by tests

Check warning on line 75 in flyteidl/clients/go/admin/pkce/auth_flow_orchestrator.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

flyteidl/clients/go/admin/pkce/auth_flow_orchestrator.go#L75 

Added line #L75 was not covered by tests

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

See this annotation in the file changed.

@codecov codecov / codecov/patch

flytepropeller/pkg/controller/controller.go#L318 

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