summaryrefslogtreecommitdiff
path: root/pkg/specgen/specgen.go
diff options
context:
space:
mode:
authorPaul Holzinger <pholzing@redhat.com>2022-04-06 11:55:40 +0200
committerPaul Holzinger <pholzing@redhat.com>2022-04-11 18:41:19 +0200
commitd8a902a167b91ca4fde4ffeef55e9aa4907ae258 (patch)
tree0a011a24406b835d3e27dd225956c8c3e18ec93b /pkg/specgen/specgen.go
parent1d01815c107c91f6cfe98446d334c94a97d11080 (diff)
downloadpodman-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/specgen.go')
-rw-r--r--pkg/specgen/specgen.go8
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.