Skip to content

Conversation

gabrielcocenza
Copy link
Member

  • prerequisites and integration steps
  • backup/restore examples with juju run-action syntax
  • explanations for cluster vs namespaced backups

- prerequisites and integration steps
- backup/restore examples with juju run-action syntax
- explanations for cluster vs namespaced backups
@gabrielcocenza gabrielcocenza requested a review from a team as a code owner August 22, 2025 21:13
@gabrielcocenza gabrielcocenza force-pushed the velero-infra-documentation branch from d5abace to 1f815e3 Compare August 22, 2025 21:24
@gabrielcocenza gabrielcocenza force-pushed the velero-infra-documentation branch from 1f815e3 to 8976b45 Compare August 22, 2025 21:25
Copy link
Contributor

@addyess addyess left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks so much

@gabrielcocenza gabrielcocenza force-pushed the velero-infra-documentation branch from 408869c to 570a4df Compare August 26, 2025 20:49
@gabrielcocenza gabrielcocenza requested a review from addyess August 26, 2025 21:17
@gabrielcocenza gabrielcocenza force-pushed the velero-infra-documentation branch from 570a4df to f434d1e Compare August 26, 2025 21:18
@addyess addyess requested a review from nhennigan August 28, 2025 18:51
@addyess
Copy link
Contributor

addyess commented Aug 28, 2025

adding @nhennigan

Copy link
Contributor

@nhennigan nhennigan left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks @gabrielcocenza for the contribution! I have given my initial review. Final review and merging of this PR should wait until we have internal approval of the spec that goes along with this work. 😸

and restore for workloads across namespaces, including non-Juju-managed ones.

This guide shows how to setup Velero Operator with the [s3-integrator charm][]
as the storage provider and with the [infra-backup-operator][] to backup the
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
as the storage provider and with the [infra-backup-operator][] to backup the
as the storage provider and the [infra-backup-operator][] to backup the

This guide shows how to setup Velero Operator with the [s3-integrator charm][]
as the storage provider and with the [infra-backup-operator][] to backup the
configuration of **any** kind of Kubernetes distribution (Canonical Kubernetes,
MicroK8s, EKS, etc.).
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

While the infra backup operator may be able to back up any config, this is the Canonical Kubernetes docs so I don't think there is a need to mention the other products here

configuration of **any** kind of Kubernetes distribution (Canonical Kubernetes,
MicroK8s, EKS, etc.).

## What the infra backup operator does
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Is the infra backup operator capitalized? We should aim for consistency across the page :)


## What you will need

- A kubernetes cluster
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
- A kubernetes cluster
- A {product} cluster


- A kubernetes cluster
- A bootstrapped K8s controller. See the [Juju documentation]
- An S3 bucket or a S3 compatible bucket like [MinIO] or [MicroCeph]
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
- An S3 bucket or a S3 compatible bucket like [MinIO] or [MicroCeph]
- A S3 bucket or a S3 compatible bucket like [MinIO] or [MicroCeph]

juju integrate infra-backup-operator:namespaced-infra-backup velero-operator
```

### Create the Backup
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
### Create the Backup
### Create the backup

### Restore

In case of disaster recovery, users can restore the cluster configuration in
the same cluster or in a different one using the Velero operator juju-action.
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
the same cluster or in a different one using the Velero operator juju-action.
the same cluster or in a different one using the Velero operator Juju action.

This will guarantee that the cluster configuration can be easily restored to
start receiving the workloads.

Before restore your cluster must have velero-operator deployed and integrated
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
Before restore your cluster must have velero-operator deployed and integrated
Before restoration, your cluster must have the Velero Operator deployed and integrated

with the same bucket of the backup.

```bash
# example output
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

We try not to put the command with the output in the same code block as our users like to use the copy button provided in the rendered RTD version of our docs. Having them together means they have to edit the copied command and this can be frustrating

phase: Completed
start-timestamp: "2025-08-07T18:42:10Z"

# restore the backups
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Again I would split this into another code block and explain now that we have verified the backup and we have it installed, we can run the restore command

- A kubernetes cluster
- A bootstrapped K8s controller. See the [Juju documentation]
- An S3 bucket or a S3 compatible bucket like [MinIO] or [MicroCeph]
- A {product} cluster
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Sorry I gave you the wrong formatting in my last review, there needs to be double brackets

Suggested change
- A {product} cluster
- A {{product}} cluster

- A bootstrapped K8s controller. See the Juju [add-k8s] documentation
- A S3 bucket or a S3 compatible bucket like [MinIO] or [MicroCeph]

### Deploy and Integrate
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
### Deploy and Integrate
### Deploy and integrate

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants