From df2c932e092201579e7f4cbff3ed04641f6fff9a Mon Sep 17 00:00:00 2001
From: Karol Latecki <karol.latecki@intel.com>
Date: Fri, 15 Jan 2021 16:32:07 +0100
Subject: [PATCH] scripts/rpc: fix framework_set_schedule period parameter

Period parameter is marked as optional in rpc.py. This works
as long the rpc call is issued with rpc.py, as argparse
sets period to "None" by default. This patch makes it
an optional argument as well directly in app.py.

Signed-off-by: Karol Latecki <karol.latecki@intel.com>
Change-Id: I2e3cae6df7edd7fc6e1120ee3d4a2655dc60813a
Reviewed-on: https://review.spdk.io/gerrit/c/spdk/spdk/+/5942
Reviewed-by: Michal Berger <michalx.berger@intel.com>
Reviewed-by: Aleksey Marchuk <alexeymar@mellanox.com>
Reviewed-by: Tomasz Zawadzki <tomasz.zawadzki@intel.com>
Reviewed-by: Jim Harris <james.r.harris@intel.com>
Tested-by: SPDK CI Jenkins <sys_sgci@intel.com>
Community-CI: Mellanox Build Bot
---
 scripts/rpc/app.py | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/scripts/rpc/app.py b/scripts/rpc/app.py
index 38cd682def..98e64569ad 100644
--- a/scripts/rpc/app.py
+++ b/scripts/rpc/app.py
@@ -37,7 +37,7 @@ def framework_get_reactors(client):
     return client.call('framework_get_reactors')
 
 
-def framework_set_scheduler(client, name, period):
+def framework_set_scheduler(client, name, period=None):
     """Select threads scheduler that will be activated and its period.
 
     Args: