Skip to content

test: Introduction of parallelization of tests to launch workspaces #13293

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 2 commits into from
Sep 26, 2022
Merged

test: Introduction of parallelization of tests to launch workspaces #13293

merged 2 commits into from
Sep 26, 2022

Conversation

utam0k
Copy link
Contributor

@utam0k utam0k commented Sep 26, 2022

Description

  • Introduction of parallelization of tests to launch workspaces
  • Use a large workspace

Note: now the integration test is broken

Related Issue(s)

Part of #13197

How to test

Run the integration test

Release Notes

NONE

Documentation

No

Werft options:

  • /werft with-local-preview
    If enabled this will build install/preview
  • /werft with-preview
  • /werft with-integration-tests=workspacce
    Valid options are all, workspace, webapp, ide

@utam0k utam0k requested a review from a team September 26, 2022 07:53
@github-actions github-actions bot added the team: workspace Issue belongs to the Workspace team label Sep 26, 2022
@utam0k
Copy link
Contributor Author

utam0k commented Sep 26, 2022

This PR https://werft.gitpod-dev.com/job/gitpod-custom-to-inte-speed.3

--- PASS: TestGitHubContexts (693.68s)
    --- PASS: TestGitHubContexts/context (693.68s)
        --- PASS: TestGitHubContexts/context/should_run_context_tests (21.89s)
            --- PASS: TestGitHubContexts/context/should_run_context_tests/github.com/gitpod-io/template-golang-cli_pvc (122.48s)
            --- PASS: TestGitHubContexts/context/should_run_context_tests/github.com/gitpod-io/template-golang-cli_classic (131.66s)
            --- PASS: TestGitHubContexts/context/should_run_context_tests/github.com/gitpod-io/empty_classic (213.45s)
            --- PASS: TestGitHubContexts/context/should_run_context_tests/github.com/atduarte/lfs-test_classic (222.62s)
            --- PASS: TestGitHubContexts/context/should_run_context_tests/github.com/gitpod-io/gitpod-test-repo/tree/integration-test-context-tag_pvc (288.75s)
            --- PASS: TestGitHubContexts/context/should_run_context_tests/github.com/gitpod-io/empty_pvc (313.51s)
            --- PASS: TestGitHubContexts/context/should_run_context_tests/github.com/atduarte/lfs-test_pvc (380.98s)
            --- PASS: TestGitHubContexts/context/should_run_context_tests/github.com/gitpod-io/gitpod-test-repo/issues/88_classic (404.77s)
            --- PASS: TestGitHubContexts/context/should_run_context_tests/github.com/gitpod-io/gitpod-test-repo/tree/integration-test-1_pvc (495.52s)
            --- PASS: TestGitHubContexts/context/should_run_context_tests/github.com/gitpod-io/gitpod-test-repo/issues/88_pvc (569.14s)
            --- PASS: TestGitHubContexts/context/should_run_context_tests/github.com/gitpod-io/gitpod-test-repo/tree/integration-test-1_classic (576.44s)
            --- PASS: TestGitHubContexts/context/should_run_context_tests/github.com/gitpod-io/gitpod-test-repo/tree/integration-test-context-tag_classic (671.79s)

Before this test took 1682 sec.

jenting
jenting approved these changes Sep 26, 2022
View reviewed changes
.werft/workspace-run-integration-tests.sh
@@ -84,7 +84,7 @@ werft log phase "build preview environment" "build preview environment"
git checkout -B "${BRANCH}" && \
git commit -m "${TEMP_COMMIT_MSG}" --allow-empty && \
git push --set-upstream origin "${BRANCH}" && \
werft run github -a with-preview=true
werft run github -a with-preview=true -a with-large-vm=true
Copy link
Contributor

Choose a reason for hiding this comment

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

Cool, I don't know we have this annotation until now.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I learned it from @vulkoingim

@roboquat roboquat merged commit 4af208a into main Sep 26, 2022
@roboquat roboquat deleted the to/inte-speed branch September 26, 2022 07:59
@roboquat roboquat added deployed: workspace Workspace team change is running in production deployed Change is completely running in production labels Sep 28, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@jenting jenting jenting approved these changes

Assignees
No one assigned
Labels
deployed: workspace Workspace team change is running in production deployed Change is completely running in production release-note-none size/S team: workspace Issue belongs to the Workspace team
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@utam0k @jenting @roboquat