Skip to content
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.

Sign up for GitHub

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

Jump to bottom

chore: revert removal of CLI report tool #13002

Merged
merged 1 commit into from May 7, 2024
Merged

chore: revert removal of CLI report tool #13002

merged 1 commit into from May 7, 2024

Conversation

lpetrovic05
Copy link
Member

closes #13000

@lpetrovic05
Revert removal of CLI report tool
607900d 
This reverts part of the commit 39c2254.

Signed-off-by: Lazar Petrovic <lpetrovic05@gmail.com>
@lpetrovic05 lpetrovic05 added this to the v0.50 milestone Apr 25, 2024
@lpetrovic05 lpetrovic05 self-assigned this Apr 25, 2024
@lpetrovic05 lpetrovic05 requested review from a team as code owners April 25, 2024 13:10
@github-actions GitHub Actions
Copy link

Node: HAPI Test (Restart) Results

2 tests   2 ✅  5m 30s ⏱️
2 suites  0 💤
2 files    0 ❌

Results for commit 607900d.

@github-actions GitHub Actions
Copy link

Node: HAPI Test (Node Death Reconnect) Results

1 tests   1 ✅  23s ⏱️
1 suites  0 💤
2 files    0 ❌
1 errors

For more details on these parsing errors, see this check.

Results for commit 607900d.

@github-actions GitHub Actions
Copy link

Node: HAPI Test (Token) Results

228 tests   227 ✅  19m 4s ⏱️
 16 suites    1 💤
 16 files      0 ❌

Results for commit 607900d.

@github-actions GitHub Actions
Copy link

Node: HAPI Test (Crypto) Results

335 tests   335 ✅  37m 42s ⏱️
 25 suites    0 💤
 25 files      0 ❌

Results for commit 607900d.

@github-actions GitHub Actions
Copy link

Node: HAPI Test (Misc) Results

457 tests   447 ✅  42m 39s ⏱️
 76 suites   10 💤
 76 files      0 ❌

Results for commit 607900d.

@github-actions GitHub Actions
Copy link

Node: HAPI Test (Time Consuming) Results

21 tests   21 ✅  54m 35s ⏱️
 3 suites   0 💤
 3 files     0 ❌

Results for commit 607900d.

@github-actions GitHub Actions
Copy link

Node: Unit Test Results

  2 268 files  +1    2 268 suites  +1   3h 6m 23s ⏱️ + 20m 44s
112 315 tests +2  112 249 ✅ +2  66 💤 ±0  0 ❌ ±0 
120 778 runs  +2  120 712 ✅ +2  66 💤 ±0  0 ❌ ±0 

Results for commit 607900d. ± Comparison against base commit a24e0d3.

This pull request removes 3982 and adds 3747 tests. Note that renamed tests count towards both. 

  
             IssuerDN: CN=s-aaaa
            SubjectDN: CN=s-aaaa
           Final Date: Fri Jan 01 00:00:00 UTC 2100
           Public Key: RSA Public Key [2e:28:bc:1e:d3:83:25:92:8e:cb:98:b1:b6:84:06:9c:d5:d8:14:d5],[56:66:d1:a4]
           Start Date: Sat Jan 01 00:00:00 UTC 2000
         SerialNumber: 12482092706667292405
        modulus: c1a0ff5d2372b53d12d12bb87dd03f5e…
        modulus: c1a0ff5d2372b53d12d12bb87dd03f5…
…

com.hedera.node.app.grpc.impl.netty.GrpcServiceBuilderTest ‑ [4] 

com.hedera.node.app.grpc.impl.netty.GrpcServiceBuilderTest ‑ [6] 

com.hedera.node.app.grpc.impl.netty.GrpcServiceBuilderTest ‑ [7]   
  
com.hedera.node.app.service.mono.state.codec.VirtualKeySerdesAdapterTest ‑ [10] com.hedera.node.app.service.mono.state.codec.VirtualBlobKey@838beb0a
com.hedera.node.app.service.mono.state.codec.VirtualKeySerdesAdapterTest ‑ [11] com.hedera.node.app.service.mono.state.codec.VirtualBlobKey@ddff4428
com.hedera.node.app.service.mono.state.codec.VirtualKeySerdesAdapterTest ‑ [12] com.hedera.node.app.service.mono.state.codec.VirtualBlobKey@85649fbd
com.hedera.node.app.service.mono.state.codec.VirtualKeySerdesAdapterTest ‑ [13] com.hedera.node.app.service.mono.state.codec.VirtualBlobKey@5243a3f6
com.hedera.node.app.service.mono.state.codec.VirtualKeySerdesAdapterTest ‑ [14] com.hedera.node.app.service.mono.state.codec.VirtualBlobKey@55d637e
com.hedera.node.app.service.mono.state.codec.VirtualKeySerdesAdapterTest ‑ [15] com.hedera.node.app.service.mono.state.codec.VirtualBlobKey@7240c2df
com.hedera.node.app.service.mono.state.codec.VirtualKeySerdesAdapterTest ‑ [16] com.hedera.node.app.service.mono.state.codec.VirtualBlobKey@352880ee
…

@github-actions GitHub Actions
Copy link

Node: HAPI Test (Smart Contract) Results

579 tests   578 ✅  1h 9m 37s ⏱️
 62 suites    0 💤
 62 files      1 ❌

For more details on these failures, see this check.

Results for commit 607900d.

mxtartaglia-sl
mxtartaglia-sl approved these changes May 6, 2024
View reviewed changes
Copy link
Contributor

@mxtartaglia-sl mxtartaglia-sl left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

approved for BaseTeam owned files

...-platform-test/src/test/java/com/swirlds/platform/test/cli/EventStreamReportingToolTest.java Show resolved Hide resolved
@lpetrovic05 lpetrovic05 merged commit e9f858e into develop May 7, 2024
53 of 57 checks passed
@lpetrovic05 lpetrovic05 deleted the 13000-D-revert-removal-of-CES-info-command branch May 7, 2024 09:25
stoyanov-st pushed a commit that referenced this pull request May 15, 2024
@lpetrovic05 @stoyanov-st
chore: revert removal of CLI report tool (#13002)
fab7a71 
Signed-off-by: Lazar Petrovic <lpetrovic05@gmail.com>
Signed-off-by: Stanimir Stoyanov <stanimir.stoyanov@limechain.tech>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@jeromy-cannon jeromy-cannon jeromy-cannon approved these changes

@cody-littley cody-littley cody-littley approved these changes

@mxtartaglia-sl mxtartaglia-sl mxtartaglia-sl approved these changes

@kfa-aguda kfa-aguda Awaiting requested review from kfa-aguda kfa-aguda is a code owner automatically assigned from hashgraph/platform-hashgraph

@timo0 timo0 Awaiting requested review from timo0 timo0 is a code owner automatically assigned from hashgraph/platform-base

@isavov isavov Awaiting requested review from isavov isavov is a code owner automatically assigned from hashgraph/release-engineering

@leninmehedy leninmehedy Awaiting requested review from leninmehedy leninmehedy is a code owner automatically assigned from hashgraph/release-engineering-managers

Assignees

@lpetrovic05 lpetrovic05

Labels
None yet
Projects
None yet
Milestone
v0.50
Development

Successfully merging this pull request may close these issues.

Revert removal of CES info command
4 participants
@lpetrovic05 @jeromy-cannon @cody-littley @mxtartaglia-sl