Skip to content

Update PodExists function to return pod#448

Merged
openshift-merge-bot[bot] merged 1 commit into
openstack-k8s-operators:mainfrom
kstrenkova:update-pod-exists-function
May 28, 2026
Merged

Update PodExists function to return pod#448
openshift-merge-bot[bot] merged 1 commit into
openstack-k8s-operators:mainfrom
kstrenkova:update-pod-exists-function

Conversation

@kstrenkova
Copy link
Copy Markdown
Contributor

Replaced PodExists(bool) with GetPodIfExists(*Pod, error) to enable callers to access the pod object while distinguishing between "not found" and actual errors.

Replaced PodExists(bool) with GetPodIfExists(*Pod, error) to enable
callers to access the pod object while distinguishing between
"not found" and actual errors.
@openshift-ci openshift-ci Bot requested review from evallesp and lpiwowar May 28, 2026 08:17
@arxcruz
Copy link
Copy Markdown
Collaborator

arxcruz commented May 28, 2026

/lgtm

Copy link
Copy Markdown
Contributor

@sdatko sdatko left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

/lgtm /approve ❤️

@openshift-ci
Copy link
Copy Markdown

openshift-ci Bot commented May 28, 2026

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: kstrenkova, sdatko

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

@openshift-merge-bot openshift-merge-bot Bot merged commit 16033b8 into openstack-k8s-operators:main May 28, 2026
7 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants