fix: ApisixRouteEvent type assertion

[tao12345666333]

Signed-off-by: Jintao Zhang zhangjintao9020@gmail.com

Type of change:

• Bugfix
• New feature provided
• Improve performance
• Backport patches

What this PR does / why we need it:

fixes #924

Pre-submission checklist:

• Did you explain what problem does this PR solve? Or what new features have been added?
• Have you added corresponding test cases?
• Have you modified the corresponding document?
• Is this PR backward compatible? If it is not backward compatible, please discuss on the mailing list first

[nayihz]

apisix-ingress-controller/pkg/ingress/apisix_route.go

Lines 261 to 271 in c9e0c96

	func (c *apisixRouteController) handleSyncErr(obj interface{}, errOrigin error) {
	ev := obj.(*types.Event)
	event := ev.Object.(kube.ApisixRouteEvent)
	if k8serrors.IsNotFound(errOrigin) && ev.Type != types.EventDelete {
	log.Infow("sync ApisixRoute but not found, ignore",
	zap.String("event_type", ev.Type.String()),
	zap.String("ApisixRoute", ev.Object.(string)),
	)
	c.workqueue.Forget(event)
	return
	}

We will delete this event from workqueue if we have an ApisixRoute resources without any valid backends. It is reasonable?

[tao12345666333]

apisix-ingress-controller/pkg/ingress/apisix_route.go

Lines 261 to 271 in c9e0c96

	func (c *apisixRouteController) handleSyncErr(obj interface{}, errOrigin error) {
	ev := obj.(*types.Event)
	event := ev.Object.(kube.ApisixRouteEvent)
	if k8serrors.IsNotFound(errOrigin) && ev.Type != types.EventDelete {
	log.Infow("sync ApisixRoute but not found, ignore",
	zap.String("event_type", ev.Type.String()),
	zap.String("ApisixRoute", ev.Object.(string)),
	)
	c.workqueue.Forget(event)
	return
	}

We will delete this event from workqueue if we have an ApisixRoute resources without any valid backends. It is reasonable?

@cmssczy This change is to fix the #810 and was introduced in #893

This does cause some problems.

For example, create an ApisixRoute first, which references a service that does not exist.
But when a Service is created, the APISIX Ingress controller can't actually create a route.

[nayihz]

But when a Service is created, the APISIX Ingress controller can't actually create a route

That's it.

[tao12345666333]

@cmssczy I created #926 to track it, let's discuss it in #926