| Message ID | 20190127163823.23382-3-freifunk@adrianschmutzler.de |
|---|---|
| State | Accepted |
| Headers | show |
diff --git a/src/packages/fff/fff-alfred-monitoring-proxy/Makefile b/src/packages/fff/fff-alfred-monitoring-proxy/Makefile index 93d982e0..7ab592d9 100644 --- a/src/packages/fff/fff-alfred-monitoring-proxy/Makefile +++ b/src/packages/fff/fff-alfred-monitoring-proxy/Makefile @@ -13,7 +13,7 @@ define Package/$(PKG_NAME) CATEGORY:=Freifunk TITLE:= Freifunk-Franken Alfred-Monitoring-Proxy URL:=https://www.freifunk-franken.de/ - DEPENDS:=+micrond +fff-random +curl +alfred-json +fff-nodewatcher + DEPENDS:=+micrond +fff-random +curl +alfred-json endef define Package/$(PKG_NAME)/description
hi Rückfrage: Der Nodewatcher hat folgende Abhängigkeiten drinnen: DEPENDS:=+fff-alfred +micrond +fff-config +fff-random +ip +ip-bridge Speziell fällt mir hier fff-alfred auf anhieb auf. Allerdings verwendet fff-alfred-monitoring-proxy nicht direkt fff-alfred sondern alfred-json greift auf den Alfred zu und alfred-json wird von fff-alfred-monitoring-proxy verwenden und haben wir hier korrekterweise dann auch mit drinnen. Meines erachtens fehlt dann aber im alfred-json die Abhängigkeit zu fff-alfred oder? Wenn ich es richtig zusammengefasst habe, sollte es hier passen (wenn es jemand bestätigt gibts auch ein Review) aber wir müssen fff-alfred noch als Abhängigkeit in alfred-json hinzufügen. Richtig soweit? Gruß Christian On 27.01.19 17:38, Adrian Schmutzler wrote: > The proxy only sends data from alfred master to the Monitoring. > It does not require the device to produce alfred-data in the > nodewatcher itself. > > Signed-off-by: Adrian Schmutzler <freifunk@adrianschmutzler.de> > --- > src/packages/fff/fff-alfred-monitoring-proxy/Makefile | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/src/packages/fff/fff-alfred-monitoring-proxy/Makefile b/src/packages/fff/fff-alfred-monitoring-proxy/Makefile > index 93d982e0..7ab592d9 100644 > --- a/src/packages/fff/fff-alfred-monitoring-proxy/Makefile > +++ b/src/packages/fff/fff-alfred-monitoring-proxy/Makefile > @@ -13,7 +13,7 @@ define Package/$(PKG_NAME) > CATEGORY:=Freifunk > TITLE:= Freifunk-Franken Alfred-Monitoring-Proxy > URL:=https://www.freifunk-franken.de/ > - DEPENDS:=+micrond +fff-random +curl +alfred-json +fff-nodewatcher > + DEPENDS:=+micrond +fff-random +curl +alfred-json > endef > > define Package/$(PKG_NAME)/description
Hallo Christian, guter Fund: Meines Erachtens brauchen sowohl alfred-json als auch der proxy jeweils die Abhängigkeit zu fff-alfred: alfred-json hast du schon begründet. alfred-monitoring-proxy ändert im uci-defaults Skript die alfred uci config, die vorher in fff-alfred erzeugt werden muss. Ich würde daher die Abhängigkeit zu fff-alfred in BEIDE Makefiles reinschreiben. Mach ich aber in nem extra Patch. Grüße Adrian > -----Original Message----- > From: Christian Dresel [mailto:fff@chrisi01.de] > Sent: Sonntag, 27. Januar 2019 17:48 > To: Adrian Schmutzler <freifunk@adrianschmutzler.de>; franken- > dev@freifunk.net > Subject: Re: [PATCH 2/6] fff-alfred-monitoring-proxy: Remove dependency > from nodewatcher > > hi > > Rückfrage: > > Der Nodewatcher hat folgende Abhängigkeiten drinnen: > > DEPENDS:=+fff-alfred +micrond +fff-config +fff-random +ip +ip-bridge > > Speziell fällt mir hier fff-alfred auf anhieb auf. Allerdings verwendet fff- > alfred-monitoring-proxy nicht direkt fff-alfred sondern alfred-json greift auf > den Alfred zu und alfred-json wird von fff-alfred-monitoring-proxy > verwenden und haben wir hier korrekterweise dann auch mit drinnen. > > Meines erachtens fehlt dann aber im alfred-json die Abhängigkeit zu fff- > alfred oder? > > Wenn ich es richtig zusammengefasst habe, sollte es hier passen (wenn es > jemand bestätigt gibts auch ein Review) aber wir müssen fff-alfred noch als > Abhängigkeit in alfred-json hinzufügen. Richtig soweit? > > Gruß > > Christian > > On 27.01.19 17:38, Adrian Schmutzler wrote: > > The proxy only sends data from alfred master to the Monitoring. > > It does not require the device to produce alfred-data in the > > nodewatcher itself. > > > > Signed-off-by: Adrian Schmutzler <freifunk@adrianschmutzler.de> > > --- > > src/packages/fff/fff-alfred-monitoring-proxy/Makefile | 2 +- > > 1 file changed, 1 insertion(+), 1 deletion(-) > > > > diff --git a/src/packages/fff/fff-alfred-monitoring-proxy/Makefile > > b/src/packages/fff/fff-alfred-monitoring-proxy/Makefile > > index 93d982e0..7ab592d9 100644 > > --- a/src/packages/fff/fff-alfred-monitoring-proxy/Makefile > > +++ b/src/packages/fff/fff-alfred-monitoring-proxy/Makefile > > @@ -13,7 +13,7 @@ define Package/$(PKG_NAME) > > CATEGORY:=Freifunk > > TITLE:= Freifunk-Franken Alfred-Monitoring-Proxy > > URL:=https://www.freifunk-franken.de/ > > - DEPENDS:=+micrond +fff-random +curl +alfred-json +fff-nodewatcher > > + DEPENDS:=+micrond +fff-random +curl +alfred-json > > endef > > > > define Package/$(PKG_NAME)/description
Hi On 27.01.19 18:00, mail@adrianschmutzler.de wrote: > Hallo Christian, > > guter Fund: > > Meines Erachtens brauchen sowohl alfred-json als auch der proxy jeweils die Abhängigkeit zu fff-alfred: > > alfred-json hast du schon begründet. > alfred-monitoring-proxy ändert im uci-defaults Skript die alfred uci config, die vorher in fff-alfred erzeugt werden muss. ein noch besserer Fund ;) Ja stimmt dann brauchen wir es hier auch. Ich überlege aber gerade ob dieses ändern hier überhaupt korrekt ist oder wo anders vielleicht besser aufgehoben ist? Bin mir da noch nicht sicher. > > Ich würde daher die Abhängigkeit zu fff-alfred in BEIDE Makefiles reinschreiben. Mach ich aber in nem extra Patch. Das heißt du willst diesen applien und dann in einen weiteren Patch jeweils fff-alfred hinzufügen? Ist in meinen Augen ok: Reviewed-by: Christian Dresel <fff@chrisi01.de> Gruß Christian > > Grüße > > Adrian > >> -----Original Message----- >> From: Christian Dresel [mailto:fff@chrisi01.de] >> Sent: Sonntag, 27. Januar 2019 17:48 >> To: Adrian Schmutzler <freifunk@adrianschmutzler.de>; franken- >> dev@freifunk.net >> Subject: Re: [PATCH 2/6] fff-alfred-monitoring-proxy: Remove dependency >> from nodewatcher >> >> hi >> >> Rückfrage: >> >> Der Nodewatcher hat folgende Abhängigkeiten drinnen: >> >> DEPENDS:=+fff-alfred +micrond +fff-config +fff-random +ip +ip-bridge >> >> Speziell fällt mir hier fff-alfred auf anhieb auf. Allerdings verwendet fff- >> alfred-monitoring-proxy nicht direkt fff-alfred sondern alfred-json greift auf >> den Alfred zu und alfred-json wird von fff-alfred-monitoring-proxy >> verwenden und haben wir hier korrekterweise dann auch mit drinnen. >> >> Meines erachtens fehlt dann aber im alfred-json die Abhängigkeit zu fff- >> alfred oder? >> >> Wenn ich es richtig zusammengefasst habe, sollte es hier passen (wenn es >> jemand bestätigt gibts auch ein Review) aber wir müssen fff-alfred noch als >> Abhängigkeit in alfred-json hinzufügen. Richtig soweit? >> >> Gruß >> >> Christian >> >> On 27.01.19 17:38, Adrian Schmutzler wrote: >>> The proxy only sends data from alfred master to the Monitoring. >>> It does not require the device to produce alfred-data in the >>> nodewatcher itself. >>> >>> Signed-off-by: Adrian Schmutzler <freifunk@adrianschmutzler.de> >>> --- >>> src/packages/fff/fff-alfred-monitoring-proxy/Makefile | 2 +- >>> 1 file changed, 1 insertion(+), 1 deletion(-) >>> >>> diff --git a/src/packages/fff/fff-alfred-monitoring-proxy/Makefile >>> b/src/packages/fff/fff-alfred-monitoring-proxy/Makefile >>> index 93d982e0..7ab592d9 100644 >>> --- a/src/packages/fff/fff-alfred-monitoring-proxy/Makefile >>> +++ b/src/packages/fff/fff-alfred-monitoring-proxy/Makefile >>> @@ -13,7 +13,7 @@ define Package/$(PKG_NAME) >>> CATEGORY:=Freifunk >>> TITLE:= Freifunk-Franken Alfred-Monitoring-Proxy >>> URL:=https://www.freifunk-franken.de/ >>> - DEPENDS:=+micrond +fff-random +curl +alfred-json +fff-nodewatcher >>> + DEPENDS:=+micrond +fff-random +curl +alfred-json >>> endef >>> >>> define Package/$(PKG_NAME)/description
Hallo zusammen, On 27.01.19 18:00, mail@adrianschmutzler.de wrote: > Hallo Christian, > > guter Fund: > > Meines Erachtens brauchen sowohl alfred-json als auch der proxy jeweils die Abhängigkeit zu fff-alfred: > > alfred-json hast du schon begründet. Von fff-alfred sehe ich keine Abhängigkeit. Eher von alfred. > alfred-monitoring-proxy ändert im uci-defaults Skript die alfred uci config, die vorher in fff-alfred erzeugt werden muss. Gut beobachtet. Ich würde einfach diese Abhängigkeit hinzufügen, fff-alfred bietet dann die Basis, fff-alfred-monitoring-proxy ist dann der Gateway-spezifische Teil. Wenn man für Client nochmal was extra braucht, dann könnte man das ja in ein Node-spezifisches eigenes Package lagern. @Tim: Bisher hast du imho immer sehr gut auf die langlebigkeit der Architektur geachtet. Wie würdest du das machen? Fabian
Applied. From: franken-dev [mailto:franken-dev-bounces@freifunk.net] On Behalf Of mail@adrianschmutzler.de Sent: Sonntag, 27. Januar 2019 18:01 To: fff@chrisi01.de; franken-dev@freifunk.net Subject: RE: [PATCH 2/6] fff-alfred-monitoring-proxy: Remove dependency from nodewatcher Hallo Christian, guter Fund: Meines Erachtens brauchen sowohl alfred-json als auch der proxy jeweils die Abhängigkeit zu fff-alfred: alfred-json hast du schon begründet. alfred-monitoring-proxy ändert im uci-defaults Skript die alfred uci config, die vorher in fff-alfred erzeugt werden muss. Ich würde daher die Abhängigkeit zu fff-alfred in BEIDE Makefiles reinschreiben. Mach ich aber in nem extra Patch. Grüße Adrian > -----Original Message----- > From: Christian Dresel [mailto:fff@chrisi01.de] > Sent: Sonntag, 27. Januar 2019 17:48 > To: Adrian Schmutzler <freifunk@adrianschmutzler.de <mailto:freifunk@adrianschmutzler.de> >; franken- > dev@freifunk.net <mailto:dev@freifunk.net> > Subject: Re: [PATCH 2/6] fff-alfred-monitoring-proxy: Remove dependency > from nodewatcher > > hi > > Rückfrage: > > Der Nodewatcher hat folgende Abhängigkeiten drinnen: > > DEPENDS:=+fff-alfred +micrond +fff-config +fff-random +ip +ip-bridge > > Speziell fällt mir hier fff-alfred auf anhieb auf. Allerdings verwendet fff- > alfred-monitoring-proxy nicht direkt fff-alfred sondern alfred-json greift auf > den Alfred zu und alfred-json wird von fff-alfred-monitoring-proxy > verwenden und haben wir hier korrekterweise dann auch mit drinnen. > > Meines erachtens fehlt dann aber im alfred-json die Abhängigkeit zu fff- > alfred oder? > > Wenn ich es richtig zusammengefasst habe, sollte es hier passen (wenn es > jemand bestätigt gibts auch ein Review) aber wir müssen fff-alfred noch als > Abhängigkeit in alfred-json hinzufügen. Richtig soweit? > > Gruß > > Christian > > On 27.01.19 17:38, Adrian Schmutzler wrote: > > The proxy only sends data from alfred master to the Monitoring. > > It does not require the device to produce alfred-data in the > > nodewatcher itself. > > > > Signed-off-by: Adrian Schmutzler <freifunk@adrianschmutzler.de <mailto:freifunk@adrianschmutzler.de> > > > --- > > src/packages/fff/fff-alfred-monitoring-proxy/Makefile | 2 +- > > 1 file changed, 1 insertion(+), 1 deletion(-) > > > > diff --git a/src/packages/fff/fff-alfred-monitoring-proxy/Makefile > > b/src/packages/fff/fff-alfred-monitoring-proxy/Makefile > > index 93d982e0..7ab592d9 100644 > > --- a/src/packages/fff/fff-alfred-monitoring-proxy/Makefile > > +++ b/src/packages/fff/fff-alfred-monitoring-proxy/Makefile > > @@ -13,7 +13,7 @@ define Package/$(PKG_NAME) > > CATEGORY:=Freifunk > > TITLE:= Freifunk-Franken Alfred-Monitoring-Proxy > > URL:=https://www.freifunk-franken.de/ > > - DEPENDS:=+micrond +fff-random +curl +alfred-json +fff-nodewatcher > > + DEPENDS:=+micrond +fff-random +curl +alfred-json > > endef > > > > define Package/$(PKG_NAME)/description
The proxy only sends data from alfred master to the Monitoring. It does not require the device to produce alfred-data in the nodewatcher itself. Signed-off-by: Adrian Schmutzler <freifunk@adrianschmutzler.de> --- src/packages/fff/fff-alfred-monitoring-proxy/Makefile | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)