Skip to content

Commit 286a6d7

Browse files
committed
set defualt logger level for feature servers as WARNING
Signed-off-by: Abdul Hameed <ahameed@redhat.com>
1 parent 1b5a8fc commit 286a6d7

File tree

5 files changed

+7
-9
lines changed

5 files changed

+7
-9
lines changed

infra/charts/feast-feature-server/values.yaml

+1-1
Original file line numberDiff line numberDiff line change
@@ -11,7 +11,7 @@ image:
1111
# image.tag -- The Docker image tag (can be overwritten if custom feature server deps are needed for on demand transforms)
1212
tag: 0.40.0
1313

14-
logLevel: "DEBUG" # Set log level
14+
logLevel: "WARNING" # Set log level DEBUG, INFO, WARNING, ERROR, and CRITICAL (case-insensitive)
1515

1616
imagePullSecrets: []
1717
nameOverride: ""

sdk/python/feast/permissions/auth/oidc_token_parser.py

-1
Original file line numberDiff line numberDiff line change
@@ -17,7 +17,6 @@
1717
from feast.permissions.user import User
1818

1919
logger = logging.getLogger(__name__)
20-
logger.setLevel(logging.INFO)
2120

2221

2322
class OidcTokenParser(TokenParser):

sdk/python/feast/permissions/server/arrow.py

+2-3
Original file line numberDiff line numberDiff line change
@@ -17,7 +17,6 @@
1717
from feast.permissions.user import User
1818

1919
logger = logging.getLogger(__name__)
20-
logger.setLevel(logging.INFO)
2120

2221

2322
class AuthorizationMiddlewareFactory(fl.ServerMiddlewareFactory):
@@ -49,7 +48,7 @@ def __init__(self, access_token: str, *args, **kwargs):
4948

5049
def call_completed(self, exception):
5150
if exception:
52-
logger.error(
51+
logger.exception(
5352
f"{AuthorizationMiddleware.__name__} encountered an exception: {exception}"
5453
)
5554

@@ -78,7 +77,7 @@ def inject_user_details(context: ServerCallContext):
7877
if sm is not None:
7978
auth_middleware = cast(AuthorizationMiddleware, context.get_middleware("auth"))
8079
current_user = asyncio.run(auth_middleware.extract_user())
81-
logger.info(f"User extracted: {current_user}")
80+
logger.debug(f"User extracted: {current_user}")
8281

8382
sm.set_current_user(current_user)
8483

sdk/python/feast/permissions/server/grpc.py

+4-3
Original file line numberDiff line numberDiff line change
@@ -9,7 +9,6 @@
99
from feast.permissions.security_manager import get_security_manager
1010

1111
logger = logging.getLogger(__name__)
12-
logger.setLevel(logging.INFO)
1312

1413

1514
class AuthInterceptor(grpc.ServerInterceptor):
@@ -22,11 +21,13 @@ def intercept_service(self, continuation, handler_call_details):
2221
metadata=dict(handler_call_details.invocation_metadata)
2322
)
2423

25-
logger.info(f"Fetching user details for token of length: {len(access_token)}")
24+
logger.debug(
25+
f"Fetching user details for token of length: {len(access_token)}"
26+
)
2627
current_user = asyncio.run(
2728
auth_manager.token_parser.user_details_from_access_token(access_token)
2829
)
29-
logger.info(f"User is: {current_user.username}")
30+
logger.debug(f"User is: {current_user}")
3031
sm.set_current_user(current_user)
3132

3233
return continuation(handler_call_details)

sdk/python/feast/permissions/server/utils.py

-1
Original file line numberDiff line numberDiff line change
@@ -30,7 +30,6 @@
3030
from feast.permissions.server.rest_token_extractor import RestTokenExtractor
3131

3232
logger = logging.getLogger(__name__)
33-
logger.setLevel(logging.INFO)
3433

3534

3635
class ServerType(enum.Enum):

0 commit comments

Comments
 (0)