Skip to content

Call e2e test workflow from main build with installer version #18381

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

Merged
merged 5 commits into from
Jul 28, 2023

Conversation

WVerlaek
Copy link
Member

@WVerlaek WVerlaek commented Jul 28, 2023

Description

Fix: in the previous PR I had configured the e2e tests to run on push events to main, but it was not using the installer version of that pushed commit (it chooses the last successful green build).

Now, instead of triggering on pushes to main, have the Build workflow trigger the e2e test workflow on main, and explicitly pass the installer version of that commit.

Summary generated by Copilot

🤖 Generated by Copilot at 8c56e91

This pull request adds the ability to run workspace integration tests on the main branch using a reusable workflow. It modifies the .github/workflows/workspace-integration-tests.yml and .github/workflows/build.yml files to enable this functionality.

Related Issue(s)

Fixes #

How to test

Documentation

Preview status

gitpod:summary

Build Options

Build
  • /werft with-werft
    Run the build with werft instead of GHA
  • leeway-no-cache
  • /werft no-test
    Run Leeway with --dont-test
Publish
  • /werft publish-to-npm
  • /werft publish-to-jb-marketplace
Installer
  • analytics=segment
  • with-dedicated-emulation
  • workspace-feature-flags
    Add desired feature flags to the end of the line above, space separated
Preview Environment / Integration Tests
  • /werft with-local-preview
    If enabled this will build install/preview
  • /werft with-preview
  • /werft with-large-vm
  • /werft with-gce-vm
    If enabled this will create the environment on GCE infra
  • with-integration-tests=all
    Valid options are all, workspace, webapp, ide, jetbrains, vscode, ssh. If enabled, with-preview and with-large-vm will be enabled.
  • with-monitoring

/hold

@WVerlaek WVerlaek changed the title Call e2e test workflow from build with installer version Call e2e test workflow from main build with installer version Jul 28, 2023
@WVerlaek WVerlaek marked this pull request as ready for review July 28, 2023 12:44
@WVerlaek WVerlaek requested a review from a team as a code owner July 28, 2023 12:44
@roboquat roboquat added size/M and removed size/S labels Jul 28, 2023
} >> $GITHUB_OUTPUT
else
{
echo "name=workspace-integration-test-${{ github.run_id }}-${{ github.run_attempt }}"
} >> $GITHUB_OUTPUT
fi

if [[ '${{ github.event.inputs.version }}' != '' ]]; then
if [[ '${{ inputs.version }}' != '' ]]; then
Copy link
Member Author

Choose a reason for hiding this comment

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

for workflow_call github.event.inputs.* doesnt exist and need to use inputs.* instead. Works now: https://github.com/gitpod-io/gitpod/actions/runs/5692256844/job/15429016661?pr=18381#step:6:12

inputs.* is also available for workflow_dispatch

@WVerlaek
Copy link
Member Author

/unhold

@roboquat roboquat merged commit eebaae4 into main Jul 28, 2023
@roboquat roboquat deleted the wv/fix-e2e-version-on-main branch July 28, 2023 13:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants