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

Migration and Update Guide for 5 to 7 #1029

Merged
merged 29 commits into from
Nov 28, 2024
Merged

Migration and Update Guide for 5 to 7 #1029

merged 29 commits into from
Nov 28, 2024

Conversation

mmattel
Copy link
Contributor

@mmattel mmattel commented Oct 25, 2024

References:
#1028 (Migration and Update Guide for 5 to 7)
#1034 (CLI: health check for services (to be added in maintenance commands)
#1038 (Update the compose examples to be ready to upgrade for the next release)
owncloud/ocis#10439 (Restart Postprocessing Properly) (CLI change)
owncloud/ocis#10499 ([docs-only] Create envvar delta files after v5 to master (v7.0.0))
owncloud/ocis#10566 ([docs-only] Improve error messages for health command output)

Updates the migration, update and envvar changes documentation.

Currently visible on staging

  • Added/Removed envvar .adoc table files in ocis/docs/services/general-info/envvar-deltas
  • Added/Changed/Removed CLI commands
  • Config changes when running ocis init --diff
  • Breaking changes
    activitylog: service_account: + value in ocis.yaml (via ocis init --diff)
    collaboration: wopi.secret + value in ocis.yaml (via ocis init --diff)
  • Added content that is ocis 7 relevant found during update writing
    (There will be most likely more content that gets added post merging)

IMPORTANT
These changes also require a final docs branching where we reference to versioned ocis content that MUST be provided to make this final. Some things get automatically referenced with branching pointing to the correct ressources included.

@mmattel mmattel added the documentation Improvements or additions to documentation label Oct 25, 2024
@mmattel mmattel force-pushed the migrate_5_to_7 branch 2 times, most recently from baf2cfa to 4b3abb5 Compare October 28, 2024 13:38
@mmattel mmattel force-pushed the migrate_5_to_7 branch 3 times, most recently from 6f12743 to 92b8ff9 Compare October 30, 2024 09:04
@mmattel mmattel force-pushed the migrate_5_to_7 branch 2 times, most recently from a840b2c to 2a0d61c Compare November 4, 2024 15:00
@mmattel mmattel marked this pull request as ready for review November 7, 2024 13:36
@mmattel mmattel requested review from micbar and kobergj November 7, 2024 13:36
@ScharfViktor ScharfViktor self-requested a review November 11, 2024 15:26
@ScharfViktor
Copy link
Contributor

step https://doc.staging.owncloud.com/ocis/next/migration/upgrading_5.0.x_7.0.0.html#update-config-settings for docker doesn't work because patch command is currently missing in the ocis docker image

image

@mmattel
Copy link
Contributor Author

mmattel commented Nov 11, 2024

for docker doesn't work because patch command is currently missing

A commit has been added with a note what to do if the patch command is missing.
Master/7 has the patch command, rolling not.

modules/ROOT/pages/maintenance/commands/changed-cli.adoc Outdated Show resolved Hide resolved
modules/ROOT/pages/maintenance/commands/changed-cli.adoc Outdated Show resolved Hide resolved
modules/ROOT/pages/maintenance/commands/commands.adoc Outdated Show resolved Hide resolved
modules/ROOT/pages/migration/upgrading_5.0.x_7.0.0.adoc Outdated Show resolved Hide resolved
modules/ROOT/pages/migration/upgrading_5.0.x_7.0.0.adoc Outdated Show resolved Hide resolved
modules/ROOT/pages/migration/upgrading_5.0.x_7.0.0.adoc Outdated Show resolved Hide resolved
modules/ROOT/pages/migration/upgrading_5.0.x_7.0.0.adoc Outdated Show resolved Hide resolved
modules/ROOT/pages/migration/upgrading_5.0.x_7.0.0.adoc Outdated Show resolved Hide resolved
Copy link
Contributor

@micbar micbar left a comment

Choose a reason for hiding this comment

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

Approving. Overall, everything is fine. Small tweaks can be made later in subsequent PRs.

@mmattel mmattel merged commit 330fdc3 into master Nov 28, 2024
1 check passed
@mmattel mmattel deleted the migrate_5_to_7 branch November 28, 2024 09:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation Improvements or additions to documentation
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants