AddUvicornApp doesn't use uvicorn from the virtual environment

[dluc]

Describe the bug

Trying to launch a python service from Aspire using AddUvicornApp(), the launch fails with:

ModuleNotFoundError: No module named 'redis'

When I launch the python service manually, using poetry run uvicorn app.main:app everything works fine instead.

I found that the different behavior is because AddUvicornApp uses uvicorn (and python) from the host, rather than the one inside .venv:

Manual => poetry run uvicorn app.main:app => Ok
AddUvicornApp => uvicorn app.main:app => Fail

Could you change the implementation to allow using uvicorn from the virtual env (.venv dir)?

Regression

No response

Steps to reproduce

Create app with Poetry, add a dependency missing in the host, integrate the service using `AddUvicornApp`. The app crashes.

Expected behavior

AddUvicornApp should leverage uvicorn from the app .venv folder

Screenshots

No response

IDE and version

Other

IDE version

CLI

Nuget packages

Aspire.Hosting.AppHost 9.1.0
CommunityToolkit.Aspire.Hosting.Python.Extensions 9.2.1

Additional context

No response

Help us help you

No, just wanted to report this

[dluc]

This change to private ... AddUvicornApp(...) method should be sufficient:

From:

var instrumentationExecutable = virtualEnvironment.GetExecutable("opentelemetry-instrument");
var projectExecutable = instrumentationExecutable ?? "uvicorn";

To:

var instrumentationExecutable = virtualEnvironment.GetExecutable("opentelemetry-instrument");
var uvicornExecutable = virtualEnvironment.GetExecutable("uvicorn") ?? "uvicorn";
var projectExecutable = instrumentationExecutable ?? uvicornExecutable;