Merge pull request #21000 from rhatdan/machine

Strip key/values of external quotes

Author: openshift-merge-bot[bot]

pkg/systemd/parser/unitfile.go

@@ -622,7 +622,7 @@ func (f *UnitFile) Lookup(groupName string, key string) (string, bool) {
 		return "", false
 	}
 
-	return strings.TrimRightFunc(v, unicode.IsSpace), true
+	return strings.Trim(strings.TrimRightFunc(v, unicode.IsSpace), "\""), true
 }
 
 // Lookup the last instance of a key and convert the value to a bool

test/e2e/quadlet/health.container

@@ -1,6 +1,6 @@
 [Container]
 Image=localhost/imagename
-## assert-podman-args "--health-cmd" "\"hello world\""
+## assert-podman-args "--health-cmd" "hello world"
 HealthCmd="hello world"
 ## assert-podman-args "--health-interval" "1m"
 HealthInterval=1m

test/e2e/quadlet/hostname.container

@@ -1,4 +1,4 @@
 [Container]
 Image=localhost/imagename
-## assert-podman-args "--hostname" "\"quadlet-host\""
-HostName="quadlet-host"
+## assert-podman-args "--hostname" "quadlet-host"
+HostName=quadlet-host

test/e2e/quadlet/quotes.container

@@ -0,0 +1,6 @@
+## assert-podman-final-args localhost/imagename
+## assert-podman-args --name=RemoveQuotes\"Name
+
+[Container]
+Image="localhost/imagename"
+ContainerName="RemoveQuotes"Name"

test/e2e/quadlet_test.go

@@ -792,6 +792,7 @@ BOGUS=foo
 		Entry("ports.container", "ports.container", 0, ""),
 		Entry("ports_ipv6.container", "ports_ipv6.container", 0, ""),
 		Entry("pull.container", "pull.container", 0, ""),
+		Entry("quotes.container", "quotes.container", 0, ""),
 		Entry("readonly.container", "readonly.container", 0, ""),
 		Entry("readonly-tmpfs.container", "readonly-tmpfs.container", 0, ""),
 		Entry("readonly-notmpfs.container", "readonly-notmpfs.container", 0, ""),