Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix: allow wait for multiple DWO webhook pods #2895

Merged
merged 1 commit into from
Jul 18, 2024

Conversation

dkwon17
Copy link
Contributor

@dkwon17 dkwon17 commented Jul 15, 2024

What does this PR do?

Fixes the DWO pod timeout issue when running chectl server:deploy -p openshift:

      ✖ Wait for Dev Workspace operator ready
        → ERR_TIMEOUT: Timeout set to pod ready timeout 60000
      Create Subscription eclipse-che
      Eclipse Che Operator pod bootstrap
      Fetch CheCluster sample from a CSV
      Create CheCluster Custom Resource
Error: Command server:deploy failed with the error: ERR_TIMEOUT: Timeout set to pod ready timeout 60000 See details: /home/vsvydenko/.cache/chectl/error.log. Eclipse Che logs: /tmp/chectl-logs/1720444057431.
    at newError (/home/vsvydenko/.local/share/chectl/client/0.0.20240702-next.8e77812/lib/utils/utls.js:39:19)
    at wrapCommandError (/home/vsvydenko/.local/share/chectl/client/0.0.20240702-next.8e77812/lib/utils/command-utils.js:54:32)
    at Deploy.<anonymous> (/home/vsvydenko/.local/share/chectl/client/0.0.20240702-next.8e77812/lib/commands/server/deploy.js:82:65)
    at Generator.throw (<anonymous>)
    at rejected (/home/vsvydenko/.local/share/chectl/client/0.0.20240702-next.8e77812/node_modules/tslib/tslib.js:167:69)
Cause: Error: ERR_TIMEOUT: Timeout set to pod ready timeout 60000
    at KubeClient.<anonymous> (/home/vsvydenko/.local/share/chectl/client/0.0.20240702-next.8e77812/lib/api/kube-client.js:826:19)
    at Generator.next (<anonymous>)
    at fulfilled (/home/vsvydenko/.local/share/chectl/client/0.0.20240702-next.8e77812/node_modules/tslib/tslib.js:166:62)

Screenshot/screencast of this PR

What issues does this PR fix or reference?

eclipse-che/che#23029

How to test this PR?

To test this PR on OpenShift, run:

yarn && ./bin/run server:deploy -p openshift

The command should complete without any problems.

PR Checklist

As the author of this Pull Request I made sure that:

Reviewers

Reviewers, please comment how you tested the PR when approving it.

Copy link

openshift-ci bot commented Jul 15, 2024

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

Copy link

openshift-ci bot commented Jul 17, 2024

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by: dkwon17, ibuziuk

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

@dkwon17
Copy link
Contributor Author

dkwon17 commented Jul 17, 2024

/retest

@openshift-ci openshift-ci bot removed the lgtm label Jul 18, 2024
Copy link

openshift-ci bot commented Jul 18, 2024

New changes are detected. LGTM label has been removed.

@dkwon17
Copy link
Contributor Author

dkwon17 commented Jul 18, 2024

I've updated this PR. This should fix the CI check failure

@dkwon17 dkwon17 merged commit 386da7e into che-incubator:main Jul 18, 2024
12 checks passed
@devstudio-release
Copy link

Build 3.16 :: dsc_3.x/1943: Console, Changes, Git Data

@devstudio-release
Copy link

Build 3.16 :: dsc_3.x/1943: SUCCESS

3.16.0-CI

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