Skip to content
This repository was archived by the owner on Apr 23, 2025. It is now read-only.

fix: execute in BGT to avoid freezing EDT (#963) #964

Merged
merged 1 commit into from
Jan 31, 2025

Conversation

adietish
Copy link
Contributor

@adietish adietish commented Jan 28, 2025

fixes #963
same problem as in #922 (review)

Copy link

openshift-ci bot commented Jan 28, 2025

Skipping CI for Draft Pull Request.
If you want CI signal for your change, please convert it to an actual PR.
You can still manually trigger a test run with /test all

@adietish adietish requested a review from sbouchet January 28, 2025 14:10
@adietish adietish added this to the 1.11.0 milestone Jan 28, 2025
@adietish adietish marked this pull request as ready for review January 28, 2025 15:35
@adietish adietish added bug Something isn't working severity/major labels Jan 28, 2025
@adietish adietish removed this from the 1.11.0 milestone Jan 28, 2025
@adietish adietish removed bug Something isn't working severity/major labels Jan 28, 2025
Copy link
Collaborator

@sbouchet sbouchet left a comment

Choose a reason for hiding this comment

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

LGTM.

Copy link

openshift-ci bot commented Jan 31, 2025

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: sbouchet

The full list of commands accepted by this bot can be found here.

The pull request process is described here

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

Copy link

openshift-ci bot commented Jan 31, 2025

New changes are detected. LGTM label has been removed.

@adietish adietish added the lgtm label Jan 31, 2025
@adietish
Copy link
Contributor Author

/override ci/prow/e2e-openshift "Integration UI Tests with Local Kind Cluster Context Report" "Integration UI Tests with No Context Report" "Aggregated Test Report"

Copy link

openshift-ci bot commented Jan 31, 2025

@adietish: Overrode contexts on behalf of adietish: Aggregated Test Report, Integration UI Tests with Local Kind Cluster Context Report, Integration UI Tests with No Context Report, ci/prow/e2e-openshift

In response to this:

/override ci/prow/e2e-openshift "Integration UI Tests with Local Kind Cluster Context Report" "Integration UI Tests with No Context Report" "Aggregated Test Report"

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.

@redhat-developer redhat-developer deleted a comment from openshift-ci bot Jan 31, 2025
Copy link

@openshift-merge-bot openshift-merge-bot bot merged commit 94ff209 into redhat-developer:main Jan 31, 2025
20 checks passed
@adietish adietish deleted the issue-963 branch January 31, 2025 09:57
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

CreateProjectAction blocks EDT for too long, causes severe warning in log
2 participants