Skip to content

Commit f9484bd

Browse files
fix(logs): Updating feature variable evaluation logs (#282)
1 parent c9fca91 commit f9484bd

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
@@ -477,16 +477,9 @@ def get_variable_value_for_variation(self, variable, variation):
477477

478478
if variable_usage:
479479
variable_value = variable_usage.value
480-
self.logger.info(
481-
'Value for variable "%s" for variation "%s" is "%s".' % (variable.key, variation.key, variable_value)
482-
)
483480

484481
else:
485482
variable_value = variable.defaultValue
486-
self.logger.info(
487-
'Variable "%s" is not used in variation "%s". Assigning default value "%s".'
488-
% (variable.key, variation.key, variable_value)
489-
)
490483

491484
return variable_value
492485

0 commit comments

Comments
 (0)