sa-mw-dach / manuela-dev

Manuela Demo Apps
Apache License 2.0
13 stars 96 forks source link

267 new repo structure #24

Closed wrichter closed 4 years ago

wrichter commented 4 years ago

Addresses https://github.com/sa-mw-dach/manuela/issues/267

ortwinschneider commented 4 years ago

Ich finde die grundsätzliche Aufteilung sehr gut, hätte noch folgende Anmerkungen / Fragen:

wrichter commented 4 years ago

Ich finde die grundsätzliche Aufteilung sehr gut, hätte noch folgende Anmerkungen / Fragen:

  • Gehört networkpathoperator nicht auch in components? Für mich ist das eigentlich eine Komponente wie die anderen auch

Ist aus meiner Sicht keine Komponente der Anwendung. Daher passt es eigentlich nicht so gut zu dem "in-storyline-dev-repo". Aus meiner Sicht eher noch zu diskutieren ob es nicht in manuela gehört.

  • Dann eher eine grundsätzliche Entscheidung: Thema Operator und die zugehörigen CRs. Wollen wir die in manuela-dev haben oder nicht? momentan haben wir in operator_subscriptions einige subs, aber nicht die zugehörigen CRs. Irgendwie macht es für mich nur Sinn entweder beides oder nichts davon zu haben. Meine persönliche Tendenz wäre es operator_subscriptions auch in das manuela repo zu verlagern. Das wäre dann der eine zentrale Ausgangspunkt alles aufzusetzen, wie bspw. iotdemo

Dann ist das Repo aber nicht mehr eigenständig, bzw. man kann ohne manuela auch nicht die Inhalte von manuela-dev aufsetzen. Die subs sind getrennt von den CRs, weil ohne dass man erst die Subs anlegt (und dem cluster dann ein paar Sekunden zeit gibt) sind die CRDs nicht registriert und ein anlegen der CRs schlägt fehl/wird zurückgewiesen. Daher müssen die separat gehalten werden.

  • in components/iot-consumer/manifests gibt es noch die amq-broker.yaml (AMQ CR), die gehört da für mich nicht hin, sondern eher an die Stelle an der die anderen CRs beheimatet sind.

Die intention war, dort alles zuhaben was die Anwendungskomponente ausmacht. Ohne Broker tut die Anwendungskomponente nicht -> daher dort. Andere CRs gibt es glaube ich nicht im Manuela-dev Projekt.