Skip to content

Commit

Permalink
fix(robot-server): fix mqtt dependencies (#14384)
Browse files Browse the repository at this point in the history
We need to add paho-mqtt as an install dep in our setup.py for
good-practice reasons and to fix it in the pipfile.lock. More
importantly, we need to add it as a buildroot dep in the local buildroot
toolchain file so the OT-2 gets it, since otherwise the server can't
boot because it won't be present when notification_client tries to
import it.

Closes RQA-2270
  • Loading branch information
sfoster1 authored Jan 30, 2024
1 parent c067cf9 commit 87eda75
Show file tree
Hide file tree
Showing 4 changed files with 73 additions and 124 deletions.
1 change: 1 addition & 0 deletions robot-server/Config.in
Original file line number Diff line number Diff line change
Expand Up @@ -15,6 +15,7 @@ config BR2_PACKAGE_PYTHON_OPENTRONS_ROBOT_SERVER
select BR2_PACKAGE_PYTHON_TYPING_EXTENSIONS # runtime
select BR2_PACKAGE_PYTHON_UVICORN # runtime
select BR2_PACKAGE_PYTHON_WSPROTO # runtime
select BR2_PACKAGE_PYTHON_PAHO_MQTT # runtime

help
Opentrons HTTP server. Controls an OT-2 robot.
Expand Down
1 change: 1 addition & 0 deletions robot-server/Pipfile
Original file line number Diff line number Diff line change
Expand Up @@ -34,6 +34,7 @@ types-requests = "==2.25.6"
types-mock = "==4.0.1"
sqlalchemy2-stubs = "==0.0.2a21"
python-box = "==5.4.1"
types-paho-mqtt = "==1.6.0.20240106"

[packages]
anyio = "==3.6.1"
Expand Down
194 changes: 70 additions & 124 deletions robot-server/Pipfile.lock

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

1 change: 1 addition & 0 deletions robot-server/setup.py
Original file line number Diff line number Diff line change
Expand Up @@ -67,6 +67,7 @@ def get_version():
"systemd-python==234; sys_platform=='linux'",
"sqlalchemy==1.4.32",
"aiosqlite==0.17.0",
"paho-mqtt==1.6.1",
]


Expand Down

0 comments on commit 87eda75

Please sign in to comment.