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) #6640

Closed
wants to merge 24 commits into from

Conversation

m-ildefons
Copy link

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

Modifications to the Longhorn helm-chart for the Object Storage gateway for Longhorn.
This includes updates to the CRDs, cluster roles and daemonsets.

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

LEP: #5832

@mergify
Copy link

mergify bot commented Sep 6, 2023

This pull request is now in conflicts. Could you fix it @m-ildefons? 🙏

@mergify
Copy link

mergify bot commented Oct 4, 2023

This pull request is now in conflicts. Could you fix it @m-ildefons? 🙏

- Add config options and flags for object gateway
- Add CRDs for the object endpoint
- Regenerate depoy/longhorn.yaml

Related-to: longhorn#5832
Signed-off-by: Moritz Röhrich <moritz.rohrich@suse.com>
Change image of the longhorn manager to custom build.

Signed-off-by: Moritz Röhrich <moritz.rohrich@suse.com>
Signed-off-by: Moritz Röhrich <moritz.rohrich@suse.com>
Signed-off-by: Moritz Röhrich <moritz.rohrich@suse.com>
Signed-off-by: Moritz Röhrich <moritz.rohrich@suse.com>
Signed-off-by: Moritz Röhrich <moritz.rohrich@suse.com>
Signed-off-by: Moritz Röhrich <moritz.rohrich@suse.com>
Accept size and storage class for the backing volume as settings for the
object endpoint. Since a specific volume is not required in this
position, instead just the size and storage class may be given. The
controller then takes care that a PVC with those resource requests will
be created, which causes the CSI driver to create the backing volume
accordingly.

Signed-off-by: Moritz Röhrich <moritz.rohrich@suse.com>
Signed-off-by: Moritz Röhrich <moritz.rohrich@suse.com>
Make the ObjectEndpoint CRD cluster scoped to allow the dependent
resources to live in the longhorn-system namespace.

Signed-off-by: Moritz Röhrich <moritz.rohrich@suse.com>
Pull custom UI builds to test modifications to Longhorn UI.

Signed-off-by: Moritz Röhrich <moritz.rohrich@suse.com>
- Update CRDs
- Update s3gw version

Signed-off-by: Moritz Röhrich <moritz.rohrich@suse.com>
- Rename ObjectEndpoint CRD to ObjectStore
- Update CRDs
- Add permissions to uninstall to cluster role
- Add UI container config values to helm chart

Signed-off-by: Moritz Röhrich <moritz.rohrich@suse.com>
Update CRDs.

Signed-off-by: Moritz Röhrich <moritz.rohrich@suse.com>
Signed-off-by: Moritz Röhrich <moritz.rohrich@suse.com>
Signed-off-by: Moritz Röhrich <moritz.rohrich@suse.com>
Update CRDs after fixing spelling

Signed-off-by: Moritz Röhrich <moritz.rohrich@suse.com>
- deploy ingress as path typre "Prefix"
- configure service account to be able to create/modify/list/get
  ingresses for LH manager
- Update CRDs

Signed-off-by: Moritz Röhrich <moritz.rohrich@suse.com>
- Re-add missing comments stripped out somehow
- Add comments to new added s3gw image repositories
- Remove previous merge conflicts

Signed-off-by: Volker Theile <vtheile@suse.com>
Copy link

mergify bot commented Oct 31, 2023

This pull request is now in conflicts. Could you fix it @m-ildefons? 🙏

Add stub storage class for object stores. This is necessary to allow
volume expansion as volumes without a storage class can not be expanded.

Signed-off-by: Moritz Röhrich <moritz.rohrich@suse.com>
- Update CRDs

- Make storage class "longhorn-objectstorage-static" a config map

Signed-off-by: Moritz Röhrich <moritz.rohrich@suse.com>
Signed-off-by: Moritz Röhrich <moritz.rohrich@suse.com>
Fix name and labels in storage class config map

Signed-off-by: Moritz Röhrich <moritz.rohrich@suse.com>
- Clean up diff for CRDs. There were some bogus diffs on this branch,
  which likely resulted from using different version of the code
  generation tool for longhorn-manager.

- Set container versions for s3gw and s3gw-ui to v0.23.0

Signed-off-by: Moritz Röhrich <moritz.rohrich@suse.com>
@innobead innobead closed this Dec 5, 2023
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.

None yet

4 participants