Skip to content

25128 reapply latency metrics in nlg#25190

Open
alex-kuzmin-hg wants to merge 2 commits intomainfrom
25128-reapply-latency-metrics-in-NLG
Open

25128 reapply latency metrics in nlg#25190
alex-kuzmin-hg wants to merge 2 commits intomainfrom
25128-reapply-latency-metrics-in-NLG

Conversation

@alex-kuzmin-hg
Copy link
Copy Markdown
Contributor

Description:

Related issue(s):

Fixes #25128

Notes for reviewer:

Checklist

  • Documented (Code comments, README, etc.)
  • Tested (unit, integration, etc.)

Signed-off-by: Alex Kuzmin <alex.kuzmin@swirldslabs.com>
Signed-off-by: Alex Kuzmin <alex.kuzmin@swirldslabs.com>
@alex-kuzmin-hg alex-kuzmin-hg added this to the v0.75 milestone Apr 30, 2026
@alex-kuzmin-hg alex-kuzmin-hg self-assigned this Apr 30, 2026
@alex-kuzmin-hg alex-kuzmin-hg requested a review from a team as a code owner April 30, 2026 17:15
@trunk-io
Copy link
Copy Markdown

trunk-io Bot commented Apr 30, 2026

Merging to main in this repository is managed by Trunk.

  • To merge this pull request, check the box to the left or comment /trunk merge below.

After your PR is submitted to the merge queue, this comment will be automatically updated with its status. If the PR fails, failure details will also be posted here

@lfdt-bot
Copy link
Copy Markdown

Snyk checks have passed. No issues have been found so far.

Status Scan Engine Critical High Medium Low Total (0)
Open Source Security 0 0 0 0 0 issues

💻 Catch issues earlier using the plugins for VS Code, JetBrains IDEs, Visual Studio, and Eclipse.

@codacy-production
Copy link
Copy Markdown

codacy-production Bot commented Apr 30, 2026

Up to standards ✅

🟢 Issues 0 issues

Results:
0 new issues

View in Codacy

🟢 Coverage ∅ diff coverage · +0.00% coverage variation

Metric Results
Coverage variation +0.00% coverage variation (-1.00%)
Diff coverage diff coverage

View coverage diff in Codacy

Coverage variation details
Coverable lines Covered lines Coverage
Common ancestor commit (cb0859c) 99965 78703 78.73%
Head commit (c05c5fe) 99965 (+0) 78703 (+0) 78.73% (+0.00%)

Coverage variation is the difference between the coverage for the head and common ancestor commits of the pull request branch: <coverage of head commit> - <coverage of common ancestor commit>

Diff coverage details
Coverable lines Covered lines Diff coverage
Pull request (#25190) 0 0 ∅ (not applicable)

Diff coverage is the percentage of lines that are covered by tests out of the coverable lines that the pull request added or modified: <covered lines added or modified>/<coverable lines added or modified> * 100%

NEW Get contextual insights on your PRs based on Codacy's metrics, along with PR and Jira context, without leaving GitHub. Enable AI reviewer
TIP This summary will be updated as you push new changes.

@codecov
Copy link
Copy Markdown

codecov Bot commented Apr 30, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.

Impacted file tree graph

@@            Coverage Diff            @@
##               main   #25190   +/-   ##
=========================================
  Coverage     74.90%   74.90%           
  Complexity    11782    11782           
=========================================
  Files          2587     2587           
  Lines        100062   100062           
  Branches      11058    11058           
=========================================
  Hits          74954    74954           
  Misses        21309    21309           
  Partials       3799     3799           

Impacted file tree graph

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@github-actions
Copy link
Copy Markdown

Flaky Test(s) Detected

One or more flaky tests were detected in this run. These tests have been reported before.

Test Ticket
com.hedera.services.bdd.suites.validation.LogValidationTest#logsContainNoUnexpectedProblems #25154

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.

To remove NLG option -L from 0.74 release

2 participants