Skip to content

Conversation

@JoaoFula
Copy link
Contributor

Description

Scaling up operator pod to run finalizer on olsconfig deletion

Type of change

  • Refactor
  • New feature
  • Bug fix
  • CVE fix
  • Optimization
  • Documentation Update
  • Configuration Update
  • Bump-up dependent library
  • Bump-up library or tool used for development (does not change the final image)
  • CI configuration change
  • Konflux configuration change

Related Tickets & Documents

  • Related Issue #
  • Closes #

Checklist before requesting a review

  • I have performed a self-review of my code.
  • PR has passed all pre-merge test jobs.
  • If it is a core feature, I have added thorough tests.

Testing

  • Please provide detailed steps to perform tests related to this code change.
  • How were the fix/results from this change verified? Please provide relevant screenshots or results.

@openshift-ci openshift-ci bot requested review from bparees and tisnik January 23, 2026 13:44
@blublinsky
Copy link
Contributor

/lgtm

@openshift-ci openshift-ci bot added the lgtm Indicates that a PR is ready to be merged. label Jan 23, 2026
@xrajesh
Copy link
Contributor

xrajesh commented Jan 23, 2026

/approve

@openshift-ci
Copy link

openshift-ci bot commented Jan 23, 2026

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: xrajesh

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-ci openshift-ci bot added the approved Indicates a PR has been approved by an approver from all required OWNERS files. label Jan 23, 2026
@openshift-ci-robot
Copy link

/retest-required

Remaining retests: 0 against base HEAD c3d176f and 2 for PR HEAD 3124b0e in total

@xrajesh
Copy link
Contributor

xrajesh commented Jan 23, 2026

/override "Red Hat Konflux / lightspeed-service-on-pull-request"
/override "Red Hat Konflux / ols-enterprise-contract / lightspeed-service"

@openshift-ci
Copy link

openshift-ci bot commented Jan 23, 2026

@xrajesh: Overrode contexts on behalf of xrajesh: Red Hat Konflux / lightspeed-service-on-pull-request, Red Hat Konflux / ols-enterprise-contract / lightspeed-service

Details

In response to this:

/override "Red Hat Konflux / lightspeed-service-on-pull-request"
/override "Red Hat Konflux / ols-enterprise-contract / lightspeed-service"

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository.

@JoaoFula JoaoFula force-pushed the fix-olsconfig-delete-timeout branch from 3124b0e to 3550fa0 Compare January 23, 2026 18:43
@openshift-ci openshift-ci bot removed the lgtm Indicates that a PR is ready to be merged. label Jan 23, 2026
@openshift-ci
Copy link

openshift-ci bot commented Jan 23, 2026

New changes are detected. LGTM label has been removed.

deactivating more tests

deactivating more tests
@JoaoFula JoaoFula force-pushed the fix-olsconfig-delete-timeout branch from 3550fa0 to 3a24962 Compare January 23, 2026 18:45
@xrajesh
Copy link
Contributor

xrajesh commented Jan 23, 2026

/hold

@openshift-ci openshift-ci bot added the do-not-merge/hold Indicates that a PR should not merge because someone has issued a /hold command. label Jan 23, 2026
@openshift-ci
Copy link

openshift-ci bot commented Jan 23, 2026

@JoaoFula: The following test failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name Commit Details Required Rerun command
ci/prow/e2e-ols-cluster 3a24962 link true /test e2e-ols-cluster

Full PR test history. Your PR dashboard.

Details

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository. I understand the commands that are listed here.

@openshift-merge-robot openshift-merge-robot added the needs-rebase Indicates a PR cannot be merged because it has merge conflicts with HEAD. label Jan 23, 2026
@openshift-merge-robot
Copy link

PR needs rebase.

Details

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

approved Indicates a PR has been approved by an approver from all required OWNERS files. do-not-merge/hold Indicates that a PR should not merge because someone has issued a /hold command. needs-rebase Indicates a PR cannot be merged because it has merge conflicts with HEAD.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants