Feat/elastic search

[BerniUtz]

Hallihallo! Würdet ihr euch das bitte mal anschauen? Ich bin mir nicht sicher ob das gerade nur bei mir läuft oder ob das eh überall laufen würde... habe da nämlich im container was angepasst und wenn da ja bei euch das image neu geholt wird sind die Einstellungen da ja nicht vorhanden 😅 Aber die integrationtests laufen bei mir zumindest schonmal und es sind dann tatsächlich Dokumente in Elastic gespeichert:

PS: nicht schrecken ich hab die branch vom vorigen PR erstellt darum sind die commits jetzt auch noch drinnen...

[leonardstruck]

Hm, wenn du einstellungen im Container gemacht hast, dann wird das bei uns nicht funktionieren... Kannst du kurz umschreiben was du genau für Einstellungen hinterlegt hast? In der Regel kann man diese nämlich mit ENV-Variablen machen, oder wir müssen einen zusätzlichen Container (ähnlich wie bei Minio) erstellen, der ausschließlich für die Konfiguration verantwortlich ist und dann stoppt.

[leonardstruck]

Ah und leider können wir keine Integrationstests machen mit NUnit, da wir ja in der Pipeline überhaupt keinen ElasticSearch Container am laufen haben. Ich würde vorschlagen dass wir die Integrationstest in Postman erstellen und NUnit ausschließlich für Unittests verwenden.

[BerniUtz]

Kannst du kurz umschreiben was du genau für Einstellungen hinterlegt hast

schlussendlich müsste es eh nur gewesen sein, dass ich das Passwort gesetzt habe. Ich hatte nämlich das Problem dass er mich ohne auth nichts machen hat lassen, ich hab aber nie ein Passwort gesetzt gehabt. Dann bin ich in den Container rein und hab das Passwort gesetzt und dann hat es geklappt

[BerniUtz]

Ah und leider können wir keine Integrationstests machen mit NUnit, da wir ja in der Pipeline überhaupt keinen ElasticSearch Container am laufen haben. Ich würde vorschlagen dass wir die Integrationstest in Postman erstellen und NUnit ausschließlich für Unittests verwenden.

Ja voll hab ich mir schon gedacht. Kann man dann eh raus nehmen, wollte das bei mir nur testen ob das tatsächlich mit dem Container funktioniert und dann eben euch auch die Möglichkeit lassen dass ihr das einfach testen könnt :)

[leonardstruck]

Ah okay, das sollte ganz einfach mit der env-variable ELASTIC_PASSWORD zu machen sein. Alternativ, falls dir noch irgendwelche Sachen einfallen, die du konfigurieren müsstest, kann man eine elasticssearch.yml Datei erstellen und diese in den Container kopieren.

[leonardstruck]

Ah und leider können wir keine Integrationstests machen mit NUnit, da wir ja in der Pipeline überhaupt keinen ElasticSearch Container am laufen haben. Ich würde vorschlagen dass wir die Integrationstest in Postman erstellen und NUnit ausschließlich für Unittests verwenden.

Ja voll hab ich mir schon gedacht. Kann man dann eh raus nehmen, wollte das bei mir nur testen ob das tatsächlich mit dem Container funktioniert und dann eben euch auch die Möglichkeit lassen dass ihr das einfach testen könnt :)

Ja, versteh ich voll, aber das ist immer schade wenn über dem PR steht dass alle checks fehlgeschlagen sind hehe

[BerniUtz]

Hat funktioniert mit env und habe jetzt noch eine delete methode hinzugefügt

[BerniUtz]

Fehlt da jetzt noch was? @leonardstruck @SirPoz

[SirPoz]

naja die checks laufen nicht durch oder? Von welchem Branch hast du das gezogen? Ich kenne die Fehlermeldungen im OCRWorker nicht

[BerniUtz]

Das liegt nicht an den Tests, es builded nichtmal.. ich habe die dev rein gemerged nachdem du deinen PR in die dev gemerged hast obwohl es nicht gebuilded hat und jetzt builded es bei mir natürlich auch nicht 😄