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

Add e2e distribution tests #160

Merged
merged 30 commits into from
Dec 20, 2022
Merged

Add e2e distribution tests #160

merged 30 commits into from
Dec 20, 2022

Conversation

degray
Copy link
Contributor

@degray degray commented Dec 16, 2022

Adds vagrant based e2e tests that run every day at 17:00 UTC via github actions.

  1. Provisions the vagrant VM (done every time to ensure we have the latest packages)
  2. Installs Orchestrator via our interactive install.sh script
  3. Runs testsignal (dotnet DLL monitor) within 30s
  4. Checks to ensure testsignal existed properly
  5. Uninstalls/purges and stops the Orchestrator
  6. Destroys the vagrant VM (clean slate next run)

Account that this sends telemetry to is Orchestrator dist e2e

There's a hard 30m timeout on the e2e job in case a parallel run gets stuck somehow (I have seen provisioning of the vagrant VM take 2m or 7m as an example on the same vagrant box)

Uses macos-12 Github Action runner image as that is the only image that supports nested virtualization at the moment. Request to support it on Linux images have so far gone unanswered. See: actions/runner-images#433

Reviewer can remove branch push trigger on the github action before merging. Left there for now so that the reviewer can trigger/try it.

  push:
    branches: [distribution-e2e-tests]

Deploy steps:

  • Merge

Note: Secrets already setup

image

Internal reference: MET-1151

Copy link
Contributor

@fmterrorf fmterrorf left a comment

Choose a reason for hiding this comment

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

:shipit:

@fmterrorf fmterrorf merged commit 3ae0407 into main Dec 20, 2022
@fmterrorf fmterrorf deleted the distribution-e2e-tests branch December 20, 2022 16:50
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.

2 participants