Add: Services to system #883

Misha-Shvets · 2026-01-13T13:21:03Z

Переименование раздела services в System по аналогии с АД

Задача: 1084

…ieval

…or services and system

…es for Kerberos configuration files

…es, enhance config update script, and simplify test case

…Service class

…krb_doc

milov-dmitriy

главное проверь это локально любым из способов, главное чтобы переименование сработало корректно и даунгрейд тоже сработал корректно

milov-dmitriy · 2026-01-13T14:45:05Z

app/alembic/versions/a1b2c3d4e5f6_rename_services_to_system.py

+    """Upgrade: Rename 'services' container to 'System'."""
+
+    async def _rename_services_to_system(connection: AsyncConnection) -> None:
+        session = AsyncSession(bind=connection)


из контейнера берем теперь сесиию

milov-dmitriy · 2026-01-13T14:58:05Z

app/alembic/versions/a1b2c3d4e5f6_rename_services_to_system.py

+            base_directories = await get_base_directories(session)
+            if not base_directories:
+                await session.commit()
+                return


это можно наружу, зачем это внутри try

или может вообще уберем try? у нас это не очень распространенная практика

milov-dmitriy · 2026-01-13T15:23:46Z

app/alembic/versions/a1b2c3d4e5f6_rename_services_to_system.py

+                system_exists = await session.scalar(
+                    select(exists(Directory)).where(
+                        and_(
+                            qa(Directory.name) == "System",
+                            qa(Directory.parent_id) == services_dir.parent_id,
+                        ),
+                    ),
+                )


select(exists().where())
а у тебя так:
select(exists()).where()

проверь работоспособность этого решения в реалтайме плз

можно создать любые директории-контейнеры и прочее руками, а затем сделать make downgrade чтоб откатить и накатить эту миграцию или можно просто сделать чекаут с dev на твою ветку

тут точно нельзя никак по другому запрос написать?

можно в запрос встроить parent_id.in_(service_dir_ids) (псевдокод)

предлагаю сделать "команда с отдельной строки" как у нас везде принято:

select where

milov-dmitriy · 2026-01-13T15:26:24Z

app/alembic/versions/a1b2c3d4e5f6_rename_services_to_system.py

+        result = await session.execute(
+            select(Attribute).where(
+                Attribute.value.ilike(f"%{old_value}%"),  # type: ignore
+            ),
+        )


предлагаю сделать "команда с отдельной строки" как у нас везде принято:

select where

milov-dmitriy · 2026-01-13T15:26:30Z

app/alembic/versions/a1b2c3d4e5f6_rename_services_to_system.py

+        children = await session.scalars(
+            select(Directory).where(qa(Directory.parent_id) == parent_id),
+        )


предлагаю сделать "команда с отдельной строки" как у нас везде принято:

select where

milov-dmitriy · 2026-01-13T15:28:14Z

app/alembic/versions/a1b2c3d4e5f6_rename_services_to_system.py

+            select(Directory).where(qa(Directory.parent_id) == parent_id),
+        )
+
+        for child in children:


child -> child_dir

milov-dmitriy · 2026-01-13T15:28:52Z

app/alembic/versions/a1b2c3d4e5f6_rename_services_to_system.py

+def downgrade(container: AsyncContainer) -> None:  # noqa: ARG001
+    """Downgrade: Rename 'System' container back to 'services'."""
+


проверь downgrade на аналогичные комментарии что и upgrade

неминг

форматирование запросов

необходимость try-блока

сессия из контейнера

milov-dmitriy · 2026-01-13T15:30:25Z

app/ldap_protocol/kerberos/service.py

        base_dn, domain = await self._get_base_dn()
        krbadmin = f"cn=krbadmin,cn=users,{base_dn}"
        krbgroup = f"cn=krbadmin,cn=groups,{base_dn}"
-        services_container = f"ou=services,{base_dn}"
+        services_container = get_services_container_dn(base_dn)


[nitpick] можем сделать для krbadmin, krbgroup, ... аналогичный метод что и для services_container

м?

milov-dmitriy · 2026-01-13T15:30:47Z

app/ldap_protocol/kerberos/utils.py

+def get_services_container_dn(base_dn: str) -> str:
+    """Get System container DN for services."""
+    return f"ou=System,{base_dn}"


а это точно к керберосу относится а не к лдапу?

milov-dmitriy · 2026-01-13T15:31:21Z

docker-compose.yml

      - ./certs:/certs
      - ./app:/app
      - ldap_keytab:/LDAP_keytab/
+      - kdc:/etc/krb5kdc/


а это нужная для этого ПР правка? не могу разобраться

milov-dmitriy · 2026-01-13T15:38:42Z

app/alembic/versions/a1b2c3d4e5f6_rename_services_to_system.py

+
+
+def upgrade(container: AsyncContainer) -> None:  # noqa: ARG001
+    """Upgrade: Rename 'services' container to 'System'."""


[nitpick] название System именно в upper кейсе?
просто у нас везде lower, и это как будто выбивается из общей практики, я бы уточнил у автора задачи

milov-dmitriy

и еще надо будет подлить к тебе мою задачу 796, т.к. я в ней добавляю понятие системная директория или нет, если системная - значит ее нельзя менять

upd. но это уже после того как ее апрувнут, но я бы предлагал сначала мою апрувать потом твою

milov-dmitriy · 2026-01-13T15:43:52Z

app/alembic/versions/a1b2c3d4e5f6_rename_services_to_system.py

+depends_on: None | list[str] = None
+
+
+def upgrade(container: AsyncContainer) -> None:  # noqa: ARG001


еще что не могу понять, а что если в лдап дереве нет директории с именем service ? мы же тогда просто выйдем из миграции, и кажется , что это не совсем правильно, наверное стоит создавать директорию System в таком случае

не знаю, стоит ли мне лезть в бизнеслогику, но не знаю в общем...

milov-dmitriy · 2026-01-13T15:45:37Z

app/alembic/versions/a1b2c3d4e5f6_rename_services_to_system.py

+                await session.flush()
+                await _update_descendants(session, services_dir.id)
+
+            await _update_attributes(session, "ou=services", "ou=System")


предлагаю строго привязывать к id той директории которую мы обновили, т.е. передавать в _update_attributes еще и directory_id, чтобы не сделать "лишних" изменений у тех директорий, которые в отбор не попали, но атрибуты которых попали под фильтр атрибутов. думаю, это повысит надежность миграции

m.shvets added 6 commits January 13, 2026 16:22

Refactor: use get_services_container_dn for service container DN retr…

eb166b1

…ieval

Refactor: improve attribute update queries in alembic migration

363c92c

Refactor: simplify directory selection queries in alembic migration f…

94ae725

…or services and system

Enhance entrypoint script: add config fix function and periodic updat…

1eeb1d5

…es for Kerberos configuration files

Update Kerberos configuration handling: add shared volume for KDC fil…

371e37f

…es, enhance config update script, and simplify test case

Remove outdated comment regarding services container name in Kerberos…

88cc71a

…Service class

Misha-Shvets force-pushed the services_to_system_1084 branch from 38592fb to 88cc71a Compare January 13, 2026 13:23

milov-dmitriy self-requested a review January 13, 2026 13:52

milov-dmitriy assigned milov-dmitriy and Misha-Shvets Jan 13, 2026

Update test case in kadmin.py to reflect new expected hash value for …

13b2265

…krb_doc

milov-dmitriy requested changes Jan 13, 2026

View reviewed changes

milov-dmitriy reviewed Jan 13, 2026

View reviewed changes

milov-dmitriy requested changes Jan 13, 2026

View reviewed changes

milov-dmitriy reviewed Jan 13, 2026

View reviewed changes

milov-dmitriy changed the title ~~Services to system 1084~~ Add: Services to system Jan 14, 2026

		def downgrade(container: AsyncContainer) -> None: # noqa: ARG001
		"""Downgrade: Rename 'System' container back to 'services'."""



		def upgrade(container: AsyncContainer) -> None: # noqa: ARG001
		"""Upgrade: Rename 'services' container to 'System'."""

		depends_on: None \| list[str] = None


		def upgrade(container: AsyncContainer) -> None: # noqa: ARG001

Add: Services to system #883

Are you sure you want to change the base?

Add: Services to system #883

Uh oh!

Conversation

Misha-Shvets commented Jan 13, 2026 • edited by milov-dmitriy Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

milov-dmitriy left a comment

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

milov-dmitriy Jan 13, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

milov-dmitriy left a comment • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Choose a reason for hiding this comment

Uh oh!

milov-dmitriy Jan 13, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

3 participants

Misha-Shvets commented Jan 13, 2026 •

edited by milov-dmitriy

Loading

milov-dmitriy Jan 13, 2026 •

edited

Loading

milov-dmitriy left a comment •

edited

Loading

milov-dmitriy Jan 13, 2026 •

edited

Loading