Skip to content

Commit 935f4e6

Browse files
Merge pull request #348 from rewantsoni/recipe-release-4.18
DFBUGS-2326: [release-4.18] bundle: add recipe op as a dependency
2 parents a6172b6 + a3aa88e commit 935f4e6

File tree

6 files changed

+47
-1
lines changed

6 files changed

+47
-1
lines changed

bundle/manifests/ocs-client-operator.clusterserviceversion.yaml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -7,7 +7,7 @@ metadata:
77
categories: Storage
88
console.openshift.io/plugins: '["odf-client-console"]'
99
containerImage: quay.io/ocs-dev/ocs-client-operator:latest
10-
createdAt: "2024-11-27T03:54:42Z"
10+
createdAt: "2025-04-21T13:41:01Z"
1111
description: OpenShift Data Foundation client operator enables consumption of
1212
storage services from a remote centralized OpenShift Data Foundation provider
1313
cluster.

bundle/metadata/dependencies.yaml

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -11,3 +11,7 @@ dependencies:
1111
value:
1212
packageName: noobaa-operator
1313
version: ">=5.17.0 <=5.18.0"
14+
- type: olm.package
15+
value:
16+
packageName: recipe
17+
version: "0.0.1"

config/metadata/dependencies.yaml

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -11,3 +11,7 @@ dependencies:
1111
value:
1212
packageName: noobaa-operator
1313
version: ">=5.17.0 <=5.18.0"
14+
- type: olm.package
15+
value:
16+
packageName: recipe
17+
version: "0.0.1"

hack/build-catalog.sh

Lines changed: 11 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -7,6 +7,7 @@ mkdir catalog
77

88
${OPM} render --output=yaml ${BUNDLE_IMG} > catalog/ocs-client-bundle.yaml
99
${OPM} render --output=yaml ${CSI_ADDONS_BUNDLE_IMG} > catalog/csi-adddons-bundle.yaml
10+
${OPM} render --output=yaml ${RECIPE_BUNDLE_IMG} > catalog/recipe.yaml
1011
${OPM} render --output=yaml ${NOOBAA_BUNDLE_IMG} > catalog/noobaa-operator-bundle.yaml
1112

1213
cat << EOF >> catalog/index.yaml
@@ -40,6 +41,16 @@ package: noobaa-operator
4041
name: alpha
4142
entries:
4243
- name: $NOOBAA_BUNDLE_NAME.v$NOOBAA_BUNDLE_VERSION
44+
---
45+
defaultChannel: alpha
46+
name: $RECIPE_BUNDLE_NAME
47+
schema: olm.package
48+
---
49+
schema: olm.channel
50+
package: recipe
51+
name: alpha
52+
entries:
53+
- name: $RECIPE_BUNDLE_NAME.v$RECIPE_BUNDLE_VERSION
4354
EOF
4455

4556
${OPM} validate catalog

hack/make-bundle-vars.mk

Lines changed: 8 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -64,6 +64,12 @@ OCS_CLIENT_CONSOLE_IMG_TAG ?= latest
6464
OCS_CLIENT_CONSOLE_IMG_LOCATION ?= $(IMAGE_REGISTRY)/$(REGISTRY_NAMESPACE)
6565
OCS_CLIENT_CONSOLE_IMG ?= $(OCS_CLIENT_CONSOLE_IMG_LOCATION)/$(OCS_CLIENT_CONSOLE_IMG_NAME):$(OCS_CLIENT_CONSOLE_IMG_TAG)
6666

67+
RECIPE_BUNDLE_NAME ?= recipe
68+
RECIPE_REGISTRY_NAMESPACE ?= ramendr
69+
RECIPE_BUNDLE_IMG_NAME ?= $(RECIPE_BUNDLE_NAME)-bundle
70+
RECIPE_BUNDLE_VERSION ?= 0.0.1
71+
RECIPE_BUNDLE_IMG_TAG ?= latest
72+
6773
# IMG defines the image used for the operator.
6874
IMG ?= $(IMAGE_REGISTRY)/$(REGISTRY_NAMESPACE)/$(IMAGE_NAME):$(IMAGE_TAG)
6975

@@ -73,6 +79,8 @@ BUNDLE_IMG ?= $(IMAGE_REGISTRY)/$(REGISTRY_NAMESPACE)/$(BUNDLE_IMAGE_NAME):$(IMA
7379
CSI_ADDONS_BUNDLE_IMG ?= $(CSI_ADDONS_IMAGE_REGISTRY)/$(CSI_ADDONS_REGISTRY_NAMESPACE)/$(CSI_ADDONS_BUNDLE_IMAGE_NAME):$(CSI_ADDONS_BUNDLE_IMAGE_TAG)
7480

7581

82+
RECIPE_BUNDLE_IMG ?= $(IMAGE_REGISTRY)/$(RECIPE_REGISTRY_NAMESPACE)/$(RECIPE_BUNDLE_IMG_NAME):$(RECIPE_BUNDLE_IMG_TAG)
83+
7684
# CATALOG_IMG defines the image used for the catalog.
7785
CATALOG_IMG ?= $(IMAGE_REGISTRY)/$(REGISTRY_NAMESPACE)/$(CATALOG_IMAGE_NAME):$(IMAGE_TAG)
7886

internal/controller/operatorconfigmap_controller.go

Lines changed: 19 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -271,6 +271,11 @@ func (c *OperatorConfigMapReconciler) Reconcile(ctx context.Context, req ctrl.Re
271271
return ctrl.Result{}, err
272272
}
273273

274+
if err := c.reconcileRecipeOperatorSubscription(); err != nil {
275+
c.log.Error(err, "unable to reconcile Recipe Operator subscription")
276+
return ctrl.Result{}, err
277+
}
278+
274279
if err := c.ensureConsolePlugin(); err != nil {
275280
c.log.Error(err, "unable to deploy client console")
276281
return ctrl.Result{}, err
@@ -676,6 +681,20 @@ func (c *OperatorConfigMapReconciler) reconcileCephCSIOperatorSubscription() err
676681
return nil
677682
}
678683

684+
func (c *OperatorConfigMapReconciler) reconcileRecipeOperatorSubscription() error {
685+
recipeOperatorSubscription, err := c.getSubscriptionByPackageName("recipe")
686+
if err != nil {
687+
return err
688+
}
689+
if c.subscriptionChannel != "" && c.subscriptionChannel != recipeOperatorSubscription.Spec.Channel {
690+
recipeOperatorSubscription.Spec.Channel = c.subscriptionChannel
691+
if err := c.update(recipeOperatorSubscription); err != nil {
692+
return fmt.Errorf("failed to update subscription channel of 'recipe' to %v: %v", c.subscriptionChannel, err)
693+
}
694+
}
695+
return nil
696+
}
697+
679698
func (c *OperatorConfigMapReconciler) reconcileWebhookService() error {
680699
svc := &corev1.Service{}
681700
svc.Name = templates.WebhookServiceName

0 commit comments

Comments
 (0)