Skip to content

Землянов Владимир Лаб. 2 Группа 6511#52

Closed
AvtoBBus wants to merge 13 commits intoitsecd:mainfrom
AvtoBBus:lab2
Closed

Землянов Владимир Лаб. 2 Группа 6511#52
AvtoBBus wants to merge 13 commits intoitsecd:mainfrom
AvtoBBus:lab2

Conversation

@AvtoBBus
Copy link

ФИО: Землянов Владимир
Номер группы: 6511
Номер лабораторной: 2
Номер варианта: 8
Краткое описание предметной области: Сотрудник компании
Краткое описание добавленных фич:

  • Реализован алгоритм балансировки QueryBased
  • Реализован ApiGateway и подняты три реплики сервиса генерации

@github-actions github-actions bot added In progress Код в процессе проверки Lab 2 Лабораторная №2. Балансировка нагрузки labels Mar 14, 2026
@github-actions github-actions bot requested a review from danlla March 14, 2026 07:03
Copy link

@danlla danlla left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

  • Почему-то нет структурированных логов генераторов в дашбордах aspire
  • Если в aspire поменять порты генераторов, то все перестанет работать, в gateway нужно брать хосты и порты генераторов из appsettings.json, они там появляются из-за .WithReference в aspire

@AvtoBBus
Copy link
Author

Не знаю на сколько правильно, но вынес ещё порт для gateway в appsettings и генераторы поднимаются аккуратно за ним, типо 5200 -> 5201, 5202 и 5203

Наверное тут по-хорошему нужно проверять ,что порты эти вообще свободны....

@AvtoBBus AvtoBBus requested a review from danlla March 16, 2026 12:54
@danlla
Copy link

danlla commented Mar 19, 2026

Наверное тут по-хорошему нужно проверять ,что порты эти вообще свободны

Ну по-хорошему наверно стоит, но сервис просто упадет с ошибкой, что порт занят, это не страшно

Здесь в принципе можно сделать настройку запуска с рандомным портом, который aspire сам выдаст, но можно так оставить

@AvtoBBus AvtoBBus requested a review from danlla March 19, 2026 09:37
Comment on lines +5 to +7
var builder = WebApplication.CreateBuilder(args);

builder.Configuration
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Тут нужно будет добавить AddServiceDefaults();

@danlla danlla added Approved Лабораторная зачтена and removed In progress Код в процессе проверки labels Mar 19, 2026
@danlla danlla closed this Mar 19, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Approved Лабораторная зачтена Lab 2 Лабораторная №2. Балансировка нагрузки

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants