Merge ~jugmac00/launchpad-buildd:add-information-about-architecture-on-production into launchpad-buildd:master

Proposed by Jürgen Gmach
Status: Merged
Approved by: Jürgen Gmach
Approved revision: 76c27cffade7dab01bf72d3b17df8a5ef1c24eba
Merge reported by: Otto Co-Pilot
Merged at revision: not available
Proposed branch: ~jugmac00/launchpad-buildd:add-information-about-architecture-on-production
Merge into: launchpad-buildd:master
Diff against target: 28 lines (+14/-1)
1 file modified
docs/explanation/deployment.rst (+14/-1)
Reviewer Review Type Date Requested Status
Guruprasad Approve
Review via email: mp+444925@code.launchpad.net

Commit message

Add additional information about production architecture

To post a comment you must log in.
Revision history for this message
Guruprasad (lgp171188) :
review: Approve
Revision history for this message
Guruprasad (lgp171188) wrote :

Jürgen, can you fix the typo in the spelling of 'architecture' in your commit message? I have fixed it already in the 'Commit message' above in this MP.

Revision history for this message
Colin Watson (cjwatson) :
Revision history for this message
Jürgen Gmach (jugmac00) wrote :

Thanks Guruprasad and Colin. I updated the wording in the documentation and the commit message.

I am waiting for some feedback from Colin on how to improve the last paragraph.

Revision history for this message
Colin Watson (cjwatson) :
Revision history for this message
Jürgen Gmach (jugmac00) :

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1diff --git a/docs/explanation/deployment.rst b/docs/explanation/deployment.rst
2index 1f0d3fa..29b5a88 100644
3--- a/docs/explanation/deployment.rst
4+++ b/docs/explanation/deployment.rst
5@@ -1,5 +1,6 @@
6+===========================
7 Production deployment notes
8-***************************
9+===========================
10
11 In Launchpad's production build farm, launchpad-buildd is deployed via base
12 virtual machine images constructed by taking standard Ubuntu cloud images
13@@ -8,3 +9,15 @@ https://launchpad.net/~canonical-is-sa/+archive/ubuntu/buildd. This is done
14 by
15 https://code.launchpad.net/~canonical-sysadmins/canonical-is-charms/launchpad-buildd-image-modifier
16 (currently private, sorry).
17+
18+We deliberately run builders in virtual machines rather than containers
19+for the following reasons:
20+
21+- avoiding issues with nested containerization
22+- containers are not secure enough against being escaped by malicious code
23+
24+------------------
25+Additional context
26+------------------
27+
28+Charm recipe builds, `Launchpad CI <https://help.launchpad.net/Code/ContinuousIntegration>`_, live filesystem builds, OCI recipe builds, and snap recipe builds all build in LXD containers. Everything else builds in chroots.

Subscribers

People subscribed via source and target branches