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

Docs: document how to PR the runtime, with the golang VC build issue #430

Closed
grahamwhaley opened this issue Jun 20, 2018 · 5 comments
Closed

Comments

@grahamwhaley
Copy link
Contributor

Description of problem

Due to some golang path deps inside virtcontainers, you cannot (trivially) build the runtime in your own fork of the repo.
This makes the build/test/PR workflow for the runtime different from the other repos. We should document that in the Runtime developer docs - a BKM for build/test/PR workflow.

I believe the best way is to work in a clone of the main repo, and then push your working branch to your user fork of the main repo (by adding your repo as a remote to your clone of the main branch).

Input on BKM's welcome here @jodh-intel @sboeuf

This was related in the past, for ref: containers/virtcontainers#655

@bergwolf
Copy link
Member

you cannot (trivially) build the runtime in your own fork of the repo.

Do you mean building from a local path other than $GOPATH/src/github.com/kata-containers/runtime? That doesn't sound like a golang development best practice. Most golang projects do not support it afaict.

@jodh-intel
Copy link
Contributor

Talking to @grahamwhaley, it seems the problem is building in:

$GOPATH/src/github.com/$github_user/$repo

Whereas if you build in the following location, there are no problems afaik:

$GOPATH/src/github.com/kata-containers/$repo

@grahamwhaley
Copy link
Contributor Author

Exactly. It is a golang thing - but, all the other repos (proxy, shim etc.) will build in the $github_user path. I think making a note somewhere in the dev docs that this is a golang project and has to build in its 'native' path might help non-golang users when they try their first PR and build.
I think there are some golang links out there, possibly only community links, that talk about this. We'll see if we can find some to place in the docs.

@gabibeyer
Copy link
Contributor

@grahamwhaley @jodh-intel could we move this to the documentation repository (I don't think I have permission to do so). I wasn't able to find the documentation surrounding this, so it should probably stay open.

@grahamwhaley
Copy link
Contributor Author

We can probably close it - it is just that the runtime repo does not fit nicely into my normal github/golang workflow, but that seems to be just me ;-)

zklei pushed a commit to zklei/runtime that referenced this issue Jun 13, 2019
specconv.CreateLibcontainerConfig returns a configuration without rlimits,
because it can't convert posixRlimits to Rlimits.
Convert posixRlimits to Rlimits and set rlimits in libcontainer configuration
before finishing container creation.

fixes kata-containers#430
fixes github.com/kata-containers#913

Signed-off-by: Julio Montes <julio.montes@intel.com>
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

4 participants