Skip to content
Snippets Groups Projects

Played around

Merged Played around
played-around into master
Merged Jakob Moser requested to merge played-around into master

Änderungen am Code

  • .gitlab-ci.yml: Images für alle Branches bauen (@tmueller)
  • Dockerfile: Variablen für PaperMC-User und PaperMC-Group eingefügt und überall verwendet
  • Dockerfile: chmod nicht mehr nötig, da die Dateien eh schon ausführbar im Git liegen
  • Dockerfile: Skriptdateien werden direkt nach /, nicht nach . (was in unserem Fall sowieso der Root ist) kopiert
  • Dockerfile und papermc.sh: wget überall durch curl ersetzt, Dependency entfernt
  • papermc.sh: baseUrl und auth als Konfigurationsvariablen an den Anfang geschrieben
  • papermc.sh: Im Namen der JAR-Datei papermc durch paper ersetzt, damit man die Variable JAR_NAME sowohl in der URL als auch zum Starten der heruntergeladenen Datei verwenden kann
  • health.sh hinzugefügt und healtcheck im dockerfile benutzt

Änderungen an Dokumentation etc.

  • .gitlab-ci.yml: Bessere Quelleangabe
  • backup.sh, restore.sh, health.sh, backup.sh und papermc.sh: Als ausführbar markiert
  • Dockerfile: Dokumentationskommentare eingefügt
  • Dockerfile: /papermc und /backup werden in einer Zeile ausgeführt
  • Dockerfile: Apt-get-Dependencies zeilenweise aufgelistet, damit es später mal bei Merge Requests einfacher wird, Änderungen zu sehen
  • papermc.sh: Dokumentationskommentare eingefügt
  • papermc.sh: Einrückungen mit Tabs im unteren Teil der Datei durch Spaces ersetzt (damit konsistent)

Grundlegende Tests die bestanden werden müssen

  • Healthcheck sollte ein healthy bzw exit code 0 zurückgeben
  • latest.log sollte in der letzten Zeile Timings Reset beinhalten (testscript folgt noch, das ist aber eher ein milestone)

Testbedingungen: am besten eine Debian VM sodass manuelle Änderungen an den Rechten o. ä. nicht die Ergebnisse verfälschen Innerhalb des Containers kann per console die test.sh ausgeführt werden um die Tests auszuführen

Optionale Tests, deren Priorität zweitrangig sind

  • Dynmap webserver files sind für webserver lesbar und .json files beschreibbar (Chatfunktion)
  • backup.sh kann von EssentialsX (Plugin) gestartet werden.
Edited by tmueller

Merge request reports

Pipeline #2627 passed

Stage: build

Pipeline passed for 6eb6ab6a on played-around

Approved by
tmueller

Merged by tmuellertmueller 4 years ago (Mar 2, 2021 3:58pm UTC)

Merge details

  • Changes merged into master with 9c72793e (commits were squashed).
  • Deleted the source branch.

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
Please register or sign in to reply