summaryrefslogtreecommitdiff
Commit message (Collapse)AuthorAge
* [CI:DOCS]Binding overhaulsBrent Baude2020-01-28
| | | | | | | | | | Add binding for networks and begin documentation for binding methods for godoc. Also, add major functions to their own subpackages so reduce the amount of of method confusion. So instead of: bindings.ListImages(), we now do a [bindings].images.List(). Also, the connection is passed to each binding method via a context to allow for future growth. Lastly, add first set of tests. There are a couple of things to work out for rootless tests yet. Signed-off-by: Brent Baude <bbaude@redhat.com>
* Merge pull request #4999 from rpjday/docs/network_rstOpenShift Merge Robot2020-01-28
|\ | | | | docs: fix incomplete heading underlining in network.rst
| * docs: fix incomplete heading underlining in network.rstRobert P. J. Day2020-01-28
| | | | | | | | | | | | Lack of proper underlining generates processing diagnostic. Signed-off-by: Robert P. J. Day <rpjday@crashcourse.ca>
* | Merge pull request #4996 from baude/rootlessnetnsOpenShift Merge Robot2020-01-28
|\ \ | | | | | | [CI:DOCS]rootless exec cannot join root namespace
| * | [CI:DOCS]rootless exec cannot join root namespaceBrent Baude2020-01-27
| | | | | | | | | | | | | | | | | | | | | | | | add a quick note in the rootless.md about usage of --net=host as rootless and subsequent podman execs. Fixes: #4473 Signed-off-by: Brent Baude <bbaude@redhat.com>
* | | Merge pull request #4995 from rpjday/docs/podman_stopOpenShift Merge Robot2020-01-28
|\ \ \ | | | | | | | | docs: add missing hyphen for '-t' option, command '$' prompts
| * | | docs: add missing hyphen for '-t' option, command '$' promptsRobert P. J. Day2020-01-28
| |/ / | | | | | | | | | Signed-off-by: Robert P. J. Day <rpjday@crashcourse.ca>
* | | Merge pull request #4974 from rhatdan/manOpenShift Merge Robot2020-01-28
|\ \ \ | |_|/ |/| | Cleanup man pages exit code descriptions
| * | Cleanup man pages exit code descriptionsDaniel J Walsh2020-01-26
| | | | | | | | | | | | | | | | | | | | | The conversion of markdown to man pages is causing "_" to cover entire lines. This PR cleans this up and fixes some of the english. Signed-off-by: Daniel J Walsh <dwalsh@redhat.com>
* | | Merge pull request #4986 from baude/showhconimageOpenShift Merge Robot2020-01-27
|\ \ \ | | | | | | | | inspect image healthchecks
| * | | inspect image healthchecksBrent Baude2020-01-27
| | | | | | | | | | | | | | | | | | | | | | | | | | | | when a docker image has a defined healthcheck, it should be displayed with inspect. this is only valid for docker images as oci images are not aware of healthchecks. Fixes: #4799 Signed-off-by: Brent Baude <bbaude@redhat.com>
* | | | Merge pull request #4992 from rhatdan/dockerOpenShift Merge Robot2020-01-27
|\ \ \ \ | | | | | | | | | | Add link from docker.sock to podman.sock
| * | | | Add link from docker.sock to podman.sockDaniel J Walsh2020-01-27
| | | | | | | | | | | | | | | | | | | | Signed-off-by: Daniel J Walsh <dwalsh@redhat.com>
* | | | | Merge pull request #4991 from baude/addcoprOpenShift Merge Robot2020-01-27
|\ \ \ \ \ | |_|_|_|/ |/| | | | [CI:DOCS]Add copr link to fedora install page
| * | | | [CI:DOCS]Add copr link to fedora install pageBrent Baude2020-01-27
| |/ / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | for non-rawhide fedora distributions, you can use the copr repo to track upstream Fixes: #4956 Signed-off-by: Brent Baude <bbaude@redhat.com>
* | | | Merge pull request #4894 from edsantiago/test_apiv2OpenShift Merge Robot2020-01-27
|\ \ \ \ | |/ / / |/| | | Tests for API v2
| * | | Tests for API v2Ed Santiago2020-01-17
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Initial framework for testing the version 2 (HTTP) API. Includes a collection of tests for some of the existing endpoints. Not all tests are currently passing. Signed-off-by: Ed Santiago <santiago@redhat.com>
* | | | Merge pull request #4982 from rpjday/rpjday/exists_sudoOpenShift Merge Robot2020-01-27
|\ \ \ \ | | | | | | | | | | docs: remove reference to "sudo" in "podman exists" examples
| * | | | docs: remove reference to "sudo" in "podman exists" examplesRobert P. J. Day2020-01-26
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | In both of "podman {container,image} exists" man pages, there are superfluous uses of "sudo" in examples -- get rid of them. Signed-off-by: Robert P. J. Day <rpjday@crashcourse.ca>
* | | | | Merge pull request #4983 from rpjday/rpjday/history_boolsOpenShift Merge Robot2020-01-27
|\ \ \ \ \ | | | | | | | | | | | | docs: add boolean values and defaults to "man podman-history" options
| * | | | | docs: add boolean values and defaults to "man podman-history" optionsRobert P. J. Day2020-01-26
| | | | | | | | | | | | | | | | | | | | | | | | Signed-off-by: Robert P. J. Day <rpjday@crashcourse.ca>
* | | | | | Merge pull request #4985 from baude/pushhiddenOpenShift Merge Robot2020-01-27
|\ \ \ \ \ \ | |/ / / / / |/| | | | | Hidden remote flags can be nil
| * | | | | Hidden remote flags can be nilBrent Baude2020-01-26
| | |_|/ / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The pull command has several options that are hidden for the remote client. In that case, when checking to see if the flag has been flipped with .Changed, we get a nil pointer error. Using IsSet is tolerant of this. Fixes: #4706 Signed-off-by: Brent Baude <bbaude@redhat.com>
* | | | | Merge pull request #4979 from rpjday/rpjday/system_pruneOpenShift Merge Robot2020-01-26
|\ \ \ \ \ | | | | | | | | | | | | docs: fix system-prune markdown; reword for clarity
| * | | | | docs: fix system-prune markdown; reword for clarityRobert P. J. Day2020-01-26
| | |/ / / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | In addition to tweaking markdown, reword the explanation of the "--volumes" option for clarity. Signed-off-by: Robert P. J. Day <rpjday@crashcourse.ca>
* | | | | Merge pull request #4977 from rpjday/rpjday/man_pagesOpenShift Merge Robot2020-01-26
|\ \ \ \ \ | |_|/ / / |/| | | | docs: clean up "man podman-rm", "man podman-rmi"
| * | | | docs: clean up "man podman-rm", "man podman-rmi"Robert P. J. Day2020-01-26
| |/ / / | | | | | | | | | | | | | | | | | | | | | | | | Standardize markdown for options and commands, and add leading dollar signs to emphasize commands being run. Signed-off-by: Robert P. J. Day <rpjday@crashcourse.ca>
* | | | Merge pull request #4975 from agners/add-openembedded-to-install.mdOpenShift Merge Robot2020-01-26
|\ \ \ \ | |/ / / |/| | | install.md: mention availability of OpenEmbedded recipes
| * | | install.md: mention availability of OpenEmbedded recipesStefan Agner2020-01-26
| | |/ | |/| | | | | | | | | | | | | | | | | | | OpenEmbedded is a build framework to build an embedded Linux distribution. Podman recipes are available in the meta-virtualization layer. Signed-off-by: Stefan Agner <stefan@agner.ch>
* | | Merge pull request #4933 from rpjday/docs/markdown/inspectOpenShift Merge Robot2020-01-26
|\ \ \ | |/ / |/| | markdown: reword 'podman-inspect' to properly explain '--size'
| * | markdown: reword 'podman-inspect' to properly explain '--size'Robert P. J. Day2020-01-22
| | | | | | | | | | | | | | | | | | | | | Reword the man page to clarify that the '--size' option displays the container size in addition to the normal output. Signed-off-by: Robert P. J. Day <rpjday@crashcourse.ca>
* | | Merge pull request #4926 from TomSweeneyRedHat/dev/tsweeney/fixsystemdfOpenShift Merge Robot2020-01-25
|\ \ \ | | | | | | | | Fix example format in system df man
| * | | Fix example format in system df manTomSweeneyRedHat2020-01-22
| | | | | | | | | | | | | | | | | | | | | | | | | | | | Fix the formatting of the examples in the podman system df man page. Signed-off-by: TomSweeneyRedHat <tsweeney@redhat.com>
* | | | Merge pull request #4965 from baude/reviewcorrections3OpenShift Merge Robot2020-01-25
|\ \ \ \ | | | | | | | | | | APIv2 review corrections #3
| * | | | APIv2 review corrections #3Brent Baude2020-01-25
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The third pass of corrections for the APIv2. Signed-off-by: Brent Baude <bbaude@redhat.com>
* | | | | Merge pull request #4941 from ↵OpenShift Merge Robot2020-01-25
|\ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | containers/dependabot/go_modules/gopkg.in/yaml.v2-2.2.8 build(deps): bump gopkg.in/yaml.v2 from 2.2.7 to 2.2.8
| * | | | | build(deps): bump gopkg.in/yaml.v2 from 2.2.7 to 2.2.8dependabot-preview[bot]2020-01-23
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Bumps [gopkg.in/yaml.v2](https://github.com/go-yaml/yaml) from 2.2.7 to 2.2.8. - [Release notes](https://github.com/go-yaml/yaml/releases) - [Commits](https://github.com/go-yaml/yaml/compare/v2.2.7...v2.2.8) Signed-off-by: dependabot-preview[bot] <support@dependabot.com> Signed-off-by: Matthew Heon <mheon@redhat.com>
* | | | | | Merge pull request #4952 from mheon/inspect_default_sharesOpenShift Merge Robot2020-01-25
|\ \ \ \ \ \ | | | | | | | | | | | | | | Default CPUShares in Inspect are 1024
| * | | | | | Default CPUShares in Inspect are 1024Matthew Heon2020-01-23
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This is purely a display change - we weren't initializing the default value to display for the CPUShares field, which defaults to 1024. Fixes #4822 Signed-off-by: Matthew Heon <mheon@redhat.com>
* | | | | | | Merge pull request #4968 from vrothberg/fork-camelcaseOpenShift Merge Robot2020-01-25
|\ \ \ \ \ \ \ | | | | | | | | | | | | | | | | fork fatih/camelcase
| * | | | | | | camelcase: fix lint reportsValentin Rothberg2020-01-25
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Signed-off-by: Valentin Rothberg <rothberg@redhat.com>
| * | | | | | | fork fatih/camelcaseValentin Rothberg2020-01-25
|/ / / / / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | faith/camelcase has been archived and is no longer maintained. The package is sufficiently small and self-contained enough to maintain it in libpod. Fixes: #4783 Signed-off-by: Valentin Rothberg <rothberg@redhat.com>
* | | | | | | Merge pull request #4966 from jwhonce/wip/filtersOpenShift Merge Robot2020-01-24
|\ \ \ \ \ \ \ | |_|_|/ / / / |/| | | | | | [CI:DOCS] Refactor time parsing to be more liberal in accepted values
| * | | | | | Refactor time parsing to be more liberal in accepted valuesJhon Honce2020-01-24
|/ / / / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | * Added helper function to allow parsing from filters Signed-off-by: Jhon Honce <jhonce@redhat.com>
* | | | | | Merge pull request #4958 from jwhonce/wip/filtersOpenShift Merge Robot2020-01-24
|\ \ \ \ \ \ | | | | | | | | | | | | | | [CI:DOCS] Add query parameter converters for complex types
| * | | | | | Add query parameter converters for complex typesJhon Honce2020-01-23
| | |/ / / / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | * Add converter for URL query parameters of type map[string][]string * Add converter for URL query parameters of type time.Time * Added function to allocate and configure schema.Decoder for API use * Updated API handlers to leverage new converters, and correct handler code for filter type An encoding example for a client using filters: v := map[string][]string{ "dangling": {"true"}, } payload, err := jsoniter.MarshalToString(v) if err != nil { panic(err) } payload = "?filters=" + url.QueryEscape(payload) Signed-off-by: Jhon Honce <jhonce@redhat.com>
* | | | | | Merge pull request #4953 from baude/reviewcorrections2OpenShift Merge Robot2020-01-23
|\ \ \ \ \ \ | |/ / / / / |/| | | | | Review corrections pass #2
| * | | | | Review corrections pass #2Brent Baude2020-01-23
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Add API review comments to correct documentation and endpoints. Also, add a libpode prune method to reduce code duplication. Only used right now for the API but when the remote client is wired, we will switch over there too. Signed-off-by: Brent Baude <bbaude@redhat.com>
* | | | | | Merge pull request #4948 from vrothberg/seccomp-labelOpenShift Merge Robot2020-01-23
|\ \ \ \ \ \ | |_|/ / / / |/| | | | | seccomp policy: expect profile in config label
| * | | | | seccomp policy: expect profile in config labelValentin Rothberg2020-01-23
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Move the seccomp profile from a manifest annotation to a config label. This way, we can support it for Docker images as well and provide an easy way to add that data via Dockerfiles. Signed-off-by: Valentin Rothberg <rothberg@redhat.com>