Merge pull request 'master' (#49) from master into prod

Reviewed-on: #49

.deploy/deploy-dev.yaml

@@ -6,12 +6,13 @@ services:
     image: mathwave/sprint-repo:pizda-bot
     command: worker
     environment:
-      MONGO_HOST: "mongo.develop.sprinthub.ru"
+      MONGO_HOST: "mongo"
       MONGO_PASSWORD: $MONGO_PASSWORD_DEV
       STAGE: "development"
     networks:
       - queues-development
       - configurator
+      - mongo-development
     deploy:
       mode: replicated
       restart_policy:
@@ -24,11 +25,12 @@ services:
     image: mathwave/sprint-repo:pizda-bot
     command: api
     environment:
-      MONGO_HOST: "mongo.develop.sprinthub.ru"
+      MONGO_HOST: "mongo"
       MONGO_PASSWORD: $MONGO_PASSWORD_DEV
       STAGE: "development"
     networks:
       - common-infra-nginx-development
+      - mongo-development
     deploy:
       mode: replicated
       restart_policy:
@@ -44,3 +46,5 @@ networks:
     external: true
   configurator:
     external: true
+  mongo-development:
+    external: true

.deploy/deploy-prod.yaml

@@ -6,12 +6,13 @@ services:
     image: mathwave/sprint-repo:pizda-bot
     command: worker
     environment:
-      MONGO_HOST: "mongo.sprinthub.ru"
+      MONGO_HOST: "mongo"
       MONGO_PASSWORD: $MONGO_PASSWORD_PROD
       STAGE: "production"
     networks:
       - queues
       - configurator
+      - mongo
     deploy:
       mode: replicated
       restart_policy:
@@ -24,10 +25,11 @@ services:
     image: mathwave/sprint-repo:pizda-bot
     command: api
     environment:
-      MONGO_HOST: "mongo.sprinthub.ru"
+      MONGO_HOST: "mongo"
       MONGO_PASSWORD: $MONGO_PASSWORD_PROD
     networks:
       - common-infra-nginx
+      - mongo
     deploy:
       mode: replicated
       restart_policy:
@@ -43,3 +45,5 @@ networks:
     external: true
   configurator:
     external: true
+  mongo:
+    external: true

.gitignore

@@ -117,3 +117,5 @@ GitHub.sublime-settings
 !.vscode/launch.json
 !.vscode/extensions.json
 .history
+
+*pb2*

.gitlab-ci.yml

@@ -1,51 +0,0 @@
-stages:
-  - build
-  - deploy-dev
-  - deploy-prod
-
-build:
-  stage: build
-  tags:
-    - dev
-  before_script:
-    - docker login -u mathwave -p $DOCKERHUB_PASSWORD
-  script:
-    - docker build -t mathwave/sprint-repo:pizda-bot .
-
-push:
-  stage: build
-  tags:
-    - dev
-  before_script:
-    - docker login -u mathwave -p $DOCKERHUB_PASSWORD
-  script:
-    - docker push mathwave/sprint-repo:pizda-bot
-
-.deploy:
-  before_script:
-    - docker login -u mathwave -p $DOCKERHUB_PASSWORD
-
-deploy-dev:
-  extends:
-    - .deploy
-  stage: deploy-dev
-  tags:
-    - dev
-  rules:
-    - if: '$CI_COMMIT_BRANCH == "master"'
-      when: on_success
-    - when: manual
-  script:
-    - docker stack deploy --with-registry-auth -c ./.deploy/deploy-dev.yaml pizda-bot
-
-deploy-prod:
-  extends:
-    - .deploy
-  stage: deploy-prod
-  tags:
-    - prod
-  only:
-    - master
-  when: manual
-  script:
-    - docker stack deploy --with-registry-auth -c ./.deploy/deploy-prod.yaml pizda-bot

daemons/api.py

@@ -2,7 +2,7 @@ from collections import defaultdict
 
 from flask import Flask
 
-from mongo import mongo
+from utils.mongo import mongo
 
 app = Flask(__name__)
 

daemons/base.py

@@ -1,3 +1,13 @@
-class BaseDaemon:
+import os
+
+
+stage = os.getenv("STAGE", 'local')
+if stage == 'local':
+    QUEUES_URL = 'localhost:50051'
+else:
+    QUEUES_URL = 'queues-grpc:50051'
+
+
+class Daemon:
     def execute(self):
-        raise NotImplementedError
+        raise NotImplemented

daemons/updater.py

@@ -2,7 +2,7 @@ import datetime
 import random
 from time import sleep
 
-from mongo import mongo
+from utils.mongo import mongo
 
 
 def update():

daemons/worker.py

@@ -6,9 +6,9 @@ from utils import queues
 from telebot.types import Message
 import json
 
-from mongo import mongo
+from utils.mongo import mongo
-from sprint_platform import PlatformClient
+from utils.sprint_platform import PlatformClient
-from storage import set_values, get_chat_info
+from utils.storage import set_values, get_chat_info
 
 security_token = os.getenv("PLATFORM_SECURITY_TOKEN")
 stage = os.getenv("STAGE", 'local')
@@ -35,11 +35,15 @@ def get_answers():
     return client.get_config('answers')
 
 
+def get_ignored_users():
+    return client.get_config('ignored_users')['users']
+
+
 def get_replies():
     return client.get_config('replies')
 
 
-class Daemon(base.BaseDaemon, queues.TasksHandlerMixin):
+class Daemon(base.Daemon, queues.TasksHandlerMixin):
     @property
     def queue_name(self):
         return 'pizda_bot_worker'
@@ -118,9 +122,9 @@ class Daemon(base.BaseDaemon, queues.TasksHandlerMixin):
             set_values(message.chat.id, state="set_probability")
 
     def process(self, payload):
+        message: Message = Message.de_json(json.dumps(payload))
         if not message.text:
             return
-        message: Message = Message.de_json(json.dumps(payload))
         if message.text.startswith('/'):
             self.process_command(message)
             return
@@ -151,6 +155,6 @@ class Daemon(base.BaseDaemon, queues.TasksHandlerMixin):
         else:
             return
         ans = get_answers().get(convert_text)
-        if ans is not None and randrange(1, 101) <= info["probability"]:
+        if ans is not None and randrange(1, 101) <= info["probability"] and message.from_user.id not in get_ignored_users():
             self.reply(ans, message.chat.id, message.message_id)
             mongo.inc(message.from_user.username, message.chat.id)

local_platform.json

@@ -1,5 +0,0 @@
-{
-    "configs": {},
-    "experiments": {},
-    "platform_staff": {}
-}

main.py

@@ -6,7 +6,7 @@ if arg == 'worker':
     from daemons import worker
     daemon = worker.Daemon()
 else:
-    from api import app
+    from daemons.api import app
     app.run(host="0.0.0.0", port=1238)
 
 daemon.execute()

settings.py

@@ -1,10 +0,0 @@
-import os
-import sys
-import zoneinfo
-
-MONGO_USER = os.getenv("MONGO_USER", "mongo")
-MONGO_PASSWORD = os.getenv("MONGO_PASSWORD", "password")
-MONGO_HOST = os.getenv("MONGO_HOST", "localhost")
-
-CACHE_SIZE = int(os.getenv("CACHE_SIZE", 1000))
-CACHE_TTL = int(os.getenv("CACHE_TTL", 3600))

utils/mongo.py

@@ -1,10 +1,14 @@
 import pymongo
-import settings
+import os
+
+MONGO_USER = os.getenv("MONGO_USER", "mongo")
+MONGO_PASSWORD = os.getenv("MONGO_PASSWORD", "password")
+MONGO_HOST = os.getenv("MONGO_HOST", "localhost")
 
 
 class Mongo:
     def __init__(self):
-        url = f"mongodb://{settings.MONGO_USER}:{settings.MONGO_PASSWORD}@{settings.MONGO_HOST}:27017/"
+        url = f"mongodb://{MONGO_USER}:{MONGO_PASSWORD}@{MONGO_HOST}:27017/"
         self.client = pymongo.MongoClient(url)
         self.database = self.client.get_database("pizda-bot")
         self.chats_collection.create_index([

utils/queues.py

@@ -1,5 +1,7 @@
-import json
+from concurrent.futures import ThreadPoolExecutor
+import datetime
 import os
+import zoneinfo
 import requests
 import time
 
@@ -16,24 +18,54 @@ 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': 'botalka',
+                '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:
-            response = requests.get(f'{QUEUES_URL}/api/v1/take', headers={'queue': self.queue_name}).json()
+            try:
+                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}')
-                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):

utils/storage.py

@@ -1,9 +1,12 @@
 from cachetools import TTLCache
+import os
 
-import settings
+from utils.mongo import mongo
-from mongo import mongo
 
-cache = TTLCache(settings.CACHE_SIZE, settings.CACHE_TTL)
+CACHE_SIZE = int(os.getenv("CACHE_SIZE", 1000))
+CACHE_TTL = int(os.getenv("CACHE_TTL", 3600))
+
+cache = TTLCache(CACHE_SIZE, CACHE_TTL)
 
 
 def get_chat_info(chat_id: int) -> dict: