Skip to content

[#247] PortManager__Generic uses lock-dirs for reserved ports #255

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
wants to merge 19 commits into
base: master
Choose a base branch
from

Conversation

dmitry-lipetsk
Copy link
Collaborator

@dmitry-lipetsk dmitry-lipetsk commented May 6, 2025

New structures

  • OsLockObj: base (abstract) class to manage fs-based lock object.

Critical changes

  • utils.bound_ports is removed

Closes #247

- [FIX] A creation of self._log_dir is moved to reserve_port method
- Code normalization [copy&paste mistakes]
- New asserts are added
- New comments are added
@dmitry-lipetsk
Copy link
Collaborator Author

dmitry-lipetsk commented May 7, 2025

Patch was tested with probackup2.

@dmitry-lipetsk
Copy link
Collaborator Author

dmitry-lipetsk commented Jun 23, 2025

У меня сомнения насчет это патча.

Штука полезная, но слишком требовательная к устойчивости тестов - если блокировочные каталоги не удалять, то их потом надо будет чистить руками. Это не критично для одноразовых тестовых машин, но все равно как-то не очень...

Пусть пока повисит.

Можно по умолчанию оставить все как есть, а эту фичу оформить как новый класс.

@demonolock
Copy link
Contributor

Какой новый класс, делали чтобы проблему устранить же. И если и с проблемой живется ок, то может просто не делать вовсе

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.

Using lock files for port reservation
2 participants