Skip to content

Conversation

jonathan-gramain
Copy link

Add support for the metadata migration components, when explicitly enabled in the features yaml.

It adds:

  • a standalone deployment for the migration bucketd and map

  • a container with the migration tools worker and API server

Add support for the metadata migration components, when explicitly
enabled in the features yaml.

It adds:

- a standalone deployment for the migration bucketd and map

- a container with the migration tools worker and API server
@tcarmet tcarmet requested a review from tmacro September 19, 2025 11:57
- ./config/migration-tools/supervisord.conf:/conf/supervisord.conf:ro
- ./config/migration-tools/migration.yml:/conf/migration.yml:ro
- ./config/migration-tools/env:/conf/env:ro
- ./logs/migration-tools:/logs:rw
Copy link
Contributor

Choose a reason for hiding this comment

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

I think there's somewhere in the code where I make the sure the folders logs/<some component> exists, (can't remember the exact folder but a grep should find it) otherwise we face some permissions issues with the containers vs host and it's annoying to deal with.

Could you please just add it into the mix? thanks

Copy link
Contributor

Choose a reason for hiding this comment

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

Found it:

func createLogDirectories(envDir string) error {

Copy link
Author

Choose a reason for hiding this comment

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

Copy link
Contributor

Choose a reason for hiding this comment

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

yes 🤦 don't know why I didn't saw it, I'm tired, ignore me

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