Merge pull request 'fix' (#12) from queues into dev

Reviewed-on: #12

.deploy/deploy-dev.yaml

@@ -28,7 +28,6 @@ services:
       PLATFORM_SECURITY_TOKEN: $PLATFORM_SECURITY_TOKEN
     networks:
       - queues-development
-      - configurator
     command: worker
     deploy:
       mode: replicated
@@ -115,5 +114,3 @@ networks:
     external: true
   queues-development:
     external: true
-  configurator:
-    external: true

.deploy/deploy-prod.yaml

@@ -28,7 +28,6 @@ services:
       PLATFORM_SECURITY_TOKEN: $PLATFORM_SECURITY_TOKEN
     networks:
       - queues
-      - configurator
     command: worker
     deploy:
       mode: replicated
@@ -128,5 +127,3 @@ networks:
     external: true
   queues:
     external: true
-  configurator:
-    external: true

daemons/mailbox.py

@@ -25,7 +25,4 @@ class Daemon(base.Daemon, queues.TasksHandlerMixin):
         reply_markup = payload.get('reply_markup')
         if reply_markup:
             body['reply_markup'] = reply_markup
-        try:
         self.bot.send_message(**body, parse_mode='Markdown')
-        except Exception as exc:
-            print('Error', str(exc))

helpers/sprint_platform.py

@@ -15,8 +15,11 @@ class PlatformClient:
         self.stage = stage
         self.configs = configs
         self.experiments = experiments
-        self.endpoint = 'http://configurator/'
+        self.endpoint = 'https://platform.sprinthub.ru/'
-        self.fetch_url = urllib.parse.urljoin(self.endpoint, '/api/v1/fetch')
+        self.configs_url = urllib.parse.urljoin(self.endpoint, 'configs/get')
+        self.experiments_url = urllib.parse.urljoin(self.endpoint, 'experiments/get')
+        self.staff_url = urllib.parse.urljoin(self.endpoint, 'is_staff')
+        self.fetch_url = urllib.parse.urljoin(self.endpoint, 'fetch')
         self.config_storage = {}
         self.experiment_storage = {}
         self.staff_storage = {}
@@ -41,6 +44,7 @@ class PlatformClient:
             try:
                 response = get(
                     url,
+                    headers={'X-Security-Token': self.platform_security_token},
                     params=params
                 )
                 if response.status_code == 200:

utils/queues.py

@@ -17,22 +17,22 @@ class QueuesException(Exception):
 class TasksHandlerMixin:
     def poll(self):
         while True:
-            response = requests.get(f'{QUEUES_URL}/api/v1/take', headers={'queue': self.queue_name}).json()
+            response = requests.get(f'{QUEUES_URL}/api/v1/take', headers={'queue': self.queue_name})
-            task = response.get('task')
+            if response.status_code == 404:
-            if not task:
                 time.sleep(0.2)
                 continue
+            data = response.json()
             try:
-                self.process(task['payload'])
+                self.process(data['payload'])
             except Exception as exc:
-                print(f'Error processing message id={task["id"]}, payload={task["payload"]}, exc={exc}')
+                print(f'Error processing message id={data["id"]}, payload={data["payload"]}, exc={exc}')
                 continue
             try:
-                resp = requests.post(f'{QUEUES_URL}/api/v1/finish', json={'id': task['id']})
+                resp = requests.post(f'{QUEUES_URL}/api/v1/finish', json={'id': data['id']})
                 if resp.status_code != 202:
                     raise QueuesException
             except:
-                print(f'Failed to finish task id={task["id"]}')
+                print(f'Failed to finish task id={data["id"]}')
 
     @property
     def queue_name(self):