Merge ~cjwatson/launchpad:charm-frontend-extras-librarian-rsync into launchpad:master
Proposed by
Colin Watson
Status: | Merged |
---|---|
Approved by: | Colin Watson |
Approved revision: | c4ebe030f0578e92fc13acc6debcc148cac95721 |
Merge reported by: | Otto Co-Pilot |
Merged at revision: | not available |
Proposed branch: | ~cjwatson/launchpad:charm-frontend-extras-librarian-rsync |
Merge into: | launchpad:master |
Diff against target: |
112 lines (+57/-2) 6 files modified
charm/launchpad-frontend-extras/README.md (+8/-2) charm/launchpad-frontend-extras/config.yaml (+6/-0) charm/launchpad-frontend-extras/layer.yaml (+1/-0) charm/launchpad-frontend-extras/metadata.yaml (+3/-0) charm/launchpad-frontend-extras/reactive/launchpad-frontend-extras.py (+25/-0) charm/launchpad-frontend-extras/templates/librarian-logs-rsync.conf.j2 (+14/-0) |
Related bugs: |
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
Guruprasad | Approve | ||
Review via email: mp+450385@code.launchpad.net |
Commit message
charm: Add librarian-logs rsync module to librarian frontend
Description of the change
The `launchpad-scripts` charm needs to be able to rsync librarian logs from the appropriate frontends so that it can parse them and update download counts. On the legacy frontends this is configured manually; on charmed frontends we need to configure it in code somewhere.
To avoid having to fork the `apache2` charm, we add a custom `librarian-logs` subordinate to `launchpad-
To post a comment you must log in.
LGTM 👍