Merge ~twom/launchpad:db-oci-recipe-build-path into launchpad:db-devel

Proposed by Tom Wardill
Status: Merged
Approved by: Tom Wardill
Approved revision: be39e7da6465416a85cff3f2079b776390451fba
Merge reported by: Otto Co-Pilot
Merged at revision: not available
Proposed branch: ~twom/launchpad:db-oci-recipe-build-path
Merge into: launchpad:db-devel
Diff against target: 16 lines (+10/-0)
1 file modified
database/schema/patch-2210-20-0.sql (+10/-0)
Reviewer Review Type Date Requested Status
William Grant db Approve
Colin Watson (community) db Approve
Review via email: mp+389228@code.launchpad.net

Commit message

Add OCIRecipe.build_path

To post a comment you must log in.
Revision history for this message
Colin Watson (cjwatson) wrote :

I'm OK with this, but get a second review from William on general principles.

OCIRecipe has 11 rows on staging, so rewriting the table shouldn't be a problem.

review: Approve (db)
Revision history for this message
William Grant (wgrant) :
review: Approve (db)

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1diff --git a/database/schema/patch-2210-20-0.sql b/database/schema/patch-2210-20-0.sql
2new file mode 100644
3index 0000000..e1e812d
4--- /dev/null
5+++ b/database/schema/patch-2210-20-0.sql
6@@ -0,0 +1,10 @@
7+-- Copyright 2020 Canonical Ltd. This software is licensed under the
8+-- GNU Affero General Public License version 3 (see the file LICENSE).
9+
10+SET client_min_messages=ERROR;
11+
12+ALTER TABLE OCIRecipe ADD COLUMN build_path text DEFAULT '.' NOT NULL;
13+
14+COMMENT ON COLUMN OCIRecipe.build_path IS 'Directory to use for build context and OCIRecipe.build_file location.';
15+
16+INSERT INTO LaunchpadDatabaseRevision VALUES (2210, 20, 0);

Subscribers

People subscribed via source and target branches

to status/vote changes: