diff options
| author | Paul Holzinger <pholzing@redhat.com> | 2022-04-06 11:55:40 +0200 |
|---|---|---|
| committer | Paul Holzinger <pholzing@redhat.com> | 2022-04-11 18:41:19 +0200 |
| commit | d8a902a167b91ca4fde4ffeef55e9aa4907ae258 (patch) | |
| tree | 0a011a24406b835d3e27dd225956c8c3e18ec93b /pkg/specgen | |
| parent | 1d01815c107c91f6cfe98446d334c94a97d11080 (diff) | |
| download | podman-d8a902a167b91ca4fde4ffeef55e9aa4907ae258.tar.gz podman-d8a902a167b91ca4fde4ffeef55e9aa4907ae258.tar.bz2 podman-d8a902a167b91ca4fde4ffeef55e9aa4907ae258.zip | |
API: use no_hosts from containers.conf
The API endpoints should properly honour the `no_hosts=true` setting in
containers.conf.
Fixes #13719
Signed-off-by: Paul Holzinger <pholzing@redhat.com>
Diffstat (limited to 'pkg/specgen')
| -rw-r--r-- | pkg/specgen/specgen.go | 8 |
1 files changed, 7 insertions, 1 deletions
diff --git a/pkg/specgen/specgen.go b/pkg/specgen/specgen.go index 27d77af9f..dfac1d457 100644 --- a/pkg/specgen/specgen.go +++ b/pkg/specgen/specgen.go @@ -467,7 +467,13 @@ type ContainerNetworkConfig struct { // UseImageHosts indicates that /etc/hosts should not be managed by // Podman, and instead sourced from the image. // Conflicts with HostAdd. - UseImageHosts bool `json:"use_image_hosts,omitempty"` + // Do not set omitempty here, if this is false it should be set to not get + // the server default. + // Ideally this would be a pointer so we could differentiate between an + // explicitly false/true and unset (containers.conf default). However + // specgen is stable so we can not change this right now. + // TODO (5.0): change to pointer + UseImageHosts bool `json:"use_image_hosts"` // HostAdd is a set of hosts which will be added to the container's // /etc/hosts file. // Conflicts with UseImageHosts. |