summaryrefslogtreecommitdiffstats
path: root/recipes-connectivity/kuksa-val/kuksa-client/0001-kuksa-client-Update-cmd2-completer-usage.patch
diff options
context:
space:
mode:
authorScott Murray <scott.murray@konsulko.com>2023-11-27 15:28:38 -0500
committerJan-Simon Moeller <jsmoeller@linuxfoundation.org>2023-11-30 23:32:10 +0000
commit470045b42e4175ed778a24664c910ed57c510c4f (patch)
tree602aae3536980515d0b45691f14e1168ffc46050 /recipes-connectivity/kuksa-val/kuksa-client/0001-kuksa-client-Update-cmd2-completer-usage.patch
parentcbeacb02ef678b4c3d53429548bb12105b22b7f0 (diff)
Upgrade KUKSA.val components to 0.4.2 release
Upgrade to the KUKSA.val 0.4.2 release that includes the changes required to build the databroker on RISC-V. Notes: - A patch is required to allow building the databroker with Rust 1.68 instead of 1.70, which is currently not available to us on top of Yocto kirkstone. The patch only fixes the versions of the clap command-line parser crate and some of its direct dependencies to what was effectively being used with 0.4.0, so the risk seems minimal at present. When time permits, backporting Rust 1.70 to kirkstone using a mixin layer will be investigated, as that seems better with respect to handling any more KUKSA.val updates during the Quillback lifecycle. - The new optional VISS support has been enabled in the databroker build, but it has not yet been tested on target (a configuration change is required). - The kuksa-client recipe has been updated to build from the new separate kuksa-python-sdk repository. This futureproofs things a bit, as any further 0.4.x releases will only be buildable from the separate repository instead of kuksa.val as before. - The kuksa-dbc-feeder recipe has been left using the 0.4.0 tag commit for now, as no 0.4.2 tag has been made in the feeders repository. As well, some of the post-0.4.0 changes will require rework of the local patches on our side. Bug-AGL: SPEC-4893 Change-Id: I5f92fdea0095a432977c9fa7da05fe5347f7e789 Signed-off-by: Scott Murray <scott.murray@konsulko.com> Reviewed-on: https://gerrit.automotivelinux.org/gerrit/c/AGL/meta-agl-demo/+/29490 Tested-by: Jenkins Job builder account ci-image-boot-test: Jenkins Job builder account Reviewed-by: Jan-Simon Moeller <jsmoeller@linuxfoundation.org> ci-image-build: Jenkins Job builder account
Diffstat (limited to 'recipes-connectivity/kuksa-val/kuksa-client/0001-kuksa-client-Update-cmd2-completer-usage.patch')
-rw-r--r--recipes-connectivity/kuksa-val/kuksa-client/0001-kuksa-client-Update-cmd2-completer-usage.patch176
1 files changed, 98 insertions, 78 deletions
diff --git a/recipes-connectivity/kuksa-val/kuksa-client/0001-kuksa-client-Update-cmd2-completer-usage.patch b/recipes-connectivity/kuksa-val/kuksa-client/0001-kuksa-client-Update-cmd2-completer-usage.patch
index d43a34322..e32e5c469 100644
--- a/recipes-connectivity/kuksa-val/kuksa-client/0001-kuksa-client-Update-cmd2-completer-usage.patch
+++ b/recipes-connectivity/kuksa-val/kuksa-client/0001-kuksa-client-Update-cmd2-completer-usage.patch
@@ -1,7 +1,7 @@
-From cc4da8b14896739ac2966ecad0a5c82ab1de025c Mon Sep 17 00:00:00 2001
+From 89aa41f52d5946796ad8dea587cd848aef71c8c0 Mon Sep 17 00:00:00 2001
From: Scott Murray <scott.murray@konsulko.com>
-Date: Sat, 5 Aug 2023 13:26:55 -0400
-Subject: [PATCH 1/2] kuksa_viss_client: Update cmd2 completer usage
+Date: Mon, 27 Nov 2023 15:31:02 -0500
+Subject: [PATCH 1/2] kuksa-client: Update cmd2 completer usage
Update cmd2 usage to avoid using internal methods and fix breakage
with newer versions of the cmd2 module.
@@ -10,31 +10,22 @@ Upstream-Status: pending
Signed-off-by: Scott Murray <scott.murray@konsulko.com>
---
- kuksa-client/kuksa_client/__main__.py | 31 +++++++++++++--------------
- 1 file changed, 15 insertions(+), 16 deletions(-)
+ kuksa-client/kuksa_client/__main__.py | 33 +++++++++++++--------------
+ 1 file changed, 16 insertions(+), 17 deletions(-)
diff --git a/kuksa-client/kuksa_client/__main__.py b/kuksa-client/kuksa_client/__main__.py
-index 948cf56..fceca41 100755
+index 032410b..1d2d1fc 100755
--- a/kuksa-client/kuksa_client/__main__.py
+++ b/kuksa-client/kuksa_client/__main__.py
-@@ -35,7 +35,6 @@ from cmd2 import Cmd
- from cmd2 import CompletionItem
+@@ -36,7 +36,6 @@ from cmd2 import CompletionItem
from cmd2 import with_argparser
from cmd2 import with_category
+ from cmd2 import constants
-from cmd2.utils import basic_complete
+ from urllib.parse import urlparse
- import kuksa_certificates
- from kuksa_client import KuksaClientThread
-@@ -126,7 +125,7 @@ class TestClient(Cmd):
- "Children of branch "+prefix+key),
- )
-
-- return basic_complete(text, line, begidx, endidx, self.pathCompletionItems)
-+ return Cmd.basic_complete(self, text, line, begidx, endidx, self.pathCompletionItems)
-
- def subscribeCallback(self, logPath, resp):
- with logPath.open('a', encoding='utf-8') as logFile:
-@@ -136,7 +135,7 @@ class TestClient(Cmd):
+ from kuksa_client import kuksa_server_certificates
+@@ -156,7 +155,7 @@ class TestClient(Cmd):
self.pathCompletionItems = []
for sub_id in self.subscribeIds:
self.pathCompletionItems.append(CompletionItem(sub_id))
@@ -43,99 +34,128 @@ index 948cf56..fceca41 100755
COMM_SETUP_COMMANDS = "Communication Set-up Commands"
VSS_COMMANDS = "Kuksa Interaction Commands"
-@@ -153,7 +152,7 @@ class TestClient(Cmd):
+@@ -171,19 +170,19 @@ class TestClient(Cmd):
+
+ ap_disconnect = argparse.ArgumentParser()
+ ap_authorize = argparse.ArgumentParser()
+- tokenfile_completer_method = functools.partial(
++ tokenfile_completer = functools.partial(
+ Cmd.path_complete,
+ path_filter=lambda path: (os.path.isdir(path) or path.endswith(".token")),
+ )
ap_authorize.add_argument(
- 'token_or_tokenfile',
- help='JWT(or the file storing the token) for authorizing the client.',
-- completer_method=tokenfile_completer_method,)
-+ completer=tokenfile_completer_method,)
- ap_setServerAddr = argparse.ArgumentParser()
- ap_setServerAddr.add_argument(
- 'IP', help='VISS/gRPC Server IP Address', default=DEFAULT_SERVER_ADDR)
-@@ -168,7 +167,7 @@ class TestClient(Cmd):
+ "token_or_tokenfile",
+ help="JWT(or the file storing the token) for authorizing the client.",
+- completer_method=tokenfile_completer_method,
++ completer=tokenfile_completer,
+ )
ap_setValue = argparse.ArgumentParser()
ap_setValue.add_argument(
-- "Path", help="Path to be set", completer_method=path_completer)
-+ "Path", help="Path to be set", completer=path_completer)
- ap_setValue.add_argument("Value", nargs='+', help="Value to be set")
+- "Path", help="Path to be set", completer_method=path_completer
++ "Path", help="Path to be set", completer=path_completer
+ )
+ ap_setValue.add_argument("Value", nargs="+", help="Value to be set")
ap_setValue.add_argument(
- "-a", "--attribute", help="Attribute to be set", default="value")
-@@ -185,19 +184,19 @@ class TestClient(Cmd):
+@@ -203,7 +202,7 @@ class TestClient(Cmd):
ap_getValue = argparse.ArgumentParser()
ap_getValue.add_argument(
-- "Path", help="Path to be read", completer_method=path_completer)
-+ "Path", help="Path to be read", completer=path_completer)
+- "Path", help="Path to be read", completer_method=path_completer
++ "Path", help="Path to be read", completer=path_completer
+ )
ap_getValue.add_argument(
- "-a", "--attribute", help="Attribute to be get", default="value")
-
- ap_getValues = argparse.ArgumentParser()
- ap_getValues.add_argument(
-- "Path", help="Path whose value is to be read", nargs='+', completer_method=path_completer)
-+ "Path", help="Path whose value is to be read", nargs='+', completer=path_completer)
+ "-a", "--attribute", help="Attribute to be get", default="value"
+@@ -214,7 +213,7 @@ class TestClient(Cmd):
+ "Path",
+ help="Path whose value is to be read",
+ nargs="+",
+- completer_method=path_completer,
++ completer=path_completer,
+ )
ap_getValues.add_argument(
- "-a", "--attribute", help="Attribute to be get", default="value")
-
- ap_setTargetValue = argparse.ArgumentParser()
+ "-a", "--attribute", help="Attribute to be get", default="value"
+@@ -224,7 +223,7 @@ class TestClient(Cmd):
ap_setTargetValue.add_argument(
-- "Path", help="Path whose target value to be set", completer_method=path_completer)
-+ "Path", help="Path whose target value to be set", completer=path_completer)
+ "Path",
+ help="Path whose target value to be set",
+- completer_method=path_completer,
++ completer=path_completer,
+ )
ap_setTargetValue.add_argument("Value", help="Value to be set")
- ap_setTargetValues = argparse.ArgumentParser()
-@@ -210,35 +209,35 @@ class TestClient(Cmd):
-
- ap_getTargetValue = argparse.ArgumentParser()
+@@ -240,7 +239,7 @@ class TestClient(Cmd):
ap_getTargetValue.add_argument(
-- "Path", help="Path whose target value is to be read", completer_method=path_completer)
-+ "Path", help="Path whose target value is to be read", completer=path_completer)
+ "Path",
+ help="Path whose target value is to be read",
+- completer_method=path_completer,
++ completer=path_completer,
+ )
ap_getTargetValues = argparse.ArgumentParser()
- ap_getTargetValues.add_argument(
-- "Path", help="Path whose target value is to be read", nargs='+', completer_method=path_completer)
-+ "Path", help="Path whose target value is to be read", nargs='+', completer=path_completer)
+@@ -248,12 +247,12 @@ class TestClient(Cmd):
+ "Path",
+ help="Path whose target value is to be read",
+ nargs="+",
+- completer_method=path_completer,
++ completer=path_completer,
+ )
ap_subscribe = argparse.ArgumentParser()
ap_subscribe.add_argument(
-- "Path", help="Path to subscribe to", completer_method=path_completer)
-+ "Path", help="Path to subscribe to", completer=path_completer)
+- "Path", help="Path to subscribe to", completer_method=path_completer
++ "Path", help="Path to subscribe to", completer=path_completer
+ )
ap_subscribe.add_argument(
- "-a", "--attribute", help="Attribute to subscribe to", default="value")
+ "-a", "--attribute", help="Attribute to subscribe to", default="value"
+@@ -268,7 +267,7 @@ class TestClient(Cmd):
ap_subscribeMultiple = argparse.ArgumentParser()
ap_subscribeMultiple.add_argument(
-- "Path", help="Path to subscribe to", nargs='+', completer_method=path_completer)
-+ "Path", help="Path to subscribe to", nargs='+', completer=path_completer)
+- "Path", help="Path to subscribe to", nargs="+", completer_method=path_completer
++ "Path", help="Path to subscribe to", nargs="+", completer=path_completer
+ )
ap_subscribeMultiple.add_argument(
- "-a", "--attribute", help="Attribute to subscribe to", default="value")
-
- ap_unsubscribe = argparse.ArgumentParser()
+ "-a", "--attribute", help="Attribute to subscribe to", default="value"
+@@ -284,18 +283,18 @@ class TestClient(Cmd):
ap_unsubscribe.add_argument(
-- "SubscribeId", help="Corresponding subscription Id", completer_method=subscriptionIdCompleter,
-+ "SubscribeId", help="Corresponding subscription Id", completer=subscriptionIdCompleter,
+ "SubscribeId",
+ help="Corresponding subscription Id",
+- completer_method=subscriptionIdCompleter,
++ completer=subscriptionIdCompleter,
)
ap_getMetaData = argparse.ArgumentParser()
ap_getMetaData.add_argument(
-- "Path", help="Path whose metadata is to be read", completer_method=path_completer)
-+ "Path", help="Path whose metadata is to be read", completer=path_completer)
+ "Path",
+ help="Path whose metadata is to be read",
+- completer_method=path_completer,
++ completer=path_completer,
+ )
ap_updateMetaData = argparse.ArgumentParser()
ap_updateMetaData.add_argument(
-- "Path", help="Path whose MetaData is to update", completer_method=path_completer)
-+ "Path", help="Path whose MetaData is to update", completer=path_completer)
+- "Path", help="Path whose MetaData is to update", completer_method=path_completer
++ "Path", help="Path whose MetaData is to update", completer=path_completer
+ )
ap_updateMetaData.add_argument(
"Json",
- help="MetaData to update. Note, only attributes can be update, if update children or the whole vss tree, use"
-@@ -249,7 +248,7 @@ class TestClient(Cmd):
- jsonfile_completer_method = functools.partial(Cmd.path_complete,
- path_filter=lambda path: (os.path.isdir(path) or path.endswith(".json")))
+@@ -304,14 +303,14 @@ class TestClient(Cmd):
+ )
+
+ ap_updateVSSTree = argparse.ArgumentParser()
+- jsonfile_completer_method = functools.partial(
++ jsonfile_completer = functools.partial(
+ Cmd.path_complete,
+ path_filter=lambda path: (os.path.isdir(path) or path.endswith(".json")),
+ )
ap_updateVSSTree.add_argument(
-- "Json", help="Json tree to update VSS", completer_method=jsonfile_completer_method)
-+ "Json", help="Json tree to update VSS", completer=jsonfile_completer_method)
+ "Json",
+ help="Json tree to update VSS",
+- completer_method=jsonfile_completer_method,
++ completer=jsonfile_completer,
+ )
# Constructor, request names after protocol to avoid errors
- def __init__(self, server_ip=None, server_port=None, server_protocol=None, *, insecure=False, token_or_tokenfile=None,
--
-2.41.0
+2.42.0