-
Notifications
You must be signed in to change notification settings - Fork 17
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
proposal mini scripts #3
Comments
This is the way to go without question.. One thing: I would prefer the different commands to be grouped in one i.e.:
|
also nice idea, i love that you basicly could use all the identifiers from the ssh-login-info (or by others the nodeinfo) ..
but you could easily get in trouble with generating plenty and maybe concurrent names like fastd autoupdater ... this gives you solution a clear plus |
I wouldn't want the login to be so bloated. That could be the output of |
Es wäre praktisch, wenn man über die Site.conf für diese Scripts ändern könnte b) würde auch ermöglichen, auf Nodes mit erwiesenermaßen "mieser Anbindung" das abzustellen. Oder dem Wunsch einzelner Nodeaufstellenden nachzukommen, die das "bei sich" nicht haben wollen mit so viel test. (oder umgekehrt: Per default aus, und nur bei denen die es möchten....) |
Hmm. Du meinst eher per UCI setting, die beim Update bestehen bleibt..? |
ich denke, man sollte nicht mehr auf dem Router speichern, als notwendig für den Betrieb ist. Wenn man solche Kürzel braucht, dann kann man sie einfach lokal auf seinem Rechner anlegen und per SSH ausführen |
@rubo77 auch eine gute idee , das als externes script zu haben, aber meine Faulheit machte das script . auf jedem node ein einfaches set an befehlen |
so i nicely noticed you loved some of my ssh infos and you advanced them heavily, which i took than for myself - thx 👍
like you can use :
viisauksena/gluon-banner@f9cf579
the git is tooo different to do simply merge-request - sorry for that (be aware of minor following bugfixes)
edit: especially the colliding names of autoupdater and fastd - which means these miniscripts have to be renamed, beside some internal bugfixes
The text was updated successfully, but these errors were encountered: