Skip to content
This repository has been archived by the owner on May 12, 2021. It is now read-only.

[RFC] Initial gRPC protocol for agent communication #2

Merged
merged 1 commit into from
Nov 23, 2017

Conversation

gnawux
Copy link
Member

@gnawux gnawux commented Nov 23, 2017

based on hyperstart gRPC protocol and the discussion in hyperhq/runv#628 and
kata-containers/runtime#2

based on hyperstart gRPC protocol and the discussion in hyperhq/runv#628 and
kata-containers/runtime#2

Signed-off-by: Wang Xu <gnawux@gmail.com>
@sameo
Copy link

sameo commented Nov 23, 2017

My only concern is the StringUser part. I believe it's not needed, so I'll open an issue for it.
For now, this LGTM.

@sameo
Copy link

sameo commented Nov 23, 2017

@laijs PTAL and if this looks good to you we'll merge.

@gnawux
Copy link
Member Author

gnawux commented Nov 23, 2017

OK. Good start point.

I add Hyper Inc copyright for the files from runV originally, and add Intel Corporation copyright for those comes from hyperhq/runv#628.

However, I am thinking if we should put some header for kata org itself.

@sameo
Copy link

sameo commented Nov 23, 2017

@gnawux Thanks for putting the right copyright, I appreciate.
I don't think we want to give code copyrights to the kata organization, but I am not a lawyer :)

@gnawux
Copy link
Member Author

gnawux commented Nov 23, 2017

@sameo afaik we may have an additional header, such as

// Copyright 2017 HyperHQ Inc.
// Copyright 2017-2018 Kata...

But I don't know how it works in detail, I am not a lawyer as well.

@laijs
Copy link
Contributor

laijs commented Nov 23, 2017

LGTM

@sameo sameo merged commit dfaceb1 into kata-containers:master Nov 23, 2017
mcastelino pushed a commit to mcastelino/agent-1 that referenced this pull request Dec 6, 2018
virtio-mmio: Add support for the virtio-mmio driver
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants