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

ynh_backup_before_upgrade #161

Merged
merged 1 commit into from May 11, 2021
Merged

ynh_backup_before_upgrade #161

merged 1 commit into from May 11, 2021

Conversation

yalh76
Copy link
Contributor

@yalh76 yalh76 commented May 9, 2021

Problem

  • Just a thinking: during upgrade we start to make some changes to ensure downward compatibility, before doing a backup and even stopping the service, I think backup and stopping the service should be done before making some changes ...
  • I'm currently working on bitwarden to vaultwarden name chnage with ynh_handle_app_migration

Solution

  • Make backup and service stop, come first

PR Status

  • Code finished.
  • Tested with Package_check.
  • Fix or enhancement tested.
  • Upgrade from last version tested.
  • Can be reviewed and tested.

Package_check results


  • An automatic package_check will be launch at https://ci-apps-dev.yunohost.org/, when you add a specific comment to your Pull Request: "!testme", "!gogogadgetoci" or "By the power of systemd, I invoke The Great App CI to test this Pull Request!"*

@yalh76 yalh76 changed the title Update upgrade ynh_backup_before_upgrade May 9, 2021
Copy link
Member

@alexAubin alexAubin left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yup, don't know why the backup was done historically after we start doing changes ...

Not sure about the service stop though, maybe it doesn't matter, idk

@yalh76 yalh76 marked this pull request as ready for review May 11, 2021 18:02
@yalh76 yalh76 merged commit e3878cd into master May 11, 2021
@yalh76 yalh76 deleted the ynh_backup_before_upgrade branch May 11, 2021 18:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
2 participants