search

thiws24 / equipli

MIT License
1 stars 0 forks source link

Service für InventoryItem #14

Closed d135-1r43 closed 3 weeks ago

d135-1r43 commented 1 month ago
d135-1r43 commented 1 month ago

Wird verblockt von #12

baurohnehof commented 1 month ago

@nadri23 @janniknu @marvinkrn

Ainges commented 4 weeks ago

Hi @marvinkrn, du hast mich doch heute gefragt wegen dem Docker-Build:

Also nachdem ich die Tests auskommentiert habe, builded das ganze mit ./mvnw package. Dann kann auch mit docker build -t ghcr.io/thiws24/inventoryitem:latest -f src/main/docker/Dockerfile.jvm . das Docker-Image erstellt werden. Ich weiß nicht ob der Imagename so passt. Hab noch nicht versucht zu pushen.

Edit: Es muss dann aber noch eine Datasource in den applications.properties hinterlegt werden, nach dem build gibt es keinen dev service mehr

Ainges commented 4 weeks ago

Ich habe auch mal wegen der CI geschaut. Mein run (https://github.com/thiws24/inv/actions/runs/11441682988/job/31830290229) lief soweit ich das sehe glatt durch, hat aber keine Berechtigungen um auf die Registry der Organisation zu pushen. @d135-1r43 kann man für eine Organisation auch einen PAT mit den Berechtigungen generieren und im Secretstore ablegen?

marvinkrn commented 4 weeks ago

Hi @marvinkrn, du hast mich doch heute gefragt wegen dem Docker-Build:

Also nachdem ich die Tests auskommentiert habe, builded das ganze mit ./mvnw package. Dann kann auch mit docker build -t ghcr.io/thiws24/inventoryitem:latest -f src/main/docker/Dockerfile.jvm . das Docker-Image erstellt werden. Ich weiß nicht ob der Imagename so passt. Hab noch nicht versucht zu pushen.

Hey @Ainges, danke fürs Testen! Funktioniert bei mir jetzt auch soweit, danke auch für den Tipp mit der application.properties. Die CI lief bei dir durch? Aber nicht mit der selben yml, oder?

Ainges commented 4 weeks ago

Ja hab die yml verändert ist aber noch nicht auf dem anderen Branch pushed.

Edit: Doch natürlich hab ich den pushed, sonst wär actions ja nicht gelaufen 😅

marvinkrn commented 4 weeks ago

Aber auf deinem Actions Branch, oder? Habs jetzt mal auf unseren kopiert mit Credits an dich 🙌 dann fehlt jetzt theoretisch nur noch die Berechtigung, dann müsste es durchlaufen

Ausversehen geschlossen, ups

d135-1r43 commented 4 weeks ago

Ich habe auch mal wegen der CI geschaut. Mein run (https://github.com/thiws24/inv/actions/runs/11441682988/job/31830290229) lief soweit ich das sehe glatt durch, hat aber keine Berechtigungen um auf die Registry der Organisation zu pushen. @d135-1r43 kann man für eine Organisation auch einen PAT mit den Berechtigungen generieren und im Secretstore ablegen?

Lass uns das morgen gemeinsam machen. Ich glaube, ich muss noch in den Berechtigungen etwas anpassen.