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

change host to node #81

Merged
merged 8 commits into from
Jun 25, 2024
Merged

change host to node #81

merged 8 commits into from
Jun 25, 2024

Conversation

sukantoraymond
Copy link
Collaborator

No description provided.

Comment on lines +43 to +72
type Node struct {
// NodeID is Avalanche Node ID of the node
NodeID string

// IP address of the host
// IP address of the node
IP string

// SSH configuration for the host
// SSH configuration for the node
SSHConfig SSHConfig

// Cloud configuration for the host
// Cloud is the cloud service that the node is on
// Full list of cloud service:
// - AWS
// - GCP
// - Docker
Cloud SupportedCloud

// CloudConfig is the cloud specific configuration for the host
// CloudConfig is the cloud specific configuration for the node
CloudConfig CloudParams

// Connection to the host
Connection *goph.Client
// connection to the node
connection *goph.Client

// Roles of the host
// Roles of the node
// Full list of node roles:
// - Validator
// - API
// - AWM Relayer
// - Load Test
// - Monitoring
Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Added more description to node

Comment on lines -18 to -20
// Name of the node
Name string

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

removing name

Comment on lines -30 to -31
// Static IP of the node
StaticIP string
Copy link
Collaborator Author

Choose a reason for hiding this comment

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

removing static ip, note: need to complete #82 otherwise create node on gcp will not work

Copy link
Collaborator

@arturrez arturrez left a comment

Choose a reason for hiding this comment

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

also pls fix staticcheck

node/templates/avalanchego.docker-compose.yml Outdated Show resolved Hide resolved
node/templates/monitoring.docker-compose.yml Outdated Show resolved Hide resolved
@arturrez arturrez merged commit 0f82630 into refactor Jun 25, 2024
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants