Skip to content

Conversation

@qmonnet
Copy link
Member

@qmonnet qmonnet commented Oct 24, 2025

When we manually trigger the dev.yml workflow to run tests, we don't have Pull Request labels in the context, and can't turn VLAB or HLAB tests on. Add options to enable these tests (and the "release" tests) via the dispatch interface.

@qmonnet qmonnet requested a review from a team as a code owner October 24, 2025 09:59
@qmonnet qmonnet added the ci Continuous Integration label Oct 24, 2025
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR enhances the manual dispatch interface for the dev.yml workflow by adding three boolean input options that allow users to enable VLAB tests, HLAB tests, and release tests when triggering the workflow manually. Previously, these test types could only be enabled through Pull Request labels, which are unavailable in manual workflow dispatches.

Key changes:

  • Added three boolean workflow dispatch inputs: run_vlab_tests, run_hlab_tests, and enable_release_tests
  • Updated the skip logic to check both the new inputs and existing PR labels using OR conditions
  • Modified the releasetest parameter to consider the new run_release_tests input alongside the existing label check

@qmonnet qmonnet force-pushed the pr/qmonnet/vlab-workflow-trigger branch from 65c816d to 638de30 Compare October 24, 2025 10:45
When we manually trigger the dev.yml workflow to run tests, we don't
have Pull Request labels in the context, and can't turn VLAB or HLAB
tests on. Add options to enable these tests (and the "release" tests)
via the dispatch interface.

Signed-off-by: Quentin Monnet <[email protected]>
@qmonnet qmonnet force-pushed the pr/qmonnet/vlab-workflow-trigger branch from 638de30 to 1b16d22 Compare October 24, 2025 11:23
@qmonnet qmonnet added the ci:+vlab Enable VLAB tests label Oct 24, 2025
@qmonnet qmonnet closed this Oct 24, 2025
@qmonnet qmonnet reopened this Oct 24, 2025
@qmonnet
Copy link
Member Author

qmonnet commented Oct 24, 2025

Fredi reported that VLAB tests are currently failing, this is not related to this PR.

Copy link
Contributor

@Fredi-raspall Fredi-raspall left a comment

Choose a reason for hiding this comment

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

Looks good to me @qmonnet , thanks!
We should document somewhere what options do we have wrt to ci pipelines/workflows etc. I see a lot of workflow files but have to admit that have very little idea of what they do.

@qmonnet qmonnet removed the ci:+vlab Enable VLAB tests label Oct 24, 2025
This adds comprehensive documentation for all GitHub workflows in the
repository, including triggers, main steps, and available options for
manual dispatch and PR labels.

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Prompt:

    Look at all GitHub workflow files under .github/workflows/, and
    produce a file .github/workflows/README.md that summarizes the
    different workflows, what they do, the main steps, and on what
    triggers. You may mention the main elements from the job matrix if
    any, but don't go super deep into the details. For the dev.yaml,
    mention the main "options" that users can use via labels (when
    running the workflow on a Pull Request) or via dispatch options. The
    objective is to help contributors not familiar with CI understand
    what workflows we have in this repository.

Quentin: I did a second pass, changed the formatting, cleaned up and
trimmed or completed some pieces of information.

Suggested-by: Fredi Raspall <[email protected]>
Co-Authored-By: Claude <[email protected]>
Signed-off-by: Quentin Monnet <[email protected]>
@qmonnet qmonnet force-pushed the pr/qmonnet/vlab-workflow-trigger branch from dbbef94 to becb0f8 Compare October 24, 2025 14:50
@qmonnet
Copy link
Member Author

qmonnet commented Oct 24, 2025

@Fredi-raspall True. I added a commit with an overview of the workflows, please take a look.

@mvachhar mvachhar added this pull request to the merge queue Oct 24, 2025
@github-merge-queue github-merge-queue bot removed this pull request from the merge queue due to failed status checks Oct 24, 2025
@qmonnet qmonnet added this pull request to the merge queue Oct 24, 2025
Merged via the queue into main with commit 983815f Oct 24, 2025
19 checks passed
@qmonnet qmonnet deleted the pr/qmonnet/vlab-workflow-trigger branch October 24, 2025 21:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ci Continuous Integration

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants