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

[CI Testing] Add manual GHA triggers for Drake PRs #371

Closed
wants to merge 6 commits into from

Conversation

tyler-yankee
Copy link
Collaborator

@tyler-yankee tyler-yankee commented Feb 24, 2025

This introduces parameters on each of the GitHub Actions examples to be able to run the CI jobs with alternative versions of Drake. Towards #22574.

Changes to each example are as follows:

  • drake_cmake_installed: install an experimental package of Drake into $HOME/drake, and use that when cmake -DCMAKE_PREFIX_PATH=<path> .. is called to configure/generate
  • drake_cmake_installed_apt: uses apt-get install <url> with a link to an experimental package
  • drake_bazel_download: similar to drake_cmake_installed, but uses Bazel's --override-repository flag when calling bazel test //...
  • drake_pip: uses pip install <url> with a link to experimental wheels (note: does this after installing drake from requirements.txt, in case any other packages are added to that file in the future)
  • drake_poetry: similar to drake_pip, but uses poetry add <url>

Additionally, modifies the CI config files to add parameters to the workflow_dispatch event.

  • linux_jammy_package_tar
  • linux_jammy_package_deb
  • mac_arm_sonoma_package_tar
  • linux_jammy_wheel
  • mac_arm_sonoma_wheel

The use case here is that a developer with a PR on Drake could test their changes on DEE with the following workflow:

  • Run experimental packaging builds on Linux Jammy and macOS Sonoma and obtain the artifact URLs.
  • Run experimental wheel builds on Linux Jammy and macOS Sonoma and obtain the artifact URLs.
  • Navigate to DEE's Actions page and kick off a build of all GHA examples, inputting the URLs from above.

Tested via GHA on my forked repo (TODO will insert link here).

This change is Reviewable

@tyler-yankee tyler-yankee force-pushed the drake-pr-ci branch 2 times, most recently from 47812b1 to 13d53ff Compare February 24, 2025 14:03
tyler-yankee added a commit that referenced this pull request Feb 24, 2025
Needed for #371 (and also just generally good to have).
tyler-yankee added a commit to tyler-yankee/drake-external-examples that referenced this pull request Feb 24, 2025
Needed for RobotLocomotion#371 (and also just generally good to have).
sammy-tri pushed a commit that referenced this pull request Feb 24, 2025
Needed for #371 (and also just generally good to have).
@tyler-yankee tyler-yankee force-pushed the drake-pr-ci branch 2 times, most recently from 31d771c to ea6fffb Compare February 25, 2025 15:57
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.

1 participant