Eric Erhardt

Results 266 comments of Eric Erhardt

@vhvb1989 @ellismg - does `azd` support default values in the manifest as above? Or do we need to log an issue in azd to get this support?

> + public static IResourceBuilder AddParameter(this IDistribuedApplicationBuilder builder, string value, bool secret = false); > + public static IResourceBuilder AddParameter(this IDistribuedApplicationBuilder builder, Func valueGetter, bool secret = false); > +...

> Code wins. Hmmm 🤔, maybe it shouldn't be named `defaultValue` then...

Following things like #7005 #6605, this seems like something we should enable. cc @sebastienros

Elasticsearch seems to be failing.

> And more needed by TestShop: All 3 of these are done.

> @eerhardt what are your thoughts on this? Should we introduce such a breaking change or perhaps just extend the existing ways with a new overload? My default opinion is...

I don't think this is a HealthChecks issue, but instead the fact that you are mis-matching `Npgsql.EntityFrameworkCore.PostgreSQL` v8 with `Npgsql` v9. AFAICT this is not a supported combination. You need...

> I would guess this is because Azure.AI.OpenAI only targets netstandard2.0, the `#if NET6_0_OR_GREATER` here: https://github.com/Azure/azure-sdk-for-net/blob/main/sdk/openai/Azure.AI.OpenAI/src/Generated/AzureOpenAIChatError.Serialization.cs#L58 and in similar places is ignored since there is no .NET6 or greater target...

> @eerhardt could you please import `docker.io/sosedoff/pgweb/latest` to make ci green? I've pushed it to the netaspireci.azurecr.io registry. However, I don't believe the test will pass because the CI machines...