Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Pakeisti clients migracijų paleidimo logiką #764

Closed
4 tasks done
JustinasKen opened this issue Sep 4, 2024 · 0 comments · Fixed by #767
Closed
4 tasks done

Pakeisti clients migracijų paleidimo logiką #764

JustinasKen opened this issue Sep 4, 2024 · 0 comments · Fixed by #767
Labels

Comments

@JustinasKen
Copy link
Contributor

JustinasKen commented Sep 4, 2024

Ši užduotis kilo iš #721 (comment) komentaro.

#122 Užduotis pakeitė kaip saugome clients failus, vietoj to, kad saugotume viename clients aplankale, dabar turime id aplankalą, kuriame pagal xx/xx/rest-of-uuid.yml formatą saugome klientus.

Pereinant iš senos sistemos į naują, buvo sukurta migracijos funkcija, kuri yra paleidžiama kaskartą užkraunant Config klasę. Tai nėra labai gera implementacija, kadangi, jei paleidžiame kelis tuos pačius procesus vienu metu, gauname klaidas, nes bandome atlikti rašymo ir skaitymo operacijas tiem patiems failams, kelis kartus vienu metu.

Reikia pakeisti, kad nebevyktų automatinės migracijos.
Turime aptikti, ar sistema reikalauja migracijų, jei taip, metame klaidą, kurioje nurodome, kad vartotojas turi panaudoti spinta upgrade komandą, kuri atliks migracijas tik vieną kartą.

spinta upgrade komanda bus atsakinga už visas tokio pobūdžio migracijas ir atnaujinimus. Ši komanda turėtų automatiškai aptikti, kas bus migruojama. Jei yra poreikis paleisti tam tikrą migraciją be patikrinimų, reikia nurodyti -r, --run MIGRATION_SCRIPT argumentą.

Užduotys

  • Panaikinti automatines migracijas.
  • Pridėti spinta upgrade komandą.
  • Įdėti tikrinimus, ar sistema laikosi naujo standarto.
  • Pridėti testus.

Susiję

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants