aboutsummaryrefslogtreecommitdiff
Commit message (Collapse)AuthorAge
* 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
* Add cpu-profiling to kpodbaude2017-11-09
| | | | | | | | | | | | | | Add a global flag for cpu-profiling to allow us to profile kpod for performance issues. To parse its results, use: go tool pprof --text <profile_path> Signed-off-by: baude <bbaude@redhat.com> Closes: #36 Approved by: mheon
* README.md: Typo fixes and small tweaksColin Walters2017-11-09
| | | | | | | | | | Should be mostly self-evident. I dropped the `is meant to` as it feels too passive. Signed-off-by: Colin Walters <walters@verbum.org> Closes: #35 Approved by: mheon
* Update tarsplit vendor to address CVE-2017-14992Matthew Heon2017-11-08
| | | | | | | Signed-off-by: Matthew Heon <matthew.heon@gmail.com> Closes: #34 Approved by: rhatdan
* Do not attempt to use command --args as cli flagsbaude2017-11-08
| | | | | | | | | i.e. kpod run fedora python --version Signed-off-by: baude <bbaude@redhat.com> Closes: #33 Approved by: rhatdan
* Fix panic when no image is givenDaniel J Walsh2017-11-08
| | | | | | | Signed-off-by: Daniel J Walsh <dwalsh@redhat.com> Closes: #31 Approved by: mheon
* Cleanup missing whitespaceDaniel J Walsh2017-11-07
| | | | | | | Signed-off-by: Daniel J Walsh <dwalsh@redhat.com> Closes: #20 Approved by: rhatdan
* Add 'kpod import' commandUrvashi Mohnani2017-11-07
| | | | | | | | | Imports a tarball and saves it as a filesystem image Signed-off-by: Urvashi Mohnani <umohnani@redhat.com> Closes: #12 Approved by: rhatdan
* Vendor in latest containers/imageUrvashi Mohnani2017-11-07
| | | | | | | | | Adds the tarball transport, which is used by the kpod import command Signed-off-by: Urvashi Mohnani <umohnani@redhat.com> Closes: #12 Approved by: rhatdan
* Respect the quiet flag.Daniel J Walsh2017-11-07
| | | | | | | | | | | When setting the `kpod load -q` flag, output was still going to the screen. This patch adds a check to not output to the terminal unless there is an io.Writer specified, and then to write to the io.Writer. Signed-off-by: Daniel J Walsh <dwalsh@redhat.com> Closes: #23 Approved by: rhatdan
* SELinux tests are mistakenly not running.Daniel J Walsh2017-11-07
| | | | | | | Signed-off-by: Daniel J Walsh <dwalsh@redhat.com> Closes: #22 Approved by: rhatdan
* Convert tmpfs mounts to use generateDaniel J Walsh2017-11-06
| | | | | | | Signed-off-by: Daniel J Walsh <dwalsh@redhat.com> Closes: #19 Approved by: baude