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

🐛 Minor fix: adding log_format_local_dev_enabled env-var to the dynamic sidecar #4173

Conversation

matusdrobuliak66
Copy link
Contributor

@matusdrobuliak66 matusdrobuliak66 commented Apr 27, 2023

What do these changes do?

  • MINOR: Adding LOG_FORMAT_LOCAL_DEV_ENABLED to the sidecar

Related issue/s

How to test

DevOps Checklist

@matusdrobuliak66 matusdrobuliak66 self-assigned this Apr 27, 2023
@matusdrobuliak66 matusdrobuliak66 changed the title WIP: adding log_format_local_dev_enebaled ENV var to the dynamic sidecar minor: adding log_format_local_dev_enebaled ENV var to the dynamic sidecar Apr 27, 2023
@codecov
Copy link

codecov bot commented Apr 27, 2023

Codecov Report

Merging #4173 (4237d4f) into master (0b69cfb) will decrease coverage by 15.9%.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff            @@
##           master   #4173      +/-   ##
=========================================
- Coverage    84.7%   68.9%   -15.9%     
=========================================
  Files         958     369     -589     
  Lines       41558   19115   -22443     
  Branches      955       0     -955     
=========================================
- Hits        35236   13185   -22051     
+ Misses       6105    5930     -175     
+ Partials      217       0     -217     
Flag Coverage Δ
integrationtests 60.1% <ø> (-2.3%) ⬇️
unittests 86.3% <ø> (+3.9%) ⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files Coverage Δ
...es/dynamic_sidecar/docker_service_specs/sidecar.py 79.1% <ø> (ø)

... and 754 files with indirect coverage changes

@matusdrobuliak66 matusdrobuliak66 marked this pull request as ready for review May 3, 2023 09:13
@matusdrobuliak66 matusdrobuliak66 added the t:maintenance Some planned maintenance work label May 3, 2023
@matusdrobuliak66 matusdrobuliak66 added this to the The Next Milestone milestone May 3, 2023
@matusdrobuliak66 matusdrobuliak66 requested a review from GitHK May 3, 2023 10:55
@pcrespov pcrespov changed the title minor: adding log_format_local_dev_enebaled ENV var to the dynamic sidecar 🐛 Minor fix: adding log_format_local_dev_enabled env-var to the dynamic sidecar May 3, 2023
@matusdrobuliak66 matusdrobuliak66 enabled auto-merge (squash) May 4, 2023 08:17
@codeclimate
Copy link

codeclimate bot commented May 4, 2023

Code Climate has analyzed commit 4237d4f and detected 0 issues on this pull request.

View more on Code Climate.

@sonarcloud
Copy link

sonarcloud bot commented May 4, 2023

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

No Coverage information No Coverage information
0.0% 0.0% Duplication

@matusdrobuliak66 matusdrobuliak66 merged commit 7a7c601 into ITISFoundation:master May 4, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
t:maintenance Some planned maintenance work
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Maintenance: revisit logging in osparc
4 participants