Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[WebToolsE2E][Aspire] Deploying Aspire-Samples/Metrics solution using 'azd up', the status of container app shows failed: ContainerCrashing #3321

Closed
v-cazhang opened this issue Feb 8, 2024 · 11 comments
Assignees
Labels
Milestone

Comments

@v-cazhang
Copy link

REGRESSION INFO: Also repro on Aspire 8.0.0-preview.2.23619.3

INSTALL STEPS

  1. Clean machine: Win11 x64 23h2 ENU
  2. Install VS 17.10 Preview 1 [latest build, includes Aspire 8.0.0-preview.3.24105.21 ]
  3. Apply NuGet Feeds
  4. Install Docker Desktop for Windows

REPRO STEPS

  1. Clone aspire-samples repo from preview.3 branch
  2. Update NuGet.config to use the correct feeds
  3. Open samples\Metrics.sln
  4. Build
  5. Right-click solution > Open Folder in File Explorer. In the opened folder, click any place > Open in Terminal
  6. Runing 'azd init', select 'Use code in the current directory' > Confirm and continue initializing my app, type a new environment name, press enter.
  7. When running azd init choose to expose 'app' to the Internet
  8. Run azd up to deploy to Azure
  9. Open the resource group, click the 'app' container app.

ACTUAL:
The status shows failed: Failed:appContainerCrashing:
image

EXPECTED
No issues
image

@rajeshkamal5050
Copy link

@vhvb1989 can you try this?

@davidfowl
Copy link
Member

I don’t think this is a new problem. We don’t have a way to represent volumes in the manifest and this sample relies on them

@v-sherryfan
Copy link

This issue also reproduce on samples\AspireWithDapr.sln solution.

@davidfowl
Copy link
Member

@v-sherryfan can you paste the output of streaming logs here?

@v-sherryfan
Copy link

v-sherryfan commented Feb 21, 2024

@davidfowl The log stream is as below, Is this what you want?

2024-02-21T10:01:44.10957 Connecting to the container 'webfrontend'...

2024-02-21T10:01:44.13786 Successfully Connected to container: 'webfrontend' [Revision: 'webfrontend--p5ywlwb-7c6b8ccf77-lwln2', Replica: 'webfrontend--p5ywlwb']
2024-02-21T09:59:40.228555288Z Unhandled exception. System.InvalidOperationException: This service descriptor is keyed. Your service provider may not support keyed services.
2024-02-21T09:59:40.228591636Z at Microsoft.Extensions.DependencyInjection.ServiceDescriptor.ThrowKeyedDescriptor()
2024-02-21T09:59:40.228595493Z at Microsoft.Extensions.DependencyInjection.ServiceDescriptor.get_ImplementationType()
2024-02-21T09:59:40.228598849Z at Microsoft.Extensions.DependencyInjection.DaprServiceCollectionExtensions.<>c.b__0_0(ServiceDescriptor s)
2024-02-21T09:59:40.228602035Z at System.Linq.Enumerable.Any[TSource](IEnumerable1 source, Func2 predicate)
2024-02-21T09:59:40.228604961Z at Microsoft.Extensions.DependencyInjection.DaprServiceCollectionExtensions.AddDaprClient(IServiceCollection services, Action`1 configure)
2024-02-21T09:59:40.228608577Z at Program.

$(String[] args) in C:\Users\v-sherryfan\source\repos\aspire-samples\samples\AspireWithDapr\AspireWithDapr.Web\Program.cs:line 13

@rajeshkamal5050 rajeshkamal5050 added this to the February 2024 milestone Feb 26, 2024
@rajeshkamal5050
Copy link

The issue was initially filed for Aspire-Samples/Metrics which is not supported in Azd since it requires modeling on Aspire

@vhvb1989 can you try to reproduce the AspireWithDapr sample? if it reproduces, lets create a new issue.

@vhvb1989
Copy link
Member

The sample from https://github.com/dotnet/aspire-samples/tree/main/samples/AspireWithDapr might be out of date for preview4.

The code and API looks different from the one at https://github.com/dotnet/aspire/tree/main/playground/dapr

@philliphoff , are you keeping both samples? Do you know which one is the right one? Should we keep just one?

@vhvb1989
Copy link
Member

Root caused to: dapr/dotnet-sdk#1235

There is a workaround to use version 1.13.0-rc03 for the dapr samples. I'm closing this as not-azd related.

@vhvb1989
Copy link
Member

@v-cazhang for any future container crashing, can we have a new/unique issue please?

Let's avoid combining issues from different samples. The container crashing could be happening for a lot of reasons, so it is better to have unique issues with the logstream from the container

@v-cazhang
Copy link
Author

@v-cazhang for any future container crashing, can we have a new/unique issue please?

Let's avoid combining issues from different samples. The container crashing could be happening for a lot of reasons, so it is better to have unique issues with the logstream from the container

Got it, we will open separate issue for any container crashing in the future.

@v-sherryfan
Copy link

This issue is not reproduced on Aspire 8.0 P3 and 8.0 P4 after updating 'Dapr.AspNetCore' version to 1.13.0-rc03.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

5 participants