Skip to content

feat: adding a common insertion interface to the database#257

Open
alwaysunhappy wants to merge 47 commits into
mainfrom
fio-db-integration
Open

feat: adding a common insertion interface to the database#257
alwaysunhappy wants to merge 47 commits into
mainfrom
fio-db-integration

Conversation

@alwaysunhappy

Copy link
Copy Markdown
Collaborator

Closes #221
Добавила единый интерфейс для записи в БД, изменила fio_disks, чтобы результаты записывались в БД.

@alwaysunhappy alwaysunhappy requested a review from Artanias March 8, 2026 14:33
@alwaysunhappy alwaysunhappy self-assigned this Mar 8, 2026
@alwaysunhappy alwaysunhappy added the enhancement New feature or request label Mar 8, 2026
@Artanias Artanias changed the title Adding a commom insertion interface to thr DB feat: adding a common insertion interface to the database Mar 30, 2026
@alwaysunhappy

Copy link
Copy Markdown
Collaborator Author

@Artanias в main ветке в tests/image/performance/fio_disks.py в классе FioDisksDMDust была ошибка, FioSuite(...).run() - это генератор, а в коде он просто создавался без итерации:

try:
    FioSuite(client, read_cfg).run()
except RuntimeError:
    logger.info("Error above is intended, dm-dust works.")

То есть read часть не выполнялась, я попыталась исправить и заодно добавила новый абстрактный класс _FioDisksBaseTest, чтобы было меньше дублирования кода.

@Artanias

Artanias commented Apr 1, 2026

Copy link
Copy Markdown
Collaborator

@Artanias в main ветке в tests/image/performance/fio_disks.py в классе FioDisksDMDust была ошибка, FioSuite(...).run() - это генератор, а в коде он просто создавался без итерации:

try:
    FioSuite(client, read_cfg).run()
except RuntimeError:
    logger.info("Error above is intended, dm-dust works.")

То есть read часть не выполнялась, я попыталась исправить и заодно добавила новый абстрактный класс _FioDisksBaseTest, чтобы было меньше дублирования кода.

Отличная находка, стоит вынести фикс в PR для слития в main ветку.

Base automatically changed from test_plan_parameterization to main April 11, 2026 15:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants