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

nextcloud migration #2377

Merged
merged 84 commits into from May 2, 2024
Merged

nextcloud migration #2377

merged 84 commits into from May 2, 2024

Conversation

stavros-k
Copy link
Contributor

@stavros-k stavros-k commented Apr 12, 2024

No description provided.

@stavros-k stavros-k added the WIP Work In Progress label Apr 12, 2024
@stavros-k stavros-k self-assigned this Apr 12, 2024
@stavros-k
Copy link
Contributor Author

stavros-k commented Apr 16, 2024

TODO's:

  • Port all settings from previous version
  • Add tests for most common config scenarios (http, https, etc)
  • Figure out the storage part (Take into account add additional mount for user data #1557)
  • Write migration script
  • Check nginx config that is up-to-date with nextcloud's recommendations (check overview for warnings) Will be done on separate PR as I'd like to also take into account NC 29 potential suggestions
  • Update to_keep_versions accordingly Will do right after nextcloud: pre-migration actions #2412 is merged
  • Handle the db back, because nextcloud creates its own user and database on intial setup and ignores the predifined one we setup
  • Test upgrades from 2.0.0 to 2.0.1 (backups performed etc)
  • Test upgrades from 1.x.x to 2.x.x
  • After everything is up and running, plug redis in [Issue] NextCloud Application - Missing Redis DB #1304, which should be less than 15lines of code and a nice improvement.~

@frederickjansen
Copy link

@stavros-k could you please add this as well if you haven't already? #2078
Thanks!

@stavros-k
Copy link
Contributor Author

@stavros-k could you please add this as well if you haven't already? #2078 Thanks!

I can do right after this PR, but I believe that nextcloud container does not have the required packages installed by default for that to work.

@stavros-k stavros-k requested a review from Qubad786 April 29, 2024 12:44
@stavros-k stavros-k removed the WIP Work In Progress label Apr 29, 2024
@stavros-k stavros-k marked this pull request as ready for review April 29, 2024 12:44
@stavros-k stavros-k linked an issue Apr 29, 2024 that may be closed by this pull request
@stavros-k stavros-k linked an issue Apr 29, 2024 that may be closed by this pull request
@stavros-k stavros-k merged commit fbc87af into master May 2, 2024
10 checks passed
@stavros-k stavros-k deleted the nextcloud-migration branch May 2, 2024 20:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Nextcloud data volume unexpected behavior [Issue] NextCloud Application - Missing Redis DB
3 participants