From 335de960d9b3f10dfd12b2972b04d18a86d27201 Mon Sep 17 00:00:00 2001 From: Rajesh Rajendran Date: Thu, 9 Mar 2023 13:07:25 +0100 Subject: [PATCH 1/2] Skip sign confirmation, and adding chalice liveness probe (#1026) * chore(helm): chalice updating liveness probe * chore(build): Skip confirmation for signing --------- Signed-off-by: rjshrjndrn --- scripts/helmcharts/build_deploy.sh | 1 + scripts/helmcharts/openreplay/charts/chalice/values.yaml | 8 ++++---- 2 files changed, 5 insertions(+), 4 deletions(-) diff --git a/scripts/helmcharts/build_deploy.sh b/scripts/helmcharts/build_deploy.sh index c922878d4..f37f26c94 100644 --- a/scripts/helmcharts/build_deploy.sh +++ b/scripts/helmcharts/build_deploy.sh @@ -10,6 +10,7 @@ docker rmi alpine || true # Signing image # cosign sign --key awskms:///alias/openreplay-container-sign image_url:tag +export COSIGN_YES=true # Skip confirmation export SIGN_IMAGE=1 export PUSH_IMAGE=1 export AWS_DEFAULT_REGION="eu-central-1" diff --git a/scripts/helmcharts/openreplay/charts/chalice/values.yaml b/scripts/helmcharts/openreplay/charts/chalice/values.yaml index 3269aa503..c639d9cbd 100644 --- a/scripts/helmcharts/openreplay/charts/chalice/values.yaml +++ b/scripts/helmcharts/openreplay/charts/chalice/values.yaml @@ -121,11 +121,11 @@ affinity: {} healthCheck: livenessProbe: httpGet: - path: / + path: /signup port: 8000 - initialDelaySeconds: 100 - periodSeconds: 15 - timeoutSeconds: 10 + initialDelaySeconds: 120 + periodSeconds: 30 + timeoutSeconds: 15 pvc: From c4ae41b54df4dddd0919fd56cccf94d8f1d2ed46 Mon Sep 17 00:00:00 2001 From: Taha Yassine Kraiem Date: Fri, 10 Mar 2023 11:45:33 +0100 Subject: [PATCH 2/2] feat(chalice): enhanced helper functions --- api/chalicelib/utils/helper.py | 2 ++ ee/api/chalicelib/core/unlock.py | 3 ++- ee/api/routers/crons/core_dynamic_crons.py | 2 +- 3 files changed, 5 insertions(+), 2 deletions(-) diff --git a/api/chalicelib/utils/helper.py b/api/chalicelib/utils/helper.py index 369aff40a..62fe6c248 100644 --- a/api/chalicelib/utils/helper.py +++ b/api/chalicelib/utils/helper.py @@ -318,4 +318,6 @@ def get_domain(): def obfuscate(text, keep_last: int = 4): if text is None or not isinstance(text, str): return text + if len(text) <= keep_last: + return "*" * len(text) return "*" * (len(text) - keep_last) + text[-keep_last:] diff --git a/ee/api/chalicelib/core/unlock.py b/ee/api/chalicelib/core/unlock.py index d656edf8a..646cea681 100644 --- a/ee/api/chalicelib/core/unlock.py +++ b/ee/api/chalicelib/core/unlock.py @@ -4,6 +4,7 @@ from os import environ import requests from decouple import config +from chalicelib.utils import helper from chalicelib.utils.TimeUTC import TimeUTC @@ -22,7 +23,7 @@ def check(): environ["expiration"] = "-1" environ["numberOfSeats"] = "0" return - print(f"validating: {license}") + print(f"validating: {helper.obfuscate(license)}") r = requests.post('https://api.openreplay.com/os/license', json={"mid": __get_mid(), "license": get_license()}) if r.status_code != 200 or "errors" in r.json() or not r.json()["data"].get("valid"): print("license validation failed") diff --git a/ee/api/routers/crons/core_dynamic_crons.py b/ee/api/routers/crons/core_dynamic_crons.py index 0ea096546..5d13c90d1 100644 --- a/ee/api/routers/crons/core_dynamic_crons.py +++ b/ee/api/routers/crons/core_dynamic_crons.py @@ -26,7 +26,7 @@ def unlock_cron() -> None: cron_jobs = [ - {"func": unlock_cron, "trigger": "cron", "hour": "*"}, + {"func": unlock_cron, "trigger": CronTrigger(day="*")}, ] SINGLE_CRONS = [{"func": telemetry_cron, "trigger": CronTrigger(day_of_week="*"),