This repository was archived by the owner on Feb 8, 2021. It is now read-only.
Refactor hyperd daemon package #445
Merged
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
|
Still have something to fix, including
and will rebase to current master after the fix |
|
All the major parts have been done. now
|
gnawux
force-pushed
the
refactor_core
branch
2 times, most recently
from
17c766c to
fe91567
Compare
|
rebased to master ( #447 ), however, still lot of things to be fixed |
gnawux
force-pushed
the
refactor_core
branch
from
158d89d to
7dcb2b1
Compare
|
fixed the build, but still need to correct the behaviors. |
gnawux
force-pushed
the
refactor_core
branch
from
7dcb2b1 to
4a4b295
Compare
no panic at least, better than go on |
|
two more |
|
two more again.
|
gnawux
force-pushed
the
refactor_core
branch
2 times, most recently
from
1c7c921 to
0808092
Compare
|
OOPS: 15 passed, 1 skipped, 2 FAILED
|
|
rename container is fixed
|
|
Cool |
There are still lots of issues to fix, however, at least the integration cases are passed. Cheers and go on. |
gnawux
force-pushed
the
refactor_core
branch
from
6f57fa4 to
440b4aa
Compare
... |
Passed one round, still need fix the save/re-load issues |
gnawux
force-pushed
the
refactor_core
branch
from
3c1d19b to
74021cc
Compare
|
WOW! It is the first time this branch pass the test in the past 5 months. There are still lot of known issues to be solved, however, it has never been so close to be completed. |
gnawux
force-pushed
the
refactor_core
branch
from
d6ff981 to
a6b7ff4
Compare
31 tasks
|
Added save/load to/from leveldb |
Signed-off-by: Wang Xu <gnawux@gmail.com>
Signed-off-by: Wang Xu <gnawux@gmail.com>
Signed-off-by: Wang Xu <gnawux@gmail.com>
|
Squashed the commits. Same contents with the previous commits. And the old commits are kept here. |
14 tasks
gnawux
changed the title
|
I listed remain tasks in #462 , and we may merge this and keep working on fix all issues. |
Signed-off-by: Wang Xu <gnawux@gmail.com>
|
Cool. LGTM. |
Sign up for free
to subscribe to this conversation on GitHub.
Already have an account?
Sign in.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
//WIP, could not merge yet.