Conversation
github-actions
Bot
added
In progress
| public Task<Response<ServiceHostAndPort>> LeaseAsync(HttpContext httpContext) | ||
| { | ||
| var cumulative = 0.0; | ||
| var roll = Random.Shared.NextDouble(); |
There was a problem hiding this comment.
Можно тут домножать на сумму элементов weights, которую по-хорошему вынести в приватное поле
Тогда не нужно будет следить за тем, что сумма весов равна 1
| .WithHttpEndpoint(port: port, name: "api-endpoint", isProxied: false) | ||
| .WithExternalHttpEndpoints(); | ||
|
|
||
| gateway.WaitFor(replica); |
There was a problem hiding this comment.
Здесь не хватает .WithReference(replica), чтобы aspire прокинул адрес апи в api gateway
Если в aspire поменять порты, то все должно работать, api gateway проект должен достать из переменных окружения, которые пришли от aspire, адреса сервисов и поменять конфигурацию самого api gateway
danlla
left a comment
danlla
left a comment
There was a problem hiding this comment.
Если в aspire поменять порты у replica, то все перестанет работать, потому что сейчас все завязано на то, что написано в файле ocelot.json, нужно сделать так, чтобы эта конфигурация переопределялась, если в переменных окружения есть адреса сервисов, которые aspire туда будет прокидывать при использовании WithReference
|
|
||
| for (var i = 1; ; i++) | ||
| { | ||
| var url = builder.Configuration[$"services__vehicleapi-{i}__http__0"]; |
There was a problem hiding this comment.
Если посмотреть по трассировкам, то сейчас только один сервис обрабатывает все запросы
Здесь нужно указать: builder.Configuration[$"services:vehicleapi-{i}:api-endpoint:0"], так как эндпоинт в aspire назван как api-endpoint
В виде $"services__vehicleapi-{i}__http__0" у меня не запустилось нормально, такой переменной не находит в конфигурации
| { | ||
| options.AddDefaultPolicy(policy => | ||
| { | ||
| policy.AllowAnyOrigin() |
There was a problem hiding this comment.
Здесь стоит использовать origins клиента вместо AllowAnyOrigin
2 participants
ФИО: Чукарев Михаил
Номер группы: 6511
Номер лабораторной: 2
Номер варианта: 17
Краткое описание предметной области: Транспортное средство
Краткое описание добавленных фич: