Skip to content

fix(ApisixRoute): failed to delete stream_route#2624

Merged
AlinsRan merged 1 commit intoapache:masterfrom
AlinsRan:fix/streamroute-unable-del
Oct 29, 2025
Merged

fix(ApisixRoute): failed to delete stream_route#2624
AlinsRan merged 1 commit intoapache:masterfrom
AlinsRan:fix/streamroute-unable-del

Conversation

@AlinsRan
Copy link
Copy Markdown
Contributor

@AlinsRan AlinsRan commented Oct 27, 2025

Type of change:

  • Bugfix
  • New feature provided
  • Improve performance
  • Backport patches
  • Documentation
  • Refactor
  • Chore
  • CI/CD or Tests

What this PR does / why we need it:

helm install -n ingress-apisix apisix apisix/apisix \
  --namespace ingress-apisix \
  --create-namespace \
  --set apisix.deployment.role=traditional \
  --set apisix.deployment.role_traditional.config_provider=yaml \
  --set etcd.enabled=false \
  --set ingress-controller.enabled=true \
  --set ingress-controller.config.provider.type=apisix-standalone \
  --set ingress-controller.apisix.adminService.namespace=ingress-apisix \
  --set ingress-controller.gatewayProxy.createDefault=true \
  --set "service.stream.enabled=true" \
  --set "service.stream.tcp[0]=9100"

kubectl apply:

apiVersion: apisix.apache.org/v2
kind: ApisixRoute
metadata:
  name: stream-route-httpbin
spec:
  ingressClassName: apisix
  stream:
    - name: stream-route-httpbin
      protocol: TCP
      match:
        ingressPort: 9100
      backend:
        serviceName: httpbin
        servicePort: 80
image

expect stream-route-httpbin deleted.

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

@AlinsRan AlinsRan changed the title fix(ApisixRoute): failed delete stream_route fix(ApisixRoute): failed to delete stream_route Oct 29, 2025
@AlinsRan AlinsRan merged commit 36a3196 into apache:master Oct 29, 2025
28 of 35 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants