Andal.LND/edx-platform
6
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

fix temp_flag_no_request_default_match (#24409)

Browse Source 
Fix bug in metric temp_flag_no_request_default_match.
Metric renamed to temp_flag_no_request_default_match_2
to ensure we are looking at the right data.

This extends Phase 1 of the rollout, as documented in:
https://github.com/edx/edx-platform/pull/24392

ARCHBOM-1331
This commit is contained in:
Robert Raposa
2020-07-07 16:32:11 -04:00
committed by GitHub
parent 10ab63995a
commit 081a751d2c
1 changed files with 2 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files

4
openedx/core/djangoapps/waffle_utils/__init__.py
View File

@@ -297,8 +297,8 @@ class WaffleFlagNamespace(six.with_metaclass(ABCMeta, WaffleNamespace)):
# the default value.
value = bool(flag_undefined_default)
self._set_waffle_flag_metric(namespaced_flag_name, value)
no_request_default_match = flag_undefined_default == value
set_custom_metric('temp_flag_no_request_default_match', no_request_default_match)
no_request_default_match = is_flag_active_for_everyone == value
set_custom_metric('temp_flag_no_request_default_match_2', no_request_default_match)
set_custom_metric('warn_flag_no_request_return_value', value)
return value