Use --yes flag for migrations everywhere

Signed-off-by: aeneasr <[email protected]>
ory · Apr 27, 2019 · c7e7aa0 · c7e7aa0
1 parent 2c09d20
commit c7e7aa0
Show file tree

Hide file tree

Showing 6 changed files with 17 additions and 7 deletions.
diff --git a/UPGRADE.md b/UPGRADE.md
@@ -139,6 +139,15 @@ secure deployment with zero effort? We can run it for you! If you're interested,
 
 ## 1.0.0-rc.10
 
+### Schema Changes
+
+Please read all paragraphs of this section with the utmost care, before executing `hydra migrate sql`. Do
+not take this change lightly and create a backup of the database before you begin. To be sure, copy the database
+and do a dry-run locally.
+
+> Be aware that running these migrations might take some time when using large databases. Do a dry-run before hammering
+your production database.
+
 ### SQL Migrations now require user-input or `--yes` flag
 
 `hydra migrate sql` now shows an execution plan and asks for confirmation before executing the migrations. To run

diff --git a/quickstart-mysql.yml b/quickstart-mysql.yml
@@ -19,7 +19,7 @@ services:
     environment:
       - DSN=mysql://root:secret@tcp(mysqld:3306)/mysql?max_conns=20&max_idle_conns=4
     command:
-      migrate sql -e
+      migrate sql -e --yes
     restart: on-failure
 
   hydra:

diff --git a/quickstart-postgres.yml b/quickstart-postgres.yml
@@ -19,7 +19,7 @@ services:
     environment:
       - DSN=postgres://hydra:secret@postgresd:5432/hydra?sslmode=disable&max_conns=20&max_idle_conns=4
     command:
-      migrate sql -e
+      migrate sql -e --yes
     restart: on-failure
 
   hydra:

diff --git a/test/e2e/circle-ci.bash b/test/e2e/circle-ci.bash
@@ -42,14 +42,14 @@ case "$1" in
             ;;
 
         postgres)
-            hydra migrate sql $TEST_DATABASE_POSTGRESQL
+            hydra migrate sql --yes $TEST_DATABASE_POSTGRESQL
             DSN=$TEST_DATABASE_POSTGRESQL \
                 hydra serve all --dangerous-force-http --disable-telemetry >> ./hydra.e2e.log 2>&1 &
             export CYPRESS_jwt_enabled=false
             ;;
 
         postgres-jwt)
-            hydra migrate sql $TEST_DATABASE_POSTGRESQL
+            hydra migrate sql --yes $TEST_DATABASE_POSTGRESQL
             DSN=$TEST_DATABASE_POSTGRESQL \
                 OAUTH2_ACCESS_TOKEN_STRATEGY=jwt \
                 OIDC_SUBJECT_IDENTIFIERS_ENABLED=public \
@@ -58,13 +58,14 @@ case "$1" in
             ;;
 
         mysql)
-            hydra migrate sql $TEST_DATABASE_MYSQL
+            hydra migrate sql --yes $TEST_DATABASE_MYSQL
             DSN=$TEST_DATABASE_MYSQL \
                 hydra serve all --dangerous-force-http --disable-telemetry >> ./hydra.e2e.log 2>&1 &
             export CYPRESS_jwt_enabled=false
             ;;
 
         mysql-jwt)
+            hydra migrate sql --yes $TEST_DATABASE_MYSQL
             DSN=$TEST_DATABASE_MYSQL \
                 OAUTH2_ACCESS_TOKEN_STRATEGY=jwt \
                 OIDC_SUBJECT_IDENTIFIERS_ENABLED=public \

diff --git a/test/e2e/docker-compose.mysql.yml b/test/e2e/docker-compose.mysql.yml
@@ -7,7 +7,7 @@ services:
     environment:
       - DSN=mysql://root:secret@tcp(mysqld:3306)/mysql?max_conns=20&max_idle_conns=4
     command:
-      migrate sql -e
+      migrate sql -e --yes
     restart: on-failure
 
   hydra:

diff --git a/test/e2e/docker-compose.postgres.yml b/test/e2e/docker-compose.postgres.yml
@@ -7,7 +7,7 @@ services:
     environment:
       - DSN=postgres://hydra:secret@postgresd:5432/hydra?sslmode=disable&max_conns=20&max_idle_conns=4
     command:
-      migrate sql -e
+      migrate sql -e --yes
     restart: on-failure
 
   hydra: