feat: added dotnet aspire support#1132
Conversation
|
I tested this locally against the current Aspire stack and a local Podman-backed Kubernetes cluster. Environment:
What worked:
Two things I noticed:
I will try to improve these two points. |
|
not to bad 😄 thanks for your effort!
ah - yes that's my fault - I didn't wanted to tie both samples together and messed it up 🙈 may be we can have an extra aspire operator calling |
|
I could not push directly to https://github.com/msexxeta/dotnet-operator-sdk/tree/pr-1132-aspire-kubeops-publish The branch contains commit To pull it into this PR branch: git fetch https://github.com/msexxeta/dotnet-operator-sdk.git pr-1132-aspire-kubeops-publish
git cherry-pick cb7a720I verified this locally with:
The solution build also passes, with the existing net8/net9 package TFM warnings from |
|
I think there are even more implications to consider here: the operating model of aspire is to run .net services locally. I don't think the |
|
@msexxeta thanks for the pull request - but that one has exactly what I didn't want to do. see: #1132 (comment)
the "normal" operator sample now has a reference to aspire - I wanted to avoid that as we have different scenarios to serve. I think it would be a better approach to have an extra aspire.operator for the aspire sample. |
if you're referring to the examples operator #if DEBUG I've added - this was just purely cosmetics as the crd installer is also added with a preprocessor directive and the using statement becomes unused when switching to debug mode. I think this won't be an issue once the aspire operator is separated |
|
#1135 In short: This supports the operating model with a local and a cloud kubernetes environment, as well as without a kubernetes environment, just directly generating manifests. It also has the usual methods of configuring such as found with other azure models, such as Run... Publish... with specific configuration overrides inside. |
|
closed in favor of #1135 |
closes #926