fix(build, stapel, import): import from root causes runner to hang (add: /) (#6906)

Signed-off-by: Alexandr Zaytsev <alexandr.zaytsev@flant.com>

Author: nervgh

pkg/build/stage/dependencies.go

@@ -382,6 +382,8 @@ func generateChecksumCommand(from string, includePaths, excludePaths []string, r
 		findCommandParts = append(findCommandParts, fmt.Sprintf("\\( %s \\)", strings.Join(nameIncludeArgs, " -or ")))
 	}
 
+	excludePaths = append(excludePaths, stapel.CONTAINER_MOUNT_ROOT)
+
 	var nameExcludeArgs []string
 	for _, excludePath := range excludePaths {
 		formattedPath := util.SafeTrimGlobsAndSlashesFromPath(excludePath)

pkg/config/common.go

@@ -10,6 +10,8 @@ import (
 	"github.com/werf/common-go/pkg/util"
 )
 
+//go:generate mockgen -source common.go -package config -destination common_mock.go
+
 type rawOrigin interface {
 	configSection() interface{}
 	doc() *doc

pkg/config/common_mock.go

@@ -124,6 +124,8 @@ func (c *ExportBase) validate() error {
 		return newDetailedConfigError("`includePaths: [PATH, ...]|PATH` should be relative paths!", c.raw.rawOrigin.configSection(), c.raw.rawOrigin.doc())
 	case !allRelativePaths(c.ExcludePaths):
 		return newDetailedConfigError("`excludePaths: [PATH, ...]|PATH` should be relative paths!", c.raw.rawOrigin.configSection(), c.raw.rawOrigin.doc())
+	case c.Add == "/" && len(c.IncludePaths) == 0:
+		return newDetailedConfigError("`add: '/'` requires not empty includePaths to interpret copy sources unambiguously", c.raw.rawOrigin.configSection(), c.raw.rawOrigin.doc())
 	default:
 		return nil
 	}

pkg/config/export_base.go

@@ -0,0 +1,47 @@
+package config
+
+import (
+	. "github.com/onsi/ginkgo/v2"
+	. "github.com/onsi/gomega"
+	"go.uber.org/mock/gomock"
+	"gopkg.in/yaml.v2"
+)
+
+var _ = Describe("ExportBase", func() {
+	DescribeTable("validate()",
+		func(yamlMap map[string]any, errMessage string) {
+			Expect(yamlMap).NotTo(HaveLen(0), "yamlMap should not be empty")
+
+			testConfigSection := "test config"
+			testDoc := &doc{
+				Content:        []byte("some content"),
+				Line:           -1,
+				RenderFilePath: "some file path",
+			}
+
+			rawOriginMock := NewMockrawOrigin(gomock.NewController(GinkgoT()))
+			rawOriginMock.EXPECT().configSection().Return(testConfigSection)
+			rawOriginMock.EXPECT().doc().Return(testDoc)
+
+			rawExBase := &rawExportBase{rawOrigin: rawOriginMock}
+			exBase := &ExportBase{raw: rawExBase}
+
+			rawYaml, err := yaml.Marshal(yamlMap)
+			Expect(err).To(Succeed())
+
+			Expect(yaml.UnmarshalStrict(rawYaml, &exBase)).To(Succeed())
+
+			expectedErr := newDetailedConfigError(errMessage, testConfigSection, testDoc)
+
+			Expect(exBase.validate()).To(Equal(expectedErr))
+		},
+		Entry(
+			"should return err if b.Add = / and b.IncludePaths=[]",
+			map[string]any{
+				"add": "/",
+				"to":  "/some/path",
+			},
+			"`add: '/'` requires not empty includePaths to interpret copy sources unambiguously",
+		),
+	)
+})