Skip to content

Run the local development initializations in isolated container#95

Merged
lsipii merged 1 commit intomainfrom
fix/local-dev-dotnet-restores
Oct 23, 2023
Merged

Run the local development initializations in isolated container#95
lsipii merged 1 commit intomainfrom
fix/local-dev-dotnet-restores

Conversation

@lsipii
Copy link
Copy Markdown
Contributor

@lsipii lsipii commented Oct 20, 2023

  • docker-compose script usersapi-local-initialization runs now without using bind-mounts
    • no longer touches the hosts dotnet project binaries (no need to locally run dotnet restore after docker brought up)
    • docker compose no longer messes up the code editor linter situation

@lsipii lsipii requested a review from LauriGofore October 20, 2023 08:09
@lsipii
Copy link
Copy Markdown
Contributor Author

lsipii commented Oct 20, 2023

Tää liittyy siihen että kun ollaan siirretty loputkin appiksen ulkotoiminnot kuten migraatiot pois app-runtimestä niin lokaalissa kuitenkin pitäis jossain kohti päästä ne ajamaan ja sitä varten on tehty docker-composeen oma osio joka ne sit ajelee..

Tämä osio usersapi-local-initialization on tähän asti ajautunut siten että kontti mounttaa hostin kansion jossa usersapin koodit ja sit ajelee dotnettiä suoraan siellä: tässä on ollut sellainen ongelma hostiin nähden (ongelma nähty linuks ja windowsilla jossa pörrätään wsl-linuksia) että kun koodieditoria varten on asennettu hostiin sen omalla arkkisella versio dotnetistä niin docker-kontin dotnet restore sekoittaa hostissa asiat ja siitä syystä koodieditorissa kaikki depsit vilkuttelee punaista.

Tämä PR korjaa asiaa siten että init-scripti ei käytä kontissa bind-mounttia vaan sen sijasta hieman raskaasti buildaa kuvan samaan tapaan kuin itse usersapi-kuvakin eli kopsaamalla koko kontekstin kuvaan mukaan. Vaatisi ehkä hieman optimointia, koska tuohan syö devauskoneelta levytilaa, mutta ei tässä PR:ssä.

@LauriGofore
Copy link
Copy Markdown
Contributor

LauriGofore commented Oct 23, 2023

👍

E: nevermind! maanantai-🧠, asiat toimii.

@lsipii
Copy link
Copy Markdown
Contributor Author

lsipii commented Oct 23, 2023

Hmm > Initialization script executed - shutting down ja sen jälkeinen exit code 0 olisi muuten ihan kuten kuuluu mutta kyl siinä pitäisi ennen sitä hieman asioita tapahtua. Pitääköhän taas tehdä mäkki-armille jotain taikatemppuja..

.. ah!

E: nevermind! maantain-🧠, asiat toimii.

Very kud!

@lsipii lsipii merged commit bc72bae into main Oct 23, 2023
@lsipii lsipii deleted the fix/local-dev-dotnet-restores branch October 23, 2023 10:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants