summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorMatthew Heon <mheon@redhat.com>2020-07-14 09:44:56 -0400
committerMatthew Heon <matthew.heon@pm.me>2020-07-22 14:20:14 -0400
commitf74a28e08a920dd50eb9377336a6cabbe2d43ff1 (patch)
tree261e3d3459061ade9ae9bc365dc4dceb9472b811
parentcad36d4b7ad47ac0150c5b55472e92ca524c5b4d (diff)
downloadpodman-f74a28e08a920dd50eb9377336a6cabbe2d43ff1.tar.gz
podman-f74a28e08a920dd50eb9377336a6cabbe2d43ff1.tar.bz2
podman-f74a28e08a920dd50eb9377336a6cabbe2d43ff1.zip
Fix lint
Signed-off-by: Matthew Heon <mheon@redhat.com>
-rw-r--r--libpod/pod_api.go20
-rw-r--r--test/e2e/pod_inspect_test.go7
2 files changed, 9 insertions, 18 deletions
diff --git a/libpod/pod_api.go b/libpod/pod_api.go
index f889ba789..f2ef81bec 100644
--- a/libpod/pod_api.go
+++ b/libpod/pod_api.go
@@ -494,33 +494,23 @@ func (p *Pod) Inspect() (*define.InspectPodData, error) {
if len(p.config.InfraContainer.DNSServer) > 0 {
infraConfig.DNSServer = make([]string, 0, len(p.config.InfraContainer.DNSServer))
- for _, i := range p.config.InfraContainer.DNSServer {
- infraConfig.DNSServer = append(infraConfig.DNSServer, i)
- }
+ infraConfig.DNSServer = append(infraConfig.DNSServer, p.config.InfraContainer.DNSServer...)
}
if len(p.config.InfraContainer.DNSSearch) > 0 {
infraConfig.DNSSearch = make([]string, 0, len(p.config.InfraContainer.DNSSearch))
- for _, i := range p.config.InfraContainer.DNSSearch {
- infraConfig.DNSSearch = append(infraConfig.DNSSearch, i)
- }
+ infraConfig.DNSSearch = append(infraConfig.DNSSearch, p.config.InfraContainer.DNSSearch...)
}
if len(p.config.InfraContainer.DNSOption) > 0 {
infraConfig.DNSOption = make([]string, 0, len(p.config.InfraContainer.DNSOption))
- for _, i := range p.config.InfraContainer.DNSOption {
- infraConfig.DNSOption = append(infraConfig.DNSOption, i)
- }
+ infraConfig.DNSOption = append(infraConfig.DNSOption, p.config.InfraContainer.DNSOption...)
}
if len(p.config.InfraContainer.HostAdd) > 0 {
infraConfig.HostAdd = make([]string, 0, len(p.config.InfraContainer.HostAdd))
- for _, i := range p.config.InfraContainer.HostAdd {
- infraConfig.HostAdd = append(infraConfig.HostAdd, i)
- }
+ infraConfig.HostAdd = append(infraConfig.HostAdd, p.config.InfraContainer.HostAdd...)
}
if len(p.config.InfraContainer.Networks) > 0 {
infraConfig.Networks = make([]string, 0, len(p.config.InfraContainer.Networks))
- for _, i := range p.config.InfraContainer.Networks {
- infraConfig.Networks = append(infraConfig.Networks, i)
- }
+ infraConfig.Networks = append(infraConfig.Networks, p.config.InfraContainer.Networks...)
}
infraConfig.PortBindings = makeInspectPortBindings(p.config.InfraContainer.PortBindings)
diff --git a/test/e2e/pod_inspect_test.go b/test/e2e/pod_inspect_test.go
index 64f943c80..16bf1c4c9 100644
--- a/test/e2e/pod_inspect_test.go
+++ b/test/e2e/pod_inspect_test.go
@@ -1,6 +1,7 @@
package integration
import (
+ "encoding/json"
"os"
"github.com/containers/libpod/v2/libpod/define"
@@ -84,7 +85,7 @@ var _ = Describe("Podman pod inspect", func() {
It("podman pod inspect outputs port bindings", func() {
podName := "testPod"
- create := podmanTest.Podman([]string{"pod", "create", "--name", testPod, "-p", "8080:80"})
+ create := podmanTest.Podman([]string{"pod", "create", "--name", podName, "-p", "8080:80"})
create.WaitWithDefaultTimeout()
Expect(create.ExitCode()).To(Equal(0))
@@ -96,7 +97,7 @@ var _ = Describe("Podman pod inspect", func() {
err := json.Unmarshal(inspectOut.Out.Contents(), inspectJSON)
Expect(err).To(BeNil())
Expect(inspectJSON.InfraConfig).To(Not(BeNil()))
- Expect(len(inspectJSON.PortBindings["80/tcp"])).To(Equal(1))
- Expect(inspectJSON.PortBindings["80/tcp"].HostPort).To(Equal("8080"))
+ Expect(len(inspectJSON.InfraConfig.PortBindings["80/tcp"])).To(Equal(1))
+ Expect(inspectJSON.InfraConfig.PortBindings["80/tcp"][0].HostPort).To(Equal("8080"))
})
})