-
Notifications
You must be signed in to change notification settings - Fork 115
Add metadata store and basic unikernel service #219
Add metadata store and basic unikernel service #219
Conversation
c9d28d2
to
c21de0c
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM in general, could you add a following PR we could review the whole workflow?
- hack/install-protoc.sh | ||
- hack/install-hypercontainer.sh | ||
- go install -v $(go list ./... | grep -v /vendor/) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
why this change?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@feiskyer I've introduce libvirt-go dependency in this patch, when go install libvirt-go
, it will try to build it, and it need libvirt-dev lib to do that. We installed this lib in hack/install-hypercontainer.sh
, so I changed this order to install libvirt-dev first.
BTW, new patch is on the way, thanks for reviewing.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
OK, that makes sense. thanks
Also this PR needs a rebase now. |
Signed-off-by: Crazykev <crazykev@zju.edu.cn>
Signed-off-by: Crazykev <crazykev@zju.edu.cn>
Signed-off-by: Crazykev <crazykev@zju.edu.cn>
Signed-off-by: Crazykev <crazykev@zju.edu.cn>
Signed-off-by: Crazykev <crazykev@zju.edu.cn>
Signed-off-by: Crazykev <crazykev@zju.edu.cn>
c21de0c
to
1d4cd06
Compare
/lgtm |
Will bring container and VM workflow when test is enough
@feiskyer @resouer PTAL