Skip to content

[TECH] Ajout d'un feature toggle de restoration incrémentale de la BDD - #41

Merged
VincentHardouin merged 7 commits into
masterfrom
add-feature-toggle-for-incremental-restore
Oct 29, 2020
Merged

[TECH] Ajout d'un feature toggle de restoration incrémentale de la BDD#41
VincentHardouin merged 7 commits into
masterfrom
add-feature-toggle-for-incremental-restore

Conversation

@VincentHardouin

@VincentHardouin VincentHardouin commented Oct 28, 2020

Copy link
Copy Markdown
Contributor

🦄 Problème

Nous voulons mettre en place un système de restoration de façon incrémental. Cependant le code actuel ne nous permet pas de le faire car il supprime le schéma de la base.

🤖 Solution

Ajout d'un feature toggle afin de choisir si on supprime le schéma ou non.

💯 Pour tester

Test sur l'application pix-datawarehouse-test sur la zone osc-secnum-fr1 pour avoir les données de production

Vérifier que la log contient le message
"msg":"Full replication and enrichment done"

Non-régression: vérifier que les données de la veille au soir (ex: le 29 => 28 au soir) sont présentes dans la table users

scalingo --region osc-secnum-fr1 --app pix-datawarehouse-test pgsql-console
SELECT MAX("createdAt") from users;
 2020-10-28 22:59:24.66568+00

Evolution: vérifier que les données de la veille au soir (ex: le 29 => 28 au soir) ne sont pas présentes dans la table answsers

scalingo --region osc-secnum-fr1 --app pix-datawarehouse-test pgsql-console
SELECT MAX("createdAt") from answers;
 2020-10-28 01:08:00.353+00

@VincentHardouin
VincentHardouin force-pushed the add-feature-toggle-for-incremental-restore branch from 49f20a9 to c14e82a Compare October 28, 2020 14:00
@octo-topi
octo-topi force-pushed the add-feature-toggle-for-incremental-restore branch from 82da81f to 97f2c8b Compare October 28, 2020 16:50
@octo-topi octo-topi changed the title [TECH] Ajout d'un feature toggle pour faire une restoration de la base de façon incrémental [TECH] Ajout d'un feature toggle de restoration incrémentale de la BDD Oct 28, 2020
@VincentHardouin
VincentHardouin force-pushed the add-feature-toggle-for-incremental-restore branch from 97f2c8b to fc33b98 Compare October 28, 2020 18:17
Comment thread test/integration/steps_test.js Outdated
@VincentHardouin
VincentHardouin force-pushed the add-feature-toggle-for-incremental-restore branch from fc33b98 to 3f5225c Compare October 29, 2020 07:44
Comment thread README.md Outdated
@VincentHardouin
VincentHardouin force-pushed the add-feature-toggle-for-incremental-restore branch from c9ba626 to 39d2d9f Compare October 29, 2020 08:46
@VincentHardouin
VincentHardouin merged commit 325b54e into master Oct 29, 2020
@VincentHardouin
VincentHardouin deleted the add-feature-toggle-for-incremental-restore branch October 29, 2020 08:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

cross-team Toutes les équipes de dev

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants