8000 [WIP] Add support for Vagrant VMs for testing by NouemanKHAL · Pull Request #20353 · DataDog/integrations-core · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content

[WIP] Add support for Vagrant VMs for testing #20353

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
Draft
wants to merge 29 commits into
base: noueman/glusterfs-vendor-gstatus
Choose a base branch
from
Draft

[WIP] Add support for Vagrant VMs for testing #20353

wants to merge 29 commits into from

Conversation

NouemanKHAL
Copy link
Member
@NouemanKHAL NouemanKHAL commented May 22, 2025 •
edited
Loading

What does this PR do?

Add support for running E2E tests on VMs, using the E2E_METADATA as an interface to customize the VM via the following properties:

  • vagrant_box (VM image)
  • start_commands
  • post_install_commands
  • env
  • vagrant_guest_os
  • vagrant_synced_folders (similar to docker_volumes)

Motivation

Provide better E2E testing for host based integrations.

Review checklist (to be filled by reviewers)

  • Feature or bugfix MUST have appropriate tests (unit, integration, e2e)
  • Add the qa/skip-qa label if the PR doesn't need to be tested during QA.
  • If you need to backport this PR to another branch, you can add the backport/<branch-name> label to the PR and it will automatically open a backport PR once this one is merged

@NouemanKHAL NouemanKHAL changed the title [WIP] Add support for vm_run context manager for testing [WIP] Add support for Vagrant VMs for testing Jun 4, 2025
@codecov Codecov
Copy link
codecov bot commented Jun 17, 2025 •
edited
Loading

Codecov Report

Attention: Patch coverage is 20.00000% with 28 lines in your changes missing coverage. Please review.

Please upload report for BASE (noueman/glusterfs-vendor-gstatus@9220006). Learn more about missing BASE report.

Additional details and impacted files
Flag Coverage Δ
datadog_checks_dev 77.58% <ø> (?)
glusterfs 52.07% <20.00%> (?)

Flags with carried forward coverage won't be shown. Click here to find out more.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
ddev dev_package dev/tooling documentation integration/glusterfs
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@NouemanKHAL
0