-
Notifications
You must be signed in to change notification settings - Fork 338
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
feat: expose more prometheus metrics #670
Conversation
… codes, also fix TLS typo
d76168b
to
bbe13c9
Compare
@tokers done. |
@Sindweller Please no longer force push, you can just use merge (from master). |
@Sindweller Changes LGTM for me but some CI failures. |
please check the lint and unit test errors |
Should have resolved the error in unit-test. It's due to the lack of cluster
It seems my changes may not affect these files and cause the error. |
That's right, it's due to the speed of GitHub action, we have some plans to optimize it, for now, please just retry it. |
Since #627 has been merged, please do with conflicting files. |
I'm used to using
|
Codecov Report
@@ Coverage Diff @@
## master #670 +/- ##
==========================================
+ Coverage 31.70% 32.21% +0.50%
==========================================
Files 66 66
Lines 6640 6768 +128
==========================================
+ Hits 2105 2180 +75
- Misses 4280 4333 +53
Partials 255 255
Continue to review full report at Codecov.
|
You don't need release, just merge the master branch is OK, don't be overwhelmed by the extra Merge commit, we use squash merge strategy so finally only a commit will be merged into the master branch and the tree will be clean. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
panic: runtime error: invalid memory address or nil pointer dereference
[signal SIGSEGV: segmentation violation code=0x1 addr=0x48 pc=0x155bc1b]
goroutine 680 [running]:
github.com/apache/apisix-ingress-controller/pkg/apisix.(*cluster).getList(0xc0004f1680, 0x1c687d0, 0xc000044118, 0xc0003873b0, 0x30, 0x19bdf54, 0x6, 0x0, 0x0, 0x0, ...)
/home/tao/go/src/github.com/apache/apisix-ingress-controller/pkg/apisix/cluster.go:711 +0x19b
github.com/apache/apisix-ingress-controller/pkg/apisix.(*pluginClient).List(0xc0005c1440, 0x1c687d0, 0xc000044118, 0x0, 0x0, 0x0, 0x0, 0x2)
/home/tao/go/src/github.com/apache/apisix-ingress-controller/pkg/apisix/plugin.go:44 +0x3b9
github.com/apache/apisix-ingress-controller/pkg/apisix.(*cluster).syncSchemaOnce(0xc0004f1680, 0x1c687d0, 0xc000044118, 0x0, 0xc0008af798)
/home/tao/go/src/github.com/apache/apisix-ingress-controller/pkg/apisix/cluster.go:365 +0x74f
github.com/apache/apisix-ingress-controller/pkg/apisix.(*cluster).syncSchema(0xc0004f1680, 0x1c687d0, 0xc000044118, 0x13a52453c000)
/home/tao/go/src/github.com/apache/apisix-ingress-controller/pkg/apisix/cluster.go:329 +0xa5
created by github.com/apache/apisix-ingress-controller/pkg/apisix.newCluster
/home/tao/go/src/github.com/apache/apisix-ingress-controller/pkg/apisix/cluster.go:150 +0x852
Please fix the e2e test case.
you can run
for local test. |
thx. |
hi, this problem is caused by |
This is not the logic of E2E test cases but the apisix-ingress-controller itself, see https://github.com/apache/apisix-ingress-controller/blob/master/pkg/ingress/controller.go#L382 for the details. |
CI failed |
Here is the CI log:
Could you help to resolve error |
Could you merge master branch? |
Already done. Here is the
|
It's Green! |
@Sindweller Merged, thanks for your great job! |
Please answer these questions before submitting a pull request
Why submit this pull request?
Bugfix
New feature provided
Improve performance
Backport patches
Related issues
New feature or improvement
Expose several metrics for apisix-ingress-controller. This PR has a prior PR: feat: add metric 'check_cluster_health_total' and 'sync_operation_total' #627
HELP apisix_ingress_controller_is_leader Whether the role of controller instance is leader
TYPE apisix_ingress_controller_is_leader guage
Labels:
HELP apisix_ingress_controller_apisix_bad_status_codes Bad status codes of requests to APISIX
TYPE apisix_ingress_controller_apisix_bad_status_codes guage
Labels:
HELP apisix_ingress_controller_apisix_request_latencies Request latencies with APISIX
TYPE apisix_ingress_controller_apisix_request_latencies summary
Labels:
HELP apisix_ingress_controller_apisix_requests Number of requests to APISIX
TYPE apisix_ingress_controller_apisix_requests counter
Labels:
HELP apisix_ingress_controller_check_cluster_health_total Number of cluster health check operations
TYPE apisix_ingress_controller_check_cluster_health_total counter
Labels:
HELP apisix_ingress_controller_sync_operation_total Number of sync operations
TYPE apisix_ingress_controller_sync_operation_total counter
Labels:
HELP apisix_ingress_controller_cache_sync_total Number of cache sync operations
TYPE apisix_ingress_controller_cache_sync_total counter
Labels:
HELP apisix_ingress_controller_events_total Number of events handled by the controller
TYPE apisix_ingress_controller_events_total counter
Labels: