Skip to content

chore(block-node): validate version v0.62.0 works with block node #1941

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
Merged
merged 32 commits into from
May 19, 2025
Merged

chore(block-node): validate version v0.62.0 works with block node #1941

merged 32 commits into from
May 19, 2025

Conversation

instamenta
Copy link
Contributor

Description

Validates that block node is working correctly with version v0.62.0

@instamenta
changed version to v0.62.0
ad073da 
Signed-off-by: Zhan Milenkov <[email protected]>
@instamenta instamenta self-assigned this May 7, 2025
@instamenta instamenta requested review from a team as code owners May 7, 2025 11:31
@github-actions GitHub Actions
Copy link
Contributor

github-actions bot commented May 7, 2025 •
edited
Loading

Unit Test Results - Linux

32 tests  ±0   32 ✅ ±0   0s ⏱️ ±0s
16 suites ±0    0 💤 ±0 
 1 files   ±0    0 ❌ ±0 

Results for commit e66cd42. ± Comparison against base commit 98bae09.

♻️ This comment has been updated with latest results.

@github-actions GitHub Actions
Copy link
Contributor

github-actions bot commented May 7, 2025 •
edited
Loading

Unit Test Results - Windows

  1 files  ± 0   85 suites  +3   2s ⏱️ -2s
360 tests +22  360 ✅ +22  0 💤 ±0  0 ❌ ±0 
365 runs  +22  365 ✅ +22  0 💤 ±0  0 ❌ ±0 

Results for commit e66cd42. ± Comparison against base commit 98bae09.

♻️ This comment has been updated with latest results.

instamenta and others added 24 commits May 8, 2025 09:10
@instamenta
bump platform version 0.62.0 -> 0.62.2
1c37d08 
Signed-off-by: Zhan Milenkov <[email protected]>
@instamenta
fix block-node test
2e69934 
Signed-off-by: Zhan Milenkov <[email protected]>
@instamenta
fix block-node test
5353637 
Signed-off-by: Zhan Milenkov <[email protected]>
@instamenta
downgrade version from 0.62.1 -> 0.62.0
7059444 
Signed-off-by: Zhan Milenkov <[email protected]>
@instamenta
try v0.60.0
8ea4f5e 
Signed-off-by: Zhan Milenkov <[email protected]>
@instamenta
bump versions
bbb42d2 
Signed-off-by: Zhan Milenkov <[email protected]>
@instamenta
attempt to fix block-node using the snapshot
b0dd08e 
Signed-off-by: Zhan Milenkov <[email protected]>
@JeffreyDallas
Merge branch '01756-implement-solo-block-nodes-destroy' into test-blo…
d9e6c08 
…ck-node-version
@instamenta
Merge remote-tracking branch 'origin/01756-implement-solo-block-nodes…
8f80928 
…-destroy' into test-block-node-version

# Conflicts:
#	src/commands/block-node.ts
#	version.ts
@instamenta
bump version
708822f 
Signed-off-by: Zhan Milenkov <[email protected]>
@JeffreyDallas @jeromy-cannon
fix: fix node update after delete (#1969)
eadedb2 
Signed-off-by: Jeromy Cannon <[email protected]>
Signed-off-by: Jeffrey Tang <[email protected]>
Co-authored-by: Jeromy Cannon <[email protected]>
@jeromy-cannon
chore: refactor docs to use docsy them in preparation for multiple re…
3e98bdc 
…visions support (#1960)

Signed-off-by: Jeromy Cannon <[email protected]>
@dependabot
chore(deps-dev): bump @typescript-eslint/utils from 8.32.0 to 8.32.1 (#…
7db7bfc 
…1967)

Signed-off-by: dependabot[bot] <[email protected]>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
@dependabot
chore(deps-dev): bump typescript-eslint from 8.32.0 to 8.32.1 (#1966)
e2ab27c 
Signed-off-by: dependabot[bot] <[email protected]>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
@JeffreyDallas
feat: update upgrade e2e test (#1964)
b5fa69a 
Signed-off-by: Jeffrey Tang <[email protected]>
@instamenta
fix test for running block node validation test
84cf02f 
Signed-off-by: Zhan Milenkov <[email protected]>
@dependabot
chore(deps): bump semver from 7.7.1 to 7.7.2 (#1968)
af95469 
Signed-off-by: dependabot[bot] <[email protected]>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
@dependabot
chore(deps): bump @hashgraph/sdk from 2.63.0 to 2.64.5 (#1970)
d596c86 
Signed-off-by: dependabot[bot] <[email protected]>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
@dependabot
chore(deps-dev): bump @types/node from 22.15.17 to 22.15.18 (#1971)
9a35d99 
Signed-off-by: dependabot[bot] <[email protected]>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
@instamenta
Merge remote-tracking branch 'origin/main' into test-block-node-version
cb0dd59
@nathanklick @jeromy-cannon
refactor: split local config runtime state into appropriately shielde…
54f2a63 
…d implementations (#1965)

Signed-off-by: Nathan Klick <[email protected]>
Signed-off-by: Jeromy Cannon <[email protected]>
Co-authored-by: Jeromy Cannon <[email protected]>
@dependabot
chore(deps): bump yaml from 2.7.1 to 2.8.0 (#1974)
fc3fcb3 
Signed-off-by: dependabot[bot] <[email protected]>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
@hendrikebbers @jwagantall
docs: create MAINTAINERS.md (#1931)
1a51877 
Signed-off-by: Hendrik Ebbers <[email protected]>
Signed-off-by: Jessica G <[email protected]>
Co-authored-by: Jessica G <[email protected]>
@instamenta
Merge remote-tracking branch 'origin/main' into test-block-node-version
7d2cafc 
# Conflicts:
#	src/commands/network.ts
#	src/core/config/remote/remote-config-validator.ts
#	test/unit/commands/network.test.ts
#	test/unit/data/schema/migration/impl/remote/remote-config-v1-migration.test.ts
@instamenta
fix merge issues and format
1927885 
Signed-off-by: Zhan Milenkov <[email protected]>
@instamenta instamenta requested a review from a team as a code owner May 16, 2025 11:14
@instamenta instamenta requested a review from jeromy-cannon May 16, 2025 11:14
@github-actions GitHub Actions
Copy link
Contributor

E2E Test Report

 18 files  124 suites   2h 4m 7s ⏱️
327 tests 327 ✅ 0 💤 0 ❌
342 runs  342 ✅ 0 💤 0 ❌

Results for commit e66cd42.

@github-actions GitHub Actions
Copy link
Contributor

E2E Test Report

 18 files  124 suites   2h 4m 7s ⏱️
327 tests 327 ✅ 0 💤 0 ❌
342 runs  342 ✅ 0 💤 0 ❌

Results for commit e66cd42.

@codacy-production Codacy Production
Copy link

Coverage summary from Codacy

See diff coverage on Codacy

Coverage variation Diff coverage
Report missing for 98bae091 ✅ 90.35%
Coverage variation details
Coverable lines Covered lines Coverage
Common ancestor commit (98bae09) Report Missing Report Missing Report Missing
Head commit (e66cd42) 36954 31090 84.13%

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 (#1941) 1026 927 90.35%

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%

See your quality gate settings    Change summary preferences

Footnotes

  1. Codacy didn't receive coverage data for the commit, or there was an error processing the received data. Check your integration for errors and validate that your coverage setup is correct. ↩

@instamenta instamenta added PR: Needs Team Approval A pull request that needs review from a team member. PR: Needs Manager Approval A pull request that needs review from a manager. labels May 19, 2025
@instamenta instamenta merged commit 162d70d into 01756-implement-solo-block-nodes-destroy May 19, 2025
38 of 41 checks passed
@instamenta instamenta deleted the test-block-node-version branch May 19, 2025 14:09
@swirlds-automation
Copy link
Contributor

🎉 This PR is included in version 0.36.0 🎉

The release is available on:

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@jeromy-cannon jeromy-cannon Awaiting requested review from jeromy-cannon jeromy-cannon was automatically assigned from hiero-ledger/github-maintainers

Assignees

@instamenta instamenta

Labels
PR: Needs Manager Approval A pull request that needs review from a manager. PR: Needs Team Approval A pull request that needs review from a team member. released
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
6 participants
@instamenta @swirlds-automation @JeffreyDallas @jeromy-cannon @nathanklick @hendrikebbers