CLI Subscribe fixes

[SebastianSchildt]

This fixes some issues in the CLI tool (does not touch API code itself)

Issues fixed:

• V2 API does not accept wildcards, therefore wildcard/branch subscribes did not work with a v2 API databroker. This is fixed by also adding wildcard expansion similar as we already have in the getValue path
• subscribeMultiple had severly broken v1/v2 routing logic. Also it was a weird command. Instead now the nromal subscribe CLI command can optionally also accept multiple paths

To test

• start current databroker i.e. docker run -it --rm --net=host ghcr.io/eclipse-kuksa/kuksa-databroker:0.7 --port 55558
• start old client, i.e. docker run --rm -it --net=host ghcr.io/eclipse-kuksa/kuksa-python-sdk/kuksa-client:main grpc://<your_host_ip>:55558
  • do sth like subscribe Vehicle.ADAS.** or subscribe Vehicle.Chassis and see it doesn't work
• start version from this PR (i.e. with checkout pip install, or build docker locally first like docker build -t kctest -f kuksa-client/Dockerfile .
  • do sth like subscribe Vehicle.ADAS.** or subscribe Vehicle.Chassis , do some setValue and see it works now

[nayakned]

Tested. Found working as described.

[lukasmittag]

should we thread subscribe call with multiple args to subscribe multiple via overloading the function arguments to not break api but keep aligned with cli calls?

[SebastianSchildt]

This API/SDK gotten to be a big mess and is due for some serious rewrite/redesign... But I think it is fine.

If you use subscribe_current_values([...]) from grpc API (not CLI) it works as the fix in #59 actually introduced the "fallback/extend" magic

If using the lower level subscribe(entries=[SubscribeEntry(...)])(called "full-fledged API in docs), well... that does not support v2 so will always use v1 (which does expansion databroker side)

I guess we would still need to repair subscribe_target_values([...]).... but maybe not here. It was broken before, I did not break it....

[lukasmittag]

Ahh I see the function is in the backend - so not exposed over sdk so my worries were unnecessary. I agree with the meme :)

[lukasmittag]

LGTM