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

Convert e2e to workflow_dispatch #623

Merged
merged 3 commits into from
Dec 21, 2023
Merged

Conversation

kmdkuk
Copy link
Contributor

@kmdkuk kmdkuk commented Dec 11, 2023

  • "OWNER", "COLLABORATOR", and "CONTRIBUTOR" comment /test in the PR, the e2e test is executed.

  • In converting ci-e2e to workflow_dispatch, status checks are now updated manually in workflow.

    • using myrotvorets/set-commit-status-action
  • Added process to ci-e2e to retrieve branches to be tested.

    • With on.pull_request, the post-merge branch was used for testing without the trigger being aware of it, but with on.workflow_dispath, the github.ref is the branch that was executed and the post-merge branch is not used.
  • The .on.issue_comment workflow is not executed until merged into the main branch, so commenting /test on this PR will not execute the e2e test.

Signed-off-by: kouki [email protected]

@kmdkuk kmdkuk self-assigned this Dec 11, 2023
* Fixed checkout of branch after merge in e2e

Signed-off-by: kouki <[email protected]>
@kmdkuk kmdkuk force-pushed the convert_e2e_to_workflow_dispatch branch from 5ab44d5 to e07f91a Compare December 12, 2023 08:48
@kmdkuk kmdkuk changed the title wip: Convert e2e to workflow_dispatch Convert e2e to workflow_dispatch Dec 12, 2023
@kmdkuk kmdkuk marked this pull request as ready for review December 12, 2023 08:54
@kmdkuk
Copy link
Contributor Author

kmdkuk commented Dec 13, 2023

The E2E test for this PR was run manually from local with gh workflow run --ref convert_e2e_to_workflow_dispatch ci-e2e.yaml.
https://github.com/cybozu-go/moco/actions/runs/7179103457

@kmdkuk kmdkuk requested a review from yamatcha December 13, 2023 00:35
yamatcha
yamatcha previously approved these changes Dec 15, 2023
@kmdkuk kmdkuk merged commit a0e39b5 into main Dec 21, 2023
4 checks passed
@kmdkuk kmdkuk deleted the convert_e2e_to_workflow_dispatch branch December 21, 2023 00:31
yamatcha added a commit that referenced this pull request Jan 30, 2024
…w_dispatch"

This reverts commit a0e39b5, reversing
changes made to 018fbeb.
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