Merge pull request #17 from amoniacou/release/v1.2.7

simonoff · web-flow · commit 05c3f0106583 · 2024-09-17T17:11:44.000+03:00
Release/v1.2.7
diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -7,6 +7,11 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 
 Generated by [`auto-changelog`](https://github.com/CookPete/auto-changelog).
 
+## 1.2.7
+
+- fix saving status of domain on mailgun
+- replacing DNS records to understand what mailgun is seeing
+
 ## 1.2.6
 
 - more optimizations for status update and reconciling only when we need it
diff --git a/internal/controller/domain/domain_controller.go b/internal/controller/domain/domain_controller.go
@@ -66,7 +66,9 @@ func (r *DomainReconciler) Reconcile(ctx context.Context, req ctrl.Request) (ctr
 
 	// lookup for item
 	if err := r.Get(ctx, req.NamespacedName, mailgunDomain); err != nil {
-		log.Error(err, "unable to fetch Domain")
+		if client.IgnoreNotFound(err) != nil {
+			log.Error(err, "unable to fetch Domain")
+		}
 		return ctrl.Result{}, client.IgnoreNotFound(err)
 	}
 
diff --git a/releases/1.2.7/kustomization.yaml b/releases/1.2.7/kustomization.yaml
@@ -0,0 +1,5 @@
+apiVersion: kustomize.config.k8s.io/v1beta1
+kind: Kustomization
+
+resources:
+  - manager.yaml
diff --git a/releases/1.2.7/manager.yaml b/releases/1.2.7/manager.yaml

Original file line number	Diff line number	Diff line change
`@@ -66,7 +66,9 @@ func (r *DomainReconciler) Reconcile(ctx context.Context, req ctrl.Request) (ctr`
`66`	`66`
`67`	`67`	`// lookup for item`
`68`	`68`	`if err := r.Get(ctx, req.NamespacedName, mailgunDomain); err != nil {`
`69`		`- log.Error(err, "unable to fetch Domain")`
	`69`	`+ if client.IgnoreNotFound(err) != nil {`
	`70`	`+ log.Error(err, "unable to fetch Domain")`
	`71`	`+ }`
`70`	`72`	`return ctrl.Result{}, client.IgnoreNotFound(err)`
`71`	`73`	`}`
`72`	`74`