From 21f2f389183cfedb4d752944c6150f68623555f4 Mon Sep 17 00:00:00 2001
From: matbu <mat.bultel@gmail.com>
Date: Thu, 3 Jun 2021 18:43:45 +0200
Subject: [PATCH] Override cliff command parser to handle argarser
 conflict_handler

The argparse conflict_handler is hardcoded in Cliff, this patch
override the command.Command().get_parser from Cliff in order
to change the value.

The conflict_handler from argparse is important for the VF because
it's allow callers to override VF CLI args.

Change-Id: I72524eeb3fc1ac0f384dcef25de61089ff15f67f
---
 validations_libs/cli/base.py | 36 ++++++++++++++++++++++++++++++++++++
 validations_libs/cli/run.py  |  5 ++---
 2 files changed, 38 insertions(+), 3 deletions(-)
 create mode 100644 validations_libs/cli/base.py

diff --git a/validations_libs/cli/base.py b/validations_libs/cli/base.py
new file mode 100644
index 00000000..7702b15a
--- /dev/null
+++ b/validations_libs/cli/base.py
@@ -0,0 +1,36 @@
+#!/usr/bin/env python
+
+#   Copyright 2021 Red Hat, Inc.
+#
+#   Licensed under the Apache License, Version 2.0 (the "License"); you may
+#   not use this file except in compliance with the License. You may obtain
+#   a copy of the License at
+#
+#        http://www.apache.org/licenses/LICENSE-2.0
+#
+#   Unless required by applicable law or agreed to in writing, software
+#   distributed under the License is distributed on an "AS IS" BASIS, WITHOUT
+#   WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied. See the
+#   License for the specific language governing permissions and limitations
+#   under the License.
+
+
+from cliff import _argparse
+from cliff.command import Command
+
+
+class BaseCommand(Command):
+    """Base Command client implementation class"""
+
+    def get_parser(self, parser):
+        """Argument parser for base command"""
+        parser = _argparse.ArgumentParser(
+            description=self.get_description(),
+            epilog=self.get_epilog(),
+            prog=prog_name,
+            formatter_class=_argparse.SmartHelpFormatter,
+            conflict_handler='resolve',
+        )
+        for hook in self._hooks:
+            hook.obj.get_parser(parser)
+        return parser
diff --git a/validations_libs/cli/run.py b/validations_libs/cli/run.py
index 6d314f01..ca4ca5b9 100644
--- a/validations_libs/cli/run.py
+++ b/validations_libs/cli/run.py
@@ -17,15 +17,14 @@
 import getpass
 import sys
 
-from cliff.command import Command
-
 from validations_libs import constants
 from validations_libs.validation_actions import ValidationActions
 from validations_libs.cli import common
+from validations_libs.cli.base import BaseCommand
 from validations_libs.cli.parseractions import CommaListAction, KeyValueAction
 
 
-class Run(Command):
+class Run(BaseCommand):
     """Validation Run client implementation class"""
 
     def get_parser(self, parser):