Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Object Storage (LEP 20230430) #148

Closed
wants to merge 1 commit into from

Conversation

m-ildefons
Copy link

@m-ildefons m-ildefons commented Sep 14, 2023

Implementation of backup functionality for the object store.
The object store consists of a normal longhorn volume and an object storage gateway (s3gw) instance. In case of a backup, the details of the object storage gateway deployment must be backed up as well, to be able to restore the entire object store. Therefore an object store backup carries additional metadata in the form of the deployment details of the object storage gateway.

Related: longhorn/longhorn#6640
Related: longhorn/longhorn-manager#2136
Related: longhorn/longhorn-ui#649
Related: longhorn/longhorn-engine#934
Related: longhorn/longhorn-instance-manager#285

LEP: longhorn/longhorn#5832

@@ -23,6 +23,7 @@ type Volume struct {
CompressionMethod string `json:",string"`
StorageClassName string `json:",string"`
BackendStoreDriver string `json:",string"`
ObjectStoreBackup string `json:",string"`
Copy link
Member

Choose a reason for hiding this comment

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

What's the value of this property? ObjectStoreBackup seems too general, so not sure what things will be saved here.

@@ -46,6 +47,7 @@ type Backup struct {
Labels map[string]string
IsIncremental bool
CompressionMethod string
ObjectStoreBackup string
Copy link
Member

Choose a reason for hiding this comment

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

What's the value of this property?

Signed-off-by: Moritz Röhrich <moritz.rohrich@suse.com>
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.

2 participants