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

TEM-62 daemon: implement mvp functionality #6

Merged
merged 41 commits into from Nov 7, 2018

Conversation

@bobheadxi
Copy link
Contributor

bobheadxi commented Nov 5, 2018

Testing Instructions

  1. make testenv
  2. add an entry to the hosted_ip_fs_private_networks with a name, eg test-network
  3. make config, edit config settings as desired (most importantly the database part)
  4. ./ipfs-orchestrator -dev daemon
  5. start up a network: ./ipfs-orchestrator -dev ctl StartNetwork Network=test-network
  6. see stats: ./ipfs-orchestrator -dev ctl NetworkStats Network=test-network
  7. stop the network: ./ipfs-orchestrator -dev ctl StopNetwork Network=test-network
@bobheadxi bobheadxi requested a review from bonedaddy Nov 5, 2018
@codecov

This comment has been minimized.

Copy link

codecov bot commented Nov 5, 2018

Codecov Report

Merging #6 into master will decrease coverage by 2.02%.
The diff coverage is 76.7%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master       #6      +/-   ##
==========================================
- Coverage    86.3%   84.29%   -2.01%     
==========================================
  Files          13       13              
  Lines         467      579     +112     
==========================================
+ Hits          403      488      +85     
- Misses         37       57      +20     
- Partials       27       34       +7
Impacted Files Coverage Δ
ipfs/utils.go 100% <100%> (ø) ⬆️
registry/registry.go 100% <100%> (ø) ⬆️
ipfs/ipfs.go 73.67% <70.59%> (-4.04%) ⬇️
config/config.go 90.91% <83.34%> (+0.91%) ⬆️
orchestrator/orchestrator.go 88.47% <94.88%> (+2.89%) ⬆️

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 35d9291...9f8658d. Read the comment docs.

@bobheadxi bobheadxi force-pushed the daemon/mvp branch from 02f559b to b71492e Nov 5, 2018
@bobheadxi bobheadxi changed the title daemon: implement basic endpoints for network start and stop daemon: implement mvp functionality Nov 6, 2018
@bobheadxi bobheadxi force-pushed the daemon/mvp branch from 59535ce to 1107830 Nov 6, 2018
@bobheadxi bobheadxi force-pushed the daemon/mvp branch from 1107830 to aba1c4f Nov 6, 2018
@bobheadxi bobheadxi added the pr: review label Nov 6, 2018
Copy link
Member

bonedaddy left a comment

@bobheadxi bobheadxi merged commit 20896b9 into master Nov 7, 2018
3 of 4 checks passed
3 of 4 checks passed
codecov/project 84.29% (-2.01%) compared to 35d9291
Details
Travis CI - Branch Build Passed
Details
Travis CI - Pull Request Build Passed
Details
codecov/patch 76.7% of diff hit (target 50%)
Details
@bobheadxi bobheadxi deleted the daemon/mvp branch Nov 7, 2018
@bobheadxi bobheadxi changed the title daemon: implement mvp functionality TEM-62 daemon: implement mvp functionality Nov 11, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
2 participants
You can’t perform that action at this time.