Skip to content

Conversation

@nixpanic
Copy link
Member

@nixpanic nixpanic commented Jan 23, 2026

It seems that in some scenarios the kubectl replace command fails
because the ConfigMap does not exist (yet?). By checking for the
existance of the ConfigMap, and running either create or replace,
this spurious issue should not happen anymore.

See-also: #5962

It seems that in some scenarios the `kubectl replace` command fails
because the ConfigMap does not exist (yet?). By checking for the
existance of the ConfigMap, and running either `create` or `replace`,
this spurious issue should not happen anymore.

Signed-off-by: Niels de Vos <ndevos@ibm.com>
@nixpanic nixpanic added the ci/skip/multi-arch-build skip building on multiple architectures label Jan 23, 2026
@mergify mergify bot added the component/testing Additional test cases or CI work label Jan 23, 2026
@nixpanic
Copy link
Member Author

/test ci/centos/k8s-e2e-external-storage/1.33

@nixpanic
Copy link
Member Author

The CI job that failed constantly now passed a manual run: https://jenkins-ceph-csi.apps.ocp.cloud.ci.centos.org/job/k8s-e2e-external-storage-1.33/547/display/redirect

This should be good to get merged and make it possible for other PRs to pass CI again.

@nixpanic nixpanic requested a review from a team January 23, 2026 09:07
@mergify mergify bot added the queued label Jan 23, 2026
mergify bot added a commit that referenced this pull request Jan 23, 2026
@mergify
Copy link
Contributor

mergify bot commented Jan 23, 2026

Merge Queue Status

✅ The pull request has been merged at c258387

This pull request spent 34 minutes 7 seconds in the queue, including 33 minutes 50 seconds running CI.
The checks were run on draft #5969.

Required conditions to merge
  • #approved-reviews-by >= 2 [🛡 GitHub branch protection]
  • #changes-requested-reviews-by = 0 [🛡 GitHub branch protection]
  • any of:
    • all of:
      • base=devel
      • status-success=codespell
      • status-success=go-test
      • status-success=golangci-lint
      • status-success=lint-extras
      • status-success=mod-check
      • status-success=multi-arch-build
      • status-success=uncommitted-code-check
      • any of:
        • label=ci/skip/e2e
        • all of:
          • status-success=ci/centos/k8s-e2e-external-storage/1.32
          • status-success=ci/centos/k8s-e2e-external-storage/1.33
          • status-success=ci/centos/k8s-e2e-external-storage/1.34
          • status-success=ci/centos/mini-e2e-helm/k8s-1.32
          • status-success=ci/centos/mini-e2e-helm/k8s-1.33
          • status-success=ci/centos/mini-e2e-helm/k8s-1.34
          • status-success=ci/centos/mini-e2e/k8s-1.32
          • status-success=ci/centos/mini-e2e/k8s-1.33
          • status-success=ci/centos/mini-e2e/k8s-1.34
          • status-success=ci/centos/upgrade-tests-cephfs
          • status-success=ci/centos/upgrade-tests-rbd
    • all of:
      • base~=^(release-.+)$
      • status-success=codespell
      • status-success=go-test
      • status-success=golangci-lint
      • status-success=lint-extras
      • status-success=mod-check
      • status-success=multi-arch-build
      • status-success=uncommitted-code-check
      • any of:
        • label=ci/skip/e2e
        • all of:
          • status-success=ci/centos/k8s-e2e-external-storage/1.32
          • status-success=ci/centos/k8s-e2e-external-storage/1.33
          • status-success=ci/centos/k8s-e2e-external-storage/1.34
          • status-success=ci/centos/mini-e2e-helm/k8s-1.32
          • status-success=ci/centos/mini-e2e-helm/k8s-1.33
          • status-success=ci/centos/mini-e2e-helm/k8s-1.34
          • status-success=ci/centos/mini-e2e/k8s-1.32
          • status-success=ci/centos/mini-e2e/k8s-1.33
          • status-success=ci/centos/mini-e2e/k8s-1.34
          • status-success=ci/centos/upgrade-tests-cephfs
          • status-success=ci/centos/upgrade-tests-rbd
    • all of:
      • base=release-v3.15
      • status-success=codespell
      • status-success=go-test
      • status-success=golangci-lint
      • status-success=lint-extras
      • status-success=mod-check
      • status-success=multi-arch-build
      • status-success=uncommitted-code-check
      • any of:
        • label=ci/skip/e2e
        • all of:
          • status-success=ci/centos/k8s-e2e-external-storage/1.31
          • status-success=ci/centos/k8s-e2e-external-storage/1.32
          • status-success=ci/centos/k8s-e2e-external-storage/1.33
          • status-success=ci/centos/mini-e2e-helm/k8s-1.31
          • status-success=ci/centos/mini-e2e-helm/k8s-1.32
          • status-success=ci/centos/mini-e2e-helm/k8s-1.33
          • status-success=ci/centos/mini-e2e/k8s-1.31
          • status-success=ci/centos/mini-e2e/k8s-1.32
          • status-success=ci/centos/mini-e2e/k8s-1.33
          • status-success=ci/centos/upgrade-tests-cephfs
          • status-success=ci/centos/upgrade-tests-rbd
    • all of:
      • base=ci/centos
      • status-success=ci/centos/jjb-validate
      • status-success=ci/centos/job-validation

@mergify mergify bot merged commit cba25cb into ceph:devel Jan 23, 2026
29 checks passed
@mergify mergify bot removed the queued label Jan 23, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ci/skip/e2e skip running e2e CI jobs ci/skip/multi-arch-build skip building on multiple architectures component/testing Additional test cases or CI work

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants