Skip to content

Commit e0aedda

Browse files
committed
fix(logs): Updating feature variable evaluation logs (#282)
1 parent a9217c4 commit e0aedda

File tree

3 files changed

+128
-164
lines changed

3 files changed

+128
-164
lines changed

optimizely/optimizely.py

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -252,8 +252,8 @@ def _get_feature_variable_for_type(
252252
)
253253
else:
254254
self.logger.info(
255-
'Feature "%s" for variation "%s" is not enabled. '
256-
'Returning the default variable value "%s".' % (feature_key, decision.variation.key, variable_value)
255+
'Feature "%s" is not enabled for user "%s". '
256+
'Returning the default variable value "%s".' % (feature_key, user_id, variable_value)
257257
)
258258
else:
259259
self.logger.info(
@@ -329,11 +329,11 @@ def _get_all_feature_variables_for_type(
329329
feature_enabled = decision.variation.featureEnabled
330330
if feature_enabled:
331331
self.logger.info(
332-
'Feature "%s" for variation "%s" is enabled.' % (feature_key, decision.variation.key)
332+
'Feature "%s" is enabled for user "%s".' % (feature_key, user_id)
333333
)
334334
else:
335335
self.logger.info(
336-
'Feature "%s" for variation "%s" is not enabled.' % (feature_key, decision.variation.key)
336+
'Feature "%s" is not enabled for user "%s".' % (feature_key, user_id)
337337
)
338338
else:
339339
self.logger.info(

optimizely/project_config.py

Lines changed: 0 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -465,16 +465,9 @@ def get_variable_value_for_variation(self, variable, variation):
465465

466466
if variable_usage:
467467
variable_value = variable_usage.value
468-
self.logger.info(
469-
'Value for variable "%s" for variation "%s" is "%s".' % (variable.key, variation.key, variable_value)
470-
)
471468

472469
else:
473470
variable_value = variable.defaultValue
474-
self.logger.info(
475-
'Variable "%s" is not used in variation "%s". Assigning default value "%s".'
476-
% (variable.key, variation.key, variable_value)
477-
)
478471

479472
return variable_value
480473

0 commit comments

Comments
 (0)