Skip to content

Commit

Permalink
fix: Removed the k8s dependency from required dependencies (#4519)
Browse files Browse the repository at this point in the history
* Removed the k8s dependency from required dependencies

Signed-off-by: Abdul Hameed <[email protected]>

* updated requirments lock files

Signed-off-by: Abdul Hameed <[email protected]>

---------

Signed-off-by: Abdul Hameed <[email protected]>
  • Loading branch information
redhatHameed committed Sep 17, 2024
1 parent 06eade3 commit 3073ea5
Show file tree
Hide file tree
Showing 8 changed files with 525 additions and 178 deletions.
5 changes: 4 additions & 1 deletion sdk/python/feast/permissions/server/utils.py
Original file line number Diff line number Diff line change
Expand Up @@ -11,7 +11,6 @@
AuthManager,
set_auth_manager,
)
from feast.permissions.auth.kubernetes_token_parser import KubernetesTokenParser
from feast.permissions.auth.oidc_token_parser import OidcTokenParser
from feast.permissions.auth.token_extractor import TokenExtractor
from feast.permissions.auth.token_parser import TokenParser
Expand Down Expand Up @@ -116,6 +115,10 @@ def init_auth_manager(
raise ValueError(f"Unmanaged server type {server_type}")

if auth_type == AuthManagerType.KUBERNETES:
from feast.permissions.auth.kubernetes_token_parser import (
KubernetesTokenParser,
)

token_parser = KubernetesTokenParser()
elif auth_type == AuthManagerType.OIDC:
assert isinstance(auth_config, OidcAuthConfig)
Expand Down
Loading

0 comments on commit 3073ea5

Please sign in to comment.