Skip to content

Feat: Aspire for development#1419

Open
Zalmez wants to merge 13 commits intoValour-Software:mainfrom
Zalmez:main
Open

Feat: Aspire for development#1419
Zalmez wants to merge 13 commits intoValour-Software:mainfrom
Zalmez:main

Conversation

@Zalmez
Copy link

@Zalmez Zalmez commented Feb 18, 2026

This PR configures aspire for Valour primarily setup for development as to not disturb production, but long term it could be a good idea to look into implementing aspire for deployment as well (https://aspire.dev/deployment/overview/). There are also support for kubernetes deployment (https://aspire.dev/integrations/compute/kubernetes/)

There's a few benefits with having aspire with development alone such as not having to install other services locally other than a docker service (standard docker or other containerd services like podman). In addition aspire can take care of local secret management which would move it out of appsettings in a secrets storage locally on the device (https://aspire.dev/fundamentals/external-parameters/#secret-values). Then there's traces, default otel configuration etc

Developer onboarding could also be easier. I do recommend using the aspire cli for the best experience tho (https://aspire.dev/reference/cli/overview/)

@micah686
Copy link
Contributor

For getting this working, what did you do to ignore the rowversion columns, so it would work? I would like to add that to my PR (https://github.com/Valour-Software/Valour/pull/1380/changes), so I can finally test aspire.

@micah686
Copy link
Contributor

micah686 commented Mar 4, 2026

I think you may need to add a fallback, as when I tested aspire out, trying to run the server with docker containers (without using aspire/apphost) led to an exception regarding connecting to postgres.

@Zalmez
Copy link
Author

Zalmez commented Mar 4, 2026

For getting this working, what did you do to ignore the rowversion columns, so it would work? I would like to add that to my PR (https://github.com/Valour-Software/Valour/pull/1380/changes), so I can finally test aspire.

I just removed it entirely on these two lines:
https://github.com/Zalmez/Valour/blob/38ad6afe6f6f1251080adee0777034a80b927b10/Valour/Database/Migrations/20250820152533_MigrateRefactor0.cs#L418C21-L418C38
https://github.com/Zalmez/Valour/blob/38ad6afe6f6f1251080adee0777034a80b927b10/Valour/Database/Migrations/20250820152533_MigrateRefactor0.cs#L87

I think you may need to add a fallback, as when I tested aspire out, trying to run the server with docker containers (without using aspire/apphost) led to an exception regarding connecting to postgres.

I noticed there was a hard coded API url. I've updated that now

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants