Azure-Samples / azure-intelligent-edge-patterns

Samples for Intelligent Edge Patterns
MIT License
112 stars 136 forks source link

Factory AI Vision Jetson Templates Have Merge Markers In #316

Open jedimasterjonny opened 3 years ago

jedimasterjonny commented 3 years ago

Please provide us with the following information:

This issue is for a: (mark with an x)

- [ ] bug report -> please search issues before submitting
- [ ] feature request
- [ ] documentation issue or request
- [X] regression (a behavior that used to work and stopped in a new release)

Both https://github.com/Azure-Samples/azure-intelligent-edge-patterns/blob/master/factory-ai-vision/EdgeSolution/deployment.jetson.template.json and https://github.com/Azure-Samples/azure-intelligent-edge-patterns/blob/master/factory-ai-vision/EdgeSolution/deployment.jetson.opencv.template.json have merge conflict markers in on the master branch.

I would offer a PR to resolve this - however the kakalin/ovms-server docker image the conflict references has no container for the ARMv8 architecture in use by the Jetson, so if this change was merged successfully, the deployment would still fail. Therefore I'm not sure what the desired resolution is.

Minimal steps to reproduce

Attempt a deployment using the deployment.jetson.template.json or deployment.jetson.opencv.template.json template

Any log messages given by the failure

Expected/desired behavior

Deployment to the Jetson works

OS and Version?

Tegra 4 Linux (Ubuntu 18.04) on Jetson NX

penorouzi commented 3 years ago

Thanks @jedimasterjonny. Hey, @waitingkuo can you please see why we have OVMS deployment as part of master branch (especially for Jetson devices)? Master branch should not have any OVMS specific references.