Skip to content

Comments

Add storage type attribute to DevWorkspace configuration in API tests; fix EmptyWorkspace PR check#23455

Merged
dmytro-ndp merged 6 commits intomainfrom
CRW-8922
Jun 10, 2025
Merged

Add storage type attribute to DevWorkspace configuration in API tests; fix EmptyWorkspace PR check#23455
dmytro-ndp merged 6 commits intomainfrom
CRW-8922

Conversation

@dmytro-ndp
Copy link
Contributor

@dmytro-ndp dmytro-ndp commented Jun 6, 2025

What does this PR do?

  1. It updates API tests - adds storage type attribute to DevWorkspace configuration in API tests (issue CRW-8922)

  2. It fixes API test execution error when one of dashboard pod is evicted:

           ▼ ShellExecutor.executeCommand - oc get pods -n openshift-devspaces | grep dashboard | awk '{print $1}'
 devspaces-dashboard-9cbb75c7d-84qq8
 devspaces-dashboard-9cbb75c7d-db8l9
           ▼ ShellExecutor.executeArbitraryShellScript - oc get pod -n openshift-devspaces devspaces-dashboard-9cbb75c7d-84qq8
           devspaces-dashboard-9cbb75c7d-db8l9 -o jsonpath='{.spec.containers[*].name}'
           ▼ ShellExecutor.executeCommand - oc get pod -n openshift-devspaces devspaces-dashboard-9cbb75c7d-84qq8
           devspaces-dashboard-9cbb75c7d-db8l9 -o jsonpath='{.spec.containers[*].name}'
 NAME                                  READY   STATUS   RESTARTS   AGE
 devspaces-dashboard-9cbb75c7d-84qq8   0/1     Error    0          13h
 /bin/sh: 2: devspaces-dashboard-9cbb75c7d-db8l9: not found
  1. It stabilize 'Empty Workspace' test PR check (issue Fix the 'Empty Workspace' test PR check failure that occurs during test workspace start due to no space left on the Minikube instance #23457)

Screenshot/screencast of this PR

What issues does this PR fix or reference?

How to test this PR?

Successful DevFile API test run: https://jenkins-csb-crwqe-main.dno.corp.redhat.com/job/Testing/job/e2e/job/complex/job/next/job/next-ds-functional-tests/1075/

Successful PR Check run: https://github.com/eclipse-che/che/actions/runs/15569210809/job/43840783566

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.

@dmytro-ndp dmytro-ndp requested review from SkorikSergey and artaleks9 and removed request for musienko-maxim June 6, 2025 19:42
@dmytro-ndp dmytro-ndp requested a review from SDawley as a code owner June 10, 2025 18:50
@dmytro-ndp dmytro-ndp removed the request for review from SDawley June 10, 2025 20:39
@dmytro-ndp dmytro-ndp merged commit 9a5c6e1 into main Jun 10, 2025
6 checks passed
@dmytro-ndp dmytro-ndp deleted the CRW-8922 branch June 10, 2025 20:42
@dmytro-ndp dmytro-ndp changed the title Add storage type attribute to DevWorkspace configuration in API tests Add storage type attribute to DevWorkspace configuration in API tests; fix EmptyWorkspace PR check Jun 10, 2025
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