edgexfoundry / edgex-compose

EdgeX Foundry Docker Compose release compose files and tools for building EdgeX compose files
Apache License 2.0
81 stars 115 forks source link

fix: [Main] Set EXTRA_PROXY_ROUTE_LIST for all add on services except Device Virtual #406

Closed lenny-goodell closed 10 months ago

lenny-goodell commented 10 months ago

fixes #405

If your build fails due to your commit message not passing the build checks, please review the guidelines here: https://github.com/edgexfoundry/edgex-compose/blob/main/.github/Contributing.md

PR Checklist

Please check if your PR fulfills the following requirements:

Testing Instructions

From compose builder run make run ds-onvif-camera ds-usb-camera ds-bacnet-ip ds-modbus ds-mqtt ds-rest ds-snmp ds-virtual ds-llrp ds-coap ds-gpio asc-http asc-mqtt asc-sample asc-metrics as-llrp asc-ex-mqtt mqtt-broker For each of the above services (expect mqtt-broker) verify https://localhost:8443/<service-key>/api/v3/ping results in 200 response.

lenny-goodell commented 10 months ago

@cherrycl , are the 2 failing tests on this PR known failures? THX!

lenny-goodell commented 10 months ago

do we apply this ADD Extra route logic into taf-secty ?

TAF team already added it where they needed it.

cherrycl commented 10 months ago

recheck