Skip to content
Snippets Groups Projects
Commit c69fcead authored by Eterevskiy Georgiy's avatar Eterevskiy Georgiy
Browse files

Merge branch 'feature/disable_unregister_extensions' into 'master'

Disable unregister method for extensions

See merge request perxis/perxis-python!43
parents f5dad13a d20b4cfe
Branches
Tags v0.0.9
No related merge requests found
...@@ -29,18 +29,19 @@ def bootstrap( ...@@ -29,18 +29,19 @@ def bootstrap(
registered_extensions: manager_pb2.ListExtensionsResponse = ext_manager_stub.ListExtensions(manager_pb2.ListExtensionsRequest()) registered_extensions: manager_pb2.ListExtensionsResponse = ext_manager_stub.ListExtensions(manager_pb2.ListExtensionsRequest())
for ext in registered_extensions.extensions: # todo enable after fix - https://tracker.yandex.ru/PRXS-1507
if ext.extension == ext_descriptor.extension: # for ext in registered_extensions.extensions:
if ext.version != ext_descriptor.version: # if ext.extension == ext_descriptor.extension:
ext_manager_stub.UnregisterExtensions( # if ext.version != ext_descriptor.version:
manager_pb2.UnregisterExtensionsRequest( # ext_manager_stub.UnregisterExtensions(
extensions=[ext_descriptor] # manager_pb2.UnregisterExtensionsRequest(
) # extensions=[ext_descriptor]
) # )
# )
logger.info( #
f"Изменилась версия расширения {ext_descriptor.extension} ({ext.version} -> {ext_descriptor.version})" # logger.info(
) # f"Изменилась версия расширения {ext_descriptor.extension} ({ext.version} -> {ext_descriptor.version})"
# )
ext_manager_stub.RegisterExtensions(manager_pb2.RegisterExtensionsRequest( ext_manager_stub.RegisterExtensions(manager_pb2.RegisterExtensionsRequest(
extensions=[ext_descriptor] extensions=[ext_descriptor]
......
...@@ -14,7 +14,7 @@ def load_requirements(): ...@@ -14,7 +14,7 @@ def load_requirements():
setup( setup(
name='perxis', name='perxis',
version='0.0.21', version='0.0.22',
description='Perxis python client', description='Perxis python client',
long_description=long_description, long_description=long_description,
long_description_content_type='text/markdown', long_description_content_type='text/markdown',
......
0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Please register or to comment