LGTM, thanks

On Thu, Jan 16, 2014 at 3:26 PM, Klaus Aehlig <[email protected]> wrote:

> ...so that this opcode parameter can become available for 'gnt-cluster
> modify'.
>
> Signed-off-by: Klaus Aehlig <[email protected]>
> ---
>  lib/cli.py                | 5 +++++
>  lib/client/gnt_cluster.py | 5 ++++-
>  2 files changed, 9 insertions(+), 1 deletion(-)
>
> diff --git a/lib/cli.py b/lib/cli.py
> index 33e32f1..302a556 100644
> --- a/lib/cli.py
> +++ b/lib/cli.py
> @@ -180,6 +180,7 @@ __all__ = [
>    "REMOVE_RESERVED_IPS_OPT",
>    "REMOVE_UIDS_OPT",
>    "RESERVED_LVS_OPT",
> +  "RQL_OPT",
>    "RUNTIME_MEM_OPT",
>    "ROMAN_OPT",
>    "SECONDARY_IP_OPT",
> @@ -1266,6 +1267,10 @@ CP_SIZE_OPT = cli_option("-C",
> "--candidate-pool-size", default=None,
>                           dest="candidate_pool_size", type="int",
>                           help="Set the candidate pool size")
>
> +RQL_OPT = cli_option("--max-running-jobs", dest="max_running_jobs",
> +                     type="int", help="Set the maximal number of jobs to "
> +                                      "run simultaneously")
> +
>  VG_NAME_OPT = cli_option("--vg-name", dest="vg_name",
>                           help=("Enables LVM and specifies the volume
> group"
>                                 " name (cluster-wide) for disk allocation"
> diff --git a/lib/client/gnt_cluster.py b/lib/client/gnt_cluster.py
> index 9042899..0ecc9fe 100644
> --- a/lib/client/gnt_cluster.py
> +++ b/lib/client/gnt_cluster.py
> @@ -1088,6 +1088,7 @@ def SetClusterParams(opts, args):
>            opts.beparams or opts.nicparams or
>            opts.ndparams or opts.diskparams or
>            opts.candidate_pool_size is not None or
> +          opts.max_running_jobs is not None or
>            opts.uid_pool is not None or
>            opts.maintain_node_health is not None or
>            opts.add_uids is not None or
> @@ -1201,6 +1202,7 @@ def SetClusterParams(opts, args):
>      diskparams=diskparams,
>      ipolicy=ipolicy,
>      candidate_pool_size=opts.candidate_pool_size,
> +    max_running_jobs=opts.max_running_jobs,
>      maintain_node_health=mnh,
>      modify_etc_hosts=opts.modify_etc_hosts,
>      uid_pool=uid_pool,
> @@ -2175,7 +2177,8 @@ commands = {
>    "modify": (
>      SetClusterParams, ARGS_NONE,
>      [FORCE_OPT,
> -     BACKEND_OPT, CP_SIZE_OPT, ENABLED_HV_OPT, HVLIST_OPT,
> MASTER_NETDEV_OPT,
> +     BACKEND_OPT, CP_SIZE_OPT, RQL_OPT,
> +     ENABLED_HV_OPT, HVLIST_OPT, MASTER_NETDEV_OPT,
>       MASTER_NETMASK_OPT, NIC_PARAMS_OPT, VG_NAME_OPT,
> MAINTAIN_NODE_HEALTH_OPT,
>       UIDPOOL_OPT, ADD_UIDS_OPT, REMOVE_UIDS_OPT, DRBD_HELPER_OPT,
>       DEFAULT_IALLOCATOR_OPT, DEFAULT_IALLOCATOR_PARAMS_OPT,
> RESERVED_LVS_OPT,
> --
> 1.8.5.2
>
>

Reply via email to