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

Allow running tests from fork #1945

Merged
merged 9 commits into from
Oct 3, 2024
Merged

Conversation

kt474
Copy link
Member

@kt474 kt474 commented Sep 24, 2024

Summary

It'd be nice to have an easier way to run integration tests from a PR. Currently they are only run on push and in a nightly cron job. Github does not allow pull request access to secrets so we can't run them from ci.yml.

Instead, we can allow the integration tests to be manually triggered from a fork. The tests will only run when manually triggered, and not during the nightly cron - example: https://github.com/kt474/qiskit-ibm-runtime/actions/runs/11040251232

Details and comments

@kt474 kt474 added the integration test run integration tests label Sep 24, 2024
@kt474 kt474 removed the integration test run integration tests label Sep 25, 2024
@kt474 kt474 changed the title Test running integration tests from PR commit msg Allow running tests from fork Sep 25, 2024
Copy link
Collaborator

@ptristan3 ptristan3 left a comment

Choose a reason for hiding this comment

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

LGTM. Thanks!

@kt474 kt474 merged commit 4ae4b8b into Qiskit:main Oct 3, 2024
18 checks passed
@kt474 kt474 deleted the pr-integration-tests branch October 3, 2024 15:47
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