launchpad-reviewers team mailing list archive

[Tom Wardill <tom.wardill@xxxxxxxxxxxxx>]

Tom Wardill has proposed merging ~twom/launchpad:db-oci-recipe-should-require-git-repository into launchpad:db-devel.

Commit message:
Make git_repository column required in OCIRecipe

Requested reviews:
  Launchpad code reviewers (launchpad-reviewers)

For more details, see:
https://code.launchpad.net/~twom/launchpad/+git/launchpad/+merge/379960

Can't build an OCIRecipe without the source to build it from, so git_repository should be NOT NULL.
-- 
Your team Launchpad code reviewers is requested to review the proposed merge of ~twom/launchpad:db-oci-recipe-should-require-git-repository into launchpad:db-devel.

diff --git a/database/schema/patch-2210-08-4.sql b/database/schema/patch-2210-08-4.sql
new file mode 100644
index 0000000..2ad0e9d
--- /dev/null
+++ b/database/schema/patch-2210-08-4.sql
@@ -0,0 +1,8 @@
+-- Copyright 2020 Canonical Ltd.  This software is licensed under the
+-- GNU Affero General Public License version 3 (see the file LICENSE).
+
+SET client_min_messages=ERROR;
+
+ALTER TABLE OCIRecipe ALTER COLUMN git_repository SET NOT NULL;
+
+INSERT INTO LaunchpadDatabaseRevision VALUES (2210, 08, 4);