-
Notifications
You must be signed in to change notification settings - Fork 7
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(network node livecheck): implement a functionality to determine network node activeness with prometheus metrics #567
Conversation
…ode root container prometheus metrics Signed-off-by: instamenta <instamenta@abv.bg>
Unit Test Results - Linux 1 files 19 suites 28s ⏱️ Results for commit 52aef25. ♻️ This comment has been updated with latest results. |
Unit Test Results - Windows 1 files 19 suites 53s ⏱️ Results for commit 52aef25. ♻️ This comment has been updated with latest results. |
E2E Relay Tests Coverage Report1 files 1 suites 2m 41s ⏱️ Results for commit 52aef25. ♻️ This comment has been updated with latest results. |
E2E Node PEM Kill Tests Coverage Report13 tests 13 ✅ 3m 16s ⏱️ Results for commit 52aef25. ♻️ This comment has been updated with latest results. |
E2E Node PEM Stop Tests Coverage Report13 tests 13 ✅ 3m 24s ⏱️ Results for commit 52aef25. ♻️ This comment has been updated with latest results. |
E2E Node Local Build Tests Coverage Report10 tests 10 ✅ 2m 22s ⏱️ Results for commit 52aef25. ♻️ This comment has been updated with latest results. |
E2E Mirror Node Tests Coverage Report 1 files 1 suites 3m 42s ⏱️ Results for commit 52aef25. ♻️ This comment has been updated with latest results. |
E2E Tests Coverage Report58 tests 58 ✅ 3m 39s ⏱️ Results for commit 49159ac. |
E2E Node Update Tests Coverage Report12 tests 12 ✅ 5m 10s ⏱️ Results for commit 52aef25. ♻️ This comment has been updated with latest results. |
E2E Node Delete Tests Coverage Report10 tests 10 ✅ 6m 12s ⏱️ Results for commit 52aef25. ♻️ This comment has been updated with latest results. |
E2E Node Add Tests Coverage Report22 tests 22 ✅ 12m 1s ⏱️ Results for commit 52aef25. ♻️ This comment has been updated with latest results. |
… healthchekcs Signed-off-by: instamenta <instamenta@abv.bg>
Signed-off-by: instamenta <instamenta@abv.bg>
E2E Standard Tests Coverage Report 1 files 9 suites 4m 29s ⏱️ Results for commit 52aef25. ♻️ This comment has been updated with latest results. |
Coverage summary from CodacySee diff coverage on Codacy
Coverage variation details
Coverage variation is the difference between the coverage for the head and common ancestor commits of the pull request branch: Diff coverage details
Diff coverage is the percentage of lines that are covered by tests out of the coverable lines that the pull request added or modified: See your quality gate settings Change summary preferencesCodacy stopped sending the deprecated coverage status on June 5th, 2024. Learn more Footnotes
|
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## main #567 +/- ##
==========================================
- Coverage 82.54% 81.77% -0.78%
==========================================
Files 36 37 +1
Lines 4720 4767 +47
Branches 674 679 +5
==========================================
+ Hits 3896 3898 +2
- Misses 824 869 +45
|
Signed-off-by: instamenta <instamenta@abv.bg>
Signed-off-by: instamenta <instamenta@abv.bg>
Signed-off-by: instamenta <instamenta@abv.bg>
## [0.30.0](v0.29.0...v0.30.0) (2024-09-17) ### Features * add node add-prepare and node add-execute commands ([#533](#533)) ([a184775](a184775)) * add support for attach jvm debugger ([#549](#549)) ([35a0075](35a0075)) * add the autogen tool to easily add e2e tests and run it for the first time & fix copyFrom ([#566](#566)) ([e6c79eb](e6c79eb)) * **network node livecheck:** implement a functionality to determine network node activeness with prometheus metrics ([#567](#567)) ([6120775](6120775)) * remove pfx support ([#537](#537)) ([aff5f63](aff5f63)) ### Bug Fixes * aligned test names to naming pattern ([#554](#554)) ([365f724](365f724)) * download application.properties file if it was generated by hedera app during handling address book change ([#571](#571)) ([35c73d8](35c73d8)) * **functionality:** removed the 'z' from the 'tar' command on the K8.copyFrom ([#561](#561)) ([796474b](796474b)) * have update readme run after version calculation in deploy and release ([#586](#586)) ([42e1685](42e1685)) * jsdoc errors due to unrecognized format ([#557](#557)) ([da7113b](da7113b)) * remove of using agreement key in hedera repo test ([#563](#563)) ([05c5517](05c5517)) * separate repeated steps as individual functions ([#562](#562)) ([dcbbe5e](dcbbe5e))
🎉 This PR is included in version 0.30.0 🎉 The release is available on: Your semantic-release bot 📦🚀 |
Description
Implemented functionality that utilizes the node root container prometheus metrics endpoint to determine node's activeness
Related Issues