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.

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

Daemon_Conf: Investigate hard-coding addnode=* #18

Open
TechDufus opened this issue Apr 14, 2023 · 3 comments
Open

Daemon_Conf: Investigate hard-coding addnode=* #18

TechDufus opened this issue Apr 14, 2023 · 3 comments
Labels
good first issue Good for newcomers question Further information is requested

Comments

@TechDufus
Copy link
Owner

Is this still needed to be hardcoded? what happens if one of these IPs is dead? We do not want to maintain a gross list of IPs if possible.
@TechDufus TechDufus converted this from a draft issue Apr 14, 2023
@TechDufus TechDufus added good first issue Good for newcomers question Further information is requested labels Apr 14, 2023
@JKTUNING
Copy link
Collaborator

It will just remove it from the list if the node isn't there.

@TechDufus
Copy link
Owner Author

It sounds like we can leave these for now.... BUT the formatting of looping over the conf file with the list of IPs would be a nice ansible-ized addition.

@TechDufus
Copy link
Owner Author

Looks like this PR will help us get this taken care of: RunOnFlux/fluxnode-multitool#149

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
good first issue Good for newcomers question Further information is requested
Projects
@TechDufus's FluxNodeInstall
Status: Todo
Milestone
No milestone
Development

No branches or pull requests
2 participants
@JKTUNING @TechDufus