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

Revert "[ceph] prepare upgrade to rook v1.16" #7580

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
8 changes: 4 additions & 4 deletions system/cc-ceph/Chart.lock
Original file line number Diff line number Diff line change
Expand Up @@ -4,9 +4,9 @@ dependencies:
version: 1.0.0
- name: rook-ceph
repository: https://charts.rook.io/release
version: v1.16.0
version: v1.15.0
- name: rook-crds
repository: oci://keppel.eu-de-1.cloud.sap/ccloud-helm
version: 0.0.2-rook.1.16.0
digest: sha256:d9a8ec1509dcec3a634aada46cbf9434897bfa7bf85bad9adb8c6af157aa08d4
generated: "2024-12-18T11:34:58.26072907+01:00"
version: 0.0.2-rook.1.15.0
digest: sha256:d8d894ba706ae4a8216ecf28bb9561fafc3b88908f183ba0b6d4b77f296e92f8
generated: "2024-09-12T10:38:14.973622-04:00"
8 changes: 4 additions & 4 deletions system/cc-ceph/Chart.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -2,16 +2,16 @@ apiVersion: v2
name: cc-ceph
description: A Helm chart for the Rook / Ceph Objects inside the Storage Clusters
type: application
version: 1.1.2
appVersion: "1.16.0"
version: 1.1.1
appVersion: "1.15.0"
dependencies:
- name: owner-info
repository: oci://keppel.eu-de-1.cloud.sap/ccloud-helm
version: '>= 0.0.0'
- name: rook-ceph
# version update should be done in the rook-crds chart as well
repository: https://charts.rook.io/release
version: 1.16.0
version: 1.15.0
- name: rook-crds
repository: oci://keppel.eu-de-1.cloud.sap/ccloud-helm
version: '0.0.2-rook.1.16.0'
version: '0.0.2-rook.1.15.0'
110 changes: 0 additions & 110 deletions system/cc-ceph/templates/cephobjectstore-extra.yaml

This file was deleted.

Original file line number Diff line number Diff line change
@@ -1,3 +1,4 @@
{{- if .Values.rgwTargetPlacements.enabled }}
{{- range $target := .Values.rgwTargetPlacements.placements }}
---
apiVersion: ceph.rook.io/v1
Expand Down Expand Up @@ -79,6 +80,7 @@ spec:
nodelete: {{ $.Values.pool.nodelete | quote }}
nosizechange: {{ $.Values.pool.nosizechange | quote }}
{{- end }}
{{- end }}
{{- if .Values.rgwTargetPlacements.premiumPlacements }}
{{- range $target := .Values.rgwTargetPlacements.premiumPlacements }}
---
Expand Down
18 changes: 0 additions & 18 deletions system/cc-ceph/templates/cephobjectstore.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -4,23 +4,8 @@ metadata:
name: {{ .Values.objectstore.name }}
namespace: {{ .Release.Namespace }}
spec:
{{- if .Values.objectstore.multiInstance.enabled }}
zone:
name: {{ .Values.objectstore.name }}
{{- else if and .Values.rgwTargetPlacements.useRookCRD .Values.rgwTargetPlacements.placements }}
sharedPools:
poolPlacements:
{{- range $target := .Values.rgwTargetPlacements.placements }}
- name: {{ $target.name }}
metadataPoolName: {{ $target.name }}.rgw.buckets.index
dataPoolName: {{ $target.name }}.rgw.buckets.data
dataNonECPoolName: {{ $target.name }}.rgw.buckets.non-ec
default: {{ $target.default | default false }}
{{- end }}
{{- else }}
metadataPool: {{ toYaml .Values.objectstore.metadataPool | nindent 4 }}
dataPool: {{ toYaml .Values.objectstore.dataPool | nindent 4 }}
{{- end }}
hosting:
{{- if gt (len .Values.objectstore.gateway.dnsNames) 0 }}
advertiseEndpoint:
Expand Down Expand Up @@ -75,9 +60,6 @@ spec:
tokenCacheSize: {{ .token_cache_size }}
url: {{ .url }}
protocols:
{{- if $.Values.objectstore.enabledAPIs }}
enableAPIs: {{ toYaml $.Values.objectstore.enabledAPIs | nindent 6 }}
{{- end }}
s3:
authUseKeystone: true
swift:
Expand Down
25 changes: 0 additions & 25 deletions system/cc-ceph/templates/certificate-extra.yaml

This file was deleted.

26 changes: 0 additions & 26 deletions system/cc-ceph/templates/record-extra.yaml

This file was deleted.

25 changes: 0 additions & 25 deletions system/cc-ceph/templates/service-extra.yaml

This file was deleted.

28 changes: 1 addition & 27 deletions system/cc-ceph/values.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -90,7 +90,6 @@ dashboard:
objectstore:
enabled: true
name: objectstore
enabledAPIs: [] # empty - all enabled. See: https://docs.ceph.com/en/reef/radosgw/config-ref/#confval-rgw_enable_apis
gateway:
instances: 6
port: 80
Expand Down Expand Up @@ -150,30 +149,6 @@ objectstore:
password: XXX
domain: XXX
project: XXX
multiInstance:
enabled: false
extraInstances:
# can inherit/override all config options from objectstore:
# - name: objectstore-admin
# gateway:
# instances: 2
# sslCertificateRef: ""
# dnsNames:
# - dns1-adm
# - dns2-adm
# resources:
# requests:
# cpu: 1
# memory: 2Gi
# service:
# name: ceph-objectstore-admin-external
# port: 443
# externalIP: "10.0.0.1"
# selector:
# app: rgw
# define other RGW instances here:
# - name: other-instance-name

prysm:
enabled: true
repository:
Expand All @@ -183,8 +158,7 @@ objectstore:
interval: "10"

rgwTargetPlacements:
# enabled: false deprecate rgwTargetPlacements.enabled because it is true on all envs
useRookCRD: false # !!!WARNING set 'true' only for new clusters. Upgrade will not work now.
enabled: false

defaultRgwPools:
enabled: false # create default rgw pools, see: https://github.com/sapcc/helm-charts/issues/6670
Expand Down
Loading