Related to the changes introduced with #541
-
We do not want to have the logic about what board supports what CSI cameras inside of arduino-app-cli, but we want move that to the arduino-linux-config tool. And call it from app-cli.
-
Would be nice to have a new package that handles two related things: detecting the presence of devices (or the fact that they have been configured) and providing a list of volumes and env vars to pass to the container. This would be done during the provisioning so that all the code related to devices is in one package.
-
In addition, at the end of the provisioning, we should check if all the volumes and paths mounted to the container really exist, for security reasons we should not mount non-existing paths or files.
-
The provision.go calls DetectCSICameraDriver() independently of the Detect(ctx, platform) result. It would be nice to call Detect once, and get a result of the detected devices, then pass it to the provisioning (that will not need to call DetectCSICameraDriver() or similar functions).
Related to the changes introduced with #541
We do not want to have the logic about what board supports what CSI cameras inside of
arduino-app-cli, but we want move that to thearduino-linux-configtool. And call it from app-cli.Would be nice to have a new package that handles two related things: detecting the presence of devices (or the fact that they have been configured) and providing a list of volumes and env vars to pass to the container. This would be done during the provisioning so that all the code related to devices is in one package.
In addition, at the end of the provisioning, we should check if all the volumes and paths mounted to the container really exist, for security reasons we should not mount non-existing paths or files.
The provision.go calls DetectCSICameraDriver() independently of the Detect(ctx, platform) result. It would be nice to call Detect once, and get a result of the detected devices, then pass it to the provisioning (that will not need to call DetectCSICameraDriver() or similar functions).