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

(PPS-819): Indexd Migration Script #2599

Merged
merged 5 commits into from
Sep 10, 2024
Merged

(PPS-819): Indexd Migration Script #2599

merged 5 commits into from
Sep 10, 2024

Conversation

BinamB
Copy link
Contributor

@BinamB BinamB commented Jul 19, 2024

Link to JIRA ticket if there is one:

New Features

  • Add job to migrate to single table indexd

@BinamB BinamB changed the title Indexd Migration Script (PPS-819): Indexd Migration Script Jul 19, 2024
Comment on lines 39 to 41
- name: creds-volume-bk
secret:
secretName: "indexd-bk-creds" # pragma: allowlist secret
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 it's indexd-creds without "bk"

Copy link
Contributor Author

Choose a reason for hiding this comment

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

it should be indexd-new-creds if we're following the migration script from the doc. We need to create a new creds file for the new database we create. We do the migration on the newly created clone database and once the migration is done, we swap the creds for the environment to use the newly cloned database.

Copy link
Contributor

Choose a reason for hiding this comment

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

oh ok, that makes sense 👍 could you also add a link to the doc at the top of this file?

@AidanHilt AidanHilt merged commit 0e02d83 into master Sep 10, 2024
8 checks passed
@AidanHilt AidanHilt deleted the feat/indexd-migration branch September 10, 2024 17:22
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