Skip to content
This repository was archived by the owner on Dec 12, 2023. It is now read-only.

Hardware checker#48

Merged
adriantpaez merged 11 commits into
developfrom
feat/hardware-checker-new
Aug 1, 2023
Merged

Hardware checker#48
adriantpaez merged 11 commits into
developfrom
feat/hardware-checker-new

Conversation

@adriantpaez
Copy link
Copy Markdown
Contributor

#40

Implemented a feature that allows checking the hardware requirements of the AVS node to be installed and verifying if the server meets those requirements. Added an end-to-end test to ensure that the installation process fails if the hardware requirements are not met.

Changes:

  • Implemented a feature to check hardware requirements of the AVS node
  • Added an end-to-end test for hardware requirements validation

Types of changes

  • New feature (non-breaking change which adds functionality)
  • Testing

Requires testing Yes

In case you checked yes, did you write tests? Yes

Comments about testing, should you have some (optional)
The end-to-end test validates that the installation process fails when the hardware requirements are not met. It covers different scenarios to ensure accurate validation.

Further comments (optional)

This feature was implemented to ensure that the AVS node is installed only on servers that meet the specified hardware requirements. The end-to-end test provides confidence in the correctness of the implementation.

Comment thread internal/package_handler/manifest.go Outdated
Comment thread internal/package_handler/package.go Outdated
Comment thread internal/hardware_checker/hardware_checker.go Outdated
Comment thread internal/hardware_checker/hardware_checker_test.go Outdated
Comment thread internal/hardware_checker/hardware_checker_test.go Outdated
@adriantpaez adriantpaez merged commit 0784181 into develop Aug 1, 2023
@adriantpaez adriantpaez deleted the feat/hardware-checker-new branch August 1, 2023 16:33
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants