-
Notifications
You must be signed in to change notification settings - Fork 2
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: use NMT to install and start network nodes #178
Merged
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Signed-off-by: Lenin Mehedy <lenin.mehedy@swirldslabs.com>
Signed-off-by: Lenin Mehedy <lenin.mehedy@swirldslabs.com>
Codecov Report
Additional details and impacted files@@ Coverage Diff @@
## main #178 +/- ##
=========================================
Coverage 56.70% 56.70%
Complexity 144 144
=========================================
Files 36 36
Lines 723 723
Branches 73 73
=========================================
Hits 410 410
Misses 281 281
Partials 32 32 |
Signed-off-by: Lenin Mehedy <lenin.mehedy@swirldslabs.com>
Signed-off-by: Lenin Mehedy <lenin.mehedy@swirldslabs.com>
Signed-off-by: Lenin Mehedy <lenin.mehedy@swirldslabs.com>
Signed-off-by: Lenin Mehedy <lenin.mehedy@swirldslabs.com>
Signed-off-by: Lenin Mehedy <lenin.mehedy@swirldslabs.com>
leninmehedy
requested review from
nathanklick and
deepak-swirlds
and removed request for
nathanklick
July 18, 2023 11:09
Signed-off-by: Lenin Mehedy <lenin.mehedy@swirldslabs.com>
Signed-off-by: Lenin Mehedy <lenin.mehedy@swirldslabs.com>
Signed-off-by: Lenin Mehedy <lenin.mehedy@swirldslabs.com>
Signed-off-by: Lenin Mehedy <lenin.mehedy@swirldslabs.com>
Signed-off-by: Lenin Mehedy <lenin.mehedy@swirldslabs.com>
Signed-off-by: Lenin Mehedy <lenin.mehedy@swirldslabs.com>
Signed-off-by: Lenin Mehedy <lenin.mehedy@swirldslabs.com>
Signed-off-by: Lenin Mehedy <lenin.mehedy@swirldslabs.com>
Signed-off-by: Lenin Mehedy <lenin.mehedy@swirldslabs.com>
Signed-off-by: Lenin Mehedy <lenin.mehedy@swirldslabs.com>
Signed-off-by: Lenin Mehedy <lenin.mehedy@swirldslabs.com>
Signed-off-by: Lenin Mehedy <lenin.mehedy@swirldslabs.com>
Signed-off-by: Lenin Mehedy <lenin.mehedy@swirldslabs.com>
Signed-off-by: Lenin Mehedy <lenin.mehedy@swirldslabs.com>
Signed-off-by: Lenin Mehedy <lenin.mehedy@swirldslabs.com>
Signed-off-by: Lenin Mehedy <lenin.mehedy@swirldslabs.com>
Signed-off-by: Lenin Mehedy <lenin.mehedy@swirldslabs.com>
Signed-off-by: Lenin Mehedy <lenin.mehedy@swirldslabs.com>
Signed-off-by: Lenin Mehedy <lenin.mehedy@swirldslabs.com> # Conflicts: # charts/hedera-network/configs/test.sh
Signed-off-by: Lenin Mehedy <lenin.mehedy@swirldslabs.com>
Signed-off-by: Lenin Mehedy <lenin.mehedy@swirldslabs.com>
Signed-off-by: Lenin Mehedy <lenin.mehedy@swirldslabs.com>
Signed-off-by: Lenin Mehedy <lenin.mehedy@swirldslabs.com>
leninmehedy
changed the title
feat: use NMT to install and run network node
feat: use NMT to install and start network nodes
Jul 27, 2023
Signed-off-by: Lenin Mehedy <lenin.mehedy@swirldslabs.com>
nathanklick
reviewed
Jul 27, 2023
nathanklick
reviewed
Jul 27, 2023
Signed-off-by: Nathan Klick <nathan@swirldslabs.com>
nathanklick
force-pushed
the
176-use-nmt-to-install-node-software
branch
from
July 27, 2023 14:33
12ab02e
to
82e591b
Compare
Kudos, SonarCloud Quality Gate passed! |
nathanklick
approved these changes
Jul 27, 2023
swirlds-automation
added a commit
that referenced
this pull request
Jul 31, 2023
## [0.4.0](v0.3.0...v0.4.0) (2023-07-31) ### Features * adds OFAC block list & switch to haproxy version 2.4 ([#183](#183)) ([0b57e15](0b57e15)) * custom k8s label needs fullstack.hedera.com/ prefix ([#195](#195)) ([c3f800a](c3f800a)) * install network node software without nmt ([#197](#197)) ([b1029ea](b1029ea)) * use NMT to install and start network nodes ([#178](#178)) ([f96519e](f96519e)) ### Bug Fixes * resolve build issue with ubi8-init-java17 container image ([#201](#201)) ([9bc93c3](9bc93c3)) * update helm chart to align with design and flexibility requirements ([#198](#198)) ([6ca10ee](6ca10ee)) * update values file to remove configurable hedera UID/GID properties ([#173](#173)) ([be14c1e](be14c1e))
🎉 This PR is included in version 0.4.0 🎉 The release is available on GitHub release Your semantic-release bot 📦🚀 |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Description
This pull request adds scripts to install and run network node using NMT. This ensures we have a working hedera-network-nodes using helm chart.
It also includes some Dockerfile and keys that are needed to start the node in the
/dev
directory. I also added a Makefile and README in/dev
directory describing how to start the node. We may update the README in the root directory later once it is more polished.Notes
gsutil
to fetch required resources fromfst-resources
bucket (I created it recently and uploaded the resources. This may become obsolete, but for now this helps to ensure all required resources are in one place and it can be accessed using a single toolgsutil
)kind
,helm
andkubectl
CLI programs are installedRelated Issues