eclipse-velocitas / vehicle-app-python-sdk

vehicle-app-python-sdk
Apache License 2.0
7 stars 18 forks source link

[Bug]: Action "Project Creation" fails on main branch #133

Closed erikbosch closed 2 weeks ago

erikbosch commented 3 weeks ago

What happened?

A regression on main, it worked about a week ago

Operating System

Linux

What release version or branch did you use?

main

Relevant log output

2024-07-04T08:37:28.7486711Z app/tests/integration/integration_test.py:43: AssertionError
2024-07-04T08:37:28.7487984Z ------------------------------ Captured log call -------------------------------
2024-07-04T08:37:28.7489114Z ERROR    grpc._common:_common.py:91 Exception serializing message!
2024-07-04T08:37:28.7490001Z Traceback (most recent call last):
2024-07-04T08:37:28.7491312Z   File "/home/vscode/.local/lib/python3.10/site-packages/grpc/_common.py", line 89, in _transform
2024-07-04T08:37:28.7492410Z     return transformer(message)
2024-07-04T08:37:28.7493947Z TypeError: descriptor 'SerializeToString' for 'google._upb._message.Message' objects doesn't apply to a 'list' object
2024-07-04T08:37:28.7495395Z =========================== short test summary info ============================
2024-07-04T08:37:28.7496585Z FAILED app/tests/integration/integration_test.py::test_set_position_not_allowed
2024-07-04T08:37:28.7497740Z !!!!!!!!!!!!!!!!!!!!!!!!!! stopping after 1 failures !!!!!!!!!!!!!!!!!!!!!!!!!!!
2024-07-04T08:37:28.7498676Z ============================== 1 failed in 21.11s ==============================

I tried to do a little of troubleshooting but with little success. No log printout from seatadjuster found, so difficult to see if/where the problem is in seatadjuster app

erikbosch commented 3 weeks ago

Likely unrelated - but I noted a circular dependency. If you change the seat example in this repo, the test will anyway take an older version of the seat example and try to run it, as velocitas cli refers an older package of this repo.

A work around is to possible do something like below

# Make sure that we do not use old shit, we do not want circular dependency to old code
          if [ "${{ matrix.example }}" != "no-example" ]; then
            cp -r ../sdk/examples/${{ matrix.example }}/* ../app/app
          fi
erikbosch commented 2 weeks ago

Possible cause is https://github.com/pypa/setuptools/issues/4374

In the failing actions setuptools 70.* is used


2024-07-08T06:44:17.1670002Z [2024-07-08T06:44:17.166Z] Requirement already satisfied: setuptools in /usr/local/lib/python3.10/dist-packages (from grpcio-tools==1.59.0->-r ./requirements.txt (line 24)) (70.2.0)

2024-06-26T08:10:34.1813793Z [2024-06-26T08:10:34.180Z] Requirement already satisfied: setuptools in /usr/local/lib/python3.10/dist-packages (from grpcio-tools==1.59.0->-r ./requirements.txt (line 24)) (69.5.1)