Skip to content

Switch to Open Telemetry for logging#3762

Merged
marrobi merged 20 commits into
microsoft:mainfrom
marrobi:marrobi/issue3761
Dec 7, 2023
Merged

Switch to Open Telemetry for logging#3762
marrobi merged 20 commits into
microsoft:mainfrom
marrobi:marrobi/issue3761

Conversation

@marrobi

@marrobi marrobi commented Oct 24, 2023

Copy link
Copy Markdown
Member

Resolves #3761

Switch to Open Telemetry logging and add variable to set logging level. Also updated the docs.

This makes troubleshooting issues much easier.

Custom attributes for operation_id:
image

Application map:
image

Fast API requests:
image

@github-actions

github-actions Bot commented Oct 24, 2023

Copy link
Copy Markdown

Unit Test Results

582 tests   582 ✔️  6s ⏱️
    1 suites      0 💤
    1 files        0

Results for commit bc0e0ad.

♻️ This comment has been updated with latest results.

Comment thread resource_processor/vmss_porter/runner.py Fixed
Comment thread resource_processor/vmss_porter/runner.py Fixed
Comment thread api_app/services/logging.py Fixed
Comment thread api_app/services/logging.py Fixed
Comment thread api_app/services/aad_authentication.py Fixed
Comment thread api_app/services/aad_authentication.py Fixed
Comment thread api_app/services/aad_authentication.py Fixed
Comment thread resource_processor/vmss_porter/runner.py Fixed
@marrobi marrobi marked this pull request as ready for review November 30, 2023 11:31
@marrobi marrobi requested a review from martinpeck November 30, 2023 11:33
@marrobi

marrobi commented Nov 30, 2023

Copy link
Copy Markdown
Member Author

/test-extended

@marrobi marrobi requested a review from tamirkamara November 30, 2023 12:17
@github-actions

Copy link
Copy Markdown

🤖 pr-bot 🤖

🏃 Running extended tests: https://github.com/microsoft/AzureTRE/actions/runs/7046329693 (with refid 838d26e6)

(in response to this comment from @marrobi)

Comment thread api_app/requirements.txt
Comment thread api_app/service_bus/airlock_request_status_update.py
@marrobi

marrobi commented Dec 7, 2023

Copy link
Copy Markdown
Member Author

/test-extended

@github-actions

github-actions Bot commented Dec 7, 2023

Copy link
Copy Markdown

🤖 pr-bot 🤖

🏃 Running extended tests: https://github.com/microsoft/AzureTRE/actions/runs/7129210899 (with refid 838d26e6)

(in response to this comment from @marrobi)

@marrobi

marrobi commented Dec 7, 2023

Copy link
Copy Markdown
Member Author

/test-extended

@github-actions

github-actions Bot commented Dec 7, 2023

Copy link
Copy Markdown

🤖 pr-bot 🤖

🏃 Running extended tests: https://github.com/microsoft/AzureTRE/actions/runs/7129368946 (with refid 838d26e6)

(in response to this comment from @marrobi)

@marrobi marrobi merged commit c382f3d into microsoft:main Dec 7, 2023
@marrobi marrobi deleted the marrobi/issue3761 branch December 7, 2023 16:33
@marrobi

marrobi commented Dec 12, 2023

Copy link
Copy Markdown
Member Author

/test-destroy-env

@github-actions

Copy link
Copy Markdown

Destroying PR test environment (RG: rg-tre838d26e6)... (run: https://github.com/microsoft/AzureTRE/actions/runs/7180419361)

@github-actions

Copy link
Copy Markdown

PR test environment destroy complete (RG: rg-tre838d26e6)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Investigate to Open Telemetry for logging

3 participants