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

Update lib-go-databox to 0.5.0 #11

Merged
merged 29 commits into from
Aug 17, 2018
Merged

Update lib-go-databox to 0.5.0 #11

merged 29 commits into from
Aug 17, 2018

Conversation

Toshbrown
Copy link
Contributor

This now has a more logical structure and contains clients to wrap APIs for the core-network, core-arbiter and core-stores, which are used in the core components as well as in apps and drivers.

There will likely be more commits to this PR as the arbiter is about to be replaced and I may make changes as I port over apps to the new version to improve the libs usability for app and driver devs.

@codecov
Copy link

codecov bot commented Jun 19, 2018

Codecov Report

Merging #11 into master will increase coverage by 4.74%.
The diff coverage is 42.9%.

Impacted file tree graph

@@            Coverage Diff            @@
##           master     #11      +/-   ##
=========================================
+ Coverage   38.16%   42.9%   +4.74%     
=========================================
  Files           7       8       +1     
  Lines         773     543     -230     
=========================================
- Hits          295     233      -62     
+ Misses        412     276     -136     
+ Partials       66      34      -32
Impacted Files Coverage Δ
databoxRequest.go 0% <0%> (ø)
databoxlog.go 0% <0%> (ø)
helperFunction.go 0% <0%> (ø)
arbiterClient.go 46.42% <46.42%> (ø)
coreStoreClient.go 60.33% <60.33%> (ø)
coreStoreKV.go 68.75% <68.75%> (ø)
coreStoreTSBlob.go 76.47% <76.47%> (ø)
coreStoreTS.go 79.66% <79.66%> (ø)
... and 6 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 1441dbe...e30bcfd. Read the comment docs.

Toshbrown added 18 commits June 20, 2018 10:16
Sets up an arbiter client and store connection using the default values making thing a bit less complex
So apps launched with image complex-image-name-with-arch-amd64v8 can have a consistent name across platforms complex-image-name
Only the cmgr ever talks to the core-network so this should not be here. It also used functions from docker requiring a go get docker/docker which is very slow. This is not needed any more so apps and driver build more quickly  ;-)
@Toshbrown Toshbrown changed the title Update lib-go-databox (DNM) Update lib-go-databox to 0.5.0 Aug 17, 2018
@Toshbrown Toshbrown merged commit 0126c74 into me-box:master Aug 17, 2018
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.

1 participant