Skip to content

CLOUD-529 Preserve provider identity on cloud unbind - #78

Draft
smintank wants to merge 3 commits into
mainfrom
factory/wca-93m
Draft

CLOUD-529 Preserve provider identity on cloud unbind#78
smintank wants to merge 3 commits into
mainfrom
factory/wca-93m

Conversation

@smintank

@smintank smintank commented Sep 9, 2026

Copy link
Copy Markdown
Contributor

Что происходит; кому и зачем нужно:

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


Что поменялось для пользователей:

  • Событие delete_provider останавливает и отключает только frpc и сборщик метрик.
  • Основной агент и конфигурация провайдера сохраняются, а старое состояние туннеля и метрик
    удаляется; ссылка переключается в переходное состояние unknown.
  • Ошибка очистки не подтверждает событие, поэтому повторная доставка безопасно завершает отвязку.
  • Явные команды del-provider и del-all-providers сохраняют удаление провайдера.

Как проверял/а:

  • Регрессионные тесты проверяют сохранение конфигурации, очистку runtime-состояния, остановку
    вспомогательных сервисов, переход ссылки, повтор после частичной ошибки и порядок подтверждения.
  • Полный прогон: black, isort, pylint 10.00/10, pytest — 191 тест, покрытие 94.07%.

@coveralls

coveralls commented Sep 9, 2026

Copy link
Copy Markdown

Coverage Report for CI Build 3

Coverage increased (+33.9%) to 97.661%

Details

  • Coverage increased (+33.9%) from the base build.
  • Patch coverage: 105 of 105 lines across 6 files are fully covered (100%).
  • 2 coverage regressions across 1 file.

Uncovered Changes

No uncovered changes found.

Coverage Regressions

2 previously-covered lines in 1 file lost coverage.

File Lines Losing Coverage Coverage
wb/cloud_agent/settings.py 2 98.25%

Coverage Stats

Coverage Status
Relevant Lines: 2394
Covered Lines: 2338
Line Coverage: 97.66%
Coverage Strength: 1.95 hits per line

💛 - Coveralls

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.

2 participants