Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Logging Improvements [UAT-81] #81

Merged
merged 24 commits into from
Feb 13, 2025
Merged

Logging Improvements [UAT-81] #81

merged 24 commits into from
Feb 13, 2025

Conversation

fqjony
Copy link
Contributor

@fqjony fqjony commented Feb 12, 2025

Changes

  • Improve logging with icons and better styling
ℹ️ Welcome to UDX Worker Container. Initializing environment...
ℹ️ Starting environment configuration...
✅ Worker configuration: User configuration detected at /home/udx/worker.yaml
✅ Worker configuration: Found variables in the configuration. Exporting...
ℹ️ Authenticating actors from configuration...
✅ Authentication: Detected credentials for azure
ℹ️ Reading base64 encoded JSON credentials
ℹ️ Processing credentials for azure
ℹ️ Authenticating Azure service principal...
✅ Azure Authentication: Azure service principal authenticated and subscription set.
ℹ️ Fetching secrets from configuration...
ℹ️ Fetching secrets and setting them as environment variables.
ℹ️ Loaded module for provider: azure
ℹ️ Azure: Retrieving secret from Azure Key Vault: vault_name=kv-udx-worker-tooling, secret_name=test-secret
✅ Secrets: Resolved secret for TEST_SECRET from azure.
ℹ️ Secrets environment variables sourced successfully.
ℹ️ Cleaned up temporary file: /tmp/secret_vars.QuHdQe
ℹ️ Cleaning up sensitive data...
ℹ️ Starting cleanup of actors
ℹ️ Cleaning up Azure authentication
✅ Cleanup: Azure authentication cleaned up successfully.
ℹ️ Secure environment setup completed successfully.
ℹ️ Starting process manager...
ℹ️ Executing command: /home/udx/main.sh
  • Upgrade to [email protected]
  • Upgrade to [email protected]
  • Upgrade to [email protected]+git20250209.89ed161+ds-1ubuntu1
  • Improved shell scripts permissions
  • Simplified entrypoint and improved process manager
  • Updated supervisor configs
  • Added --nostream and --lines params to service cli logs and errors
worker service logs firebase --lines=30
worker service errors firebase --lines 30
worker service errors firebase --nostream
  • Improved Makefile with colors
  • Added services management test
ℹ️ Running 40_test_service.sh...
ℹ️ Test: Starting service tests...
ℹ️ Test: Testing service configuration...
✅ Test: Service configuration test passed
ℹ️ Test: Testing service list and waiting for service to be ready...
✅ Test: Service is now running
✅ Test: Service list test passed
ℹ️ Test: Testing service status...
✅ Test: Service status test passed
ℹ️ Test: Testing service stop...
test_service: stopped
✅ Test: Service stop test passed
ℹ️ Test: Testing service start...
test_service: started
✅ Test: Service start test passed
ℹ️ Test: Testing service logs...
✅ Test: Service logs test passed
✅ Test: All service tests completed successfully
✅ 40_test_service.sh: Test completed successfully

@fqjony fqjony self-assigned this Feb 12, 2025
@fqjony fqjony changed the title Graceful Shutdown Handler [UAT-81] Logging Improvements [UAT-81] Feb 13, 2025
@fqjony fqjony requested a review from a team February 13, 2025 19:19
@fqjony fqjony added documentation Improvements or additions to documentation enhancement New feature or request dependencies Pull requests that update a dependency file labels Feb 13, 2025
@fqjony fqjony marked this pull request as ready for review February 13, 2025 19:19
@fqjony fqjony enabled auto-merge February 13, 2025 19:19
@fqjony fqjony merged commit 43824cb into latest Feb 13, 2025
4 checks passed
@fqjony fqjony deleted the UAT-81 branch February 13, 2025 19:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
dependencies Pull requests that update a dependency file documentation Improvements or additions to documentation enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants