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

Team feature migration tool #4472

Open
wants to merge 6 commits into
base: develop
Choose a base branch
from
Open

Conversation

pcapriotti
Copy link
Contributor

@pcapriotti pcapriotti commented Feb 26, 2025

Team feature migration tool.

We decided that a data migration is not the best way to move feature configurations to the new table. Instead, this PR adds a new migrate-features tool that can be used to manually perform the migration.

https://wearezeta.atlassian.net/browse/WPB-16238

Checklist

  • Add a new entry in an appropriate subdirectory of changelog.d
  • Read and follow the PR guidelines

@zebot zebot added the ok-to-test Approved for running tests in CI, overrides not-ok-to-test if both labels exist label Feb 26, 2025
@pcapriotti pcapriotti marked this pull request as ready for review February 26, 2025 13:34
@pcapriotti pcapriotti requested review from a team as code owners February 26, 2025 13:34
@supersven
Copy link
Contributor

supersven commented Feb 26, 2025

We're using this Job manifest to run it on staging:

apiVersion: batch/v1
kind: Job
metadata:
  name: migrate-features
spec:
  template:
    spec:
      containers:
        - name: migrate-features
          image: quay.io/wire/migrate-features:0.0.2-pr.10263
          args:
            [
              --cassandra-host-galley,
              galley-galley-eks-service.databases.svc.cluster.local,
              --cassandra-port-galley,
              "9042",
              --cassandra-keyspace-galley,
              galley,
            ]
      restartPolicy: Never
  backoffLimit: 4

If it succeeds, we may add this ☝️ (with the released image tag) to the docs.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ok-to-test Approved for running tests in CI, overrides not-ok-to-test if both labels exist
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants