Merge ~cjwatson/launchpad-mojo-specs:lp-codehosting-is-subordinate into launchpad-mojo-specs:master

Proposed by Colin Watson
Status: Merged
Merged at revision: 4c3cc685cbdb95d8af1e6f1887812dcd545fe1d0
Proposed branch: ~cjwatson/launchpad-mojo-specs:lp-codehosting-is-subordinate
Merge into: launchpad-mojo-specs:master
Diff against target: 12 lines (+0/-1)
1 file modified
lp/bundle.yaml (+0/-1)
Reviewer Review Type Date Requested Status
Ines Almeida Approve
Review via email: mp+454791@code.launchpad.net

Commit message

lp: Fix launchpad-codehosting deployment

Description of the change

`launchpad-codehosting` is a subordinate charm, so it can't have `num_units`.

To post a comment you must log in.
Revision history for this message
Ines Almeida (ines-almeida) :
review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1diff --git a/lp/bundle.yaml b/lp/bundle.yaml
2index cc5a264..19dca7f 100644
3--- a/lp/bundle.yaml
4+++ b/lp/bundle.yaml
5@@ -1304,7 +1304,6 @@ applications:
6 charm: ch:launchpad-codehosting
7 revision: 10
8 channel: stable
9- num_units: 1
10 options: {{ common_options("launchpad-codehosting") }}
11 active: {{ launchpad_codehosting_active }}
12 codehosting_private_ssh_key: include-base64://{{ local_dir }}/codehosting_ssh_host_key_rsa

Subscribers

People subscribed via source and target branches