Merge pull request #100 from alibaba/tengine-ingress-3.1.0

HTTP Route: canary ingress without referrer is valid
alibaba · Oct 24, 2023 · 6e5b6ea · 6e5b6ea
2 parents 89470a6 + 0bb2d88
commit 6e5b6ea
Show file tree

Hide file tree

Showing 3 changed files with 2 additions and 7 deletions.
diff --git a/internal/checksum/ingress/apis/checksum/v1/doc.go b/internal/checksum/ingress/apis/checksum/v1/doc.go
@@ -21,4 +21,4 @@ limitations under the License.
 // +groupName=tengine.taobao.org
 
 // Package v1 holds api types of version v1 for group "tengine.taobao.org".
-package v1 // import "tengine.taobao.org/checksum/ingress/apis/checksum/v1"
+package v1 // import "k8s.io/ingress-nginx/internal/checksum/ingress/apis/checksum/v1"
diff --git a/internal/checksum/secret/apis/checksum/v1/doc.go b/internal/checksum/secret/apis/checksum/v1/doc.go
@@ -21,4 +21,4 @@ limitations under the License.
 // +groupName=tengine.taobao.org
 
 // Package v1 holds api types of version v1 for group "tengine.taobao.org".
-package v1 // import "tengine.taobao.org/checksum/secret/apis/checksum/v1"
+package v1 // import "k8s.io/ingress-nginx/internal/checksum/secret/apis/checksum/v1"
diff --git a/internal/ingress/annotations/canary/main.go b/internal/ingress/annotations/canary/main.go
@@ -235,10 +235,5 @@ func (c canary) Parse(ing *networking.Ingress) (interface{}, error) {
 		return nil, errors.NewInvalidAnnotationConfiguration("canary", "configured but not enabled")
 	}
 
-	if config.Enabled && len(config.Referrer) == 0 {
-		klog.Warningf("Canary ingress[%v/%v] with empty referrer, ignored", ing.Namespace, ing.Name)
-		return nil, errors.NewInvalidAnnotationConfiguration("canary", "referrer is empty")
-	}
-
 	return config, nil
 }