Skip to content

fix: status updater prints the kind of ApisixUpstream as unknown#2652

Closed
bzp2010 wants to merge 1 commit intomasterfrom
bzp/fix-au-status-updater-kind-unknown
Closed

fix: status updater prints the kind of ApisixUpstream as unknown#2652
bzp2010 wants to merge 1 commit intomasterfrom
bzp/fix-au-status-updater-kind-unknown

Conversation

@bzp2010
Copy link
Copy Markdown
Contributor

@bzp2010 bzp2010 commented Nov 14, 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:

https://the-asf.slack.com/archives/CUC5MN17A/p1763019082783669

https://github.com/apache/apisix-ingress-controller/blob/master/internal/controller/status/updater.go#L124

The types.KindOf function does not include the ApisixUpstream type, causing the state updater's logs to print its kind as Unknown.

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

@ronething
Copy link
Copy Markdown
Contributor

#2622 resolved the same problem.

@bzp2010
Copy link
Copy Markdown
Contributor Author

bzp2010 commented Nov 14, 2025

@ronething Then we ought to merge that pull request as soon as possible; it has already been approved.

@bzp2010 bzp2010 closed this Nov 14, 2025
@bzp2010 bzp2010 deleted the bzp/fix-au-status-updater-kind-unknown branch November 14, 2025 01:54
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.

2 participants