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.

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(cli): allow deploying different number network nodes #726

Merged
merged 22 commits into from
Feb 3, 2024

Conversation

leninmehedy
Copy link
Member

@leninmehedy leninmehedy commented Feb 2, 2024

Description

We can now just set the node IDs to deploy different number of network nodes,
e.g. solo network deploy -i node0,node1,node2,node3,node4......nodeN

This pull request changes the following:

  • allow deploying different number network nodes
  • only output extra logs if --dev flag is set
  • updated solo README with console output logs for reference

Related Issues

Copy link

github-actions bot commented Feb 2, 2024

Unit Test Results

  26 files  ±0    26 suites  ±0   53s ⏱️ ±0s
109 tests ±0  105 ✔️ ±0  4 💤 ±0  0 ±0 
110 runs  ±0  106 ✔️ ±0  4 💤 ±0  0 ±0 

Results for commit fd01b4f. ± Comparison against base commit 5769942.

♻️ This comment has been updated with latest results.

Copy link

codecov bot commented Feb 2, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Comparison is base (5769942) 0.00% compared to head (fd01b4f) 67.51%.

Additional details and impacted files

Impacted file tree graph

@@             Coverage Diff             @@
##             main     #726       +/-   ##
===========================================
+ Coverage        0   67.51%   +67.51%     
- Complexity      0      339      +339     
===========================================
  Files           0       64       +64     
  Lines           0     1336     +1336     
  Branches        0      157      +157     
===========================================
+ Hits            0      902      +902     
- Misses          0      371      +371     
- Partials        0       63       +63     

see 64 files with indirect coverage changes

Impacted file tree graph

@leninmehedy leninmehedy force-pushed the 582-Allow-deploying-different-number-network-nodes branch 2 times, most recently from 3786943 to 1b4af8a Compare February 2, 2024 04:03
@leninmehedy leninmehedy force-pushed the 582-Allow-deploying-different-number-network-nodes branch from 1b4af8a to 7c43e81 Compare February 2, 2024 04:04
solo/README.md Outdated Show resolved Hide resolved
Signed-off-by: Jeromy Cannon <[email protected]>
solo/README.md Outdated Show resolved Hide resolved
Signed-off-by: Jeromy Cannon <[email protected]>
solo/README.md Outdated Show resolved Hide resolved
Signed-off-by: Jeromy Cannon <[email protected]>
solo/README.md Outdated Show resolved Hide resolved
solo/README.md Outdated Show resolved Hide resolved
solo/README.md Outdated Show resolved Hide resolved
solo/README.md Outdated Show resolved Hide resolved
solo/README.md Outdated Show resolved Hide resolved
solo/README.md Outdated Show resolved Hide resolved
solo/README.md Outdated Show resolved Hide resolved
solo/README.md Outdated Show resolved Hide resolved
Signed-off-by: Jeromy Cannon <[email protected]>
Signed-off-by: Jeromy Cannon <[email protected]>
Signed-off-by: Jeromy Cannon <[email protected]>
Signed-off-by: Jeromy Cannon <[email protected]>
Signed-off-by: Jeromy Cannon <[email protected]>
Signed-off-by: Jeromy Cannon <[email protected]>
Signed-off-by: Jeromy Cannon <[email protected]>
Signed-off-by: Jeromy Cannon <[email protected]>
solo/README.md Outdated Show resolved Hide resolved
jeromy-cannon and others added 5 commits February 2, 2024 10:11
Signed-off-by: Jeromy Cannon <[email protected]>
Signed-off-by: Lenin Mehedy <[email protected]>
…user > cached config > defaults (#731)

Signed-off-by: Jeromy Cannon <[email protected]>
…t tests (#729)

Signed-off-by: Lenin Mehedy <[email protected]>
Signed-off-by: Jeromy Cannon <[email protected]>
Co-authored-by: Jeromy Cannon <[email protected]>
Signed-off-by: Lenin Mehedy <[email protected]>
@leninmehedy leninmehedy force-pushed the 582-Allow-deploying-different-number-network-nodes branch from ee2357e to fd01b4f Compare February 3, 2024 04:09
Copy link

sonarqubecloud bot commented Feb 3, 2024

Quality Gate Passed Quality Gate passed

Kudos, no new issues were introduced!

0 New issues
0 Security Hotspots
No data about Coverage
No data about Duplication

See analysis details on SonarCloud

@leninmehedy leninmehedy merged commit 410d499 into main Feb 3, 2024
18 of 19 checks passed
@leninmehedy leninmehedy deleted the 582-Allow-deploying-different-number-network-nodes branch February 3, 2024 10:02
swirlds-automation added a commit that referenced this pull request Feb 5, 2024
## [0.20.0](v0.19.1...v0.20.0) (2024-02-05)

### Features

* **cli:** allow deploying different number network nodes ([#726](#726)) ([410d499](410d499))
* **cli:** generate Gossip keys and gRPC TLS keys in PEM format ([#629](#629)) ([d970796](d970796))

### Bug Fixes

* update solo readme with examples ([#725](#725)) ([dd8bbd7](dd8bbd7))
@swirlds-automation
Copy link
Contributor

🎉 This PR is included in version 0.20.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
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

CLI[Node]: Allow deploying different number network nodes
3 participants