-
Notifications
You must be signed in to change notification settings - Fork 2.8k
remove not used plugin heartbeat.lua #1845
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
Conversation
|
@mtx2d please take a look at the output of CI, eg: https://github.com/apache/incubator-apisix/pull/1845/checks?check_run_id=871655239#step:6:272 |
Cannot run the test locally, so running this with CI. |
|
Trying to figure out how to trigger CI. |
Seems like the CI picks it up now. |
|
The CI fails due to CI server network issue: https://github.com/apache/incubator-apisix/pull/1845/checks?check_run_id=874200614#step:3:68 Will make a small change and trigger another CI run. |
|
According to this doc, we can close and reopen, which will trigger a new CI run without making changes. |
|
Hi @membphis, |
moonming
left a comment
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.
LGTM
|
@mtx2d many thx, merged already |
Fix #1176
Pre-submission checklist:
Please answer each of these after submitting your pull request:
change: remove the plugin
heartbeat#1776 Says that heartbeat is no longer needed, we can remove this.Checked that there is not test cases related to heartbeat.
Removing this file does not break existing test suit.
Link to document including
heartbeart.There are three instances, two of them in the change log, and last one discussing in general service discovery.
No need to modify any one of them.
Not sure how to check this.