quadlet: handle generate environment params that inherit from host

Fixes: #26247

Signed-off-by: Volodymyr Pankin <[email protected]>

Author: sleevysleever

pkg/systemd/parser/unitfile.go

@@ -839,9 +839,9 @@ func (f *UnitFile) LookupLastArgs(groupName string, key string) ([]string, bool)
 }
 
 // Look up 'Environment' style key-value keys
-func (f *UnitFile) LookupAllKeyVal(groupName string, key string) (map[string]string, error) {
+func (f *UnitFile) LookupAllKeyVal(groupName string, key string) (map[string]*string, error) {
 	var warnings error
-	res := make(map[string]string)
+	res := make(map[string]*string)
 	allKeyvals := f.LookupAll(groupName, key)
 	for _, keyvals := range allKeyvals {
 		assigns, err := splitString(keyvals, WhitespaceSeparators, SplitRelax|SplitUnquote|SplitCUnescape)
@@ -852,9 +852,9 @@ func (f *UnitFile) LookupAllKeyVal(groupName string, key string) (map[string]str
 		for _, assign := range assigns {
 			key, value, found := strings.Cut(assign, "=")
 			if found {
-				res[key] = value
+				res[key] = &value
 			} else {
-				warnings = errors.Join(warnings, fmt.Errorf("separator was not found for %s", assign))
+				res[key] = nil
 			}
 		}
 	}

pkg/systemd/quadlet/podmancmdline.go

@@ -33,15 +33,19 @@ func (c *PodmanCmdline) addf(format string, a ...interface{}) {
 	c.add(fmt.Sprintf(format, a...))
 }
 
-func (c *PodmanCmdline) addKeys(arg string, keys map[string]string) {
+func (c *PodmanCmdline) addKeys(arg string, keys map[string]*string) {
 	ks := make([]string, 0, len(keys))
 	for k := range keys {
 		ks = append(ks, k)
 	}
 	sort.Strings(ks)
 
 	for _, k := range ks {
-		c.add(arg, fmt.Sprintf("%s=%s", k, keys[k]))
+		if keys[k] != nil {
+			c.add(arg, fmt.Sprintf("%s=%s", k, *keys[k]))
+		} else {
+			c.add(arg, k)
+		}
 	}
 }
 

pkg/systemd/quadlet/quadlet.go

@@ -810,8 +810,8 @@ func ConvertContainer(container *parser.UnitFile, isUser bool, unitsInfoMap map[
 	if ok && len(update) > 0 {
 		podman.addKeys(
 			"--label",
-			map[string]string{
-				autoUpdateLabel: update,
+			map[string]*string{
+				autoUpdateLabel: &update,
 			},
 		)
 	}