Skip to content

feat: improve logs & services config - #3507

Merged
dnechay merged 7 commits into
developfrom
dnechay/fs-130825
Aug 18, 2025
Merged

feat: improve logs & services config#3507
dnechay merged 7 commits into
developfrom
dnechay/fs-130825

Conversation

@dnechay

@dnechay dnechay commented Aug 13, 2025

Copy link
Copy Markdown
Collaborator

Issue tracking

Freestyle

Context behind the change

  1. added version to all services logs in order to easily filter by version in DD
  2. adjusted migrations commands and config for JL & Fortune to avoid unnecessary builds
  3. removed unused "serve-static" in JL, because it's not necessary for swagger
  4. updated logger lib with some convenience tooling
  5. noisy cron logs from info to debug as we don't need them in DD (at least now)
  6. added ability to override log level in services via LOG_LEVEL env variable

Decided to not apply tsconfig & eslint updates for now because it's going to cause way too many changes.

How has this been tested?

  • run service, check that logs output has proper version binding now
  • run RepO & HUMAN App, check that /health/ping returns version
  • run JL and open swagger, make sure it works
  • run JL and Fortune migrations commands, make sure they work
  • run RepO with LOG_LEVEL=info yarn start, make sure that cron jobs are not logged

Release plan

  • update start command for services on Render to include GIT_HASH=$RENDER_GIT_COMMIT
  • release new logger version

Potential risks; What to monitor; Rollback plan

No

@vercel

vercel Bot commented Aug 13, 2025

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Project Deployment Preview Comments Updated (UTC)
human-app Ready Preview Comment Aug 14, 2025 11:03am
human-dashboard-frontend Ready Preview Comment Aug 14, 2025 11:03am
staking-dashboard Ready Preview Comment Aug 14, 2025 11:03am
2 Skipped Deployments
Project Deployment Preview Comments Updated (UTC)
faucet-frontend ⬜️ Ignored Preview Aug 14, 2025 11:03am
faucet-server ⬜️ Ignored Preview Aug 14, 2025 11:03am

@vercel
vercel Bot temporarily deployed to Preview – human-dashboard-frontend August 13, 2025 19:12 Inactive
@vercel
vercel Bot temporarily deployed to Preview – human-app August 13, 2025 19:12 Inactive
@vercel
vercel Bot temporarily deployed to Preview – staking-dashboard August 13, 2025 19:12 Inactive
@dnechay dnechay self-assigned this Aug 13, 2025
@dnechay
dnechay requested a review from Dzeranov August 13, 2025 19:15
@dnechay dnechay moved this to In Review in Core-tech - 2025 Aug 13, 2025
@dnechay
dnechay marked this pull request as ready for review August 13, 2025 19:15
@vercel
vercel Bot temporarily deployed to Preview – human-app August 14, 2025 10:49 Inactive
@vercel
vercel Bot temporarily deployed to Preview – human-dashboard-frontend August 14, 2025 10:49 Inactive
@vercel
vercel Bot temporarily deployed to Preview – staking-dashboard August 14, 2025 10:49 Inactive

@flopez7 flopez7 left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM 👍

Comment thread packages/apps/job-launcher/server/src/app.module.ts
@dnechay
dnechay merged commit 5aaee46 into develop Aug 18, 2025
18 checks passed
@github-project-automation github-project-automation Bot moved this from In Review to Done in Core-tech - 2025 Aug 18, 2025
@dnechay
dnechay deleted the dnechay/fs-130825 branch August 18, 2025 08:40
@dnechay dnechay mentioned this pull request Aug 18, 2025
10 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

2 participants