fix(webhook): support relative parent paths

padok-team · Jul 10, 2023 · ab53e48 · ab53e48
1 parent ece5804
commit ab53e48
Show file tree

Hide file tree

Showing 4 changed files with 15 additions and 13 deletions.
diff --git a/internal/webhook/event/common.go b/internal/webhook/event/common.go
@@ -67,6 +67,8 @@ func layerFilesHaveChanged(layer configv1alpha1.TerraformLayer, changedFiles []s
 		if val, ok := layer.Annotations[annotations.AdditionnalTriggerPaths]; ok {
 			for _, p := range strings.Split(val, ",") {
 				p = ensureAbsPath(p)
+				// Handle relative parent paths (like "../")
+				p = filepath.Clean(filepath.Join(layer.Spec.Path, p))
 				if strings.Contains(f, p) {
 					return true
 				}

diff --git a/internal/webhook/event/event_test.go b/internal/webhook/event/event_test.go
@@ -200,17 +200,17 @@ var _ = Describe("Webhook", func() {
 					Expect(layer.Annotations[annotations.LastRelevantCommit]).To(Equal(PushEventAdditionalPathChanges.ChangeInfo.ShaAfter))
 				})
 				// TODO: make this test pass
-				// It("should have updated commit annotations for a relative change path", func() {
-				// 	layer := &configv1alpha1.TerraformLayer{}
-				// 	err := k8sClient.Get(context.TODO(), types.NamespacedName{
-				// 		Namespace: "default",
-				// 		Name:      "layer-additional-paths-2",
-				// 	}, layer)
-				// 	Expect(err).NotTo(HaveOccurred())
-				// 	Expect(handleErr).NotTo(HaveOccurred())
-				// 	Expect(layer.Annotations[annotations.LastBranchCommit]).To(Equal(PushEventAdditionalPathChanges.ChangeInfo.ShaAfter))
-				// 	Expect(layer.Annotations[annotations.LastRelevantCommit]).To(Equal(PushEventAdditionalPathChanges.ChangeInfo.ShaAfter))
-				// })
+				It("should have updated commit annotations for a relative change path", func() {
+					layer := &configv1alpha1.TerraformLayer{}
+					err := k8sClient.Get(context.TODO(), types.NamespacedName{
+						Namespace: "default",
+						Name:      "layer-additional-paths-2",
+					}, layer)
+					Expect(err).NotTo(HaveOccurred())
+					Expect(handleErr).NotTo(HaveOccurred())
+					Expect(layer.Annotations[annotations.LastBranchCommit]).To(Equal(PushEventAdditionalPathChanges.ChangeInfo.ShaAfter))
+					Expect(layer.Annotations[annotations.LastRelevantCommit]).To(Equal(PushEventAdditionalPathChanges.ChangeInfo.ShaAfter))
+				})
 			})
 			Describe("Multiple paths have been modified", Ordered, func() {
 				BeforeAll(func() {

diff --git a/internal/webhook/event/pullrequest.go b/internal/webhook/event/pullrequest.go
@@ -102,7 +102,7 @@ func (e *PullRequestEvent) getAffectedRepositories(repositories []configv1alpha1
 	affectedRepositories := []configv1alpha1.TerraformRepository{}
 	for _, repo := range repositories {
 		log.Infof("evaluating terraform repository %s for url %s", repo.Name, repo.Spec.Repository.Url)
-		log.Infof("comparing noramlized url %s with received URL from paylaod %s", NormalizeUrl(repo.Spec.Repository.Url), e.URL)
+		log.Infof("comparing normalized url %s with received URL from payload %s", NormalizeUrl(repo.Spec.Repository.Url), e.URL)
 		if e.URL == NormalizeUrl(repo.Spec.Repository.Url) {
 			affectedRepositories = append(affectedRepositories, repo)
 		}

diff --git a/internal/webhook/event/testdata/layers.yaml b/internal/webhook/event/testdata/layers.yaml
@@ -117,7 +117,7 @@ metadata:
   name: layer-additional-paths-2
   namespace: default
   annotations:
-    config.terraform.padok.cloud/additionnal-trigger-paths: "../module.hcl"
+    config.terraform.padok.cloud/additionnal-trigger-paths: "../../terragrunt.hcl,../module.hcl"
 spec:
   branch: main
   path: terragrunt/layer-path-changed/prod/