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

Revert "Remove sample configs from libcontainer" #107

Closed
wants to merge 1 commit into from

Conversation

mrunalp
Copy link
Contributor

@mrunalp mrunalp commented Jul 9, 2015

This is breaking runc builds as libcontainer tests depend on some of these configs.

@GordonTheTurtle
Copy link

Please sign your commits following these rules:
https://github.com/docker/docker/blob/master/CONTRIBUTING.md#sign-your-work
The easiest way to do this is to amend the last commit:

$ git clone -b "revert-89-hq_add_sample_config" git@github.com:opencontainers/runc.git somewhere
$ cd somewhere
$ git commit --amend -s --no-edit
$ git push -f

Ammending updates the existing PR. You DO NOT need to open a new one.

@hqhq
Copy link
Contributor

hqhq commented Jul 10, 2015

@mrunalp Looks like we have a new branch in repo because of this, can you remove it?

@mrunalp mrunalp deleted the revert-89-hq_add_sample_config branch July 10, 2015 02:17
@mrunalp
Copy link
Contributor Author

mrunalp commented Jul 10, 2015

@hqhq done.

stefanberger pushed a commit to stefanberger/runc that referenced this pull request Sep 8, 2017
glossary: Provide a quick overview of important terms
stefanberger pushed a commit to stefanberger/runc that referenced this pull request Sep 8, 2017
At the end of the list, to match its position in the README.  This
catches opencontainers#107 up with opencontainers#263, which I'd missed during one of the opencontainers#107
rebases.

Signed-off-by: W. Trevor King <wking@tremily.us>
stefanberger pushed a commit to stefanberger/runc that referenced this pull request Sep 8, 2017
So we have something to cite to avoid rehashing established decisions.
Provide some motivation and links to the backing discussion so folks
can re-open these if they have new information that wasn't covered in
the original decision.

Like the glossary (1873498, glossary: Provide a quick overview of
important terms, 2015-08-11, opencontainers#107), I've used subsection titles for
each entry to get link anchors.

Signed-off-by: W. Trevor King <wking@tremily.us>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants