Skip to content

feat(workflow/e2e): allow workflow_dispatch trigger tests#3109

Merged
SkyeYoung merged 1 commit intoapache:masterfrom
SkyeYoung:young/feat/workflow/e2e-workflow-dispatch
Jun 9, 2025
Merged

feat(workflow/e2e): allow workflow_dispatch trigger tests#3109
SkyeYoung merged 1 commit intoapache:masterfrom
SkyeYoung:young/feat/workflow/e2e-workflow-dispatch

Conversation

@SkyeYoung
Copy link
Member

Please answer these questions before submitting a pull request, or your PR will get closed.

Why submit this pull request?

  • Bugfix
  • New feature provided
  • Improve performance
  • Backport patches

What changes will this PR take into?

This is part of the apache/apisix#12234

According to Events that trigger workflows , this will allow us to trigger the test manually.

Why

This is very suitable for manually running a test to ensure that the dashboard can adapt to the latest APISIX, either on the eve of APISIX's release or after the APISIX:dev image is updated.

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

Enable manual invocation of the end-to-end test workflow alongside scheduled and label-triggered runs

  • Add workflow_dispatch event trigger for on-demand test execution
  • Update job if condition to include workflow_dispatch alongside schedule and label checks
Comments suppressed due to low confidence (2)

.github/workflows/e2e.yml:24

  • [nitpick] Clarify the comment to mention manual triggers explicitly, e.g., # run when e2e-test label is added, on schedule, or via manual workflow_dispatch.
# only run when e2e-test label is added, scheduled, workflow_dispatch

.github/workflows/e2e.yml:16

  • [nitpick] Consider adding a brief comment above workflow_dispatch to explain that it enables manual triggering from the GitHub UI.
workflow_dispatch:

Copy link
Member

@juzhiyuan juzhiyuan left a comment

Choose a reason for hiding this comment

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

Thanks

@SkyeYoung SkyeYoung merged commit b3cf746 into apache:master Jun 9, 2025
5 checks passed
@SkyeYoung SkyeYoung deleted the young/feat/workflow/e2e-workflow-dispatch branch June 9, 2025 09:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants