Skip to content

Feature: Bind9 to PowerDNS migration#954

Merged
Naksen merged 6 commits intodevfrom
feature/bind_to_pdns_migration
Mar 4, 2026
Merged

Feature: Bind9 to PowerDNS migration#954
Naksen merged 6 commits intodevfrom
feature/bind_to_pdns_migration

Conversation

@iyashnov
Copy link
Collaborator

@iyashnov iyashnov commented Mar 3, 2026

Задача: 1048

Добавлена автоматическая миграция с Bind9 на PowerDNS. Происходит проверка наличия файлов зоны и файлов-маркеров. Если файл-маркер отсутствует и присутствую файлы зоны, то происходит парс файлов бинда и создание всех найденых зон в PowerDNS.

Добавлен менеджер миграции
Добавлен контейнер только в продовый компоуз

Copilot AI review requested due to automatic review settings March 3, 2026 14:25
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Adds an automated one-shot migration path to import BIND9 zones/records into PowerDNS, exposed as a new CLI entrypoint and runnable via a dedicated Docker Compose service.

Changes:

  • Added BindToPDNSMigrationManager to parse BIND config/zone files and create zones/records in PowerDNS.
  • Added --migrate_dns entrypoint wiring in multidirectory.py and DI provider wiring in ioc.py.
  • Added a dns_migration service to .package/docker-compose.yml to run the migration container after DB migrations and PowerDNS startup.

Reviewed changes

Copilot reviewed 4 out of 4 changed files in this pull request and generated 13 comments.

File Description
app/multidirectory.py Adds a new --migrate_dns entrypoint/factory to run the migration manager.
app/ldap_protocol/dns/managers/bind_to_pdns_migrations_manager.py Implements the BIND→PowerDNS migration logic (config parsing, zone parsing, record creation, marker files).
app/ioc.py Registers a DI provider to construct BindToPDNSMigrationManager with PowerDNS clients/settings.
.package/docker-compose.yml Adds a production/package compose service to execute the DNS migration container.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@Naksen Naksen merged commit 5594f43 into dev Mar 4, 2026
6 checks passed
@Naksen Naksen deleted the feature/bind_to_pdns_migration branch March 4, 2026 10:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants