Merge pull request 'fix' (#22) from master into dev

Reviewed-on: #22

.deploy/deploy-dev.yaml

@@ -6,12 +6,12 @@ services:
   worker:
     image: mathwave/sprint-repo:ruz-bot
     environment:
+      MONGO_HOST: "mongo.develop.sprinthub.ru"
       STAGE: "development"
       MONGO_PASSWORD: $MONGO_PASSWORD_DEV
     networks:
       - queues-development
       - configurator
-      - mongo-development
     command: worker
     deploy:
       mode: replicated
@@ -24,11 +24,12 @@ services:
   fetch:
     image: mathwave/sprint-repo:ruz-bot
     environment:
+      MONGO_HOST: "mongo.develop.sprinthub.ru"
       STAGE: "development"
       MONGO_PASSWORD: $MONGO_PASSWORD_DEV
+      PLATFORM_SECURITY_TOKEN: $PLATFORM_SECURITY_TOKEN
     networks:
       - queues-development
-      - mongo-development
     command: fetch
     deploy:
       mode: replicated
@@ -41,11 +42,12 @@ services:
   notify:
     image: mathwave/sprint-repo:ruz-bot
     environment:
+      MONGO_HOST: "mongo.develop.sprinthub.ru"
       STAGE: "development"
       MONGO_PASSWORD: $MONGO_PASSWORD_DEV
+      PLATFORM_SECURITY_TOKEN: $PLATFORM_SECURITY_TOKEN
     networks:
       - queues-development
-      - mongo-development
     command: notify
     deploy:
       mode: replicated
@@ -58,12 +60,14 @@ services:
   ruz-bot-nginx:
     image: mathwave/sprint-repo:ruz-bot
     networks:
-      - common-infra-nginx-development
+      - common-infra-nginx
-      - queues-development
-      - mongo-development
     environment:
+      MONGO_HOST: "mongo.develop.sprinthub.ru"
       STAGE: "development"
       MONGO_PASSWORD: $MONGO_PASSWORD_DEV
+      PLATFORM_SECURITY_TOKEN: $PLATFORM_SECURITY_TOKEN
+    networks:
+      - queues-development
     command: api
     deploy:
       mode: replicated
@@ -80,5 +84,3 @@ networks:
     external: true
   configurator:
     external: true
-  mongo-development:
-    external: true

.deploy/deploy-prod.yaml

@@ -3,15 +3,32 @@ version: "3.4"
 
 services:
 
-  worker:
+  poll:
     image: mathwave/sprint-repo:ruz-bot
     environment:
+      STAGE: "production"
+      TELEGRAM_TOKEN: $TELEGRAM_TOKEN_PROD
+    networks:
+      - queues
+    command: poll
+    deploy:
+      mode: replicated
+      restart_policy:
+        condition: any
+      update_config:
+        parallelism: 1
+        order: start-first
+
+  worker:
+    image: mathwave/sprint-repo:ruz-bot
+    environment:
+      MONGO_HOST: "mongo.sprinthub.ru"
       STAGE: "production"
       MONGO_PASSWORD: $MONGO_PASSWORD_PROD
+      PLATFORM_SECURITY_TOKEN: $PLATFORM_SECURITY_TOKEN
     networks:
       - queues
       - configurator
-      - mongo
     command: worker
     deploy:
       mode: replicated
@@ -21,21 +38,41 @@ services:
         parallelism: 1
         order: start-first
 
+  mailbox:
+    image: mathwave/sprint-repo:ruz-bot
+    environment:
+      STAGE: "production"
+      TELEGRAM_TOKEN: $TELEGRAM_TOKEN_PROD
+    networks:
+      - queues
+    command: mailbox
+    deploy:
+      mode: replicated
+      restart_policy:
+        condition: any
+      update_config:
+        parallelism: 1
+        order: start-first
+
   fetch:
     image: mathwave/sprint-repo:ruz-bot
     environment:
+      MONGO_HOST: "mongo.sprinthub.ru"
       STAGE: "production"
       MONGO_PASSWORD: $MONGO_PASSWORD_PROD
+      PLATFORM_SECURITY_TOKEN: $PLATFORM_SECURITY_TOKEN
       DEBUG: "false"
     networks:
       - queues
-      - configurator
-      - mongo
     command: fetch
     deploy:
       mode: replicated
       restart_policy:
         condition: any
+      placement:
+        constraints:
+          - node.role == worker
+          - node.labels.zone == ru
       update_config:
         parallelism: 1
         order: start-first
@@ -43,18 +80,22 @@ services:
   notify:
     image: mathwave/sprint-repo:ruz-bot
     environment:
+      MONGO_HOST: "mongo.sprinthub.ru"
       STAGE: "production"
       MONGO_PASSWORD: $MONGO_PASSWORD_PROD
+      PLATFORM_SECURITY_TOKEN: $PLATFORM_SECURITY_TOKEN
       DEBUG: "false"
     networks:
       - queues
-      - configurator
-      - mongo
     command: notify
     deploy:
       mode: replicated
       restart_policy:
         condition: any
+      placement:
+        constraints:
+          - node.role == worker
+          - node.labels.zone == ru
       update_config:
         parallelism: 1
         order: start-first
@@ -63,17 +104,21 @@ services:
     image: mathwave/sprint-repo:ruz-bot
     networks:
       - common-infra-nginx
-      - configurator
-      - mongo
     environment:
+      MONGO_HOST: "mongo.sprinthub.ru"
       STAGE: "production"
       MONGO_PASSWORD: $MONGO_PASSWORD_PROD
+      PLATFORM_SECURITY_TOKEN: $PLATFORM_SECURITY_TOKEN
       DEBUG: "false"
     command: api
     deploy:
       mode: replicated
       restart_policy:
         condition: any
+      placement:
+        constraints:
+          - node.role == worker
+          - node.labels.zone == ru
       update_config:
         parallelism: 1
         order: start-first
@@ -85,5 +130,3 @@ networks:
     external: true
   configurator:
     external: true
-  mongo:
-    external: true

Dockerfile

@@ -7,7 +7,6 @@ RUN apt-get install -y locales locales-all
 ENV LANGUAGE ru_RU.UTF-8
 ENV LANG ru_RU.UTF-8
 ENV LC_ALL ru_RU.UTF-8
-ENV PYTHONUNBUFFERED=1
 COPY requirements.txt requirements.txt
 RUN pip install -r requirements.txt
 COPY . .

helpers/answer.py

@@ -55,6 +55,13 @@ class Answer:
 
     def process(self):
         user = User(self.user['chat_id'])
+        try:
+            bot_enabled_exp = platform.get_experiment('bot_enabled')
+            if not bot_enabled_exp['enabled'] or not eval(bot_enabled_exp['condition']):
+                return
+        except Exception as exc:
+            logging.info(exc)
+            return
         getattr(
             self,
             "handle_state_" + self.user['state'],
@@ -70,7 +77,7 @@ class Answer:
         body = {'text': text, 'chat_id': self.user['chat_id'], 'parse_mode': 'Markdown'}
         if reply_markup:
             body['reply_markup'] = reply_markup.to_json()
-        queues.set_task('botalka_mailbox', {'project': 'ruz-bot', 'name': 'telegram-bot', 'body': body}, 5)
+        queues.set_task('botalka_mailbox', {'project': 'ruz-bot', 'name': 'telegram-bot', 'body': body}, 1)
 
     def set_state(self, state: str):
         self.user['state'] = state

helpers/mongo.py

@@ -9,7 +9,7 @@ from helpers import now
 
 class Mongo:
     def __init__(self):
-        url = f"mongodb://{settings.MONGO_USER}:{settings.MONGO_PASSWORD}@mongo:27017/"
+        url = f"mongodb://{settings.MONGO_USER}:{settings.MONGO_PASSWORD}@{settings.MONGO_HOST}:27017/"
         self.client = pymongo.MongoClient(url)
         self.database = self.client.get_database("ruz-bot")
         self.users_collection.create_index([

helpers/ruz.py

@@ -12,8 +12,7 @@ fields = [
     'date_start',
     'date_end',
     'lecturer_profiles',
-    'stream_links',
+    'stream_links'
-    'type',
 ]
 
 

utils/queues.py

@@ -1,8 +1,4 @@
-from concurrent.futures import ThreadPoolExecutor
-import datetime
 import os
-import traceback
-import zoneinfo
 import requests
 import time
 
@@ -19,55 +15,24 @@ class QueuesException(Exception):
 
 
 class TasksHandlerMixin:
-    def __init__(self, *args, **kwargs):
-        super().__init__(*args, **kwargs)
-        self.executor = ThreadPoolExecutor(max_workers=1)
-
-    def _send_metric(self, start: datetime.datetime, end: datetime.datetime, success: bool):
-        def send():
-            requests.post(f'{QUEUES_URL}/api/v1/metric', json={
-                'service': 'ruz-bot',
-                'queue': self.queue_name,
-                'success': success,
-                'timestamp': start.strftime("%Y-%m-%dT%H:%M:%S") + "Z",
-                "success": success,
-                "execution_time_ms": (end - start).microseconds // 1000,
-                "environment": stage,
-            })
-        
-        self.executor.submit(send)
-
     def poll(self):
         while True:
-            try:
+            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}).json()
-            except requests.JSONDecodeError:
-                print('Unable to decode json')
-                time.sleep(3)
-                continue
             task = response.get('task')
             if not task:
                 time.sleep(0.2)
                 continue
-            start = datetime.datetime.now(zoneinfo.ZoneInfo("Europe/Moscow"))
             try:
-                print(f'process task with id {task["id"]}, attempt {task["attempt"]}')
                 self.process(task['payload'])
-                success = True
             except Exception as exc:
                 print(f'Error processing message id={task["id"]}, payload={task["payload"]}, exc={exc}')
-                traceback.print_exc()
+                continue
-                success = False
+            try:
-            end = datetime.datetime.now(zoneinfo.ZoneInfo("Europe/Moscow"))
+                resp = requests.post(f'{QUEUES_URL}/api/v1/finish', json={'id': task['id']})
-            if success:
+                if resp.status_code != 202:
-                try:
+                    raise QueuesException
-                    resp = requests.post(f'{QUEUES_URL}/api/v1/finish', json={'id': task['id']})
+            except:
-                    if resp.status_code != 202:
+                print(f'Failed to finish task id={task["id"]}')
-                        raise QueuesException
-                    print(f'finish task with id {task["id"]}')
-                except:
-                    print(f'Failed to finish task id={task["id"]}')
-            self._send_metric(start, end, success)
 
     @property
     def queue_name(self):
@@ -84,4 +49,4 @@ def set_task(queue_name: str, payload: dict, seconds_to_execute: int, delay: int
         'delay': delay,
     })
     if resp.status_code != 202:
-        raise QueuesException
+        raise QueuesException