summaryrefslogtreecommitdiff
Commit message (Collapse)AuthorAge
* libpod/runtime.go: Stick paths in arraysbaude2018-01-31
| | | | | | | | | | | To account for more path possibilities, we now iterate a string array of possible paths to try and find paths to runc and conmon. Signed-off-by: baude <bbaude@redhat.com> Closes: #278 Approved by: baude
* Ginkgo Tests: ps, pull, push and rmbaude2018-01-31
| | | | | | | | | | | | | Migrate ps, pull, push, and rm from bats to ginkgo. Also, fixed a conditional issue with adding ports when an image defines the port and the user wants to override it. Signed-off-by: baude <bbaude@redhat.com> Closes: #277 Approved by: baude
* libpod/runtime.go: runtime pathbaude2018-01-31
| | | | | | | | | | | | Ubuntu installs runc to /usr/sbin/runc so we now account for that. Also, added small check when creating a new runtime that if we cannot find the runc binary, we bail out. Signed-off-by: baude <bbaude@redhat.com> Closes: #276 Approved by: baude
* docs/tutorials/podman_tutorial.md: Add Ubuntubaude2018-01-30
| | | | | | | | | Adding in how to install and run podman on Ubuntu. Signed-off-by: baude <bbaude@redhat.com> Closes: #275 Approved by: baude
* libpod/container_api: Set hostname in ENVbaude2018-01-30
| | | | | | | | | | The container's hostname should be set as an environment variable for the container. Signed-off-by: baude <bbaude@redhat.com> Closes: #273 Approved by: baude
* Migrate start, stats, stop to Ginkgobaude2018-01-30
| | | | | | | | | | Migrate the start, stats, and stop integration tests to the Ginkgo style. Signed-off-by: baude <bbaude@redhat.com> Closes: #274 Approved by: mheon
* Add StopWithTimeout API function for containersMatthew Heon2018-01-30
| | | | | | | | | | Normal Stop should not need a timeout, and should use the default Add a function that does accept a timeout aside it Signed-off-by: Matthew Heon <mheon@redhat.com> Closes: #272 Approved by: rhatdan
* Initial gingko workbaude2018-01-29
| | | | | | | | | | | | | | | | | This implements the ginkgo integration test framework for podman. As tests are migrated from bats to ginkgo, we will still run both integration suites. When a test is migrated, we remove the tests from bats at that time. All new tests should be just for the ginkgo framework. One exception is that we only run the ginkgo suit in the travis/ubuntu environment. The CentOS and Fedora PAPR nodes will more than cover those. Signed-off-by: baude <bbaude@redhat.com> Closes: #261 Approved by: baude
* Merge pull request #267 from mheon/remove_libkpodMatthew Heon2018-01-29
|\ | | | | Remove libkpod
| * Honor storage-driver flagMatthew Heon2018-01-29
| | | | | | | | Signed-off-by: Matthew Heon <matthew.heon@gmail.com>
| * Remove libkpod. Replace runtime generation function.Matthew Heon2018-01-29
| | | | | | | | Signed-off-by: Matthew Heon <matthew.heon@gmail.com>
* | Ran spell checker through all man pagesDaniel J Walsh2018-01-29
| | | | | | | | | | | | | | Signed-off-by: Daniel J Walsh <dwalsh@redhat.com> Closes: #269 Approved by: TomSweeneyRedHat
* | Merge pull request #266 from TomSweeneyRedHat/dev/tsweeney/installfixDaniel J Walsh2018-01-28
|\ \ | |/ |/| Add containernetworking-cni to install reqs
| * Add containernetworking-cni to install reqsTomSweeneyRedHat2018-01-27
|/ | | | Signed-off-by: TomSweeneyRedHat <tsweeney@redhat.com>
* Fix tab issue with --format flagumohnani82018-01-27
| | | | | | | | | | | When the Go template was given with \t in between fields, the \t was not being recognized as a tab. Fixes that issue. Signed-off-by: umohnani8 <umohnani@redhat.com> Closes: #265 Approved by: rhatdan
* Remove conmon, get package from CRI-ODaniel J Walsh2018-01-27
| | | | | | | | | | | | conmon should not be built in two different places. conmon is now a separate package in Fedora so we can just add requires, for use on Ubuntu we can just require cri-o to be installed. Signed-off-by: Daniel J Walsh <dwalsh@redhat.com> Closes: #151 Approved by: mheon
* Strip unused format specifierMatthew Heon2018-01-27
| | | | | | | Signed-off-by: Matthew Heon <mheon@redhat.com> Closes: #263 Approved by: umohnani8
* Close attach control fileMatthew Heon2018-01-27
| | | | | | | | | From CRI-O #1288 Signed-off-by: Matthew Heon <mheon@redhat.com> Closes: #263 Approved by: umohnani8
* Add results check and debug output in podman_stopYiqiao Pu2018-01-26
| | | | | | | | | | | Add results check to make sure the container is not shows up after it is stopped. And also add some debug lines to help to figure out which step has problem if case failed. Signed-off-by: Yiqiao Pu <ypu@redhat.com> Closes: #264 Approved by: rhatdan
* Save --privileged stateDaniel J Walsh2018-01-25
| | | | | | | Signed-off-by: Daniel J Walsh <dwalsh@redhat.com> Closes: #255 Approved by: mheon
* Fix podman stats based on QE feedbackumohnani82018-01-25
| | | | | | | | | | | | QE found issues with formatting the go template and the man page was lacking information. Changed the format of the output to match latest docker. Add shortID function that returns the truncated ID Signed-off-by: umohnani8 <umohnani@redhat.com> Closes: #258 Approved by: rhatdan
* Fixed Created At and Running For valueumohnani82018-01-25
| | | | | | | | | | | | Changed the Created At value to match the format of docker ps and fixed the value for Running For to reflect the time elapsed since the container has been started. Initially it was showing the time elapsed since the container was created. Signed-off-by: umohnani8 <umohnani@redhat.com> Closes: #257 Approved by: rhatdan
* Fix issue with order of flagsumohnani82018-01-25
| | | | | | | | | | The order of the flags was casuing issue. Enabled SkipArgReorder to fix the problem. Signed-off-by: umohnani8 <umohnani@redhat.com> Closes: #259 Approved by: rhatdan
* Merge pull request #252 from baude/portDaniel J Walsh2018-01-24
|\ | | | | podman port
| * podman portbaude2018-01-23
| | | | | | | | | | | | | | | | | | | | | | podman port reports the port mappings per container. it can be used to report the ports ofa single container or latest container or all containers. in the case of a single container, the user can add an option filter for port and protocol. Signed-off-by: baude <bbaude@redhat.com>
* | Merge pull request #251 from baude/psbatchDaniel J Walsh2018-01-23
|\ \ | | | | | | Use batched operations in ps
| * | Use batched operations in psbaude2018-01-22
| | | | | | | | | | | | | | | | | | | | | | | | Because the podman ps command has to collection a lot of information from various places, many of which are controlled by locks, it is a good candidate for doing batch operations under a single lock. Signed-off-by: baude <bbaude@redhat.com>
* | | Merge pull request #256 from baude/imageperfDaniel J Walsh2018-01-23
|\ \ \ | | | | | | | | Streamline image resolution
| * | | Streamline image resolutionbaude2018-01-23
| | |/ | |/| | | | | | | | | | | | | | | | | | | | | | When trying to determine if a user-provided string that describes an image (ID, fq name, shortname, tagged), there were some inefficiencies where we looked up images multiple times to derive information about local images. Signed-off-by: baude <bbaude@redhat.com>
* | | Merge pull request #224 from ypu/import_testDaniel J Walsh2018-01-23
|\ \ \ | |/ / |/| | Update podman_import test to use podman
| * | Update podman_import test to use podmanYiqiao Pu2018-01-23
| | | | | | | | | | | | | | | | | | | | | Update the podman_import test to use podman run, and update the check method for string check method. Signed-off-by: Yiqiao Pu <ypu@redhat.com>
* | | libpod/oci.go: Perf issuebaude2018-01-23
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Matt Heon and I found that a defer statement was costing podman run dearly. We dont think the defer function was working (nor needed) and was timing out as well. Removing this defer statement decreased podman runtime by 1.5s or more. Signed-off-by: baude <bbaude@redhat.com> Closes: #253 Approved by: baude
* | | Fix issues with podman ps from QEumohnani82018-01-22
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | QE pointed out a few things missing/wrong with ps This PR addresses those issues. Added functionality for getting mounts and size also Fixed a few issues with the --filter params, for example filter with partial information. Signed-off-by: umohnani8 <umohnani@redhat.com> Closes: #250 Approved by: rhatdan
* | | Expose ports from imagebaude2018-01-22
| |/ |/| | | | | | | | | | | | | | | | | | | | | When an image has a port to expose, we need to expose it. User's input overrides the image's port information. Also, enable port information in ps so we can see which random port is assigned. Signed-off-by: baude <bbaude@redhat.com> Closes: #249 Approved by: rhatdan
* | Override hostname for containerbaude2018-01-21
| | | | | | | | | | | | | | | | | | | | | | Adds the ability to override the container's hostname. Also, uses the first twelve characters of the container ID as the default hostname if none is provided. Signed-off-by: baude <bbaude@redhat.com> Closes: #248 Approved by: baude
* | Enable port bindingsbaude2018-01-20
| | | | | | | | | | | | | | | | | | | | | | | | | | Set up nbetworking ports for the following use cases: * bind the same port between host and container * bind a specific host port to a different container port * bind a random host port to a specific container port Signed-off-by: baude <bbaude@redhat.com> Closes: #214 Approved by: baude
* | Touchup build to add missing -q flagTomSweeneyRedHat2018-01-20
| | | | | | | | | | | | | | Signed-off-by: TomSweeneyRedHat <tsweeney@redhat.com> Closes: #247 Approved by: rhatdan
* | Add dns testsbaude2018-01-19
| | | | | | | | | | | | | | | | | | | | In the dns commit, tests were accidently omitted due to a rebase against master. Signed-off-by: baude <bbaude@redhat.com> Closes: #245 Approved by: baude
* | Cleanup of podman statsDaniel J Walsh2018-01-19
| | | | | | | | | | | | | | | | | | | | | | | | Fix errors when containers are not running. --all, --latest, containers can not be used at same time. Should match the output of docker stats, 0 values replaced by "--" Should return stats right away if container is not running. Signed-off-by: Daniel J Walsh <dwalsh@redhat.com> Closes: #244 Approved by: TomSweeneyRedHat
* | Add --dns-search, --dns-opt, --dns-server and --add-host.baude2018-01-19
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Each of these options are destructive in nature, meaning if the user adds one of them, all current ones are removed from the produced resolv.conf. * dns-server allows the user to specify dns servers. * dns-opt allows the user to specify special resolv.conf options * dns-search allows the user to specify search domains The add-host option is not destructive and truly just adds the host to /etc/hosts. Signed-off-by: baude <bbaude@redhat.com> Closes: #231 Approved by: mheon
* | Set NOTEST variable to skip tests in paprbaude2018-01-19
| | | | | | | | | | | | | | | | | | | | | | | | The papr test with ansible is really handy for setting up test nodes. However, most of the time you want to run the tests manually, specifically the integration tests. Passing NOTEST=1 will allow you to skip gofmt, lint, and friends and will exit before the integration tests. Signed-off-by: baude <bbaude@redhat.com> Closes: #242 Approved by: mheon
* | Touchup README.md and fix whitespace issuesTomSweeneyRedHat2018-01-19
| | | | | | | | | | | | | | Signed-off-by: TomSweeneyRedHat <tsweeney@redhat.com> Closes: #243 Approved by: mheon
* | Merge pull request #240 from mheon/refactor_containerDaniel J Walsh2018-01-18
|\ \ | | | | | | Refactor container.go
| * | Fix gofmtMatthew Heon2018-01-18
| | | | | | | | | | | | Signed-off-by: Matthew Heon <matthew.heon@gmail.com>
| * | Rename containerRuntimeInfo to containerState for clarityMatthew Heon2018-01-18
| | | | | | | | | | | | Signed-off-by: Matthew Heon <matthew.heon@gmail.com>
| * | Rename ContainerState to ContainerStatusMatthew Heon2018-01-18
| | | | | | | | | | | | Signed-off-by: Matthew Heon <matthew.heon@gmail.com>
| * | Split container.go into three filesMatthew Heon2018-01-18
| | | | | | | | | | | | | | | | | | | | | | | | | | | Weighing in at ~1700 lines, container.go is just too big. Split it into three files: core structs and accessors (container.go), public API (container_api.go), and internal functions (container_internal.go). Signed-off-by: Matthew Heon <matthew.heon@gmail.com>
* | | Merge pull request #216 from rhatdan/dockerDaniel J Walsh2018-01-18
|\ \ \ | | | | | | | | Add support for mimicing docker CLI
| * | | Add support for mimicing docker CLIDaniel J Walsh2018-01-18
| | | | | | | | | | | | | | | | Signed-off-by: Daniel J Walsh <dwalsh@redhat.com>
* | | | Merge pull request #230 from TomSweeneyRedHat/dev/tsweeney/readmeDaniel J Walsh2018-01-18
|\ \ \ \ | | | | | | | | | | Slim down README.md and make tutorials page