Navigation Menu

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

[Bugi] Käynnistysskriptin parantelua #6

Closed
CodoDerDritte opened this issue Nov 14, 2022 · 1 comment
Closed

[Bugi] Käynnistysskriptin parantelua #6

CodoDerDritte opened this issue Nov 14, 2022 · 1 comment
Assignees
Labels
bug Something isn't working local enhancement New feature or request

Comments

@CodoDerDritte
Copy link

Mikropalvelun käynnistysskriptissä (koha_suomi_services_daemon.sh) on muutamia ongelmia:

  1. Lukitus puuttuu, skriptillä on mahdollista käynnistää "ylimääräisiä" taustaskriptejä tai hypnotoad-instansseja, jolloin valutuksessa tietuemäärät tuplaantuvat ja tapahtuu muita kummitteluja.
  2. Palveluja pysäytettäessä skripti ei odota että taustaprosessit ja hypnotoad pysähtyvät, jolloin esimerkiksi reload (=stop/start) voivat käynnistää ylimääräisiä taustaskriptejä ja hypnotoad-instansseja.
  3. Skriptissä on turhaa "koukeroisuutta" esimerkiksi käyttäjien vaihtamisen kanssa.

Lisäksi on olemassa erillinen run_background_scripts.sh, jonka ajatus on ilmeisestikin käynnistää mahdollisesti kaatuneet taustaskriptit uudelleen.

Korjataan nämä ongelmat seuraavasti:

  1. Skriptiin tarkistus/lukitus, joka ei käynnistä uusia taustaskriptejä/hypnotoadeja jos palvelu on jo käynnissä.
  2. Muutetaan skripti odottamaan kaikkien taustaskriptien ja hypnotoadin pysähtymistä 'stop' parametrillä, jolloin reloadkin toimii oikein (i.e. käynnistää uudet prosessit vasta kun vanhat ovat pysähtyneet).
  3. Siivotaan turhat käyttäjänvaihtomutkat pois.

Lisäksi kirjoitetaan run_background_scripts.sh:n toiminnallisuus yksinkertaistettuna suoraan koha_suomi_services_daemon.sh -skriptiin, jolloin taustaskriptien "elossapito" voidaan hoitaa koha_suomi_services_daemon.sh reload eikä erillistä run_background_scripts.sh:ta enää tarvita.

@CodoDerDritte CodoDerDritte self-assigned this Nov 14, 2022
@CodoDerDritte CodoDerDritte added bug Something isn't working local enhancement New feature or request labels Nov 14, 2022
@CodoDerDritte
Copy link
Author

Korjaus sisältää myös muuta koodin siistimistä.

@CodoDerDritte CodoDerDritte changed the title Käynnistysskriptin parantelua [Bugi] Käynnistysskriptin parantelua Nov 17, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working local enhancement New feature or request
Projects
Status: Suljettu
Development

No branches or pull requests

1 participant