Skip to content

[DNM] Check s2i switch - #804

Open
karelyatin wants to merge 1 commit into
openstack-k8s-operators:mainfrom
karelyatin:s2i_check
Open

karelyatin wants to merge 1 commit into
openstack-k8s-operators:mainfrom
karelyatin:s2i_check

Conversation

@karelyatin

@karelyatin karelyatin commented Sep 18, 2026 •

Copy link
Copy Markdown
Contributor

Depends-On: openstack-k8s-operators/openstack-operator#2070

Describe your changes

Jira Ticket Link

Jira:

Checklist before requesting a review

  • I have performed a self-review of my code and confirmed it passes tests
  • Performed pre-commit run --all
  • Tested operator image in a test/dev environment. It can be CRC via install_yamls or a hotstack instance (optional)
  • Verified that no failures present in logs(optional):
    • ironic-operator-build-deploy-kuttl
    • podified-multinode-ironic-deployment

@openshift-ci
openshift-ci Bot requested review from mumesan and steveb September 18, 2026 05:11
@coderabbitai

coderabbitai Bot commented Sep 18, 2026 •

Copy link
Copy Markdown

Important

  • 🔍 Trigger review

This repository does not receive automatic reviews because it has fewer than 10 stars.

⚙️ Run configuration

Configuration used: Central YAML (base), Organization UI (inherited)

Review profile: CHILL

Plan: Advanced

Run ID: 135e81ae-8be2-4d76-96c6-3b0ca4304a82


Comment @coderabbitai help to get the list of available commands.

@openshift-ci

openshift-ci Bot commented Sep 18, 2026

Copy link
Copy Markdown
Contributor

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by: karelyatin
Once this PR has been reviewed and has the lgtm label, please assign stuggi for approval. For more information see the Code Review Process.

The full list of commands accepted by this bot can be found 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

@centosinfra-prod-github-app

Copy link
Copy Markdown

Build failed (check pipeline). Post recheck (without leading slash)
to rerun all jobs. Make sure the failure cause has been resolved before
you rerun jobs.

https://gateway-cloud-softwarefactory.apps.ocp.cloud.ci.centos.org/zuul/t/rdoproject.org/buildset/79d54ddf74024e55bedd795aec0f9940

✔️ openstack-k8s-operators-content-provider SUCCESS in 1h 27m 35s
❌ podified-multinode-ironic-deployment FAILURE in 1h 08m 04s
✔️ cifmw-pod-zuul-files SUCCESS in 5m 14s

@abhibongale

abhibongale commented Sep 22, 2026 •

Copy link
Copy Markdown
Contributor

/recheck

@centosinfra-prod-github-app

Copy link
Copy Markdown

Build failed (check pipeline). Post recheck (without leading slash)
to rerun all jobs. Make sure the failure cause has been resolved before
you rerun jobs.

https://gateway-cloud-softwarefactory.apps.ocp.cloud.ci.centos.org/zuul/t/rdoproject.org/buildset/275b06c4c0ae40f29670b3f1d3ae4d52

✔️ openstack-k8s-operators-content-provider SUCCESS in 6h 04m 26s
❌ podified-multinode-ironic-deployment FAILURE in 1h 08m 23s
✔️ cifmw-pod-zuul-files SUCCESS in 4m 39s

@karelyatin

Copy link
Copy Markdown
Contributor Author

recheck additional depends-on

@centosinfra-prod-github-app

Copy link
Copy Markdown

Build failed (check pipeline). Post recheck (without leading slash)
to rerun all jobs. Make sure the failure cause has been resolved before
you rerun jobs.

https://gateway-cloud-softwarefactory.apps.ocp.cloud.ci.centos.org/zuul/t/rdoproject.org/buildset/1c1be779eb114242a1a3a6883a81188e

❌ openstack-k8s-operators-content-provider FAILURE in 15m 10s
⚠️ podified-multinode-ironic-deployment SKIPPED Skipped due to failed job openstack-k8s-operators-content-provider
✔️ cifmw-pod-zuul-files SUCCESS in 4m 46s

Depends-On: openstack-k8s-operators/openstack-operator#2070
Signed-off-by: Yatin Karel <ykarel@redhat.com>
@centosinfra-prod-github-app

Copy link
Copy Markdown

Build failed (check pipeline). Post recheck (without leading slash)
to rerun all jobs. Make sure the failure cause has been resolved before
you rerun jobs.

https://gateway-cloud-softwarefactory.apps.ocp.cloud.ci.centos.org/zuul/t/rdoproject.org/buildset/3b298408fb384f28a51ceebd611985c4

✔️ openstack-k8s-operators-content-provider SUCCESS in 1h 26m 41s
❌ podified-multinode-ironic-deployment FAILURE in 1h 07m 47s
✔️ cifmw-pod-zuul-files SUCCESS in 4m 32s

@mumesan

mumesan commented Sep 24, 2026

Copy link
Copy Markdown
Contributor

recheck

@abhibongale

Copy link
Copy Markdown
Contributor

recheck copy annotation to api and conductors CR

@centosinfra-prod-github-app

Copy link
Copy Markdown

Merge Failed.

This change or one of its cross-repo dependencies was unable to be automatically merged with the current state of its repository. Please rebase the change and upload a new patchset.
Warning:
Error merging github.com/openstack-k8s-operators/openstack-operator for 2087,be552804f3c6263ea97bcfa4b2bbd960c07573a2

@mumesan

mumesan commented Sep 24, 2026

Copy link
Copy Markdown
Contributor

recheck

@abhibongale

Copy link
Copy Markdown
Contributor

o/ @karelyatin, can you please rebase the PR. @mumesan provided the annotation fix, but need a rebase. ty :)

@karelyatin

Copy link
Copy Markdown
Contributor Author

2087 is rebased so we are good now

@centosinfra-prod-github-app

Copy link
Copy Markdown

This change depends on a change that failed to merge.

Change openstack-k8s-operators/openstack-operator#2087 is needed.

@mumesan

mumesan commented Sep 24, 2026

Copy link
Copy Markdown
Contributor

recheck

@centosinfra-prod-github-app

Copy link
Copy Markdown

@karelyatin

Copy link
Copy Markdown
Contributor Author

recheck final concluded patch

@centosinfra-prod-github-app

Copy link
Copy Markdown

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.

3 participants