servicecatalog / oscm-app

OSCM Asynchronous Provisioning Proxy.
Apache License 2.0
1 stars 3 forks source link

Log4j2 update #75

Closed kowalczyka closed 4 years ago

kowalczyka commented 4 years ago

Changes in this Pull Request:

Mandatory checks:

OSCM Build References:


This change is Reviewable

codecov[bot] commented 4 years ago

Codecov Report

Merging #75 into master will decrease coverage by 0.12%. The diff coverage is 92.3%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master      #75      +/-   ##
==========================================
- Coverage   36.32%   36.19%   -0.13%     
==========================================
  Files         254      254              
  Lines       16417    16369      -48     
  Branches     2163     2162       -1     
==========================================
- Hits         5963     5925      -38     
+ Misses      10141    10130      -11     
- Partials      313      314       +1
Impacted Files Coverage Δ
...scm-app/src/main/java/org/oscm/app/dao/BesDAO.java 69.53% <100%> (+0.1%) :arrow_up:
...p/src/main/java/org/oscm/app/domain/Operation.java 88.46% <33.33%> (-0.22%) :arrow_down:
...main/java/org/oscm/app/dao/ServiceInstanceDAO.java 21.53% <50%> (-0.6%) :arrow_down:
...c/main/java/org/oscm/app/business/Initializer.java 94.82% <94.44%> (+0.64%) :arrow_up:
...src/main/java/org/oscm/app/common/Initializer.java 95.08% <94.73%> (+2.58%) :arrow_up:
...n/java/org/oscm/app/azure/proxy/ProxySettings.java 86.95% <0%> (+10.86%) :arrow_up:
GoebelL commented 4 years ago

I guess a log4j2.properties.template file needs to be added for the sample adapter.

kowalczyka commented 4 years ago

I guess a log4j2.properties.template file needs to be added for the sample adapter.

This will be handled in separate issue: #76