Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Config variable Path changed to Paths to support real sense cameras #278

Closed
presatish opened this issue Aug 30, 2023 · 0 comments · Fixed by #290
Closed

Config variable Path changed to Paths to support real sense cameras #278

presatish opened this issue Aug 30, 2023 · 0 comments · Fixed by #290
Assignees
Labels
bug Something isn't working
Milestone

Comments

@presatish
Copy link
Contributor

presatish commented Aug 30, 2023

🐞 Bug Report

Affected Services [REQUIRED]

device-usb-camera

Is this a regression?

No

Description and Minimal Reproduction [REQUIRED]

After starting all the core services and the usb service when the usb device is manually added using Path instead of Paths this issue is seen.

🔥 Exception or Error


curl -X POST -H 'Content-Type: application/json'  \
http://localhost:59881/api/v3/device \
-d '[
    {
    "apiVersion" : "v3",
    "device": {
        "name": "Camera001",
        "serviceName": "device-usb-camera",
        "profileName": "USB-Camera-General",
        "description": "My test camera",
        "adminState": "UNLOCKED",
        "operatingState": "UP",
        "protocols": {
            "USB": {
            "CardName": "UVC Camera (012d:0678)",
            "SerialNumber": "71C0AE6","
            "Path": "/dev/video0",
            "AutoStreaming": "false"
            }
        }
    }
    }
]'

[{"apiVersion":"v3","message":"Device Camera001 validation failed: property Paths of protocol USB is missing. Please check device configuration","statusCode":500}]

🌍 Your Environment

Deployment Environment: Docker, Docker + Dev Hybrid

EdgeX Version [REQUIRED]: main

Anything else relevant?
Path Deprecated info needs to be added in the code and docs. For now the code has to support both Path and Paths to avoid breaking change until the next release.

@presatish presatish added the bug Something isn't working label Aug 30, 2023
@presatish presatish self-assigned this Sep 8, 2023
@lindseysimple lindseysimple added this to the Napa milestone Sep 12, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment