Skip to content

Conversation

@IgorBenko
Copy link
Contributor

When importing the migration tar archive we expect there to be two folders: dist and artifact

That's why the composer folder must be renamed to dist and artifacts folder to artifact.

@glaubinix the only thing I don't quite understand is that the SelfHostedStorageMigrationCommand is also expecting these two folders on the source system - at least when using FlySystem/S3. I presume there is some logic somewhere that maps composer and artifacts folders in local FS accordingly, but couldn't find that. Please let me know if I'm missing something here.

@IgorBenko IgorBenko requested a review from a team November 27, 2024 13:56
@glaubinix
Copy link
Member

@IgorBenko not quite sure what you mean. The tar command with --transform renames the folders to what we expect. The import command then extracts those files onto the filesystem before uploading them to the desired storage.

Copy link
Member

@glaubinix glaubinix left a comment

Choose a reason for hiding this comment

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

Looks good. Feel free to merge, if the previous comment clears up everything.

@glaubinix glaubinix merged commit 1188d26 into main Nov 27, 2024
@glaubinix glaubinix deleted the docs/add-info-on-storage-migration branch November 27, 2024 15:26
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

Successfully merging this pull request may close these issues.

3 participants