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

[Core feature] Allow flyteadmin to start even if OIDC is unavailable (Improve flyteadmin startup resiliency) #5702

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
+58 −33

Allow flyteadmin to start even if OIDC is unavailable (improve flytea…

080a4cf
Select commit
Loading
Failed to load commit list.
Open

[Core feature] Allow flyteadmin to start even if OIDC is unavailable (Improve flyteadmin startup resiliency) #5702

Allow flyteadmin to start even if OIDC is unavailable (improve flytea…
080a4cf
Select commit
Loading
Failed to load commit list.
Codecov / codecov/patch failed Aug 28, 2024 in 1s

2.38% of diff hit (target 60.92%)

View this Pull Request on Codecov

2.38% of diff hit (target 60.92%)

Annotations

Check warning on line 53 in flyteadmin/auth/auth_context.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

flyteadmin/auth/auth_context.go#L53 

Added line #L53 was not covered by tests

Check warning on line 76 in flyteadmin/auth/auth_context.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

flyteadmin/auth/auth_context.go#L71-L76 

Added lines #L71 - L76 were not covered by tests

Check warning on line 83 in flyteadmin/auth/auth_context.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

flyteadmin/auth/auth_context.go#L82-L83 

Added lines #L82 - L83 were not covered by tests

Check warning on line 86 in flyteadmin/auth/auth_context.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

flyteadmin/auth/auth_context.go#L85-L86 

Added lines #L85 - L86 were not covered by tests

Check warning on line 90 in flyteadmin/auth/auth_context.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

flyteadmin/auth/auth_context.go#L88-L90 

Added lines #L88 - L90 were not covered by tests

Check warning on line 94 in flyteadmin/auth/auth_context.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

flyteadmin/auth/auth_context.go#L93-L94 

Added lines #L93 - L94 were not covered by tests

Check warning on line 97 in flyteadmin/auth/auth_context.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

flyteadmin/auth/auth_context.go#L97 

Added line #L97 was not covered by tests

Check warning on line 101 in flyteadmin/auth/auth_context.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

flyteadmin/auth/auth_context.go#L101 

Added line #L101 was not covered by tests

Check warning on line 105 in flyteadmin/auth/auth_context.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

flyteadmin/auth/auth_context.go#L105 

Added line #L105 was not covered by tests

Check warning on line 109 in flyteadmin/auth/auth_context.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

flyteadmin/auth/auth_context.go#L109 

Added line #L109 was not covered by tests

Check warning on line 113 in flyteadmin/auth/auth_context.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

flyteadmin/auth/auth_context.go#L113 

Added line #L113 was not covered by tests

Check warning on line 117 in flyteadmin/auth/auth_context.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

flyteadmin/auth/auth_context.go#L117 

Added line #L117 was not covered by tests

Check warning on line 121 in flyteadmin/auth/auth_context.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

flyteadmin/auth/auth_context.go#L121 

Added line #L121 was not covered by tests

Check warning on line 125 in flyteadmin/auth/auth_context.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

flyteadmin/auth/auth_context.go#L125 

Added line #L125 was not covered by tests

Check warning on line 129 in flyteadmin/auth/auth_context.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

flyteadmin/auth/auth_context.go#L129 

Added line #L129 was not covered by tests

Check warning on line 134 in flyteadmin/auth/auth_context.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

flyteadmin/auth/auth_context.go#L134 

Added line #L134 was not covered by tests

Check warning on line 139 in flyteadmin/auth/auth_context.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

flyteadmin/auth/auth_context.go#L139 

Added line #L139 was not covered by tests

Check warning on line 144 in flyteadmin/auth/auth_context.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

flyteadmin/auth/auth_context.go#L144 

Added line #L144 was not covered by tests

Check warning on line 150 in flyteadmin/auth/auth_context.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

flyteadmin/auth/auth_context.go#L150 

Added line #L150 was not covered by tests

Check warning on line 164 in flyteadmin/auth/auth_context.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

flyteadmin/auth/auth_context.go#L164 

Added line #L164 was not covered by tests

Check warning on line 166 in flyteadmin/auth/auth_context.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

flyteadmin/auth/auth_context.go#L166 

Added line #L166 was not covered by tests

Check warning on line 174 in flyteadmin/auth/auth_context.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

flyteadmin/auth/auth_context.go#L174 

Added line #L174 was not covered by tests

Check warning on line 182 in flyteadmin/auth/auth_context.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

flyteadmin/auth/auth_context.go#L182 

Added line #L182 was not covered by tests

Check warning on line 206 in flyteadmin/auth/auth_context.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

flyteadmin/auth/auth_context.go#L201-L206 

Added lines #L201 - L206 were not covered by tests

Check warning on line 210 in flyteadmin/auth/auth_context.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

flyteadmin/auth/auth_context.go#L210 

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