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

Reparatur-Script erstellt #17

Closed
wants to merge 1 commit into from
Closed

Conversation

pavl21
Copy link
Contributor

@pavl21 pavl21 commented Jun 27, 2022

Es soll den Benutzer erleichtern, MySpeed bei Problemen zu reparieren. Alle Fortschritte und Infos werden angezeigt, farbliche Infos wurden ebenfalls (wie im Installations- und Deinstallationsscript) angepasst. Zudem werden Dienste (wie npm und Build Essentials) und das Linux System auf dem neuesten Stand gebracht, damit mögliche Bugfixes installiert werden können. Am Ende wird zur Sicherheit MySpeed neu gestartet, am Ende der Reparatur wird auch noch einmal die IP mit dem Port gezeigt, wo die Weboberfläche zu finden ist. Außerdem wird einem auch noch einmal hingewiesen, das man ein Issue erstellen kann, wenn andere unbekannte Fehler auftreten sollten, die dieses Script nicht beheben kann.

Es soll den Benutzer erleichtern, MySpeed bei Problemen zu reparieren. Alle Fortschritte und Infos werden angezeigt, farbliche Infos wurden ebenfalls (wie im Installations- und Deinstallationsscript) angepasst. Zudem werden Dienste (wie npm und Build Essentials) und das Linux System auf dem neuesten Stand gebracht, damit mögliche Bugfixes installiert werden können. Am Ende wird zur Sicherheit MySpeed neu gestartet, am Ende der Reparatur wird auch noch einmal die IP mit dem Port gezeigt, wo die Weboberfläche zu finden ist. Außerdem wird einem auch noch einmal hingewiesen, das man ein Issue erstellen kann, wenn andere unbekannte Fehler auftreten sollten, die dieses Script nicht beheben kann.
Copy link
Owner

@gnmyt gnmyt left a comment

Choose a reason for hiding this comment

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

Erstmal möchte ich dir natürlich für die Arbeit danken. Das ist nicht selbstverständlich.
Ich finde die Idee von einem Installationsskript gut, allerdings gibt es ab der neueren Version die Bugs, die hier behoben werden, nich mehr.
Ich bin dabei in der Review nochmal auf einige dieser Probleme eingegangen.

Trotzdem vielen Dank für die Idee. Vielleicht führe ich zukünftig ein solches Skript mal ein, mal sehen :)

clear
echo -e "$BLUE🔎 Status:$NORMAL Umgebungsvariable wird geändert..."
# see in wiki Fehlerbehebung -> Cannot get /
export NODE_ENV=production
Copy link
Owner

Choose a reason for hiding this comment

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

Das setzen dieser Variable ändert die Umgebung leider nur temporär (Bis das Skript endet). Ist das Skript fertig, wird diese Variable wieder entfernt

echo -e "$BLUE🔎 Status:$NORMAL Vorhandene Module werden entfernt..."
cd /
cd /opt/myspeed
rm -R node_modules
Copy link
Owner

Choose a reason for hiding this comment

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

Seit dem Commit 6525ef7 wird der Prozess vom Neuinstallieren der Pakete direkt im Installationsskript ausgeführt, wird also hier nicht mehr benötigt

cd /
clear
echo -e "$BLUE🔎 Status:$NORMAL npm wird aktualisiert..."
curl https://www.npmjs.com/install.sh | sh
Copy link
Owner

Choose a reason for hiding this comment

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

Das installieren von npm verwaltet hier in dem Fall das System selbst. Wir brauchen also nicht nochmals npm installieren

clear
echo -e "$BLUE🔎 Status:$NORMAL Berechtigungen werden festgelegt..."
# setup permissions
chmod 700 /opt/myspeed
Copy link
Owner

Choose a reason for hiding this comment

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

Die Berechtigungen werden ja bereits im Installationsskript geändert. Passt man das manuell nicht an, so sollten sich die Berechtigungen hier normal auch nicht ändern

curl https://www.npmjs.com/install.sh | sh
clear
echo -e "$BLUE🔎 Status:$NORMAL Es wird nach Updates für das Linux-System gesucht..."
sudo apt-get update
Copy link
Owner

Choose a reason for hiding this comment

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

Das Installieren der neusten Softwarepakete sollte schon bereits in der Installation passiert sein und sollte nach der Installation keine Folgen haben, warum MySpeed nicht mehr funktioniert

chmod 700 /opt/myspeed
clear
echo -e "$BLUE🔎 Status:$NORMAL Es wird überprüft, ob Build Essantials installiert ist..."
sudo apt-get install build-essential
Copy link
Owner

Choose a reason for hiding this comment

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

Seit dem neusten Update am Server (#25) wird dieses Paket nun nicht mehr benötigt, da wir von speedtest-net auf ein eigenes System umgestiegen sind, da das alte Projekt leider nicht mehr verwaltet wird

@gnmyt gnmyt added the enhancement New feature or request label Aug 14, 2022
@gnmyt gnmyt closed this Aug 14, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants