Hallo zusammen,
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.
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.
Danke.
Stefan
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
Hi,
Am 21.06.2017 um 14:28 schrieb Jan-Tarek Butt via Dev:
Ok, das ging aus dem code nicht hervor.
Magst du die variablen entsprechend kommentieren, damit das in Zukunft nicht wieder passiert.
Wir benutzen kaum globale Variablen außer alles was in Facter auch steht ;)
Sofern ein MR erstellt wird, schauen wir danach schon. Nur bei dem Modul hatte jemand anders diesen akzeptiert.
VG
Stefan
Kaum ist süß, ich glaube gar nicht trifft es eher ????
Am 21.06.2017 16:51 schrieb "Stefan via Dev" dev@lists.ffnw.de:
Hi,
Am 21.06.2017 um 14:28 schrieb Jan-Tarek Butt via Dev:
Ok, das ging aus dem code nicht hervor.
Magst du die variablen entsprechend kommentieren, damit das in Zukunft nicht wieder passiert.
Wir benutzen kaum globale Variablen außer alles was in Facter auch steht ;)
Sofern ein MR erstellt wird, schauen wir danach schon. Nur bei dem Modul hatte jemand anders diesen akzeptiert.
VG
Stefan
Dev mailing list Dev@lists.ffnw.de https://lists.ffnw.de/mailman/listinfo/dev
-
Jan-Tarek Butt -
Simon Kurka -
Stefan