diff options
author | 2023-11-27 15:28:38 -0500 | |
---|---|---|
committer | 2023-11-30 23:32:10 +0000 | |
commit | 470045b42e4175ed778a24664c910ed57c510c4f (patch) | |
tree | 602aae3536980515d0b45691f14e1168ffc46050 /recipes-connectivity/kuksa-val/kuksa-client/0001-kuksa-client-Update-cmd2-completer-usage.patch | |
parent | cbeacb02ef678b4c3d53429548bb12105b22b7f0 (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.patch | 176 |
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 d43a3432..e32e5c46 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 |