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

Bij update controleren op lokale openstaande wijzigingen #1259

Closed
mthmulders opened this issue Jan 16, 2021 · 6 comments
Closed

Bij update controleren op lokale openstaande wijzigingen #1259

mthmulders opened this issue Jan 16, 2021 · 6 comments
Milestone

Comments

@mthmulders
Copy link
Contributor

In de webinterface zie ik dat er een update beschikbaar is:

image

Dan log ik met SSH in en volg de gebruikelijke stappen voor het uitvoeren van een upgrade:

sudo su - dsmr
./deploy.sh

De uitvoer eindigt dan met:

 --- Clearing cache...


 --- Deployed version: 
4.9


 >>> Deployment complete. <<<

Ik heb dit een paar keer geprobeerd maar ik blijf te zien krijgen dat 4.9 uitgerold is.

@dennissiemensma
Copy link
Member

dennissiemensma commented Jan 17, 2021 via email

@mthmulders
Copy link
Contributor Author

Hmmm, deze komt voorbij:

--- Merging/updating checkout.
Updating 044c8127..2c1ce855
error: Your local changes to the following files would be overwritten by merge:
        dsmrreader/config/defaults.py
Please commit your changes or stash them before you merge.
Aborting

Ik heb de wijzigingen in dsmrreader/config/defaults.py ongedaan gemaakt en nu slaagt de deployment wel. Dat moet de oorzaak geweest zijn.

Interessant detail: ondanks dat de merge naar v.4.10.0 niet lukt gaat het script wel door. Deze fout stond vrij vroeg in de uitvoer, maar het script gaat door en geeft uiteindelijk geen duidelijk signaal dat één van de stappen mislukt is.

Los van zelf zorgvuldig de logs inspecteren (wijze les voor de volgende keer) zou het ook wel mooi zijn als het deploy script zelf signaleert dat er waarschijnlijk iets niet in orde is.

@dennissiemensma
Copy link
Member

Bedankt voor je terugkoppeling. Zo'n check lijkt me een goede toevoeging. Ik zal kijken of ik iets kan toevoegen.

@dennissiemensma dennissiemensma added this to the 4.12 milestone Jan 17, 2021
@dennissiemensma
Copy link
Member

Aangezien je bezig was met updaten, er is nu ook v4.11

@dennissiemensma dennissiemensma changed the title Update naar v4.10.0 lukt niet Bij update controleren voor lokale openstaande wijzigingen Jan 17, 2021
@dennissiemensma dennissiemensma changed the title Bij update controleren voor lokale openstaande wijzigingen Bij update controleren op lokale openstaande wijzigingen Jan 17, 2021
@dennissiemensma
Copy link
Member

Vanaf de v4.12 release zit er straks een extra check in:

 --- Checking for local file changes: 
 M deploy.sh

[!] Local file changes detected, aborting deployment
    To revert all files to their default, WITHOUT backup, execute:

    git reset --hard HEAD

@dennissiemensma
Copy link
Member

Onderdeel van v4.12

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

No branches or pull requests

2 participants