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

Fix H2 URI in Dockerfile #2830

Merged
merged 1 commit into from
Jan 21, 2020
Merged

Conversation

lorthirk
Copy link

Since January 15, 2020, The Maven Central Repository access requires https to download artifacts. This PR fixes the URI for H2 Jar that would fail otherwise.

Related Issue
No related issues

Signed-off-by: Claudio Mezzasalma <claudio.mezzasalma@eurotech.com>
@lorthirk lorthirk added the build label Jan 20, 2020
@lorthirk lorthirk requested a review from Coduz January 20, 2020 16:39
@LeoNerdoG
Copy link
Contributor

Hi @lorthirk,
I can confirm that build passes successfully on my machine.
There is only one failing tests so maybe you should re-run it and merge after it completes?

@codecov
Copy link

codecov bot commented Jan 21, 2020

Codecov Report

Merging #2830 into develop will increase coverage by 0.21%.
The diff coverage is n/a.

Impacted file tree graph

@@              Coverage Diff              @@
##             develop    #2830      +/-   ##
=============================================
+ Coverage      55.08%   55.29%   +0.21%     
- Complexity      2337     2351      +14     
=============================================
  Files           1008     1008              
  Lines          21769    21769              
  Branches        1972     1972              
=============================================
+ Hits           11991    12037      +46     
+ Misses          8858     8818      -40     
+ Partials         920      914       -6
Impacted Files Coverage Δ Complexity Δ
...e/kapua/broker/core/BrokerJAXBContextProvider.java 70% <0%> (-10%) 0% <0%> (ø)
...e/registry/internal/DeviceRegistryServiceImpl.java 50% <0%> (-3.23%) 17% <0%> (-1%)
...egistry/event/internal/DeviceEventServiceImpl.java 76.78% <0%> (ø) 13% <0%> (ø) ⬇️
.../broker/core/plugin/KapuaSecurityBrokerFilter.java 75.23% <0%> (+0.92%) 0% <0%> (ø) ⬇️
...e/jbatch/driver/utils/JobDefinitionBuildUtils.java 90.24% <0%> (+1.21%) 9% <0%> (+1%) ⬆️
...obDeviceManagementOperationManagerServiceImpl.java 77.5% <0%> (+1.25%) 13% <0%> (+1%) ⬆️
...anager/DeviceManagementRegistryManagerService.java 73.07% <0%> (+1.28%) 0% <0%> (ø) ⬇️
...a/service/device/call/kura/KuraDeviceCallImpl.java 82.69% <0%> (+3.84%) 10% <0%> (ø) ⬇️
...a/org/eclipse/kapua/transport/mqtt/MqttFacade.java 76% <0%> (+4%) 10% <0%> (+1%) ⬆️
...a/job/engine/jbatch/listener/KapuaJobListener.java 90.27% <0%> (+4.16%) 28% <0%> (+1%) ⬆️
... and 8 more

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 9a23884...cffbe69. Read the comment docs.

@Coduz Coduz merged commit 26222b6 into eclipse-kapua:develop Jan 21, 2020
@lorthirk lorthirk deleted the fix-h2UriDocker branch January 24, 2020 09:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants