From 47cfed2a7db30b775b8c386a4374b04d6bcbf5a6 Mon Sep 17 00:00:00 2001 From: Emil Lundmark Date: Tue, 9 Apr 2024 10:40:37 +0200 Subject: [PATCH] Add flag to exclude policy exempt field trials when listing expired ones Bug: None Change-Id: I07bc9f3ad1172bcdaf205937fb518ec295c022bf No-Try: True Reviewed-on: https://webrtc-review.googlesource.com/c/src/+/346420 Commit-Queue: Emil Lundmark Reviewed-by: Jeremy Leconte Cr-Commit-Position: refs/heads/main@{#42019} --- experiments/field_trials.py | 9 ++++++++- 1 file changed, 8 insertions(+), 1 deletion(-) diff --git a/experiments/field_trials.py b/experiments/field_trials.py index 8274c03a8d..4bf2e62fef 100755 --- a/experiments/field_trials.py +++ b/experiments/field_trials.py @@ -1027,7 +1027,9 @@ def cmd_header(args: argparse.Namespace) -> None: def cmd_expired(args: argparse.Namespace) -> None: today = todays_date() diff = datetime.timedelta(days=args.in_days) - expired = expired_field_trials(today + diff) + expired = expired_field_trials( + today + diff, + REGISTERED_FIELD_TRIALS if args.exempt else ACTIVE_FIELD_TRIALS) if len(expired) <= 0: return @@ -1072,6 +1074,11 @@ def main() -> None: Lists all expired field trials. Exits with a non-zero exit status if any field trials has expired, ignoring the --in-days argument. ''') + parser_expired.add_argument( + '--exempt', + default=True, + action=argparse.BooleanOptionalAction, + help='whether to include policy exempt field trials') parser_expired.add_argument( '--in-days', default=0,