From 3bf0fbf6f6159107c0a2a120ee9f381675af53c0 Mon Sep 17 00:00:00 2001
From: zhangguanzhang <zhangguanzhang@qq.com>
Date: Mon, 26 Apr 2021 20:07:21 +0800
Subject: [PATCH] Fixes generate kube incorrect when bind-mounting "/" and
 "/root"

Signed-off-by: zhangguanzhang <zhangguanzhang@qq.com>
---
 libpod/kube.go                 |  9 ++++++++-
 test/e2e/generate_kube_test.go | 23 +++++++++++++++++++++++
 2 files changed, 31 insertions(+), 1 deletion(-)

diff --git a/libpod/kube.go b/libpod/kube.go
index 11ccaeadc7..adcfe92c9d 100644
--- a/libpod/kube.go
+++ b/libpod/kube.go
@@ -1,6 +1,7 @@
 package libpod
 
 import (
+	"fmt"
 	"math/rand"
 	"os"
 	"strconv"
@@ -539,11 +540,17 @@ func libpodMountsToKubeVolumeMounts(c *Container) ([]v1.VolumeMount, []v1.Volume
 	namedVolumes, mounts := c.sortUserVolumes(c.config.Spec)
 	vms := make([]v1.VolumeMount, 0, len(mounts))
 	vos := make([]v1.Volume, 0, len(mounts))
-	for _, m := range mounts {
+
+	var suffix string
+	for index, m := range mounts {
 		vm, vo, err := generateKubeVolumeMount(m)
 		if err != nil {
 			return vms, vos, err
 		}
+		// Name will be the same, so use the index as suffix
+		suffix = fmt.Sprintf("-%d", index)
+		vm.Name += suffix
+		vo.Name += suffix
 		vms = append(vms, vm)
 		vos = append(vos, vo)
 	}
diff --git a/test/e2e/generate_kube_test.go b/test/e2e/generate_kube_test.go
index c3586d9b68..611e8ddac1 100644
--- a/test/e2e/generate_kube_test.go
+++ b/test/e2e/generate_kube_test.go
@@ -496,6 +496,29 @@ var _ = Describe("Podman generate kube", func() {
 		Expect(inspect.OutputToString()).To(ContainSubstring(vol1))
 	})
 
+	It("podman generate kube when bind-mounting '/' and '/root' at the same time ", func() {
+		// Fixes https://github.com/containers/podman/issues/9764
+
+		ctrName := "mount-root-ctr"
+		session1 := podmanTest.Podman([]string{"run", "-d", "--pod", "new:mount-root-conflict", "--name", ctrName,
+			"-v", "/:/volume1/",
+			"-v", "/root:/volume2/",
+			"alpine", "top"})
+		session1.WaitWithDefaultTimeout()
+		Expect(session1.ExitCode()).To(Equal(0))
+
+		kube := podmanTest.Podman([]string{"generate", "kube", "mount-root-conflict"})
+		kube.WaitWithDefaultTimeout()
+		Expect(kube.ExitCode()).To(Equal(0))
+
+		pod := new(v1.Pod)
+		err := yaml.Unmarshal(kube.Out.Contents(), pod)
+		Expect(err).To(BeNil())
+
+		Expect(len(pod.Spec.Volumes)).To(Equal(2))
+
+	})
+
 	It("podman generate kube with persistent volume claim", func() {
 		vol := "vol-test-persistent-volume-claim"