charm-juju-backup-all

Merge ~sudeephb/charm-juju-backup-all/+git/charm-juju-backup-all-1:documentation into charm-juju-backup-all:master

  1. Git
  2. lp:~sudeephb/charm-juju-backup-all/+git/charm-juju-backup-all-1
  3. documentation
  4. Merge into master
Proposed by Sudeep Bhandari
Status: Merged
Approved by: Robert Gildein
Approved revision: dd80c4f78326229084efcc1bdbb7c62c47c90897
Merged at revision: 04af661e2a6d67485f1fbcb98329999303b6623f
Proposed branch: ~sudeephb/charm-juju-backup-all/+git/charm-juju-backup-all-1:documentation
Merge into: charm-juju-backup-all:master
Diff against target: 12 lines (+1/-0)
1 file modified
metadata.yaml (+1/-0)
Reviewer Review Type Date Requested Status
Tianqi Xiao (community) Approve
Robert Gildein Approve
🤖 prod-jenkaas-bootstack (community) continuous-integration Needs Fixing
Review via email: mp+448748@code.launchpad.net

Commit message

Add discourse documentation

To post a comment you must log in.
Revision history for this message
🤖 Canonical IS Merge Bot (canonical-is-mergebot) wrote :

This merge proposal is being monitored by mergebot. Change the status to Approved to merge.

Revision history for this message
🤖 prod-jenkaas-bootstack (prod-jenkaas-bootstack) wrote :

FAILED: Continuous integration, rev:dd80c4f78326229084efcc1bdbb7c62c47c90897
https://jenkins.canonical.com/bootstack/job/lp-charm-juju-backup-all-ci/15/
Executed test runs:
    FAILURE: https://jenkins.canonical.com/bootstack/job/lp-charm-test-functest/1478/
    None: https://jenkins.canonical.com/bootstack/job/lp-update-mp/401/

Click here to trigger a rebuild:
https://jenkins.canonical.com/bootstack/job/lp-charm-juju-backup-all-ci/15//rebuild

review: Needs Fixing (continuous-integration)
Revision history for this message
🤖 prod-jenkaas-bootstack (prod-jenkaas-bootstack) wrote :

FAILED: Continuous integration, rev:dd80c4f78326229084efcc1bdbb7c62c47c90897
https://jenkins.canonical.com/bootstack/job/lp-charm-juju-backup-all-ci/16/
Executed test runs:
    FAILURE: https://jenkins.canonical.com/bootstack/job/lp-charm-test-functest/1479/
    None: https://jenkins.canonical.com/bootstack/job/lp-update-mp/402/

Click here to trigger a rebuild:
https://jenkins.canonical.com/bootstack/job/lp-charm-juju-backup-all-ci/16//rebuild

review: Needs Fixing (continuous-integration)
Revision history for this message
🤖 prod-jenkaas-bootstack (prod-jenkaas-bootstack) wrote :

FAILED: Continuous integration, rev:dd80c4f78326229084efcc1bdbb7c62c47c90897
https://jenkins.canonical.com/bootstack/job/lp-charm-juju-backup-all-ci/17/
Executed test runs:
    FAILURE: https://jenkins.canonical.com/bootstack/job/lp-charm-test-functest/1481/
    None: https://jenkins.canonical.com/bootstack/job/lp-update-mp/404/

Click here to trigger a rebuild:
https://jenkins.canonical.com/bootstack/job/lp-charm-juju-backup-all-ci/17//rebuild

review: Needs Fixing (continuous-integration)
Revision history for this message
🤖 prod-jenkaas-bootstack (prod-jenkaas-bootstack) wrote :

FAILED: Continuous integration, rev:dd80c4f78326229084efcc1bdbb7c62c47c90897
https://jenkins.canonical.com/bootstack/job/lp-charm-juju-backup-all-ci/18/
Executed test runs:
    FAILURE: https://jenkins.canonical.com/bootstack/job/lp-charm-test-functest/1482/
    None: https://jenkins.canonical.com/bootstack/job/lp-update-mp/405/

Click here to trigger a rebuild:
https://jenkins.canonical.com/bootstack/job/lp-charm-juju-backup-all-ci/18//rebuild

review: Needs Fixing (continuous-integration)
Revision history for this message
Robert Gildein (rgildein) wrote :

LGTM

review: Approve
Revision history for this message
Tianqi Xiao (txiao) wrote :

LGTM. CI resulted in failure, though unrelated to the change, we should at least create a task to look into cause.

review: Approve
Revision history for this message
🤖 Canonical IS Merge Bot (canonical-is-mergebot) wrote :

Change successfully merged at revision 04af661e2a6d67485f1fbcb98329999303b6623f

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1diff --git a/metadata.yaml b/metadata.yaml
2index 608043b..384a4f1 100644
3--- a/metadata.yaml
4+++ b/metadata.yaml
5@@ -6,6 +6,7 @@
6 name: juju-backup-all
7 display-name: juju-backup-all
8 summary: A charm to backup Juju config and databases
9+docs: https://discourse.charmhub.io/t/juju-backup-all-docs-index/11321
10 description: |
11 This charm leverges Juju-backup-all to allow operators to perform backups
12 across all controllers and models. It provides autodiscovery, backing up all

Subscribers

People subscribed via source and target branches

to all changes: