Merge ~cjwatson/lp-codeimport:charm-storage-brz into lp-codeimport:master

Proposed by Colin Watson
Status: Merged
Approved by: Colin Watson
Approved revision: 5f9949eb9b737972a6fb0da4c64d88312a5ce533
Merge reported by: Otto Co-Pilot
Merged at revision: not available
Proposed branch: ~cjwatson/lp-codeimport:charm-storage-brz
Merge into: lp-codeimport:master
Diff against target: 12 lines (+5/-0)
1 file modified
charm/lp-codeimport-storage/layer.yaml (+5/-0)
Reviewer Review Type Date Requested Status
Colin Watson (community) Approve
Review via email: mp+439941@code.launchpad.net

Commit message

charm: Install brz on lp-codeimport-storage units

Description of the change

Otherwise bzr imports fail with `sh: 1: bzr: not found`, because pushing Bazaar branches needs `bzr` to be present on the server side.

(`brz` is not a typo; on jammy, the `bzr` command is provided by the `brz` package, a.k.a. Breezy.)

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

Trivial enough that I'll just go ahead and land this.

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1diff --git a/charm/lp-codeimport-storage/layer.yaml b/charm/lp-codeimport-storage/layer.yaml
2index 590c59f..7b51a45 100644
3--- a/charm/lp-codeimport-storage/layer.yaml
4+++ b/charm/lp-codeimport-storage/layer.yaml
5@@ -1,2 +1,7 @@
6 includes:
7 - layer:basic
8+ - layer:apt
9+options:
10+ apt:
11+ packages:
12+ - brz

Subscribers

People subscribed via source and target branches