Skip to content

improve: use onList() for ghost resource cleanup#3348

Merged
csviri merged 1 commit into
operator-framework:mainfrom
csviri:onlist-for-ghost-resources
May 14, 2026
Merged

improve: use onList() for ghost resource cleanup#3348
csviri merged 1 commit into
operator-framework:mainfrom
csviri:onlist-for-ghost-resources

Conversation

@csviri
Copy link
Copy Markdown
Collaborator

@csviri csviri commented May 13, 2026

Signed-off-by: Attila Mészáros a_meszaros@apple.com

@openshift-ci openshift-ci Bot added the do-not-merge/work-in-progress Indicates that a PR should not merge because it is a work in progress. label May 13, 2026
@csviri csviri changed the title improve: use onList() for host resource handling [WIP] improve: use onList() for host resource handling May 13, 2026
@csviri csviri marked this pull request as ready for review May 13, 2026 10:20
Copilot AI review requested due to automatic review settings May 13, 2026 10:20
@openshift-ci openshift-ci Bot requested review from metacosm and xstefank May 13, 2026 10:20
Signed-off-by: Attila Mészáros <a_meszaros@apple.com>
@csviri csviri force-pushed the onlist-for-ghost-resources branch from 768d5da to 4875146 Compare May 13, 2026 10:20
@csviri csviri changed the title [WIP] improve: use onList() for host resource handling improve: use onList() for ghost resource handling May 13, 2026
@openshift-ci openshift-ci Bot removed the do-not-merge/work-in-progress Indicates that a PR should not merge because it is a work in progress. label May 13, 2026
@csviri csviri requested a review from shawkins May 13, 2026 10:20
@csviri csviri changed the title improve: use onList() for ghost resource handling improve: use onList() for ghost resource cleanup May 13, 2026
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Copilot encountered an error and was unable to review this pull request. You can try again by re-requesting a review.

@csviri
Copy link
Copy Markdown
Collaborator Author

csviri commented May 14, 2026

@shawkins if you have the bandwidth pls take a look

@csviri
Copy link
Copy Markdown
Collaborator Author

csviri commented May 14, 2026

@metacosm can we merge this or you need more time to review?

Copy link
Copy Markdown
Collaborator

@shawkins shawkins left a comment

Choose a reason for hiding this comment

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

LGTM

@csviri
Copy link
Copy Markdown
Collaborator Author

csviri commented May 14, 2026

thank you!

@csviri csviri merged commit d9a476d into operator-framework:main May 14, 2026
34 of 37 checks passed
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.

Followup on ghost resource handling regarding fabric8 client improvement

4 participants