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

Advanced workflow for complete e2e test #584

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

khannakshat7
Copy link
Contributor

What this PR does / why we need it:
This workflow is needed when we want to run full e2e tests for the PR before merging.

Additional information
We have to add label pr/run-all-e2e-test to run all the e2e tests on the PR.

Special notes for your reviewer

I tried many different implementations-

  1. Using flag /test-full-e2e to run e2e test but it failed as the event was triggered but was not updating in checks in the PR.
  2. Using flag /test-full-e2e to create a workflow to add label to PR by Github bot and then running the e2e tests but it failed becoz a GitHub workflow cannot trigger another workflow.

@khannakshat7 khannakshat7 changed the title Advanced workflow for complete e2e Advanced workflow for complete e2e test Jun 8, 2022
@sachinkumarsingh092
Copy link
Contributor

Just as a reminder: after closing this issue, also close the WIP PR #452.

@dharmjit dharmjit added the pr/run-all-e2e-test Label to run all e2e tests label Jun 8, 2022
@anusha94
Copy link
Contributor

anusha94 commented Jun 9, 2022

@khannakshat7 Can we add this info to CONTRIBUTING.md ?

Copy link
Contributor

@dharmjit dharmjit left a comment

Choose a reason for hiding this comment

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

LGTM

@anusha94 anusha94 marked this pull request as draft June 9, 2022 05:28
@anusha94 anusha94 mentioned this pull request Jun 9, 2022
@anusha94 anusha94 mentioned this pull request Jun 24, 2022
@shivi28
Copy link
Contributor

shivi28 commented Nov 19, 2022

Hi @khannakshat7, thanks for the changes. If you are still interested, then can you please convert this to PR?

@khannakshat7 khannakshat7 marked this pull request as ready for review November 19, 2022 16:52
@khannakshat7
Copy link
Contributor Author

Hi @shivi28, I have converted this to PR. Thanks

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
cla-not-required pr/run-all-e2e-test Label to run all e2e tests
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants