summaryrefslogtreecommitdiff
Commit message (Collapse)AuthorAge
* 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
* Remove defaults and use runtime-tools/generate for specDaniel J Walsh2017-11-06
| | | | | | | Signed-off-by: Daniel J Walsh <dwalsh@redhat.com> Closes: #19 Approved by: baude
* Merge pull request #17 from rhatdan/capsDaniel J Walsh2017-11-05
|\ | | | | Add support for Caps Options.
| * Handle Linux Capabilities from command lineDaniel J Walsh2017-11-04
| | | | | | | | | | | | Had to revendor in docker/docker again, which dropped a bunch of packages Signed-off-by: Daniel J Walsh <dwalsh@redhat.com>
* | Merge pull request #8 from baude/clicontextDaniel J Walsh2017-11-05
|\ \ | |/ |/| Clicontext
| * libpod/runtime_img_test.go Unit Testsbaude2017-11-04
| | | | | | | | | | | | Unit tests for getRegistry related functions. Signed-off-by: baude <bbaude@redhat.com>
| * spec_test.go: Unit tests for spec.gobaude2017-11-03
| | | | | | | | | | | | | | Unit tests for spec.go which includes testing parts of the oci spec. Signed-off-by: baude <bbaude@redhat.com>
| * spec.go: Remove cli context as func argbaude2017-11-03
| | | | | | | | | | | | | | Remove cli context as a func arg to make unit tests easier. Signed-off-by: baude <bbaude@redhat.com>
| * util_test.go: Unittests for util.gobaude2017-11-03
| | | | | | | | | | | | Add unit tests for func StringInSlice. Signed-off-by: baude <bbaude@redhat.com>
| * create_cli_test.go: Unittestsbaude2017-11-03
| | | | | | | | | | | | | | | | | | Create unittests for the create_cli helper functions. As such, remove cli context usage from the functions to further divide between cli input and parsing. Also, simplifies unit testing. Signed-off-by: baude <bbaude@redhat.com>
| * Vendor in testify/assert and depsbaude2017-11-03
|/ | | | Signed-off-by: baude <bbaude@redhat.com>
* Parse SecurityOptsDaniel J Walsh2017-11-03
| | | | | | | | | This should turn on handling of SELinux, NoNewPrivs, seccomp and Apparmor Signed-off-by: Daniel J Walsh <dwalsh@redhat.com> Closes: #15 Approved by: rhatdan
* Merge pull request #14 from baude/gitvalidationMatthew Heon2017-11-03
|\ | | | | Makefile: Add repo EPOCH
| * Makefile: Add repo EPOCHbaude2017-11-03
|/ | | | | | | The git ID for the repo's EPOCH was incorrect in the Makefile so .gitvalidation would not run correctly. Signed-off-by: baude <bbaude@redhat.com>
* Merge pull request #6 from rhatdan/paprDaniel J Walsh2017-11-02
|\ | | | | Add Papr support
| * Remove tip from testing in yamlDaniel J Walsh2017-11-02
| | | | | | | | Signed-off-by: Daniel J Walsh <dwalsh@redhat.com>