9963 Commits

Author SHA1 Message Date
080ccf512d Remove build \!remote flags from test
Add some more tests, document cases where remote will not work
Add FIXMEs for tests that should work on podman-remote but currently
do not.

Signed-off-by: Daniel J Walsh <dwalsh@redhat.com>
2020-11-18 10:44:35 -05:00
286d356db0 Merge pull request #8337 from cevich/weekly_nix_cache_invalidate
Cirrus: Invalidate static cache on VM image update
2020-11-18 12:42:31 +01:00
e4e92d6104 Merge pull request #8386 from zhangguanzhang/pod-inspect-mac
Fix podman pod inspect show wrong StaticMAC
2020-11-18 12:19:57 +01:00
76a38fc3f2 Fix podman pod inspect show wrong MAC string
Signed-off-by: zhangguanzhang <zhangguanzhang@qq.com>
2020-11-18 18:06:06 +08:00
a6f7afd417 Merge pull request #8382 from dbaker-rh/helppage
[CI:DOCS] Fix example for manifest push
2020-11-18 04:33:49 +01:00
bd104d2948 Fix example for manifest push
Signed-off-by: Dave Baker <dbaker@redhat.com>
2020-11-17 20:54:37 -05:00
4bbf2b64ca Merge pull request #8378 from jwhonce/issues/8366
Swap out json-iterator for golang default
2020-11-18 01:21:19 +01:00
8a0c3d878b Merge pull request #8355 from baude/compatnetworkconnectdisconnect
add network connect|disconnect compat endpoints
2020-11-17 23:18:48 +01:00
0f745272e7 Merge pull request #8381 from Luap99/rename-test-files
Rename e2e test files to include _test.go suffix
2020-11-17 22:42:21 +01:00
770b03a50c Merge pull request #8363 from AlbanBedel/play-kube-create-only
Add an option to control if play kube should start the pod
2020-11-17 22:40:28 +01:00
0886fabbc2 Merge pull request #8380 from AlbanBedel/fix-client-play-kube-tls-verify
Client call to /play/kube incorrectly set tlsVerify
2020-11-17 22:38:28 +01:00
d3e794bda3 add network connect|disconnect compat endpoints
this enables the ability to connect and disconnect a container from a
given network. it is only for the compatibility layer. some code had to
be refactored to avoid circular imports.

additionally, tests are being deferred temporarily due to some
incompatibility/bug in either docker-py or our stack.

Signed-off-by: baude <bbaude@redhat.com>
2020-11-17 14:22:39 -06:00
d4446501f3 Rename e2e test files to include _test.go suffix
The test were working fine. Just my IDE doesn't like the files
without the suffix and I don't like red errors.

Signed-off-by: Paul Holzinger <paul.holzinger@web.de>
2020-11-17 21:18:27 +01:00
c6c6b23393 Client call to /play/kube incorrectly set tlsVerify
The API parameter `tlsVerify` should be the invert of the internal
option `SkipTLSVerify`, fix this conversion.

Signed-off-by: Alban Bedel <albeu@free.fr>
2020-11-17 20:33:33 +01:00
65880e5563 Merge pull request #8377 from jwhonce/wip/formats
Fix missing headers in `network ls`
2020-11-17 20:30:56 +01:00
f500fc955c Merge pull request #8371 from vrothberg/rm-gate
remove contrib/gate
2020-11-17 20:29:04 +01:00
eefa18f2b6 Merge pull request #8356 from rhatdan/docs
[CI:DOCS] Cleanup tutorials
2020-11-17 20:11:51 +01:00
7ab936eafa Add an option to control if play kube should start the pod
Having play kube start the pod is not always appropriate, one might
for example like to have the pod running as a set of systemd services.
Add a `start` option to the command line and API to control if the pod
should be started or not; it defaults to true for backward
compatibility.

Signed-off-by: Alban Bedel <albeu@free.fr>
2020-11-17 20:00:58 +01:00
4ed1ef533a Swap out json-iterator for golang default
json-iterator library failed to pretty print json for all inspection
types.

Fixes #8366

Signed-off-by: Jhon Honce <jhonce@redhat.com>
2020-11-17 11:45:03 -07:00
3a172c5999 Merge pull request #8373 from TomSweeneyRedHat/dev/tsweeney/nitpick
[CI:DOCS] fix an apostrophe nit in man page
2020-11-17 19:42:03 +01:00
ca9276640f Fix missing headers in network ls
Signed-off-by: Jhon Honce <jhonce@redhat.com>
2020-11-17 11:13:32 -07:00
12de330094 Merge pull request #8370 from rhatdan/wrap
Remove some more excessive wrapping and stuttering
2020-11-17 18:45:36 +01:00
586a7d97c2 [CI:DOCS] fix an apostrophe nit in man page
A quick follow up to #8367 to add a missing apostrophe in three
man pages.

Signed-off-by: TomSweeneyRedHat <tsweeney@redhat.com>
2020-11-17 11:43:03 -05:00
c2c7dd8ff3 remove contrib/gate
Remove references on contrib/gate.  The gating image is currently not
maintained and was replaced in the CI rewrite.  We can disable builds
in Quay once the change is merged.

Note that running `make validate` in a container is still desired.
A similar approach may be re-added in the future.

Signed-off-by: Valentin Rothberg <rothberg@redhat.com>
2020-11-17 15:21:50 +01:00
389dcb5c29 Remove some more excessive wrapping and stuttering
Stop over wrapping API Calls

The API calls will return an appropriate error, and this wrapping
just makes the error message look like it is stuttering and a
big mess.

Signed-off-by: Daniel J Walsh <dwalsh@redhat.com>
2020-11-17 09:19:22 -05:00
42ec4cf87f Merge pull request #8290 from vrothberg/fix-8265
use container cgroups path
2020-11-17 14:00:09 +01:00
d200801152 Cleanup tutorials
Inspired by @kannkyo PR.

Eliminate sudo when commands will work fine in rootless mode.

Make all commands in tutorials easily cut and pastable, by eliminating
$ and > symbols.

This should make them all consistant agross different tutorials.

Also make all systemctl enable calls use the --now option.

Signed-off-by: Daniel J Walsh <dwalsh@redhat.com>
2020-11-17 06:49:53 -05:00
7d067afac7 Merge pull request #8347 from rhatdan/hostname
Make sure /etc/hosts populated correctly with networks
2020-11-17 12:45:50 +01:00
d30f9ae8b6 Merge pull request #8346 from Luap99/shell-completion
Improve the shell completion api
2020-11-17 12:42:06 +01:00
c92bdf550c Merge pull request #8359 from TomSweeneyRedHat/dev/tsweeney/buildahv1.18.0
Bump Buildah to v1.18.0, c/storage to v1.24.0
2020-11-17 12:36:19 +01:00
d7a547952c Merge pull request #8367 from Luap99/docs-network-pod
[CI:DOCS] Explain the relation between --pod and --network
2020-11-17 12:34:22 +01:00
39bf07694c use container cgroups path
When looking up a container's cgroup path, parse /proc/[PID]/cgroup.
This will work across all cgroup managers and configurations and is
supported on cgroups v1 and v2.

Fixes: #8265
Signed-off-by: Valentin Rothberg <rothberg@redhat.com>
2020-11-17 12:29:50 +01:00
31ceaf7bd4 Explain the relation between --pod and --network
Signed-off-by: Paul Holzinger <paul.holzinger@web.de>
2020-11-17 11:57:28 +01:00
3502860e1c Merge pull request #8360 from edsantiago/logformatter
logformatter: highlight timing results
2020-11-17 03:42:50 +01:00
4ca4234af1 Make sure /etc/hosts populated correctly with networks
The --hostname and containername should always be added to containers.

Added some tests to make sure you can always ping the hostname and container
name from within the container.

Fixes: https://github.com/containers/podman/issues/8095

Signed-off-by: Daniel J Walsh <dwalsh@redhat.com>
2020-11-16 16:40:50 -05:00
0da801a446 logformatter: highlight timing results
Add a :hover style to rows in the 'integration timing results'
section. Without that, it's really hard for my eye to scan
across and match a time to a test name.

Signed-off-by: Ed Santiago <santiago@redhat.com>
2020-11-16 14:34:59 -07:00
b78a90cbde Bump Buildah to v1.18.0, c/storage to v1.24.0
Update to Buildah v1.18.0 and c/storage to v1.24

Signed-off-by: TomSweeneyRedHat <tsweeney@redhat.com>
2020-11-16 15:50:14 -05:00
e59394973a Merge pull request #8335 from Luap99/fix-html-tables
[CI:DOCS] Fix markdown tables on docs.podman.io
2020-11-16 21:40:10 +01:00
b8d25bd2cd Cirrus: Invalidate static cache on VM image update
It's important to periodically update the nix cache (about 1GB in size).
If not, it can grow stale and has been observed causing task failures.
Associating the nix cache update with a VM/Container image update,
ensures it happens first in PRs where environment-related failures
are less surprising.

Signed-off-by: Chris Evich <cevich@redhat.com>
2020-11-16 15:23:08 -05:00
5843f33a12 Merge pull request #8339 from rhatdan/df
Wrap missing container errors with container ID
2020-11-16 19:37:56 +01:00
6f08f2a238 Merge pull request #8324 from baude/speedupe2eremote
use lookaside storage for remote tests
2020-11-16 17:47:12 +01:00
59b0a0bd72 Merge pull request #8351 from containers/dependabot/go_modules/k8s.io/apimachinery-0.19.4
Bump k8s.io/apimachinery from 0.19.3 to 0.19.4
2020-11-16 17:38:28 +01:00
cf4967de4d Improve the shell completion api
One main advantage of the new shell completion logic is that
we can easly parse flags and adjust based on the given flags
the suggestions. For example some commands accept the
`--latest` flag only if no arguments are given.

This commit implements this logic in a simple maintainable way
since it reuses the already existing `Args` function in the
cmd struct.

I also refactored the `getXXX` function to match based on the
namei/id which could speed up the shell completion with many
containers, images, etc...

I also added the degraded status to the valid pod status
filters which was implemented in #8081.

Signed-off-by: Paul Holzinger <paul.holzinger@web.de>
2020-11-16 16:14:42 +01:00
15539c1c4b use lookaside storage for remote tests
in an effort to speed up the remote testing, we should be using
lookaside storage to avoid pull images as well as importing multiple
images into the RW store.

one test was removed and added into system test by Ed in #8325

Signed-off-by: baude <bbaude@redhat.com>
2020-11-16 08:15:44 -06:00
f9af427d59 Bump k8s.io/apimachinery from 0.19.3 to 0.19.4
Bumps [k8s.io/apimachinery](https://github.com/kubernetes/apimachinery) from 0.19.3 to 0.19.4.
- [Release notes](https://github.com/kubernetes/apimachinery/releases)
- [Commits](https://github.com/kubernetes/apimachinery/compare/v0.19.3...v0.19.4)

Signed-off-by: dependabot-preview[bot] <support@dependabot.com>
Signed-off-by: Daniel J Walsh <dwalsh@redhat.com>
2020-11-16 09:14:24 -05:00
392075631a Merge pull request #8345 from afbjorklund/volume-filter
Add support for volume ls --filter label=key=value
2020-11-15 15:13:51 +01:00
ff65d6095d Wrap missing container errors with container ID
While playing around with podman system df, I saw that my container
database was in bad state.  Basically podman new about containers
that were no longer in container/storage.

The podman system df was just erroring out early stating "container does not exist"
with no indicator of which container.

This Patch wraps the podman system df errors to indicate which container does not exist.
It also logs errors on containers that get into this state, but continues on to work on
all containers.

Signed-off-by: Daniel J Walsh <dwalsh@redhat.com>
2020-11-15 06:45:21 -05:00
9fa09a83b3 Merge pull request #8325 from edsantiago/bats
system tests: various
2020-11-15 12:38:40 +01:00
a4c2471944 system tests: various
- images: confirm that 'podman images' emits headings
  even if there are no images present. Intended to
  replace e2e test which is difficult to get working
  under podman-remote.

- build: add test for #8092, podman-build gobbling stdin.
  Workaround needed for issues #8342 and #8343, in which
  podman-remote output differs from podman local.

- volumes: add test for #8307, double-lock on same volume.

Signed-off-by: Ed Santiago <santiago@redhat.com>
2020-11-14 14:55:48 -07:00
0dad2499a3 Add support for volume ls --filter label=key=value
Supposed to be able to search for labels with a given value.

Previously it meant searching for label key and label value:

--filter label=key --filter label=value

Add some documentation and integration tests for it as well.

Signed-off-by: Anders F Björklund <anders.f.bjorklund@gmail.com>
2020-11-14 20:18:52 +01:00