Hi,

ich muss grad mal ein wenig Dampf hier ablassen. Simon und ich haben in den letzen 2 Wochen wirklich viel Arbeit ins Puppet sowie unser AS gesteckt.

Heute Morgen habe ich mich gewundert, warum beim FFNW Modul Puppet stockte. Nachdem ich mir den Source von https://git.ffnw.de/ffnw-puppet/puppet-ffnw/commit/088409c72e8b75dfb699ea0cc... angesehen habe, war es klar. Im Quellcode werden globale Variable genutzt (z.B $::{nat_ip}) die aber gar nicht global sind, sondern nur vom FFNW Modul gesetzt werden.

Ok, das ging aus dem code nicht hervor.

Magst du die variablen entsprechend kommentieren, damit das in Zukunft nicht wieder passiert.

Meine Große Bitte wäre es hier, jede Änderung über Merge Requests zu löschen und das auch nur, wenn man weis was man tut. Das ist doppelte Arbeit die wir uns hier grade wieder machen.

bzgl. der checks schicke ich dann noch einen MR der die betroffenen variablen vom Check ausschließt.

vg Tarek