toolbox-it / toolbox-netzwerk

Ansible Playbook um das Toolbox Netzwerk zu deployen
MIT License
0 stars 3 forks source link

Dieses Repository ist der falschen Gruppe zugeordnet #75

Closed oldwired closed 4 years ago

oldwired commented 4 years ago

Dieses Repository gehört thematisch und inhaltlich zu https://github.com/ToolboxBodensee/
Dort war es auch gestern noch zugeordnet. Auf welcher Grundlage wurde dieses Repository umgezogen?

DO1JLR commented 4 years ago

Aufgrund der Umstellung nach gitlab, der zu langsammen gitlab-UI, den fehlenden Berechtigungen im gitlab (halb-gefixt) und der "Alle commits auf github werden force-überschrieben"-Policy.

maxbachmann commented 4 years ago

? du hast auf gitlab die Berechtigungen um bei allen Infrastruktur und Streaming Dingen auf den Master branch zu pushen. Wenn da noch etwas fehlt, so wurde das zumindest mal nicht in Form eines issues eingebracht

rtt-mmk commented 4 years ago

also diese Repo gehört nicht in den Chaos-Bodensee - mit solchen Moves bekommt man in offiziellen Toolbox-Umgebungen auch sicher keine Berechtigungen mehr. Es macht nur unnötig viel Arbeit das wiederherstustellen - Herr zu Gott-Himmels...! Überleg mal kurz, was du bisher und damit jetzt so vollführst...

rtt-mmk commented 4 years ago

du hast auf gitlab die Berechtigungen um bei allen Infrastruktur und Streaming Dingen auf den Master branch zu pushen. Wenn da noch etwas fehlt, so wurde das zumindest mal nicht in Form eines issues eingebracht

Sprich genauso die gleichen Rechte mit denen alle Mitglieder außer dir schon seit jeher zurecht kommen müssen. Änderungen vorschlagen, auf Review warten. Ich glaube, das ist ziemlich einleuchtend. Und auch richtig so. Das so zu handhaben ist wohl der einzige Weg, dass solche unguten Sachen nicht mehr passieren.

maxbachmann commented 4 years ago

@rtt-mmk bei Streaming und Infrastructure muss er das nicht mal, sondern kann eben auch auf den Master pushen. Er kann nur nicht auf den master force pushen, oder repo settings ändern (also beispielsweise das repo löschen).

xythobuz commented 4 years ago

Also ich muss schon auch sagen, das klingt nach vorgeschobenen Gründen und in Wahrheit sind wir hier wirklich auf bestem Wege zu einem BOFH.

maxbachmann commented 4 years ago

ja ich finde auch https://github.com/chaos-bodensee/lektor-theme-landed sollte wieder in das Webseiten Repo zurück. 1) wird das nur von der Toolbox genutzt (und auch nur auf der TB Webseite) 2) ist es stressig die Webseite jetzt and zwei Stellen zu verwalten und zum Beispiel Änderungen an dem theme nur noch manuell durch getestet werden kann 3) ist seitdem das rausbewegt wurde die Map auf der TB Webseite kaputt (hatte noch keine Zeit mir anzuschauen woran das genau liegt)

rtt-mmk commented 4 years ago

https://toolbox-bodensee.de/verein/Benutzungsordnung_Mitglieder.pdf @DO1JLR --> § 2.2. Ist, nicht nur damit, vollends nicht eingehalten. Die anderen relevanten Ordnungen und Regelungen hast du sicher auch schon gesehen. Also halte dich bitte dran. Es wird sonst echt noch unschöner.

DO1JLR commented 4 years ago

@rtt-mmk hör auf mich zu belästigen!!!

DO1JLR commented 4 years ago

ja ich finde auch https://github.com/chaos-bodensee/lektor-theme-landed sollte wieder in das Webseiten Repo zurück.

1. wird das nur von der Toolbox genutzt (und auch nur auf der TB Webseite)

2. ist es stressig die Webseite jetzt and zwei Stellen zu verwalten und zum Beispiel Änderungen an dem theme nur noch manuell durch getestet werden kann

3. ist seitdem das rausbewegt wurde die Map auf der TB Webseite kaputt (hatte noch keine Zeit mir anzuschauen woran das genau liegt)

Zu erstens habe ich dir schon gesagt, was lage ist.

Aber ja, es verursacht mehr probleme als es aktuell löst. Zurück wäre mit unter sinvoll und würde das ganze CI und verständis und nahezu alles andere auch vereinfachen!

maxbachmann commented 4 years ago

ja das stimmt schon, macht den Punkt 1 aber ja nicht weniger korrekt ;)

DO1JLR commented 4 years ago

also diese Repo gehört nicht in den Chaos-Bodensee - mit solchen Moves bekommt man in offiziellen Toolbox-Umgebungen auch sicher keine Berechtigungen mehr. Es macht nur unnötig viel Arbeit das wiederherstustellen - Herr zu Gott-Himmels...! Überleg mal kurz, was du bisher und damit jetzt so vollführst...

Nein. Es gehört in ein Toolbox git in dem man arbeiten kann.

Wenn man allerdings durch "deine änderungen werden übrigens per force-push entfernt" davon mitbekommt, dass man jetzt gefälligst gitlab zu nutzen hat, was ihr hier übrigens nicht macht, dann bleiben da wenige möglichkeiten.

Bei Repos wie der Webseite wo man eine funktionierende CI braucht und die nun per gitlab realisiert wird oder schon wurde macht das ja sinn.

Bei Repos wie ansible, das nicht per CI deployed wird ist es unnötig. Und wenn das dann ohne ankündigung, die die dort am aktivsten arbeitenden menschen erreicht (ja, mich), dann muss halt etwas getan werden um großen schaden zu verhindern.

Und gitlab...

Hab ich ausprobiert. Ist irre langsam über die WebUI.

DO1JLR commented 4 years ago

ja das stimmt schon, macht den Punkt 1 aber ja nicht weniger korrekt ;)

Nunja.. DOCH. Es macht den Punkt 1 unwahr. Es ist zwar noch kein Masterbranch aber rein technisch ein zweiter user. Aber ist auch egal weil wir hier in diesem Punkt die selbe lösung wollen!

Das "theme" Repo würde ich trotzdem nicht löschen sondern maximal archiviern oder erstmal so lassen und mit einem kommentar versehen!

DO1JLR commented 4 years ago

solved by creating toolbox-it