-
Notifications
You must be signed in to change notification settings - Fork 7
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
Open
mmattel
wants to merge
29
commits into
master
Choose a base branch
from
migrate_5_to_7
base: master
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Open
Changes from all commits
Commits
Show all changes
29 commits
Select commit
Hold shift + click to select a range
bda0bde
Migration and Update Guide Changes
mmattel 4d33718
overview update
mmattel 7234243
ocis init --diff (not finished)
mmattel e335f89
update details (breaking changes)
mmattel 83f4018
updates
mmattel 3db96d8
Apply suggestions from code review
mmattel 7d43f4b
add changed cli commands
mmattel d0075a0
more cli command updates
mmattel b2c9884
add searvice health
mmattel d2f7561
link service health
mmattel 40e6066
text fixes
mmattel 3cb4a4b
docker run changes
mmattel 6c363aa
move cli changes
mmattel fd5ef9b
Apply suggestions from code review
mmattel b667915
cli restart and resume
mmattel d9523b4
small fixes
mmattel e26696a
update envvars changes
mmattel 8fea923
numbered sections (unfinished)
mmattel 619ed12
numbered sections (finished)
mmattel ef8ce22
update binary upgrade steps
mmattel 0e8e22b
add note if patch command is missing
mmattel c3fa54a
Merge branch 'master' into migrate_5_to_7
mmattel 80a3bc2
add collaboration related stuff
mmattel 734fee2
fix collaboration reason
mmattel d5f058c
fixes service health text
mmattel fab442c
Merge branch 'master' into migrate_5_to_7
mmattel ed37ccb
docs: avoid sloppy language. Throu -> trough; dangling adverb 'first'…
jnweiger e1be535
Merge pull request #1049 from owncloud/language-review-5-7
mmattel 2360692
Apply suggestions from code review
mmattel File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,37 @@ | ||
= Changed or Added CLI Commands | ||
:toc: right | ||
:description: This page contains a list with added, changed or removed CLI commands between Infinite Scale version 5.0.x and 7.0.0. | ||
|
||
== Introduction | ||
|
||
{description} | ||
|
||
== Affected CLI Commands | ||
|
||
See the link for a detailed description of the respective CLI command if available. | ||
|
||
* xref:maintenance/commands/commands.adoc#manage-expired-uploads[Manage Expired Uploads] + | ||
The `ocis storage-users uploads sessions --restart` command got an alternative `--resume` option. + | ||
Resume can be used in the same way as restart with a slighly different behavior change. | ||
|
||
* xref:maintenance/commands/commands.adoc#purge-expired-space-trash-bin-items[Purge Expired Space Trash-Bin Items] + | ||
The `ocis storage-users uploads` got restructured. + | ||
The deprecated `list` option is now removed, the `clean` option is now part of the `sessions command`. | ||
|
||
* xref:maintenance/commands/commands.adoc#resume-post-processing[Resume Post-Processing] + | ||
The `ocis postprocessing restart` command got an alternative `resume` option. + | ||
Resume can be used in the same way as restart with a slighly different behavior change. | ||
|
||
* xref:maintenance/commands/commands.adoc#reset-password-for-idm-users[Reset Password for IDM Users] + | ||
The `ocis idm resetpassword` can now specify the user name via the `--user-name` (`-u`) flag. | ||
|
||
* xref:maintenance/commands/commands.adoc#revisions-cleanup[Revisions Cleanup] + | ||
The `ocis revisions purge` command allows removing revisions of files in the storage. Note that this command has also been backported to version 5 available with its latest release. | ||
|
||
* xref:maintenance/commands/commands.adoc#service-health[Service Health] + | ||
A `health` command has been added to each service: `ocis <service-name> health`. | ||
|
||
* xref:maintenance/commands/commands.adoc#trash-purge[Trash Purge]. + | ||
The `ocis trash purge-empty-dirs` command allows removing empty folders from the trashbin. | ||
|
||
* The `ocis graph list-unified-roles` command simplifies the process of finding out which UID belongs to which role. Note that this command is described in the https://github.com/owncloud/ocis/tree/master/ocis#list-unified-roles[ocis repository, window=_blank] and has been added for completeness only. |
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Making a copy in this way does not help. the original binary file is still locked for swapping by the kernel.
I'd prefer to move away the running binary - then we can add a new version with the original name.
E.g. the following commands can be done without stopping ocis:
The symlink is used here to document the versions, active and inactive.
(A simpler sequence - with a longer absence - would just use wget -O after mv.)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This procedure is a good idea, if that works - needs testing, I like it.
With that, we can easily switch versions and keep track of original naming.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Works fine -- for me 🤷♂️ at least.
I simply avoid the ETXTBSY overwrite by moving the old binary away. That works, because a busy text file is about the inode of the file, not the name of the file (kernel only cares about inodes here).
Symlinks sometimes cause confusion; but here it looks smooth, and I could not think of anything that would break.
(All this is still not an atomic upgrade. The technically best approach is to use a hardlink and a rename system call.)