summaryrefslogtreecommitdiff
Commit message (Collapse)AuthorAge
* Merge pull request #80 from umohnani8/kpod_imagesDaniel J Walsh2017-11-27
|\ | | | | Fix output of kpod images
| * Fix output of kpod imagesumohnani82017-11-27
| | | | | | | | | | | | Update the output of kpod images to match that of docker images Signed-off-by: umohnani8 <umohnani@redhat.com>
* | Merge pull request #79 from mheon/fix_createDaniel J Walsh2017-11-27
|\ \ | |/ |/| Make 'kpod create' also create the container in runc
| * Temporarily disable kpod create tests until kpod rm is readyMatthew Heon2017-11-27
| | | | | | | | Signed-off-by: Matthew Heon <matthew.heon@gmail.com>
| * Fix kpod pause tests - pausing a created container is validMatthew Heon2017-11-27
| | | | | | | | Signed-off-by: Matthew Heon <matthew.heon@gmail.com>
| * Make 'kpod create' also create the container in runcMatthew Heon2017-11-27
| | | | | | | | Signed-off-by: Matthew Heon <matthew.heon@gmail.com>
* | kpod_waitbaude2017-11-27
| | | | | | | | | | | | | | | | | | Convert to libpod container backend Signed-off-by: baude <bbaude@redhat.com> Closes: #70 Approved by: rhatdan
* | Narrow gofmt targetsbaude2017-11-27
| | | | | | | | | | | | | | | | | | Disregard _output for gofmt'ing Signed-off-by: baude <bbaude@redhat.com> Closes: #77 Approved by: rhatdan
* | kpod_rm: Add option for --allbaude2017-11-27
|/ | | | | | | | | | | | Remove all containers with -a, --all. Enable kpod rm tests which were all set to skip. Add two tests for -a Signed-off-by: baude <bbaude@redhat.com> Closes: #74 Approved by: rhatdan
* Create|Run container from Image IDbaude2017-11-27
| | | | | | | | | | | As reported in Issue #50, we need to be able to run or create a container based on an image ID (as well as name). Signed-off-by: baude <bbaude@redhat.com> Closes: #76 Approved by: rhatdan
* kpod run, needs to shutdown the storage driver when doneDaniel J Walsh2017-11-27
| | | | | | | Signed-off-by: Daniel J Walsh <dwalsh@redhat.com> Closes: #73 Approved by: mheon
* Update kpod pause and unpause to use new container stateumohnani82017-11-24
| | | | | | | Signed-off-by: umohnani8 <umohnani@redhat.com> Closes: #66 Approved by: mheon
* Copy some verification code out of Docker to verify user inputDaniel J Walsh2017-11-22
| | | | | | | | | | Added lots of verification code to make sure resourses asociated with containers is correct. Signed-off-by: Daniel J Walsh <dwalsh@redhat.com> Closes: #60 Approved by: umohnani8
* Update vendoringDaniel J Walsh2017-11-22
| | | | | | | | | | | | Update version of docker to pull in lates code Remove kubernetes since libpod is not tied to it. Remove a few other packages that we don't seem to use. Left in the networking stuff, since we will hopefully be wiring that together. Signed-off-by: Daniel J Walsh <dwalsh@redhat.com> Closes: #60 Approved by: umohnani8
* Update kpod logs to use the new container state and runtimeUrvashi Mohnani2017-11-22
| | | | | | | Signed-off-by: Urvashi Mohnani <umohnani@redhat.com> Closes: #62 Approved by: rhatdan
* Update kpod mount and umount to use the new stateumohnani82017-11-22
| | | | | | | | | | The new state for containers has been added moved kpod mount and umount over to use it Signed-off-by: Urvashi Mohnani <umohnani@redhat.com> Closes: #57 Approved by: rhatdan
* Order containers returned from state and make container config publicMatthew Heon2017-11-22
| | | | | | | Signed-off-by: Matthew Heon <matthew.heon@gmail.com> Closes: #63 Approved by: baude
* Add support for pid nsDaniel J Walsh2017-11-22
| | | | | | | Signed-off-by: Daniel J Walsh <dwalsh@redhat.com> Closes: #54 Approved by: umohnani8
* Update kpod export to use the new container state and runtimeUrvashi Mohnani2017-11-22
| | | | | | | Signed-off-by: Urvashi Mohnani <umohnani@redhat.com> Closes: #59 Approved by: rhatdan
* Need to block access to kernel file systems in /proc and /sysDaniel J Walsh2017-11-22
| | | | | | | | | | Users of kpod run could use these file systems to perform a breakout or to learn valuable system information. Signed-off-by: Daniel J Walsh <dwalsh@redhat.com> Closes: #61 Approved by: mheon
* Update kpod rm to use new container stateUrvashi Mohnani2017-11-21
| | | | | | | | | kpod rm now uses the new container state and runtime Signed-off-by: Urvashi Mohnani <umohnani@redhat.com> Closes: #58 Approved by: mheon
* Add PID to the containers used to test SQLite stateMatthew Heon2017-11-21
| | | | | | | Signed-off-by: Matthew Heon <matthew.heon@gmail.com> Closes: #56 Approved by: rhatdan
* Add ability to retrieve container's log path to APIMatthew Heon2017-11-21
| | | | | | | Signed-off-by: Matthew Heon <matthew.heon@gmail.com> Closes: #56 Approved by: rhatdan
* Add ability to update container status from runcMatthew Heon2017-11-21
| | | | | | | | | | | | Wire this in to all state-bound container operations to ensure syncronization of container state. Also exposes PID of running containers via API. Signed-off-by: Matthew Heon <matthew.heon@gmail.com> Closes: #56 Approved by: rhatdan
* Mention docker login for creds authentication in documentationUrvashi Mohnani2017-11-21
| | | | | | | | | | Since we fall back to checking $HOME/.docker/config.json, which is set by docker login, if the creds are not found in $XDG_RUNTIME_DIR/containers/auth.json, which is set by kpod login Signed-off-by: Urvashi Mohnani <umohnani@redhat.com> Closes: #55 Approved by: rhatdan
* Add support for oom functionsDaniel J Walsh2017-11-21
| | | | | | | | | Add tests for oom-kill-disable and oom-kill-adj Signed-off-by: Daniel J Walsh <dwalsh@redhat.com> Closes: #52 Approved by: TomSweeneyRedHat
* Add cgroup fs by defaultDaniel J Walsh2017-11-20
| | | | | | | | | Docker defaults to mounting the cgroup file system. Signed-off-by: Daniel J Walsh <dwalsh@redhat.com> Closes: #53 Approved by: mheon
* Merge pull request #51 from mheon/sql_state_testsDaniel J Walsh2017-11-20
|\ | | | | Add tests for SQL-backed state impl
| * Add tests for SQL-backed state implMatthew Heon2017-11-20
|/ | | | | | | Minor changes to container.go and sql_state.go to fix issues identified by the tests Signed-off-by: Matthew Heon <matthew.heon@gmail.com>
* Add support for Ulimits/Rlimits to kpod create/runDaniel J Walsh2017-11-20
| | | | | | | Signed-off-by: Daniel J Walsh <dwalsh@redhat.com> Closes: #24 Approved by: mheon
* Merge pull request #26 from mheon/sql_stateDaniel J Walsh2017-11-20
|\ | | | | Implementation of SQL-backed state
| * Fix lint errorsMatthew Heon2017-11-18
| | | | | | | | Signed-off-by: Matthew Heon <matthew.heon@gmail.com>
| * Need to provide ID of container being updatedMatthew Heon2017-11-18
| | | | | | | | Signed-off-by: Matthew Heon <matthew.heon@gmail.com>
| * StopSignal is allowed to be 0Matthew Heon2017-11-18
| | | | | | | | | | | | If StopSignal is 0, it is assumed that the default signal will be used. Signed-off-by: Matthew Heon <matthew.heon@gmail.com>
| * Resolve another segfaultMatthew Heon2017-11-18
| | | | | | | | | | | | This one cleans up after container creation fails Signed-off-by: Matthew Heon <matthew.heon@gmail.com>
| * Fix lingering SQL errorMatthew Heon2017-11-18
| | | | | | | | Signed-off-by: Matthew Heon <matthew.heon@gmail.com>
| * Fix segfault cleaning up in after failing to create a RuntimeMatthew Heon2017-11-18
| | | | | | | | Signed-off-by: Matthew Heon <matthew.heon@gmail.com>
| * Rename Create to Init for containers to be less confusingMatthew Heon2017-11-18
| | | | | | | | Signed-off-by: Matthew Heon <matthew.heon@gmail.com>
| * Wire SQL backed state into rest of libpodMatthew Heon2017-11-18
| | | | | | | | Signed-off-by: Matthew Heon <matthew.heon@gmail.com>
| * Address review comments, fix gofmt and lintMatthew Heon2017-11-18
| | | | | | | | Signed-off-by: Matthew Heon <matthew.heon@gmail.com>
| * Compile-tested implementation of SQL-backed stateMatthew Heon2017-11-18
| | | | | | | | Signed-off-by: Matthew Heon <matthew.heon@gmail.com>
* | Use cached images for integration testsbaude2017-11-20
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Add alpine and busybox as the two seeded images that can be put into the tests' storage without the need to pull it over the network. Add the following to the kpod_TEST.bats file: function setup() { copy_images } Also, export several image names for shorter reference: * BB -> docker.io/library/busybox:latest * BB_GLIBC -> docker.io/library/busybox:glibc * ALPINE -> docker.io/library/alpine:latest * FEDORA_MINIMAL -> registry.fedoraproject.org/fedora-minimal:latest And finally, there were two pervasive changes I made in tests: * Because we use temporary storage, we do not need to rmi at the end of each test * We no longer need to pull most images because they are added via copy_images Signed-off-by: baude <bbaude@redhat.com> Closes: #43 Approved by: mheon
* | Fix up handling of environment variablesDaniel J Walsh2017-11-20
|/ | | | | | | | | | | | | | The way docker works is if a user specifies a non `-e Name=Value`, IE just a `-e Name`, then the environment variable Name from the clients OS.ENV is used. Also by default Docker containers run with the HOSTNAME environment set to the HOSTNAME specified for the container. Signed-off-by: Daniel J Walsh <dwalsh@redhat.com> Closes: #21 Approved by: baude
* Turn off IRC reporting until we have our own channelDaniel J Walsh2017-11-17
| | | | | | | Signed-off-by: Daniel J Walsh <dwalsh@redhat.com> Closes: #49 Approved by: rhatdan
* Enhancements to papr testsbaude2017-11-17
| | | | | | | | | | | | | | | The PR contains several enhancements to our CI testing. - enable lint testing on Fedora - add Centos Atomic as test platform - integration tests on run on the OS natively (uncontainerized) - builds are done in containers - inclusion of Vagrant file for local testing Signed-off-by: baude <bbaude@redhat.com> Closes: #18 Approved by: mheon
* Merge pull request #47 from baude/terminal_attachDaniel J Walsh2017-11-15
|\ | | | | Fix terminal attach
| * Fix terminal attachbaude2017-11-15
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Re-order the startup of a new container via run from initialize > start > attach to initialize > attach > start. This fixes output when running: kpod run -i -t IMAGE command and kpod run IMAGE command Signed-off-by: baude <bbaude@redhat.com>
* | Fix bash commandDaniel J Walsh2017-11-15
|/ | | | | | | Signed-off-by: Daniel J Walsh <dwalsh@redhat.com> Closes: #45 Approved by: umohnani8
* Address run/create performance issuesbaude2017-11-14
| | | | | | | | | | | | Fixed the logic where we observed different performance results when running an image by its fqname vs a shortname. In the case of the latter, we resolve the name without using the network. Signed-off-by: baude <bbaude@redhat.com> Closes: #37 Approved by: rhatdan
* Remove all imagesbaude2017-11-14
| | | | | | | | | | Add -a/--all to rmi so a user can remove all images quickly. Signed-off-by: baude <bbaude@redhat.com> Closes: #41 Approved by: mheon