Skip to content

Commit

Permalink
Monitoring/cron duration (#117)
Browse files Browse the repository at this point in the history
* monitor: cron duration

* fix: monitor cron duration
  • Loading branch information
polomarcus authored Feb 25, 2024
1 parent 90ab6cc commit e9ffc16
Show file tree
Hide file tree
Showing 3 changed files with 5 additions and 12 deletions.
1 change: 1 addition & 0 deletions docker-compose.yml
Original file line number Diff line number Diff line change
Expand Up @@ -134,6 +134,7 @@ services:
POSTGRES_PORT: 5432
PORT: 5050 # healthcheck
HEALTHCHECK_SERVER: "0.0.0.0"
SENTRY_DSN: prod_only
# START_DATE: 1704576615 # to test batch import
# UPDATE: "true" # to batch update PG
# CHANNEL : fr3-idf # to reimport only one channel
Expand Down
4 changes: 2 additions & 2 deletions quotaclimat/data_ingestion/ingest_db/ingest_sitemap_in_db.py
Original file line number Diff line number Diff line change
Expand Up @@ -70,8 +70,8 @@ async def main():
time.sleep(60 * minutes)

res=health_check_task.cancel()
logging.info("Exiting with success")
sys.exit(0)
logging.info("Exiting with success")
sys.exit(0)

if __name__ == "__main__":
# create logger with 'spam_application'
Expand Down
12 changes: 2 additions & 10 deletions quotaclimat/data_processing/mediatree/api_import.py
Original file line number Diff line number Diff line change
Expand Up @@ -262,17 +262,9 @@ async def main():
# Wait for both tasks to complete
await event_finish.wait()

# only for scaleway - delay for serverless container
# Without this we have a CrashLoopBackOff (Kubernetes health error)
if (os.environ.get("ENV") != "dev" and os.environ.get("ENV") != "docker"):
minutes = 15
seconds_to_minute = 60
logging.warning(f"Sleeping {minutes} before safely exiting scaleway container")
sleep(seconds_to_minute * minutes)

res=health_check_task.cancel()
logging.info("Exiting with success")
sys.exit(0)
logging.info("Exiting with success")
sys.exit(0)

if __name__ == "__main__":
# create logger with 'spam_application'
Expand Down

1 comment on commit e9ffc16

@github-actions
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Coverage

Coverage Report
FileStmtsMissCoverMissing
postgres
   insert_data.py46785%38–40, 59–61, 66
   insert_existing_data_example.py20385%25–27
postgres/schemas
   models.py711579%74–81, 91–92, 101–111
quotaclimat/data_analytics
   analytics_signataire_charte.py29290%1–67
   bilan.py1081080%2–372
   data_coverage.py34340%1–94
   exploration.py1251250%1–440
   sitemap_analytics.py1181180%1–343
quotaclimat/data_ingestion
   categorization_program_type.py110%1
   config_youtube.py110%1
   scaleway_db_backups.py34340%1–74
   scrap_chartejournalismeecologie_signataires.py50500%1–169
   scrap_sitemap.py1341787%27–28, 33–34, 66–71, 95–97, 138–140, 202, 223–228
   scrap_tv_program.py62620%1–149
   scrap_youtube.py1141140%1–238
quotaclimat/data_ingestion/ingest_db
   ingest_sitemap_in_db.py584129%30–51, 54–74, 78–89
quotaclimat/data_ingestion/scrap_html
   scrap_description_article.py36392%19–20, 32
quotaclimat/data_processing/mediatree
   api_import.py17810243%45–49, 54–57, 61–64, 70, 73–98, 104–119, 124–126, 151–158, 162–165, 169–175, 186–197, 200–204, 210, 235–236, 242, 244, 248–267, 271–282
   config.py15287%7, 16
   detect_keywords.py103694%115–122
   utils.py642167%27–51, 54, 73–74
quotaclimat/data_processing/sitemap
   sitemap_processing.py412734%15–19, 23–25, 29–47, 51–58, 66–96, 101–103
quotaclimat/utils
   channels.py660%1–95
   climate_keywords.py220%3–35
   healthcheck_config.py291452%22–24, 27–38
   logger.py14379%22–24
   plotly_theme.py17170%1–56
TOTAL155996238% 

Tests Skipped Failures Errors Time
43 0 💤 0 ❌ 0 🔥 10.517s ⏱️

Please sign in to comment.