Launchpad itself

Merge ~lgp171188/launchpad:fix-data-dir-typo-readme into launchpad:master

Proposed by Guruprasad on 2023-10-30

Status:	Needs review
Proposed branch:	~lgp171188/launchpad:fix-data-dir-typo-readme
Merge into:	launchpad:master
Diff against target:	28 lines (+5/-5) 1 file modified charm/launchpad-codehosting/README.md (+5/-5)
Related bugs:	Link a bug report

Reviewer	Review Type	Date Requested	Status
Colin Watson (community)		2023-10-30	Approve on 2023-10-30
Review via email: mp+454809@code.qastaging.launchpad.net

Commit message

charm/launchpad-codehosting: Fix typos in the README

The bzr repositories root directory was wrongly mentioned as
`/srv/launchpad/data/mirror` instead of `/srv/launchpad/data/mirrors`.

Revision history for this message

Colin Watson (cjwatson) wrote on 2023-10-30:

This is an improvement, but I think the volume should just be mounted on `/srv/launchpad/data`, since that matches what's done in other charms. We probably need to make sure that the `mirrors` subdirectory exists with the right ownership/permissions after mounting the volume, since the charm won't immediately be rerun to recreate it.

review: Approve

There was an error fetching revisions from git servers. Please try again in a few minutes. If the problem persists, contact Launchpad support.