Skip to content

Update backup job's default backoff limit to 1#1605

Merged
dkwon17 merged 1 commit intodevfile:mainfrom
dkwon17:update-backoff
Mar 19, 2026
Merged

Update backup job's default backoff limit to 1#1605
dkwon17 merged 1 commit intodevfile:mainfrom
dkwon17:update-backoff

Conversation

@dkwon17
Copy link
Collaborator

@dkwon17 dkwon17 commented Mar 19, 2026

What does this PR do?

Updates the default backup cron job backoff limit to 1:
image

What issues does this PR fix or reference?

Is it tested? How?

Tested with this catalog source:

apiVersion: operators.coreos.com/v1alpha1
kind: CatalogSource
metadata:
  name: devworkspace-operator-catalog
  namespace: openshift-marketplace
spec:
  displayName: DevWorkspace Operator Catalog
  image: 'quay.io/dkwon17/devworkspace-operator-index:update-backoff'
  publisher: Red Hat
  sourceType: grpc
  updateStrategy:
    registryPoll:
      interval: 5m

After installing DWO, create a new DWOC object that does not define the backup job's backoff limit:

kind: DevWorkspaceOperatorConfig
metadata:
  name: devworkspace-operator-config
  namespace: openshift-operators
apiVersion: controller.devfile.io/v1alpha1
config:                                                                                                                                         
  workspace:
    backupCronJob:
      enable: true
      registry:
        path: default-route-openshift-image-registry.apps.sno-ocp-415.my.openshift.lab
      schedule: '* * * * *'
      oras:
        extraArgs: '--insecure'

After creating the DWOC object, the backoffLimit: 1 should be added automatically.

PR Checklist

  • E2E tests pass (when PR is ready, comment /test v8-devworkspace-operator-e2e, v8-che-happy-path to trigger)
    • v8-devworkspace-operator-e2e: DevWorkspace e2e test
    • v8-che-happy-path: Happy path for verification integration with Che

@openshift-ci
Copy link

openshift-ci bot commented Mar 19, 2026

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: dkwon17

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Details Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

Signed-off-by: dkwon17 <dakwon@redhat.com>
@dkwon17 dkwon17 merged commit 2d6966e into devfile:main Mar 19, 2026
11 checks passed
dkwon17 added a commit to dkwon17/devworkspace-operator that referenced this pull request Mar 19, 2026
Signed-off-by: dkwon17 <dakwon@redhat.com>
dkwon17 added a commit that referenced this pull request Mar 19, 2026
Signed-off-by: dkwon17 <dakwon@redhat.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant