This is an automated email from the ASF dual-hosted git repository.

pcongiusti pushed a commit to branch main
in repository https://gitbox.apache.org/repos/asf/camel-k.git


The following commit(s) were added to refs/heads/main by this push:
     new fd64646  fix(cli): resource file with same content
fd64646 is described below

commit fd6464604cc6fbd764cb1b5c63f7503a9d764de6
Author: Pasquale Congiusti <pasquale.congiu...@gmail.com>
AuthorDate: Thu Mar 10 14:56:22 2022 +0100

    fix(cli): resource file with same content
    
    Closes #3077
---
 e2e/common/config/config_test.go                | 10 ++++++++++
 e2e/common/config/files/resources-data-same.txt |  1 +
 pkg/trait/mount.go                              |  6 ++----
 3 files changed, 13 insertions(+), 4 deletions(-)

diff --git a/e2e/common/config/config_test.go b/e2e/common/config/config_test.go
index 4e459d7..6904df1 100644
--- a/e2e/common/config/config_test.go
+++ b/e2e/common/config/config_test.go
@@ -209,6 +209,16 @@ func TestRunConfigExamples(t *testing.T) {
                        Eventually(AutogeneratedConfigmapsCount(ns), 
TestTimeoutShort).Should(Equal(0))
                })
 
+               t.Run("Plain text resource file with same content", func(t 
*testing.T) {
+                       Expect(Kamel("run", "-n", ns, 
"./files/resource-file-route.groovy", "--resource", 
"file:./files/resources-data.txt",
+                               "--resource", 
"file:./files/resources-data-same.txt").Execute()).To(Succeed())
+                       Eventually(IntegrationPodPhase(ns, 
"resource-file-route"), TestTimeoutMedium).Should(Equal(corev1.PodRunning))
+                       Eventually(IntegrationConditionStatus(ns, 
"resource-file-route", v1.IntegrationConditionReady), 
TestTimeoutShort).Should(Equal(corev1.ConditionTrue))
+                       Eventually(IntegrationLogs(ns, "resource-file-route"), 
TestTimeoutShort).Should(ContainSubstring("the file body"))
+                       Expect(Kamel("delete", "--all", "-n", 
ns).Execute()).To(Succeed())
+                       Eventually(AutogeneratedConfigmapsCount(ns), 
TestTimeoutShort).Should(Equal(0))
+               })
+
                // Build-Properties
                t.Run("Build time property", func(t *testing.T) {
                        Expect(Kamel("run", "-n", ns, 
"./files/build-property-route.groovy", "--build-property", 
"quarkus.application.name=my-super-application").Execute()).To(Succeed())
diff --git a/e2e/common/config/files/resources-data-same.txt 
b/e2e/common/config/files/resources-data-same.txt
new file mode 100644
index 0000000..dff7947
--- /dev/null
+++ b/e2e/common/config/files/resources-data-same.txt
@@ -0,0 +1 @@
+the file body
\ No newline at end of file
diff --git a/pkg/trait/mount.go b/pkg/trait/mount.go
index 2159692..6c8976a 100644
--- a/pkg/trait/mount.go
+++ b/pkg/trait/mount.go
@@ -183,14 +183,12 @@ func (t *mountTrait) attachResource(e *Environment, conf 
*utilResource.Config) {
 
 func (t *mountTrait) mountResource(vols *[]corev1.Volume, mnts 
*[]corev1.VolumeMount, conf *utilResource.Config) {
        refName := kubernetes.SanitizeLabel(conf.Name())
-       dstDir := ""
+       dstDir := conf.DestinationPath()
        dstFile := ""
        if conf.DestinationPath() != "" {
                if conf.Key() != "" {
-                       dstDir = filepath.Dir(conf.DestinationPath())
                        dstFile = filepath.Base(conf.DestinationPath())
                } else {
-                       dstDir = conf.DestinationPath()
                        dstFile = conf.Key()
                }
        }
@@ -200,7 +198,7 @@ func (t *mountTrait) mountResource(vols *[]corev1.Volume, 
mnts *[]corev1.VolumeM
        if conf.StorageType() == utilResource.StorageTypePVC {
                readOnly = false
        }
-       mnt := getMount(refName, mntPath, "", readOnly)
+       mnt := getMount(refName, mntPath, dstFile, readOnly)
 
        *vols = append(*vols, *vol)
        *mnts = append(*mnts, *mnt)

Reply via email to